[PATCH v18 0/3] Add i2c-mux and eeprom devices for Meta Yosemite4

Fri Sep 20 16:38:18 AEST 2024

> -----Original Message-----
> From: Andrew Jeffery <andrew at codeconstruct.com.au>
> Sent: Thursday, September 12, 2024 10:34 AM
> To: Delphine_CC_Chiu/WYHQ/Wiwynn <Delphine_CC_Chiu at wiwynn.com>;
> Ricky CX Wu <ricky.cx.wu.wiwynn at gmail.com>
> Cc: Rob Herring (Arm) <robh at kernel.org>; devicetree at vger.kernel.org; Joel
> Stanley <joel at jms.id.au>; Conor Dooley <conor+dt at kernel.org>; Krzysztof
> Kozlowski <krzk+dt at kernel.org>; linux-aspeed at lists.ozlabs.org;
> linux-arm-kernel at lists.infradead.org; patrick at stwcx.xyz;
> linux-kernel at vger.kernel.org
> Subject: Re: [PATCH v18 0/3] Add i2c-mux and eeprom devices for Meta
> Yosemite4
> 
>  [External Sender]
> 
>  [External Sender]
> 
> Hi Ricky,
> 
> On Mon, 2024-09-09 at 10:45 -0500, Rob Herring (Arm) wrote:
> > On Mon, 09 Sep 2024 18:54:15 +0800, Delphine CC Chiu wrote:
> > > From: Ricky CX Wu <ricky.cx.wu.wiwynn at gmail.com>
> > >
> ...
> > >
> > > Ricky CX Wu (3):
> > >   ARM: dts: aspeed: yosemite4: Revise i2c-mux devices
> > >   ARM: dts: aspeed: yosemite4: add mctp config and sensors for NIC
> > >   ARM: dts: aspeed: yosemite4: add fan led config
> > >
> > >  .../aspeed/aspeed-bmc-facebook-yosemite4.dts  | 706
> > > ++++++++++++++++--
> > >  1 file changed, 660 insertions(+), 46 deletions(-)
> > >
> > > --
> > > 2.25.1
> >
> >
> > My bot found new DTB warnings on the .dts files added or changed in
> > this series.
> >
> > Some warnings may be from an existing SoC .dtsi. Or perhaps the
> > warnings are fixed by another series. Ultimately, it is up to the
> > platform maintainer whether these warnings are acceptable or not. No
> > need to reply unless the platform maintainer has comments.
> >
> > If you already ran DT checks and didn't see these error(s), then make
> > sure dt-schema is up to date:
> >
> >   pip3 install dtschema --upgrade
> >
> >
> > New warnings running 'make CHECK_DTBS=y
> aspeed/aspeed-bmc-facebook-yosemite4.dtb' for
> 20240909105420.441607-1-Delphine_CC_Chiu at wiwynn.com:
> >
> > arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dtb: pwm at 20:
> '#address-cells', '#size-cells' do not match any of the regexes: '^fan-[0-9]+$',
> 'pinctrl-[0-9]+'
> >       from schema $id:
> > https://urldefense.com/v3/__http://devicetree.org/schemas/hwmon/maxim,
> >
> max31790.yaml__;!!J63qqgXj!KOrUCIgHJyjBrccMiU8ZIIKD844Aq_AqQeUZF9qer
> Ys
> > j9rqtQAcRSSRFaVck8tuE0xWFPVt0pTCb8ZptOGMzhLQCCiY$
> > arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dtb: pwm at 2f:
> '#address-cells', '#size-cells' do not match any of the regexes: '^fan-[0-9]+$',
> 'pinctrl-[0-9]+'
> >       from schema $id:
> > https://urldefense.com/v3/__http://devicetree.org/schemas/hwmon/maxim,
> >
> max31790.yaml__;!!J63qqgXj!KOrUCIgHJyjBrccMiU8ZIIKD844Aq_AqQeUZF9qer
> Ys
> > j9rqtQAcRSSRFaVck8tuE0xWFPVt0pTCb8ZptOGMzhLQCCiY$
> > arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dtb:
> > /ahb/apb/bus at 1e78a000/i2c at 780/i2c-mux at 74/i2c at 0/gpio at 61: failed to
> > match any schema with compatible: ['nxp,pca9552']
> > arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dtb: pwm at 20:
> '#address-cells', '#size-cells' do not match any of the regexes: '^fan-[0-9]+$',
> 'pinctrl-[0-9]+'
> >       from schema $id:
> > https://urldefense.com/v3/__http://devicetree.org/schemas/hwmon/maxim,
> >
> max31790.yaml__;!!J63qqgXj!KOrUCIgHJyjBrccMiU8ZIIKD844Aq_AqQeUZF9qer
> Ys
> > j9rqtQAcRSSRFaVck8tuE0xWFPVt0pTCb8ZptOGMzhLQCCiY$
> > arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dtb: pwm at 2f:
> '#address-cells', '#size-cells' do not match any of the regexes: '^fan-[0-9]+$',
> 'pinctrl-[0-9]+'
> >       from schema $id:
> > https://urldefense.com/v3/__http://devicetree.org/schemas/hwmon/maxim,
> >
> max31790.yaml__;!!J63qqgXj!KOrUCIgHJyjBrccMiU8ZIIKD844Aq_AqQeUZF9qer
> Ys
> > j9rqtQAcRSSRFaVck8tuE0xWFPVt0pTCb8ZptOGMzhLQCCiY$
> > arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dtb:
> > /ahb/apb/bus at 1e78a000/i2c at 780/i2c-mux at 74/i2c at 1/gpio at 61: failed to
> > match any schema with compatible: ['nxp,pca9552']
> > arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dtb:
> >
> /ahb/apb/bus at 1e78a000/i2c at 800/i2c-mux at 72/i2c at 0/temperature-sensor@
> 3c:
> > failed to match any schema with compatible: ['smsc,emc1403']
> > arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dtb:
> >
> /ahb/apb/bus at 1e78a000/i2c at 800/i2c-mux at 72/i2c at 1/temperature-sensor@
> 3c:
> > failed to match any schema with compatible: ['smsc,emc1403']
> > arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dtb:
> >
> /ahb/apb/bus at 1e78a000/i2c at 800/i2c-mux at 72/i2c at 2/temperature-sensor@
> 3c:
> > failed to match any schema with compatible: ['smsc,emc1403']
> > arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dtb:
> >
> /ahb/apb/bus at 1e78a000/i2c at 800/i2c-mux at 72/i2c at 3/temperature-sensor@
> 3c:
> > failed to match any schema with compatible: ['smsc,emc1403']
> >
> 
> Please either:
> 
> 1. Remove the offending nodes from your series so we don't add yet more
> warnings, or 2. Write appropriate bindings and send them as part of the series
> 
> Regarding 1, you can always update the devicetree to add the nodes after
> someone else has got the bindings merged. However, if you must have the
> nodes, then you need to do 2.
> 
> Thanks,
> 
> Andrew

Hi Andrew,

I'll remove the offending nodes first and update bindings in other patches.
Once the bindings are applied, I will update the nodes again.

Thanks.