<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 20 Jan 2021 at 00:47, 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 Tue, 19 Jan 2021 at 22:34, Tomer Maimon <<a href="mailto:tmaimon77@gmail.com" target="_blank">tmaimon77@gmail.com</a>> wrote:<br>
><br>
> Enable PECI dimm temperature nodes in Olympus<br>
> Quanta machine.<br>
><br>
> Signed-off-by: Tomer Maimon <<a href="mailto:tmaimon77@gmail.com" target="_blank">tmaimon77@gmail.com</a>><br>
> ---<br>
>  arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dts | 2 ++<br>
>  1 file changed, 2 insertions(+)<br>
><br>
> diff --git a/arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dts b/arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dts<br>
> index 1692bb7314c5..de34c9b2ff2c 100644<br>
> --- a/arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dts<br>
> +++ b/arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dts<br>
> @@ -910,10 +910,12 @@<br>
>         intel-peci-dimmtemp@30 {<br>
>                 compatible = "intel,peci-client";<br>
>                 reg = <0x30>;<br>
> +               status = "okay";<br>
<br>
Nodes are enabled by default, you shouldn't need to add "okay" for it<br>
to work. Can you confirm it works without adding this?<br>
<br></blockquote><div>Yes you right.</div><div><br></div><div>is working fine without "okay"</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
>         };<br>
>         intel-peci-dimmtemp@31 {<br>
>                 compatible = "intel,peci-client";<br>
>                 reg = <0x31>;<br>
> +               status = "okay";<br>
>         };<br>
>  };<br>
><br>
> --<br>
> 2.22.0<br>
><br></blockquote><div><br></div><div>Cheers,</div><div><br></div><div>Tomer </div></div></div>