lkml.org 
[lkml]   [2011]   [Nov]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH v2 2/8] arm/tegra: prepare early init for multiple tegra variants
From
On Wed, Nov 2, 2011 at 7:09 AM, Peter De Schrijver
<pdeschrijver@nvidia.com> wrote:
> This patch splits the early init code in a common and a tegra20 specific part.
> L2 cache initialization is generalized and discovers the cache associativity
> at runtime.
>
> Signed-off-by: Peter De Schrijver <pdeschrijver@nvidia.com>
> ---
<snip>

> -static __initdata struct tegra_clk_init_table common_clk_init_table[] = {
> +void (*arch_reset)(char mode, const char *cmd) = tegra_assert_system_reset;
> +
This is unrelated to the patch description?

> +static __initdata struct tegra_clk_init_table tegra20_clk_init_table[] = {
>        /* name         parent          rate            enabled */
>        { "clk_m",      NULL,           0,              true },
>        { "pll_p",      "clk_m",        216000000,      true },
> @@ -59,25 +61,34 @@ static __initdata struct tegra_clk_init_table common_clk_init_table[] = {
>        { NULL,         NULL,           0,              0},
>  };
>
> -static void __init tegra_init_cache(void)
> +static void __init tegra_init_cache(u32 tag_latency, u32 data_latency)
>  {
>  #ifdef CONFIG_CACHE_L2X0
>        void __iomem *p = IO_ADDRESS(TEGRA_ARM_PERIF_BASE) + 0x3000;
> +       u32 aux_ctrl;
> +
> +       writel_relaxed(tag_latency, p + L2X0_TAG_LATENCY_CTRL);
> +       writel_relaxed(data_latency, p + L2X0_DATA_LATENCY_CTRL);
>
> -       writel_relaxed(0x331, p + L2X0_TAG_LATENCY_CTRL);
> -       writel_relaxed(0x441, p + L2X0_DATA_LATENCY_CTRL);
> +       aux_ctrl = readl(p + L2X0_CACHE_TYPE);
> +       aux_ctrl = (aux_ctrl & 0x700) << (17-8);

If I'm interpreting the PL310 TRM correctly, the cache associativity
value in the L2X0_CACHE_TYPE register is read from the aux control
register, so this is just leaving the existing value in place. If the
value is not being modified, just take it out of the aux_ctrl mask
passed to l2x0_init.

> +       aux_ctrl |= 0x7C000001;

This adds the BRESP bit to the aux control register. Considering the
problems I've seen with random bits in the L2 cache causing all sorts
of instabilities, this should be split out into a separate change so
that it's obvious in a bisect.

>
> -       l2x0_init(p, 0x6C080001, 0x8200c3fe);
> +       l2x0_init(p, aux_ctrl, 0x8200c3fe);
>  #endif
>
>  }
>
> -void __init tegra_init_early(void)
> +static void __init tegra_setup_system_reset(void)
> +{
> +       arm_arch_reset = tegra_assert_system_reset;
> +}
> +
> +void __init tegra20_init_early(void)
>  {
>        tegra_init_fuse();
>        tegra_init_clock();
> -       tegra_clk_init_from_table(common_clk_init_table);
> -       tegra_init_cache();
> -
> -       arm_arch_reset = tegra_assert_system_reset;
> +       tegra_clk_init_from_table(tegra20_clk_init_table);
> +       tegra_init_cache(0x331, 0x441);
> +       tegra_setup_system_reset();

It seems likely that there will be multiple init functions that need
to be called from init_early on both tegra20 and tegra30. Maybe add a
common function that only calls tegra_setup_system_reset() for now?

>  }
> --
> 1.7.7.rc0.72.g4b5ea.dirty
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2011-11-02 19:33    [from the cache]
©2003-2011 Jasper Spaans