<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 10 April 2018 at 17:41, 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"><span class="">On Sun, Apr 8, 2018 at 4:03 PM, Tomer Maimon <<a href="mailto:tmaimon77@gmail.com">tmaimon77@gmail.com</a>> wrote:<br>
> This patch Enable ARM L2 cache module in Nuvoton NPCM7xx BMC<br>
> by adding L2 cache parameters into NPCM7xx DT machine start structure.<br>
><br>
> At patch V7 arm: npcm: add basic support for Nuvoton BMCs we got comments<br>
> regarding the flags use in L2 cache module.<br>
> - <a href="https://www.spinics.net/lists/arm-kernel/msg613212.html" rel="noreferrer" target="_blank">https://www.spinics.net/lists/<wbr>arm-kernel/msg613212.html</a><br>
><br>
> After checking again the L2 cache use in the NPCM7xx,<br>
> the only L2 cache flag we need to set is L2C_AUX_CTRL_SHARED_OVERRIDE<br>
> and it is done in the device tree:<br>
> <a href="https://patchwork.kernel.org/patch/10063497/" rel="noreferrer" target="_blank">https://patchwork.kernel.org/<wbr>patch/10063497/</a><br>
><br>
> L2 cache flag mask allowed all the flag option.<br>
<br>
</span>I've applied the patch to my fixes branch now, but took your description above<br>
instead of the two-line text that you had in the patch itself. I liked<br>
the longer<br>
text much better.<br>
<span class="HOEnZb"><font color="#888888"><br>
      Arnd<br>
</font></span></blockquote></div><br></div><div class="gmail_extra">O.K.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Thanks a lot!</div></div>