<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 29 March 2018 at 23:37, 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 Mon, Mar 26, 2018 at 12:11 PM, Tomer Maimon <<a href="mailto:tmaimon77@gmail.com">tmaimon77@gmail.com</a>> wrote:<br>
> The NPCM7xx are a family of BMC's that include several chips as:<br>
> NPCM750, NPCM730 etc.<br>
><br>
> All of the NPCM7xx BMC's have the common modules like Cortex-A9,<br>
> WDT, timers, etc, and there are a optional modules that can be added.<br>
><br>
> NPCM750 device tree already pushed to arm-soc.git:<br>
> <a href="https://git.kernel.org/pub/scm/linux/kernel/git/arm/arm-soc.git/commit/?h=for-next&id=d6bdd009c21db3f677dd1d1bbb8c20bc819074bc" rel="noreferrer" target="_blank">https://git.kernel.org/pub/<wbr>scm/linux/kernel/git/arm/arm-<wbr>soc.git/commit/?h=for-next&id=<wbr>d6bdd009c21db3f677dd1d1bbb8c20<wbr>bc819074bc</a><br>
><br>
> After discussion with Rob Herring we have modified the structure<br>
> of the NPCM7xx device tree as follow:<br>
><br>
> nuvoton-common-npcm7xx device tree include all common modules<br>
> nuvoton-npcm750 device tree include specific npcm750 modules,<br>
> the nuvoton-npcm750 device tree include nuvoton-common-npcm7xx device tree<br>
><br>
> In addition, the NPCM7xx device tree include the following changes:<br>
>         - watchdog module added<br>
>         - clock module modification<br>
>         - serial module modification<br>
>         - Modify Makefile NPCM7XX configuration name<br>
><br>
> Sorry if it cause more work after pushing NPCM750 device tree.<br>
<br>
</span>I only see one patch here, which fails to describe the changes and<br>
combines all of the above changes into one commit.<br>
<br>
Please follow the usual style for sending patches and do one thing<br>
at a time. In particular, never move large parts of a file code around<br>
in a patch that also contains other changes, those need to be split<br>
up. Each patch should also have a description that explains what<br>
it's for.<br>
<span class="HOEnZb"><font color="#888888"><br>
       Arnd<br>
</font></span></blockquote></div><br></div><div class="gmail_extra">sent again (split patches)</div><div class="gmail_extra"><br></div><div class="gmail_extra">Thanks!</div></div>