<div dir="ltr"><div>Hi all:</div><div><br></div><div>Thank you for your comments and they will be addressed.</div><div><br></div><div>Regards,</div><div>Tyrone<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Rob Herring <<a href="mailto:robh@kernel.org">robh@kernel.org</a>> 於 2022年2月7日 週一 下午10:22寫道：<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, 07 Feb 2022 14:33:33 +0800, Tyrone Ting wrote:<br>
> From: Tyrone Ting <<a href="mailto:kfting@nuvoton.com" target="_blank">kfting@nuvoton.com</a>><br>
> <br>
> This commit adds compatible and syscon description for NPCM845 i2c module.<br>
> <br>
> Fixes: 56a1485b102e ("i2c: npcm7xx: Add Nuvoton NPCM I2C controller driver")<br>
> Signed-off-by: Tyrone Ting <<a href="mailto:kfting@nuvoton.com" target="_blank">kfting@nuvoton.com</a>><br>
> Signed-off-by: Tali Perry <<a href="mailto:tali.perry1@gmail.com" target="_blank">tali.perry1@gmail.com</a>><br>
> ---<br>
> .../bindings/i2c/nuvoton,npcm7xx-i2c.yaml | 21 ++++++++++++++-----<br>
> 1 file changed, 16 insertions(+), 5 deletions(-)<br>
> <br>
<br>
Running 'make dtbs_check' with the schema in this patch gives the<br>
following warnings. Consider if they are expected or the schema is<br>
incorrect. These may not be new warnings.<br>
<br>
Note that it is not yet a requirement to have 0 warnings for dtbs_check.<br>
This will change in the future.<br>
<br>
Full log is available here: <a href="https://patchwork.ozlabs.org/patch/1589171" rel="noreferrer" target="_blank">https://patchwork.ozlabs.org/patch/1589171</a><br>
<br>
<br>
i2c@80000: 'syscon' is a required property<br>
arch/arm/boot/dts/nuvoton-npcm730-gbs.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm750-evb.dt.yaml<br>
<br>
i2c@81000: 'syscon' is a required property<br>
arch/arm/boot/dts/nuvoton-npcm730-gbs.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm730-gsj.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm750-evb.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dt.yaml<br>
<br>
i2c@82000: 'syscon' is a required property<br>
arch/arm/boot/dts/nuvoton-npcm730-gbs.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm730-gsj.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm750-evb.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dt.yaml<br>
<br>
i2c@83000: 'syscon' is a required property<br>
arch/arm/boot/dts/nuvoton-npcm730-gsj.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm750-evb.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dt.yaml<br>
<br>
i2c@84000: 'syscon' is a required property<br>
arch/arm/boot/dts/nuvoton-npcm730-gsj.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dt.yaml<br>
<br>
i2c@85000: 'syscon' is a required property<br>
arch/arm/boot/dts/nuvoton-npcm730-gbs.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm750-evb.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dt.yaml<br>
<br>
i2c@86000: 'syscon' is a required property<br>
arch/arm/boot/dts/nuvoton-npcm730-gbs.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm750-evb.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dt.yaml<br>
<br>
i2c@87000: 'syscon' is a required property<br>
arch/arm/boot/dts/nuvoton-npcm730-gbs.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm750-evb.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dt.yaml<br>
<br>
i2c@88000: 'syscon' is a required property<br>
arch/arm/boot/dts/nuvoton-npcm730-gbs.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm730-gsj.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm750-evb.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dt.yaml<br>
<br>
i2c@89000: 'syscon' is a required property<br>
arch/arm/boot/dts/nuvoton-npcm730-gbs.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm730-gsj.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm750-evb.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dt.yaml<br>
<br>
i2c@8a000: 'syscon' is a required property<br>
arch/arm/boot/dts/nuvoton-npcm730-gbs.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm730-gsj.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm750-evb.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dt.yaml<br>
<br>
i2c@8b000: 'syscon' is a required property<br>
arch/arm/boot/dts/nuvoton-npcm730-gbs.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm730-gsj.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm750-evb.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dt.yaml<br>
<br>
i2c@8c000: 'syscon' is a required property<br>
arch/arm/boot/dts/nuvoton-npcm730-gbs.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm730-gsj.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dt.yaml<br>
<br>
i2c@8d000: 'syscon' is a required property<br>
arch/arm/boot/dts/nuvoton-npcm730-gbs.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dt.yaml<br>
<br>
i2c@8e000: 'syscon' is a required property<br>
arch/arm/boot/dts/nuvoton-npcm730-gbs.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm750-evb.dt.yaml<br>
<br>
i2c@8f000: 'syscon' is a required property<br>
arch/arm/boot/dts/nuvoton-npcm730-gsj.dt.yaml<br>
arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml<br>
<br>
</blockquote></div>