On 06/19/2012 12:06 AM, Laxman Dewangan wrote:
> The Tegra20 HW issue with accessing APBIO registers (such
> as fuse registers) directly from the CPU concurrently with
> APB DMA accesses has been fixed in Tegra30 and later chips.
> 
> Access these registers directly from the CPU on Tegra30
> and later, and apply the workaround only for Tegra20.

> +void tegra_apb_io_init(void)
> +{
> +     /* Need to use dma only when it is Tegra20 based platform */
> +     if (of_machine_is_compatible("nvidia,tegra20")) {

That tests only works when the system is booted using device tree. Can
you change this to:

if (of_machine_is_compatible("nvidia,tegra20") ||
    !of_have_populated_dt()) {

... and make sure you get the expected result when booting using a board
file (e.g. Harmony). Thanks.

> +             apbio_read = tegra_apb_readl_using_dma;
> +             apbio_write = tegra_apb_writel_using_dma;
> +     } else {
> +             apbio_read = tegra_apb_readl_direct;
> +             apbio_write = tegra_apb_writel_direct;
> +     }
> +}
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to