<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 5 April 2018 at 15:54, Arnd Bergmann <span dir="ltr"><<a href="mailto:arnd@arndb.de" target="_blank">arnd@arndb.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Fri, Mar 16, 2018 at 10:51 PM, Tomer Maimon <<a href="mailto:tmaimon77@gmail.com">tmaimon77@gmail.com</a>> wrote:<br>
><br>
><br>
> On 16 March 2018 at 07:52, Brendan Higgins <<a href="mailto:brendanhiggins@google.com">brendanhiggins@google.com</a>><br>
> wrote:<br>
>><br>
>> On Thu, Mar 15, 2018 at 4:16 PM Tomer Maimon <<a href="mailto:tmaimon77@gmail.com">tmaimon77@gmail.com</a>> wrote:<br>
>><br>
>> > Enable L2 Cache in Nuvoton NPCM7xx BMC.<br>
>><br>
>> > Signed-off-by: Tomer Maimon <<a href="mailto:tmaimon77@gmail.com">tmaimon77@gmail.com</a>><br>
>> > ---<br>
>> >   arch/arm/mach-npcm/npcm7xx.c | 2 ++<br>
>> >   1 file changed, 2 insertions(+)<br>
>><br>
>> > diff --git a/arch/arm/mach-npcm/npcm7xx.c b/arch/arm/mach-npcm/npcm7xx.c<br>
>> > index 5f7cd88103ef..c5f77d854c4f 100644<br>
>> > --- a/arch/arm/mach-npcm/npcm7xx.c<br>
>> > +++ b/arch/arm/mach-npcm/npcm7xx.c<br>
>> > @@ -17,4 +17,6 @@ static const char *const npcm7xx_dt_match[] = {<br>
>> >   DT_MACHINE_START(NPCM7XX_DT, "NPCM7XX Chip family")<br>
>> >          .atag_offset    = 0x100,<br>
>> >          .dt_compat      = npcm7xx_dt_match,<br>
>> > +       .l2c_aux_val    = 0x0,<br>
>> > +       .l2c_aux_mask   = ~0x0,<br>
>><br>
>> You need to limit this to the specific bit(s) you want to set and verify<br>
>> that<br>
>> the l2c driver does not already manage that bit appropriately and that it<br>
>> can<br>
>> not be specified via the dtsi.<br>
><br>
><br>
> Do you mean that I need to specify it the same as been done in V7?<br>
> because when I run with the above l2c_aux_val and  l2c_aux_mask parameters I<br>
> get the same result.<br>
><br>
> Also if I am not adding the l2c_aux_val and  l2c_aux_mask parameters to the<br>
> DT_MACHINE_START<br>
> the L2C cache does not initialize.<br>
>><br>
>><br>
>> We discussed this a little while ago with Rob here:<br>
>> <a href="https://www.spinics.net/lists/arm-kernel/msg613372.html" rel="noreferrer" target="_blank">https://www.spinics.net/lists/<wbr>arm-kernel/msg613372.html</a><br>
><br>
><br>
> Sorry in this link I see only Russel king comment regarding the L2C<br>
<br>
</div></div>I'm not sure what the outcome is, and the patch changelog doesn't<br>
explain what the patch is for, so I've not applied it. If you need the<br>
patch for correct operation, please resend it with a proper changelog<br>
comment explaining why it's needed and why you ended up not setting<br>
any of the bits.<br></blockquote><div> </div><div>Sorry for not explain it better,</div><div><br></div><div>It seems that I have to add <span style="color:rgb(80,0,80)">l2c_aux_val, </span><span style="color:rgb(80,0,80)">l2c_aux_mask parameters to the <span style="color:rgb(80,0,80);font-family:arial,sans-serif;font-size:small;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">DT_MACHINE_START</span>

</span></div><div><span style="color:rgb(80,0,80)"><span style="color:rgb(80,0,80);font-family:arial,sans-serif;font-size:small;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">to enable the L2 Cache.</span></span></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
The last email in that thread mentions<br>
L310_AUX_CTRL_CACHE_REPLACE_<wbr>RR, is that required after all?<br></blockquote><div><br></div><div>No it is not require this is why I added:</div><div><span style="color:rgb(80,0,80)">.l2c_aux_val    = 0x0,</span></div><div><span style="color:rgb(80,0,80);font-family:arial,sans-serif;font-size:small;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">.l2c_aux_mask   = ~0x0,</span></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
     Arnd<br>
</font></span></blockquote></div><br></div><div class="gmail_extra">Can I resend it on Sunday?</div><div class="gmail_extra"><br></div><div class="gmail_extra">Thanks,</div><div class="gmail_extra"><br></div><div class="gmail_extra">Tomer</div></div>