<div dir="ltr"><div dir="ltr">Hi Daniel,<div><br></div><div>Sorry but I have probably miss it, thanks a lot for your comment</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 10 Sep 2019 at 13:25, Daniel Thompson <<a href="mailto:daniel.thompson@linaro.org">daniel.thompson@linaro.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 Mon, Sep 09, 2019 at 03:38:39PM +0300, Tomer Maimon wrote:<br>
> Added device tree binding documentation for Nuvoton BMC<br>
> NPCM Random Number Generator (RNG).<br>
> <br>
> Signed-off-by: Tomer Maimon <<a href="mailto:tmaimon77@gmail.com" target="_blank">tmaimon77@gmail.com</a>><br>
> ---<br>
>  .../bindings/rng/nuvoton,npcm-rng.txt           | 17 +++++++++++++++++<br>
>  1 file changed, 17 insertions(+)<br>
>  create mode 100644 Documentation/devicetree/bindings/rng/nuvoton,npcm-rng.txt<br>
> <br>
> diff --git a/Documentation/devicetree/bindings/rng/nuvoton,npcm-rng.txt b/Documentation/devicetree/bindings/rng/nuvoton,npcm-rng.txt<br>
> new file mode 100644<br>
> index 000000000000..a697b4425fb3<br>
> --- /dev/null<br>
> +++ b/Documentation/devicetree/bindings/rng/nuvoton,npcm-rng.txt<br>
> @@ -0,0 +1,17 @@<br>
> +NPCM SoC Random Number Generator<br>
> +<br>
> +Required properties:<br>
> +- compatible  : "nuvoton,npcm750-rng" for the NPCM7XX BMC.<br>
> +- reg         : Specifies physical base address and size of the registers.<br>
> +<br>
> +Optional property:<br>
> +- quality : estimated number of bits of true entropy per 1024 bits<br>
> +                     read from the rng.<br>
> +                     If this property is not defined, it defaults to 1000.<br>
<br>
There are pending unreplied review comments about this property (my own<br>
as it happens):<br>
<a href="https://patchwork.kernel.org/patch/11119371/" rel="noreferrer" target="_blank">https://patchwork.kernel.org/patch/11119371/</a><br>
<br></blockquote><div>No, there isn't different SoCs.</div><div>we had checked the quality of the hwrng and the results we got are set as default.</div><div>we been asked from one of our client to have a dynamic quality, they will like to be more strict when using the hwrng.</div><div>is it problematic to add it?</div><pre class="gmail-content" style="box-sizing:border-box;overflow:auto;font-family:Menlo,Monaco,Consolas,"Courier New",monospace;font-size:13px;padding:9.5px;margin-top:0px;margin-bottom:10px;line-height:14.3px;color:rgb(51,51,51);word-break:break-all;border:0px;border-radius:0px">Having a controllable quality implies that the numeric quality of the
peripheral changes when it is stamped out on different SoCs (otherwise
the driver can confidently set the quality without needing any hint
from the DT). Is that really true here? </pre><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Daniel.<br>
<br>
> +<br>
> +Example:<br>
> +<br>
> +rng: rng@f000b000 {<br>
> +     compatible = "nuvoton,npcm750-rng";<br>
> +     reg = <0xf000b000 0x8>;<br>
> +};<br>
> -- <br>
> 2.18.0<br>
> <br>
</blockquote></div></div>