Commit e761f6f3 authored by Russell King's avatar Russell King

ARM: l2c: highbank: remove explicit SMI call in L2 cache initialisation

Now that highbank uses the write_sec method, we don't need to enable
the L2 cache in SoC specific code; this can be done via the normal
mechanisms in the L2C code.

Checking with Rob Herring:
> > Can we kill the "highbank_smc1(0x102, 0x1);" here?	That means
> > l2x0_of_init() will see the L2 cache disabled, and will try to enable
> > it via the write_sec hook, so it should do the right thing.
>
> Yes, that should work. You should be able to just call l2x0_of_init
> unconditionally. The condition was really to just avoid the smc on
> Midway which does get handled on h/w, but not if running virtualized.

So also drop the DT check too.  I'm leaving the config check in place
so that if L2 is disabled, the write_sec hook can be optimised away.
Signed-off-by: default avatarRussell King <rmk+kernel@arm.linux.org.uk>
parent 0074fb2c
...@@ -68,9 +68,7 @@ static void __init highbank_init_irq(void) ...@@ -68,9 +68,7 @@ static void __init highbank_init_irq(void)
highbank_scu_map_io(); highbank_scu_map_io();
/* Enable PL310 L2 Cache controller */ /* Enable PL310 L2 Cache controller */
if (IS_ENABLED(CONFIG_CACHE_L2X0) && if (IS_ENABLED(CONFIG_CACHE_L2X0)) {
of_find_compatible_node(NULL, NULL, "arm,pl310-cache")) {
highbank_smc1(0x102, 0x1);
outer_cache.write_sec = highbank_l2c310_write_sec; outer_cache.write_sec = highbank_l2c310_write_sec;
l2x0_of_init(0, ~0); l2x0_of_init(0, ~0);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment