Hi all:

Thank you for your comments and they will be addressed.

Regards,
Tyrone

Rob Herring <robh@kernel.org> 於 2022年2月7日 週一 下午10:22寫道:
On Mon, 07 Feb 2022 14:33:33 +0800, Tyrone Ting wrote:
> From: Tyrone Ting <kfting@nuvoton.com>
>
> This commit adds compatible and syscon description for NPCM845 i2c module.
>
> Fixes: 56a1485b102e ("i2c: npcm7xx: Add Nuvoton NPCM I2C controller driver")
> Signed-off-by: Tyrone Ting <kfting@nuvoton.com>
> Signed-off-by: Tali Perry <tali.perry1@gmail.com>
> ---
>  .../bindings/i2c/nuvoton,npcm7xx-i2c.yaml     | 21 ++++++++++++++-----
>  1 file changed, 16 insertions(+), 5 deletions(-)
>

Running 'make dtbs_check' with the schema in this patch gives the
following warnings. Consider if they are expected or the schema is
incorrect. These may not be new warnings.

Note that it is not yet a requirement to have 0 warnings for dtbs_check.
This will change in the future.

Full log is available here: https://patchwork.ozlabs.org/patch/1589171


i2c@80000: 'syscon' is a required property
        arch/arm/boot/dts/nuvoton-npcm730-gbs.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm750-evb.dt.yaml

i2c@81000: 'syscon' is a required property
        arch/arm/boot/dts/nuvoton-npcm730-gbs.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm730-gsj.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm750-evb.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dt.yaml

i2c@82000: 'syscon' is a required property
        arch/arm/boot/dts/nuvoton-npcm730-gbs.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm730-gsj.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm750-evb.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dt.yaml

i2c@83000: 'syscon' is a required property
        arch/arm/boot/dts/nuvoton-npcm730-gsj.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm750-evb.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dt.yaml

i2c@84000: 'syscon' is a required property
        arch/arm/boot/dts/nuvoton-npcm730-gsj.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dt.yaml

i2c@85000: 'syscon' is a required property
        arch/arm/boot/dts/nuvoton-npcm730-gbs.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm750-evb.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dt.yaml

i2c@86000: 'syscon' is a required property
        arch/arm/boot/dts/nuvoton-npcm730-gbs.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm750-evb.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dt.yaml

i2c@87000: 'syscon' is a required property
        arch/arm/boot/dts/nuvoton-npcm730-gbs.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm750-evb.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dt.yaml

i2c@88000: 'syscon' is a required property
        arch/arm/boot/dts/nuvoton-npcm730-gbs.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm730-gsj.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm750-evb.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dt.yaml

i2c@89000: 'syscon' is a required property
        arch/arm/boot/dts/nuvoton-npcm730-gbs.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm730-gsj.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm750-evb.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dt.yaml

i2c@8a000: 'syscon' is a required property
        arch/arm/boot/dts/nuvoton-npcm730-gbs.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm730-gsj.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm750-evb.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dt.yaml

i2c@8b000: 'syscon' is a required property
        arch/arm/boot/dts/nuvoton-npcm730-gbs.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm730-gsj.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm750-evb.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dt.yaml

i2c@8c000: 'syscon' is a required property
        arch/arm/boot/dts/nuvoton-npcm730-gbs.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm730-gsj.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dt.yaml

i2c@8d000: 'syscon' is a required property
        arch/arm/boot/dts/nuvoton-npcm730-gbs.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm750-runbmc-olympus.dt.yaml

i2c@8e000: 'syscon' is a required property
        arch/arm/boot/dts/nuvoton-npcm730-gbs.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm750-evb.dt.yaml

i2c@8f000: 'syscon' is a required property
        arch/arm/boot/dts/nuvoton-npcm730-gsj.dt.yaml
        arch/arm/boot/dts/nuvoton-npcm730-kudo.dt.yaml