> -----Original Message----- > From: Andrew Jeffery <andrew@xxxxxxxxxxxxxxxxxxxx> > Sent: Thursday, September 12, 2024 10:34 AM > To: Delphine_CC_Chiu/WYHQ/Wiwynn <Delphine_CC_Chiu@xxxxxxxxxx>; > Ricky CX Wu <ricky.cx.wu.wiwynn@xxxxxxxxx> > Cc: Rob Herring (Arm) <robh@xxxxxxxxxx>; devicetree@xxxxxxxxxxxxxxx; Joel > Stanley <joel@xxxxxxxxx>; Conor Dooley <conor+dt@xxxxxxxxxx>; Krzysztof > Kozlowski <krzk+dt@xxxxxxxxxx>; linux-aspeed@xxxxxxxxxxxxxxxx; > linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; patrick@xxxxxxxxx; > linux-kernel@xxxxxxxxxxxxxxx > 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@xxxxxxxxx> > > > > ... > > > > > > 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@xxxxxxxxxx: > > > > arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dtb: pwm@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@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@1e78a000/i2c@780/i2c-mux@74/i2c@0/gpio@61: failed to > > match any schema with compatible: ['nxp,pca9552'] > > arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dtb: pwm@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@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@1e78a000/i2c@780/i2c-mux@74/i2c@1/gpio@61: failed to > > match any schema with compatible: ['nxp,pca9552'] > > arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dtb: > > > /ahb/apb/bus@1e78a000/i2c@800/i2c-mux@72/i2c@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@1e78a000/i2c@800/i2c-mux@72/i2c@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@1e78a000/i2c@800/i2c-mux@72/i2c@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@1e78a000/i2c@800/i2c-mux@72/i2c@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.
