<div dir="ltr">Hi Joel,<div><br></div><div>Thanks for bringing this to my attention,</div><div><br></div><div>I think I will leave it the same way it is now because I will like to develop the reset driver and to handle the NPCM7xx SOC resets.</div><div><br></div><div>Cheers,</div><div><br></div><div>Tomer</div><div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, 16 Jan 2019 at 01:21, Joel Stanley <<a href="mailto:joel@jms.id.au">joel@jms.id.au</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Thu, 10 Jan 2019 at 03:44, Tomer Maimon <<a href="mailto:tmaimon77@gmail.com" target="_blank">tmaimon77@gmail.com</a>> wrote:<br>
<br>
> +Required Node in the NPCM7xx BMC:<br>
> +An additional register is present in the NPCM7xx SOC which is<br>
> +assumed to be in the same device tree, with and marked as<br>
> +compatible with "nuvoton,npcm750-rst".<br>
<br>
Is there a reason you don't include a phandle to the reset node?<br>
<br>
I think doing that would make more sense.<br>
<br>
> +adc: adc@f000c000 {<br>
> +       compatible = "nuvoton,npcm750-adc";<br>
> +       reg = <0xf000c000 0x8>;<br>
> +       interrupts = <GIC_SPI 0 IRQ_TYPE_LEVEL_HIGH>;<br>
> +       clocks = <&clk NPCM7XX_CLK_ADC>;<br>
> +};<br>
<br>
> +<br>
> +rst: rst@f0801000 {<br>
> +       compatible = "nuvoton,npcm750-rst", "syscon",<br>
> +       "simple-mfd";<br>
> +       reg = <0xf0801000 0x6C>;<br>
> +};<br>
</blockquote></div>