<div dir="ltr"><div dir="ltr">Hi Philipp,</div><div dir="ltr"><br></div><div>Thanks a lot for your comments.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 29 Oct 2019 at 17:15, Philipp Zabel <<a href="mailto:p.zabel@pengutronix.de">p.zabel@pengutronix.de</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">Hi Tomer,<br>
<br>
On Mon, 2019-10-28 at 17:54 +0200, Tomer Maimon wrote:<br>
> Added device tree binding documentation for Nuvoton BMC<br>
> NPCM reset controller.<br>
> <br>
> Signed-off-by: Tomer Maimon <<a href="mailto:tmaimon77@gmail.com" target="_blank">tmaimon77@gmail.com</a>><br>
> ---<br>
>  .../bindings/reset/nuvoton,npcm-reset.txt     | 35 +++++++++++++++++++<br>
>  1 file changed, 35 insertions(+)<br>
>  create mode 100644 Documentation/devicetree/bindings/reset/nuvoton,npcm-reset.txt<br>
> <br>
> diff --git a/Documentation/devicetree/bindings/reset/nuvoton,npcm-reset.txt b/Documentation/devicetree/bindings/reset/nuvoton,npcm-reset.txt<br>
> new file mode 100644<br>
> index 000000000000..94793285a2ac<br>
> --- /dev/null<br>
> +++ b/Documentation/devicetree/bindings/reset/nuvoton,npcm-reset.txt<br>
> @@ -0,0 +1,35 @@<br>
> +Nuvoton NPCM Reset controller<br>
> +<br>
> +In the NPCM Reset controller boot the USB PHY, USB host<br>
> +and USB device initialize.<br>
<br>
Isn't this just a detail of the driver implementation?<br>
<br>
> +Required properties:<br>
> +- compatible : "nuvoton,npcm750-reset" for NPCM7XX BMC<br>
<br>
Is this driver expected to be reused for other SoCs?<br></blockquote><div>Yes. </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> +- reg : specifies physical base address and size of the register.<br>
> +- #reset-cells: must be set to 1<br>
> +<br>
> +Optional property:<br>
> +- nuvoton,sw-reset-number - Contains the software reset number to restart the SoC.<br>
> +  NPCM7xx contain four software reset that represent numbers 1 to 4.<br>
<br>
What's the difference between the four restart bits? Is this something<br>
that has to be configured per board?<br></blockquote><div>The SW reset lines are the same, but we like to give full flexibility to choose the line to use for board reset.</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> +  If 'nuvoton,sw-reset-number' is not specfied software reset is disabled.<br>
> +<br>
> +Example:<br>
> +     rstc: rstc@f0801000 {<br>
> +             compatible = "nuvoton,npcm750-reset";<br>
> +             reg = <0xf0801000 0x70>;<br>
> +             #reset-cells = <1>;<br>
> +             nuvoton,sw-reset-number = <2>;<br>
> +     };<br>
> +<br>
> +Specifying reset lines connected to IP NPCM7XX modules<br>
> +======================================================<br>
> +example:<br>
> +<br>
> +        spi0: spi@..... {<br>
> +                ...<br>
> +                resets = <&rstc NPCM7XX_RESET_PSPI1>;<br>
> +                ...<br>
> +        };<br>
> +<br>
> +The index could be found in <dt-bindings/reset/nuvoton,npcm7xx-reset.h>.<br>
<br>
regards<br>
Philipp<br>
<br></blockquote><div><br></div><div>Regards,</div><div><br></div><div>Tomer </div></div></div>