<div dir="ltr"><div dir="ltr"><br></div><div dir="ltr">Hi Rob,<br></div><div><br></div>Thanks for the clarification,<div><br></div><div>I will remove the rst node and Required Node in the NPCM7xx BMC explanation from the DT binding.</div><div> <br><div class="gmail_quote"><div dir="ltr">On Mon, 21 Jan 2019 at 18:12, Rob Herring <<a href="mailto:robh%2Bdt@kernel.org">robh+dt@kernel.org</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 Sun, Jan 20, 2019 at 7:48 PM Joel Stanley <<a href="mailto:joel@jms.id.au" target="_blank">joel@jms.id.au</a>> wrote:<br>
><br>
> On Sat, 19 Jan 2019 at 02:12, Tomer Maimon <<a href="mailto:tmaimon77@gmail.com" target="_blank">tmaimon77@gmail.com</a>> wrote:<br>
> ><br>
> > Hi Joel,<br>
> ><br>
> > Thanks for bringing this to my attention,<br>
> ><br>
> > 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.<br>
><br>
> You could also do that.<br>
><br>
> But I was suggesting you use a phandle, so you could then find the<br>
> node you want without searching the entire device tree for the node<br>
> with the correct compatible.<br>
<br>
That's not really any more efficient. You just search the entire tree<br>
for the matching phandle number instead. Well, that was true until we<br>
recently added the phandle cache.<br>
<br>
In any case, it you plan to move to the reset binding (which would be<br>
good), then it's better to have nothing in the DT and add something<br>
rather than change the DT binding.</blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> <br></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Rob<br></blockquote><div><br></div><div>Thanks,</div><div><br></div><div>Tomer </div></div></div></div>