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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID:
 <TYZPR04MB58539D6253FD5A8579FF154FD66C2@TYZPR04MB5853.apcprd04.prod.outlook.com>
Date: Fri, 20 Sep 2024 06:38:18 +0000
From: Delphine_CC_Chiu/WYHQ/Wiwynn <Delphine_CC_Chiu@...ynn.com>
To: Andrew Jeffery <andrew@...econstruct.com.au>, Delphine_CC_Chiu/WYHQ/Wiwynn
	<Delphine_CC_Chiu@...ynn.com>, Ricky CX Wu <ricky.cx.wu.wiwynn@...il.com>
CC: "Rob Herring (Arm)" <robh@...nel.org>, "devicetree@...r.kernel.org"
	<devicetree@...r.kernel.org>, Joel Stanley <joel@....id.au>, Conor Dooley
	<conor+dt@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
	"linux-aspeed@...ts.ozlabs.org" <linux-aspeed@...ts.ozlabs.org>,
	"linux-arm-kernel@...ts.infradead.org"
	<linux-arm-kernel@...ts.infradead.org>, "patrick@...cx.xyz"
	<patrick@...cx.xyz>, "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>
Subject: RE: [PATCH v18 0/3] Add i2c-mux and eeprom devices for Meta Yosemite4



> -----Original Message-----
> From: Andrew Jeffery <andrew@...econstruct.com.au>
> Sent: Thursday, September 12, 2024 10:34 AM
> To: Delphine_CC_Chiu/WYHQ/Wiwynn <Delphine_CC_Chiu@...ynn.com>;
> Ricky CX Wu <ricky.cx.wu.wiwynn@...il.com>
> Cc: Rob Herring (Arm) <robh@...nel.org>; devicetree@...r.kernel.org; Joel
> Stanley <joel@....id.au>; Conor Dooley <conor+dt@...nel.org>; Krzysztof
> Kozlowski <krzk+dt@...nel.org>; linux-aspeed@...ts.ozlabs.org;
> linux-arm-kernel@...ts.infradead.org; patrick@...cx.xyz;
> linux-kernel@...r.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@...il.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@...ynn.com:
> >
> > 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@...8a000/i2c@.../i2c-mux@...i2c@...pio@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@...8a000/i2c@.../i2c-mux@...i2c@...pio@61: failed to
> > match any schema with compatible: ['nxp,pca9552']
> > arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dtb:
> >
> /ahb/apb/bus@...8a000/i2c@.../i2c-mux@...i2c@...emperature-sensor@
> 3c:
> > failed to match any schema with compatible: ['smsc,emc1403']
> > arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dtb:
> >
> /ahb/apb/bus@...8a000/i2c@.../i2c-mux@...i2c@...emperature-sensor@
> 3c:
> > failed to match any schema with compatible: ['smsc,emc1403']
> > arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dtb:
> >
> /ahb/apb/bus@...8a000/i2c@.../i2c-mux@...i2c@...emperature-sensor@
> 3c:
> > failed to match any schema with compatible: ['smsc,emc1403']
> > arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dtb:
> >
> /ahb/apb/bus@...8a000/i2c@.../i2c-mux@...i2c@...emperature-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.