Re: [PATCH] ARM: tegra: apbio access using dma for tegra20 only

[Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>]

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;
> +	}
> +}