| 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
| ||
|
Message-ID:
<TYZPR04MB58537972801CB54C46AF0C27D6772@TYZPR04MB5853.apcprd04.prod.outlook.com>
Date: Tue, 1 Oct 2024 07:45:11 +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>, "patrick@...cx.xyz" <patrick@...cx.xyz>, Rob
Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor
Dooley <conor+dt@...nel.org>, Joel Stanley <joel@....id.au>
CC: Ricky CX Wu <ricky.cx.wu.wiwynn@...il.com>, "devicetree@...r.kernel.org"
<devicetree@...r.kernel.org>, "linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>, "linux-aspeed@...ts.ozlabs.org"
<linux-aspeed@...ts.ozlabs.org>, "linux-kernel@...r.kernel.org"
<linux-kernel@...r.kernel.org>
Subject: RE: [PATCH v1 4/8] ARM: dts: aspeed: yosemite4: Revise address of
i2c-mux for two fan boards
> -----Original Message-----
> From: Andrew Jeffery <andrew@...econstruct.com.au>
> Sent: Tuesday, October 1, 2024 8:39 AM
> To: Delphine_CC_Chiu/WYHQ/Wiwynn <Delphine_CC_Chiu@...ynn.com>;
> patrick@...cx.xyz; Rob Herring <robh@...nel.org>; Krzysztof Kozlowski
> <krzk+dt@...nel.org>; Conor Dooley <conor+dt@...nel.org>; Joel Stanley
> <joel@....id.au>
> Cc: Ricky CX Wu <ricky.cx.wu.wiwynn@...il.com>;
> devicetree@...r.kernel.org; linux-arm-kernel@...ts.infradead.org;
> linux-aspeed@...ts.ozlabs.org; linux-kernel@...r.kernel.org
> Subject: Re: [PATCH v1 4/8] ARM: dts: aspeed: yosemite4: Revise address of
> i2c-mux for two fan boards
>
> [External Sender]
>
> [External Sender]
>
> On Mon, 2024-09-30 at 15:04 +0800, Delphine CC Chiu wrote:
> > From: Ricky CX Wu <ricky.cx.wu.wiwynn@...il.com>
> >
> > Change the address of the I2C mux for two fan boards to 0x74 according
> > to schematic.
> >
> > Signed-off-by: Ricky CX Wu <ricky.cx.wu.wiwynn@...il.com>
> > Signed-off-by: Delphine CC Chiu <Delphine_CC_Chiu@...ynn.com>
> > ---
> > .../aspeed/aspeed-bmc-facebook-yosemite4.dts | 80
> > ++++++++++---------
> > 1 file changed, 43 insertions(+), 37 deletions(-)
> >
> > diff --git
> > a/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dts
> > b/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dts
> > index 26e0fcbed8d2..ecf012c9cbbc 100644
> > --- a/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dts
> > +++ b/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dts
> > @@ -32,6 +32,8 @@ aliases {
> > i2c27 = &imux27;
> > i2c28 = &imux28;
> > i2c29 = &imux29;
> > + i2c30 = &imux30;
> > + i2c31 = &imux31;
> > i2c34 = &imux34;
> > i2c35 = &imux35;
> > };
> > @@ -1062,6 +1064,8 @@ gpio@23 {
> > };
> >
> > &i2c14 {
> > + #address-cells = <1>;
> > + #size-cells = <0>;
> > status = "okay";
> > bus-frequency = <400000>;
> > adc@1d {
> > @@ -1122,20 +1126,50 @@ eeprom@51 {
> > reg = <0x51>;
> > };
> >
> > - i2c-mux@71 {
> > - compatible = "nxp,pca9846";
> > + i2c-mux@73 {
> > + compatible = "nxp,pca9544";
> > #address-cells = <1>;
> > #size-cells = <0>;
> >
> > idle-state = <0>;
> > i2c-mux-idle-disconnect;
> > - reg = <0x71>;
> > + reg = <0x73>;
> >
> > i2c@0 {
> > #address-cells = <1>;
> > #size-cells = <0>;
> > reg = <0>;
> >
> > + adc@35 {
> > + compatible = "maxim,max11617";
> > + reg = <0x35>;
> > + };
> > + };
> > +
> > + i2c@1 {
> > + #address-cells = <1>;
> > + #size-cells = <0>;
> > + reg = <0>;
> > +
> > + adc@35 {
> > + compatible = "maxim,max11617";
> > + reg = <0x35>;
> > + };
> > + };
> > + };
> > +
> > + i2c-mux@74 {
> > + compatible = "nxp,pca9546";
> > + reg = <0x74>;
> > + #address-cells = <1>;
> > + #size-cells = <0>;
> > + i2c-mux-idle-disconnect;
> > +
> > + imux30: i2c@0 {
> > + reg = <0>;
> > + #address-cells = <1>;
> > + #size-cells = <0>;
> > +
> > adc@1f {
> > compatible = "ti,adc128d818";
> > reg = <0x1f>; @@ -1152,6 +1186,8
> @@
> > pwm@20{
> > gpio@22{
> > compatible = "ti,tca6424";
> > reg = <0x22>;
> > + gpio-controller;
> > + #gpio-cells = <2>;
>
> This seems unrelated. Such changes should be in their own patch with a `Fixes:`
> tag.
>
> > };
> >
> > pwm@23{
> > @@ -1181,10 +1217,10 @@ gpio@61 {
> > };
> > };
> >
> > - i2c@1 {
> > + imux31: i2c@1 {
> > + reg = <1>;
> > #address-cells = <1>;
> > #size-cells = <0>;
> > - reg = <0>;
> >
> > adc@1f {
> > compatible = "ti,adc128d818"; @@
> -1202,6
> > +1238,8 @@ pwm@20{
> > gpio@22{
> > compatible = "ti,tca6424";
> > reg = <0x22>;
> > + gpio-controller;
> > + #gpio-cells = <2>;
>
> This should go in the same patch as the instance above.
>
Hi Andrew,
I'll split a fix patch for this in the next serial patches. Thanks.
Powered by blists - more mailing lists