* [PATCH v1 0/1] ARM: dts: aspeed: system1: IBM System1 BMC update @ 2024-07-31 21:47 ` Ninad Palsule 0 siblings, 0 replies; 18+ messages in thread From: Ninad Palsule @ 2024-07-31 21:47 UTC (permalink / raw) To: linux-aspeed Hello, This patch only applies to openbmc/linux. The max31785 pmbus driver changes are not available in the upstream yet. I will try to send those changes upstream. Ninad Palsule (1): ARM: dts: aspeed: system1: IBM System1 BMC update .../dts/aspeed/aspeed-bmc-ibm-system1.dts | 96 +++++++++++++++++++ 1 file changed, 96 insertions(+) -- 2.43.0 ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH v1 0/1] ARM: dts: aspeed: system1: IBM System1 BMC update @ 2024-07-31 21:47 ` Ninad Palsule 0 siblings, 0 replies; 18+ messages in thread From: Ninad Palsule @ 2024-07-31 21:47 UTC (permalink / raw) To: robh+dt, krzysztof.kozlowski+dt, conor+dt, joel, andrew, devicetree, linux-arm-kernel, linux-aspeed, linux-kernel, eajames Cc: Ninad Palsule Hello, This patch only applies to openbmc/linux. The max31785 pmbus driver changes are not available in the upstream yet. I will try to send those changes upstream. Ninad Palsule (1): ARM: dts: aspeed: system1: IBM System1 BMC update .../dts/aspeed/aspeed-bmc-ibm-system1.dts | 96 +++++++++++++++++++ 1 file changed, 96 insertions(+) -- 2.43.0 ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH v1 1/1] ARM: dts: aspeed: system1: IBM System1 BMC update 2024-07-31 21:47 ` Ninad Palsule @ 2024-07-31 21:47 ` Ninad Palsule -1 siblings, 0 replies; 18+ messages in thread From: Ninad Palsule @ 2024-07-31 21:47 UTC (permalink / raw) To: linux-aspeed Updated MAX31785a pmbus based fans in the device tree. Signed-off-by: Ninad Palsule <ninad@linux.ibm.com> --- .../dts/aspeed/aspeed-bmc-ibm-system1.dts | 96 +++++++++++++++++++ 1 file changed, 96 insertions(+) diff --git a/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts b/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts index f3efecc7eb8d0..f96b299d743ba 100644 --- a/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts +++ b/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts @@ -764,17 +764,113 @@ regulator at 43 { }; }; + &i2c6 { status = "okay"; fan-controller at 52 { compatible = "maxim,max31785a"; reg = <0x52>; + #address-cells = <1>; + #size-cells = <0>; + + fan0: fan at 0 { + compatible = "pmbus-fan"; + reg = <0>; + tach-pulses = <2>; + maxim,fan-rotor-input = "tach"; + maxim,fan-pwm-freq = <25000>; + maxim,fan-dual-tach; + maxim,fan-no-watchdog; + maxim,fan-no-fault-ramp; + maxim,fan-ramp = <2>; + maxim,fan-fault-pin-mon; + }; + + fan1: fan at 1 { + compatible = "pmbus-fan"; + reg = <1>; + tach-pulses = <2>; + maxim,fan-rotor-input = "tach"; + maxim,fan-pwm-freq = <25000>; + maxim,fan-dual-tach; + maxim,fan-no-watchdog; + maxim,fan-no-fault-ramp; + maxim,fan-ramp = <2>; + maxim,fan-fault-pin-mon; + }; + + fan2: fan at 2 { + compatible = "pmbus-fan"; + reg = <2>; + tach-pulses = <2>; + maxim,fan-rotor-input = "tach"; + maxim,fan-pwm-freq = <25000>; + maxim,fan-dual-tach; + maxim,fan-no-watchdog; + maxim,fan-no-fault-ramp; + maxim,fan-ramp = <2>; + maxim,fan-fault-pin-mon; + }; + + fan3: fan at 3 { + compatible = "pmbus-fan"; + reg = <3>; + tach-pulses = <2>; + maxim,fan-rotor-input = "tach"; + maxim,fan-pwm-freq = <25000>; + maxim,fan-dual-tach; + maxim,fan-no-watchdog; + maxim,fan-no-fault-ramp; + maxim,fan-ramp = <2>; + maxim,fan-fault-pin-mon; + }; + + fan4: fan at 4 { + compatible = "pmbus-fan"; + reg = <4>; + tach-pulses = <2>; + maxim,fan-rotor-input = "tach"; + maxim,fan-pwm-freq = <25000>; + maxim,fan-dual-tach; + maxim,fan-no-watchdog; + maxim,fan-no-fault-ramp; + maxim,fan-ramp = <2>; + maxim,fan-fault-pin-mon; + }; }; fan-controller at 54 { compatible = "maxim,max31785a"; reg = <0x54>; + #address-cells = <1>; + #size-cells = <0>; + + fan5: fan at 0 { + compatible = "pmbus-fan"; + reg = <0>; + tach-pulses = <2>; + maxim,fan-rotor-input = "tach"; + maxim,fan-pwm-freq = <25000>; + maxim,fan-dual-tach; + maxim,fan-no-watchdog; + maxim,fan-no-fault-ramp; + maxim,fan-ramp = <2>; + maxim,fan-fault-pin-mon; + }; + + fan6: fan at 1 { + compatible = "pmbus-fan"; + reg = <1>; + tach-pulses = <2>; + maxim,fan-rotor-input = "tach"; + maxim,fan-pwm-freq = <25000>; + maxim,fan-dual-tach; + maxim,fan-no-watchdog; + maxim,fan-no-fault-ramp; + maxim,fan-ramp = <2>; + maxim,fan-fault-pin-mon; + }; }; eeprom at 55 { -- 2.43.0 ^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH v1 1/1] ARM: dts: aspeed: system1: IBM System1 BMC update @ 2024-07-31 21:47 ` Ninad Palsule 0 siblings, 0 replies; 18+ messages in thread From: Ninad Palsule @ 2024-07-31 21:47 UTC (permalink / raw) To: robh+dt, krzysztof.kozlowski+dt, conor+dt, joel, andrew, devicetree, linux-arm-kernel, linux-aspeed, linux-kernel, eajames Cc: Ninad Palsule Updated MAX31785a pmbus based fans in the device tree. Signed-off-by: Ninad Palsule <ninad@linux.ibm.com> --- .../dts/aspeed/aspeed-bmc-ibm-system1.dts | 96 +++++++++++++++++++ 1 file changed, 96 insertions(+) diff --git a/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts b/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts index f3efecc7eb8d0..f96b299d743ba 100644 --- a/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts +++ b/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts @@ -764,17 +764,113 @@ regulator@43 { }; }; + &i2c6 { status = "okay"; fan-controller@52 { compatible = "maxim,max31785a"; reg = <0x52>; + #address-cells = <1>; + #size-cells = <0>; + + fan0: fan@0 { + compatible = "pmbus-fan"; + reg = <0>; + tach-pulses = <2>; + maxim,fan-rotor-input = "tach"; + maxim,fan-pwm-freq = <25000>; + maxim,fan-dual-tach; + maxim,fan-no-watchdog; + maxim,fan-no-fault-ramp; + maxim,fan-ramp = <2>; + maxim,fan-fault-pin-mon; + }; + + fan1: fan@1 { + compatible = "pmbus-fan"; + reg = <1>; + tach-pulses = <2>; + maxim,fan-rotor-input = "tach"; + maxim,fan-pwm-freq = <25000>; + maxim,fan-dual-tach; + maxim,fan-no-watchdog; + maxim,fan-no-fault-ramp; + maxim,fan-ramp = <2>; + maxim,fan-fault-pin-mon; + }; + + fan2: fan@2 { + compatible = "pmbus-fan"; + reg = <2>; + tach-pulses = <2>; + maxim,fan-rotor-input = "tach"; + maxim,fan-pwm-freq = <25000>; + maxim,fan-dual-tach; + maxim,fan-no-watchdog; + maxim,fan-no-fault-ramp; + maxim,fan-ramp = <2>; + maxim,fan-fault-pin-mon; + }; + + fan3: fan@3 { + compatible = "pmbus-fan"; + reg = <3>; + tach-pulses = <2>; + maxim,fan-rotor-input = "tach"; + maxim,fan-pwm-freq = <25000>; + maxim,fan-dual-tach; + maxim,fan-no-watchdog; + maxim,fan-no-fault-ramp; + maxim,fan-ramp = <2>; + maxim,fan-fault-pin-mon; + }; + + fan4: fan@4 { + compatible = "pmbus-fan"; + reg = <4>; + tach-pulses = <2>; + maxim,fan-rotor-input = "tach"; + maxim,fan-pwm-freq = <25000>; + maxim,fan-dual-tach; + maxim,fan-no-watchdog; + maxim,fan-no-fault-ramp; + maxim,fan-ramp = <2>; + maxim,fan-fault-pin-mon; + }; }; fan-controller@54 { compatible = "maxim,max31785a"; reg = <0x54>; + #address-cells = <1>; + #size-cells = <0>; + + fan5: fan@0 { + compatible = "pmbus-fan"; + reg = <0>; + tach-pulses = <2>; + maxim,fan-rotor-input = "tach"; + maxim,fan-pwm-freq = <25000>; + maxim,fan-dual-tach; + maxim,fan-no-watchdog; + maxim,fan-no-fault-ramp; + maxim,fan-ramp = <2>; + maxim,fan-fault-pin-mon; + }; + + fan6: fan@1 { + compatible = "pmbus-fan"; + reg = <1>; + tach-pulses = <2>; + maxim,fan-rotor-input = "tach"; + maxim,fan-pwm-freq = <25000>; + maxim,fan-dual-tach; + maxim,fan-no-watchdog; + maxim,fan-no-fault-ramp; + maxim,fan-ramp = <2>; + maxim,fan-fault-pin-mon; + }; }; eeprom@55 { -- 2.43.0 ^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH v1 1/1] ARM: dts: aspeed: system1: IBM System1 BMC update 2024-07-31 21:47 ` Ninad Palsule @ 2024-07-31 21:57 ` Eddie James -1 siblings, 0 replies; 18+ messages in thread From: Eddie James @ 2024-07-31 21:57 UTC (permalink / raw) To: linux-aspeed On 7/31/24 16:47, Ninad Palsule wrote: > Updated MAX31785a pmbus based fans in the device tree. > > Signed-off-by: Ninad Palsule <ninad@linux.ibm.com> > --- > .../dts/aspeed/aspeed-bmc-ibm-system1.dts | 96 +++++++++++++++++++ > 1 file changed, 96 insertions(+) > > diff --git a/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts b/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts > index f3efecc7eb8d0..f96b299d743ba 100644 > --- a/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts > +++ b/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts > @@ -764,17 +764,113 @@ regulator at 43 { > }; > }; > > + Don't really need the white space here, but no worries. Reviewed-by: Eddie James <eajames@linux.ibm.com> > &i2c6 { > status = "okay"; > > fan-controller at 52 { > compatible = "maxim,max31785a"; > reg = <0x52>; > + #address-cells = <1>; > + #size-cells = <0>; > + > + fan0: fan at 0 { > + compatible = "pmbus-fan"; > + reg = <0>; > + tach-pulses = <2>; > + maxim,fan-rotor-input = "tach"; > + maxim,fan-pwm-freq = <25000>; > + maxim,fan-dual-tach; > + maxim,fan-no-watchdog; > + maxim,fan-no-fault-ramp; > + maxim,fan-ramp = <2>; > + maxim,fan-fault-pin-mon; > + }; > + > + fan1: fan at 1 { > + compatible = "pmbus-fan"; > + reg = <1>; > + tach-pulses = <2>; > + maxim,fan-rotor-input = "tach"; > + maxim,fan-pwm-freq = <25000>; > + maxim,fan-dual-tach; > + maxim,fan-no-watchdog; > + maxim,fan-no-fault-ramp; > + maxim,fan-ramp = <2>; > + maxim,fan-fault-pin-mon; > + }; > + > + fan2: fan at 2 { > + compatible = "pmbus-fan"; > + reg = <2>; > + tach-pulses = <2>; > + maxim,fan-rotor-input = "tach"; > + maxim,fan-pwm-freq = <25000>; > + maxim,fan-dual-tach; > + maxim,fan-no-watchdog; > + maxim,fan-no-fault-ramp; > + maxim,fan-ramp = <2>; > + maxim,fan-fault-pin-mon; > + }; > + > + fan3: fan at 3 { > + compatible = "pmbus-fan"; > + reg = <3>; > + tach-pulses = <2>; > + maxim,fan-rotor-input = "tach"; > + maxim,fan-pwm-freq = <25000>; > + maxim,fan-dual-tach; > + maxim,fan-no-watchdog; > + maxim,fan-no-fault-ramp; > + maxim,fan-ramp = <2>; > + maxim,fan-fault-pin-mon; > + }; > + > + fan4: fan at 4 { > + compatible = "pmbus-fan"; > + reg = <4>; > + tach-pulses = <2>; > + maxim,fan-rotor-input = "tach"; > + maxim,fan-pwm-freq = <25000>; > + maxim,fan-dual-tach; > + maxim,fan-no-watchdog; > + maxim,fan-no-fault-ramp; > + maxim,fan-ramp = <2>; > + maxim,fan-fault-pin-mon; > + }; > }; > > fan-controller at 54 { > compatible = "maxim,max31785a"; > reg = <0x54>; > + #address-cells = <1>; > + #size-cells = <0>; > + > + fan5: fan at 0 { > + compatible = "pmbus-fan"; > + reg = <0>; > + tach-pulses = <2>; > + maxim,fan-rotor-input = "tach"; > + maxim,fan-pwm-freq = <25000>; > + maxim,fan-dual-tach; > + maxim,fan-no-watchdog; > + maxim,fan-no-fault-ramp; > + maxim,fan-ramp = <2>; > + maxim,fan-fault-pin-mon; > + }; > + > + fan6: fan at 1 { > + compatible = "pmbus-fan"; > + reg = <1>; > + tach-pulses = <2>; > + maxim,fan-rotor-input = "tach"; > + maxim,fan-pwm-freq = <25000>; > + maxim,fan-dual-tach; > + maxim,fan-no-watchdog; > + maxim,fan-no-fault-ramp; > + maxim,fan-ramp = <2>; > + maxim,fan-fault-pin-mon; > + }; > }; > > eeprom at 55 { ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v1 1/1] ARM: dts: aspeed: system1: IBM System1 BMC update @ 2024-07-31 21:57 ` Eddie James 0 siblings, 0 replies; 18+ messages in thread From: Eddie James @ 2024-07-31 21:57 UTC (permalink / raw) To: Ninad Palsule, robh+dt, krzysztof.kozlowski+dt, conor+dt, joel, andrew, devicetree, linux-arm-kernel, linux-aspeed, linux-kernel On 7/31/24 16:47, Ninad Palsule wrote: > Updated MAX31785a pmbus based fans in the device tree. > > Signed-off-by: Ninad Palsule <ninad@linux.ibm.com> > --- > .../dts/aspeed/aspeed-bmc-ibm-system1.dts | 96 +++++++++++++++++++ > 1 file changed, 96 insertions(+) > > diff --git a/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts b/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts > index f3efecc7eb8d0..f96b299d743ba 100644 > --- a/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts > +++ b/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts > @@ -764,17 +764,113 @@ regulator@43 { > }; > }; > > + Don't really need the white space here, but no worries. Reviewed-by: Eddie James <eajames@linux.ibm.com> > &i2c6 { > status = "okay"; > > fan-controller@52 { > compatible = "maxim,max31785a"; > reg = <0x52>; > + #address-cells = <1>; > + #size-cells = <0>; > + > + fan0: fan@0 { > + compatible = "pmbus-fan"; > + reg = <0>; > + tach-pulses = <2>; > + maxim,fan-rotor-input = "tach"; > + maxim,fan-pwm-freq = <25000>; > + maxim,fan-dual-tach; > + maxim,fan-no-watchdog; > + maxim,fan-no-fault-ramp; > + maxim,fan-ramp = <2>; > + maxim,fan-fault-pin-mon; > + }; > + > + fan1: fan@1 { > + compatible = "pmbus-fan"; > + reg = <1>; > + tach-pulses = <2>; > + maxim,fan-rotor-input = "tach"; > + maxim,fan-pwm-freq = <25000>; > + maxim,fan-dual-tach; > + maxim,fan-no-watchdog; > + maxim,fan-no-fault-ramp; > + maxim,fan-ramp = <2>; > + maxim,fan-fault-pin-mon; > + }; > + > + fan2: fan@2 { > + compatible = "pmbus-fan"; > + reg = <2>; > + tach-pulses = <2>; > + maxim,fan-rotor-input = "tach"; > + maxim,fan-pwm-freq = <25000>; > + maxim,fan-dual-tach; > + maxim,fan-no-watchdog; > + maxim,fan-no-fault-ramp; > + maxim,fan-ramp = <2>; > + maxim,fan-fault-pin-mon; > + }; > + > + fan3: fan@3 { > + compatible = "pmbus-fan"; > + reg = <3>; > + tach-pulses = <2>; > + maxim,fan-rotor-input = "tach"; > + maxim,fan-pwm-freq = <25000>; > + maxim,fan-dual-tach; > + maxim,fan-no-watchdog; > + maxim,fan-no-fault-ramp; > + maxim,fan-ramp = <2>; > + maxim,fan-fault-pin-mon; > + }; > + > + fan4: fan@4 { > + compatible = "pmbus-fan"; > + reg = <4>; > + tach-pulses = <2>; > + maxim,fan-rotor-input = "tach"; > + maxim,fan-pwm-freq = <25000>; > + maxim,fan-dual-tach; > + maxim,fan-no-watchdog; > + maxim,fan-no-fault-ramp; > + maxim,fan-ramp = <2>; > + maxim,fan-fault-pin-mon; > + }; > }; > > fan-controller@54 { > compatible = "maxim,max31785a"; > reg = <0x54>; > + #address-cells = <1>; > + #size-cells = <0>; > + > + fan5: fan@0 { > + compatible = "pmbus-fan"; > + reg = <0>; > + tach-pulses = <2>; > + maxim,fan-rotor-input = "tach"; > + maxim,fan-pwm-freq = <25000>; > + maxim,fan-dual-tach; > + maxim,fan-no-watchdog; > + maxim,fan-no-fault-ramp; > + maxim,fan-ramp = <2>; > + maxim,fan-fault-pin-mon; > + }; > + > + fan6: fan@1 { > + compatible = "pmbus-fan"; > + reg = <1>; > + tach-pulses = <2>; > + maxim,fan-rotor-input = "tach"; > + maxim,fan-pwm-freq = <25000>; > + maxim,fan-dual-tach; > + maxim,fan-no-watchdog; > + maxim,fan-no-fault-ramp; > + maxim,fan-ramp = <2>; > + maxim,fan-fault-pin-mon; > + }; > }; > > eeprom@55 { ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH v1 1/1] ARM: dts: aspeed: system1: IBM System1 BMC update 2024-07-31 21:57 ` Eddie James (?) @ 2024-08-01 14:32 ` Ninad Palsule -1 siblings, 0 replies; 18+ messages in thread From: Ninad Palsule @ 2024-08-01 14:32 UTC (permalink / raw) To: linux-aspeed Hi Eddie, On 7/31/24 16:57, Eddie James wrote: > >> diff --git a/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts >> b/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts >> index f3efecc7eb8d0..f96b299d743ba 100644 >> --- a/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts >> +++ b/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts >> @@ -764,17 +764,113 @@ regulator at 43 { >> ????? }; >> ? }; >> ? + > > > Don't really need the white space here, but no worries. I have fixed the whitespace in version 2. > > > Reviewed-by: Eddie James <eajames@linux.ibm.com> > > Thank you for the review! Regards, Ninad Palsule ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v1 1/1] ARM: dts: aspeed: system1: IBM System1 BMC update 2024-07-31 21:47 ` Ninad Palsule (?) (?) @ 2024-08-01 1:37 ` Andrew Jeffery 2024-08-01 14:33 ` Ninad Palsule -1 siblings, 1 reply; 18+ messages in thread From: Andrew Jeffery @ 2024-08-01 1:37 UTC (permalink / raw) To: Ninad Palsule; +Cc: openbmc, eajames, joel On Wed, 2024-07-31 at 16:47 -0500, Ninad Palsule wrote: > Updated MAX31785a pmbus based fans in the device tree. > > Signed-off-by: Ninad Palsule <ninad@linux.ibm.com> > --- > .../dts/aspeed/aspeed-bmc-ibm-system1.dts | 96 +++++++++++++++++++ > 1 file changed, 96 insertions(+) > > diff --git a/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts b/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts > index f3efecc7eb8d0..f96b299d743ba 100644 > --- a/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts > +++ b/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts > @@ -764,17 +764,113 @@ regulator@43 { > }; > }; > > + This whitespace change seems unnecessary. > &i2c6 { > status = "okay"; > > fan-controller@52 { > compatible = "maxim,max31785a"; > reg = <0x52>; > + #address-cells = <1>; > + #size-cells = <0>; > + > + fan0: fan@0 { > + compatible = "pmbus-fan"; > + reg = <0>; > + tach-pulses = <2>; > + maxim,fan-rotor-input = "tach"; > + maxim,fan-pwm-freq = <25000>; > + maxim,fan-dual-tach; > + maxim,fan-no-watchdog; > + maxim,fan-no-fault-ramp; > + maxim,fan-ramp = <2>; > + maxim,fan-fault-pin-mon; Note that none of these maxim-specific properties are documented in the upstream bindings. You will need to get that addressed either before or in the same series as this DTS patch when you send it upstream. Can you please fix up the whitespace and re-send this to the OpenBMC list, with the appropriate subject prefix, and _without_ including the upstream maintainers and lists (but still including Joel and myself)? Please reach out if there's anything that needs clarification before you do so. Andrew ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v1 1/1] ARM: dts: aspeed: system1: IBM System1 BMC update 2024-08-01 1:37 ` Andrew Jeffery @ 2024-08-01 14:33 ` Ninad Palsule 0 siblings, 0 replies; 18+ messages in thread From: Ninad Palsule @ 2024-08-01 14:33 UTC (permalink / raw) To: Andrew Jeffery; +Cc: Ninad Palsule, openbmc, eajames, joel [-- Attachment #1: Type: text/plain, Size: 733 bytes --] Hi Andrew, On 7/31/24 20:37, Andrew Jeffery wrote: >> >> + > This whitespace change seems unnecessary. Fixed the whitespace in version 2. > > Note that none of these maxim-specific properties are documented in the > upstream bindings. You will need to get that addressed either before or > in the same series as this DTS patch when you send it upstream. > > Can you please fix up the whitespace and re-send this to the OpenBMC > list, with the appropriate subject prefix, and _without_ including the > upstream maintainers and lists (but still including Joel and myself)? > > Please reach out if there's anything that needs clarification before > you do so. > > Andrew Got it. Thank you for the review. Regards, Ninad Palsule [-- Attachment #2: Type: text/html, Size: 1608 bytes --] ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v1 0/1] ARM: dts: aspeed: system1: IBM System1 BMC update 2024-07-31 21:47 ` Ninad Palsule (?) (?) @ 2024-08-01 1:30 ` Andrew Jeffery -1 siblings, 0 replies; 18+ messages in thread From: Andrew Jeffery @ 2024-08-01 1:30 UTC (permalink / raw) To: Ninad Palsule; +Cc: openbmc, eajames, joel Hi Ninad, On Wed, 2024-07-31 at 16:47 -0500, Ninad Palsule wrote: > Hello, > > This patch only applies to openbmc/linux. > If this is the case, then several points: - Please don't send it to the upstream lists and maintainers, only to openbmc@lists.ozlabs.org and the people who maintain the tree you're targeting (Joel and myself). I've removed upstream people and lists from To: and Cc: to spare them the thread :) There's no reason to bother them with this change. - You should use --subject-prefix="PATCH linux dev-6.6" to indicate where we should apply the patch, as both u-boot and linux patches are sent to the OpenBMC list, and for different branches (when we're upgrading). > The max31785 pmbus driver > changes are not available in the upstream yet. > This should have been the trigger to reconsider whether to include the upstream maintainers and lists. Andrew ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH v1 0/1] ARM: dts: aspeed: system1: IBM System1 BMC update 2024-07-31 21:47 ` Ninad Palsule @ 2024-08-01 15:29 ` Rob Herring (Arm) -1 siblings, 0 replies; 18+ messages in thread From: Rob Herring @ 2024-08-01 15:29 UTC (permalink / raw) To: linux-aspeed On Wed, 31 Jul 2024 16:47:28 -0500, Ninad Palsule wrote: > Hello, > > This patch only applies to openbmc/linux. The max31785 pmbus driver > changes are not available in the upstream yet. I will try to send those > changes upstream. > > Ninad Palsule (1): > ARM: dts: aspeed: system1: IBM System1 BMC update > > .../dts/aspeed/aspeed-bmc-ibm-system1.dts | 96 +++++++++++++++++++ > 1 file changed, 96 insertions(+) > > -- > 2.43.0 > > > 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-ibm-system1.dtb' for 20240731214737.986010-1-ninad at linux.ibm.com: arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus at 1e78a000/i2c at 380/fan-controller at 52/fan at 0: failed to match any schema with compatible: ['pmbus-fan'] arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus at 1e78a000/i2c at 380/fan-controller at 52/fan at 1: failed to match any schema with compatible: ['pmbus-fan'] arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus at 1e78a000/i2c at 380/fan-controller at 52/fan at 2: failed to match any schema with compatible: ['pmbus-fan'] arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus at 1e78a000/i2c at 380/fan-controller at 52/fan at 3: failed to match any schema with compatible: ['pmbus-fan'] arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus at 1e78a000/i2c at 380/fan-controller at 52/fan at 4: failed to match any schema with compatible: ['pmbus-fan'] arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus at 1e78a000/i2c at 380/fan-controller at 54/fan at 0: failed to match any schema with compatible: ['pmbus-fan'] arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus at 1e78a000/i2c at 380/fan-controller at 54/fan at 1: failed to match any schema with compatible: ['pmbus-fan'] ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v1 0/1] ARM: dts: aspeed: system1: IBM System1 BMC update @ 2024-08-01 15:29 ` Rob Herring (Arm) 0 siblings, 0 replies; 18+ messages in thread From: Rob Herring (Arm) @ 2024-08-01 15:29 UTC (permalink / raw) To: Ninad Palsule Cc: andrew, conor+dt, eajames, robh+dt, linux-kernel, krzysztof.kozlowski+dt, linux-arm-kernel, joel, devicetree, linux-aspeed On Wed, 31 Jul 2024 16:47:28 -0500, Ninad Palsule wrote: > Hello, > > This patch only applies to openbmc/linux. The max31785 pmbus driver > changes are not available in the upstream yet. I will try to send those > changes upstream. > > Ninad Palsule (1): > ARM: dts: aspeed: system1: IBM System1 BMC update > > .../dts/aspeed/aspeed-bmc-ibm-system1.dts | 96 +++++++++++++++++++ > 1 file changed, 96 insertions(+) > > -- > 2.43.0 > > > 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-ibm-system1.dtb' for 20240731214737.986010-1-ninad@linux.ibm.com: arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus@1e78a000/i2c@380/fan-controller@52/fan@0: failed to match any schema with compatible: ['pmbus-fan'] arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus@1e78a000/i2c@380/fan-controller@52/fan@1: failed to match any schema with compatible: ['pmbus-fan'] arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus@1e78a000/i2c@380/fan-controller@52/fan@2: failed to match any schema with compatible: ['pmbus-fan'] arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus@1e78a000/i2c@380/fan-controller@52/fan@3: failed to match any schema with compatible: ['pmbus-fan'] arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus@1e78a000/i2c@380/fan-controller@52/fan@4: failed to match any schema with compatible: ['pmbus-fan'] arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus@1e78a000/i2c@380/fan-controller@54/fan@0: failed to match any schema with compatible: ['pmbus-fan'] arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus@1e78a000/i2c@380/fan-controller@54/fan@1: failed to match any schema with compatible: ['pmbus-fan'] ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH v1 0/1] ARM: dts: aspeed: system1: IBM System1 BMC update 2024-08-01 15:29 ` Rob Herring (Arm) @ 2024-08-01 16:24 ` Ninad Palsule -1 siblings, 0 replies; 18+ messages in thread From: Ninad Palsule @ 2024-08-01 16:24 UTC (permalink / raw) To: linux-aspeed Hi Rob, On 8/1/24 10:29, Rob Herring (Arm) wrote: > On Wed, 31 Jul 2024 16:47:28 -0500, Ninad Palsule wrote: >> Hello, >> >> This patch only applies to openbmc/linux. The max31785 pmbus driver >> changes are not available in the upstream yet. I will try to send those >> changes upstream. >> >> Ninad Palsule (1): >> ARM: dts: aspeed: system1: IBM System1 BMC update >> >> .../dts/aspeed/aspeed-bmc-ibm-system1.dts | 96 +++++++++++++++++++ >> 1 file changed, 96 insertions(+) >> >> -- >> 2.43.0 >> >> >> > > 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-ibm-system1.dtb' for 20240731214737.986010-1-ninad at linux.ibm.com: > > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus at 1e78a000/i2c at 380/fan-controller at 52/fan at 0: failed to match any schema with compatible: ['pmbus-fan'] > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus at 1e78a000/i2c at 380/fan-controller at 52/fan at 1: failed to match any schema with compatible: ['pmbus-fan'] > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus at 1e78a000/i2c at 380/fan-controller at 52/fan at 2: failed to match any schema with compatible: ['pmbus-fan'] > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus at 1e78a000/i2c at 380/fan-controller at 52/fan at 3: failed to match any schema with compatible: ['pmbus-fan'] > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus at 1e78a000/i2c at 380/fan-controller at 52/fan at 4: failed to match any schema with compatible: ['pmbus-fan'] > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus at 1e78a000/i2c at 380/fan-controller at 54/fan at 0: failed to match any schema with compatible: ['pmbus-fan'] > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus at 1e78a000/i2c at 380/fan-controller at 54/fan at 1: failed to match any schema with compatible: ['pmbus-fan'] > I have seen these warning. I am not sure how to fix them as it is already documented in the following file. Documentation/devicetree/bindings/hwmon/pmbus/max31785.txt May be because its not converted to YAML yet? Thanks & Rehads, Ninad Palsule ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v1 0/1] ARM: dts: aspeed: system1: IBM System1 BMC update @ 2024-08-01 16:24 ` Ninad Palsule 0 siblings, 0 replies; 18+ messages in thread From: Ninad Palsule @ 2024-08-01 16:24 UTC (permalink / raw) To: Rob Herring (Arm) Cc: andrew, eajames, robh+dt, joel, devicetree, linux-aspeed Hi Rob, On 8/1/24 10:29, Rob Herring (Arm) wrote: > On Wed, 31 Jul 2024 16:47:28 -0500, Ninad Palsule wrote: >> Hello, >> >> This patch only applies to openbmc/linux. The max31785 pmbus driver >> changes are not available in the upstream yet. I will try to send those >> changes upstream. >> >> Ninad Palsule (1): >> ARM: dts: aspeed: system1: IBM System1 BMC update >> >> .../dts/aspeed/aspeed-bmc-ibm-system1.dts | 96 +++++++++++++++++++ >> 1 file changed, 96 insertions(+) >> >> -- >> 2.43.0 >> >> >> > > 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-ibm-system1.dtb' for 20240731214737.986010-1-ninad@linux.ibm.com: > > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus@1e78a000/i2c@380/fan-controller@52/fan@0: failed to match any schema with compatible: ['pmbus-fan'] > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus@1e78a000/i2c@380/fan-controller@52/fan@1: failed to match any schema with compatible: ['pmbus-fan'] > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus@1e78a000/i2c@380/fan-controller@52/fan@2: failed to match any schema with compatible: ['pmbus-fan'] > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus@1e78a000/i2c@380/fan-controller@52/fan@3: failed to match any schema with compatible: ['pmbus-fan'] > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus@1e78a000/i2c@380/fan-controller@52/fan@4: failed to match any schema with compatible: ['pmbus-fan'] > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus@1e78a000/i2c@380/fan-controller@54/fan@0: failed to match any schema with compatible: ['pmbus-fan'] > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus@1e78a000/i2c@380/fan-controller@54/fan@1: failed to match any schema with compatible: ['pmbus-fan'] > I have seen these warning. I am not sure how to fix them as it is already documented in the following file. Documentation/devicetree/bindings/hwmon/pmbus/max31785.txt May be because its not converted to YAML yet? Thanks & Rehads, Ninad Palsule ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH v1 0/1] ARM: dts: aspeed: system1: IBM System1 BMC update 2024-08-01 16:24 ` Ninad Palsule @ 2024-08-01 17:36 ` Rob Herring -1 siblings, 0 replies; 18+ messages in thread From: Rob Herring @ 2024-08-01 17:36 UTC (permalink / raw) To: linux-aspeed On Thu, Aug 1, 2024 at 10:25?AM Ninad Palsule <ninad@linux.ibm.com> wrote: > > Hi Rob, > > On 8/1/24 10:29, Rob Herring (Arm) wrote: > > On Wed, 31 Jul 2024 16:47:28 -0500, Ninad Palsule wrote: > >> Hello, > >> > >> This patch only applies to openbmc/linux. The max31785 pmbus driver > >> changes are not available in the upstream yet. I will try to send those > >> changes upstream. > >> > >> Ninad Palsule (1): > >> ARM: dts: aspeed: system1: IBM System1 BMC update > >> > >> .../dts/aspeed/aspeed-bmc-ibm-system1.dts | 96 +++++++++++++++++++ > >> 1 file changed, 96 insertions(+) > >> > >> -- > >> 2.43.0 > >> > >> > >> > > > > 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-ibm-system1.dtb' for 20240731214737.986010-1-ninad at linux.ibm.com: > > > > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus at 1e78a000/i2c at 380/fan-controller at 52/fan at 0: failed to match any schema with compatible: ['pmbus-fan'] > > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus at 1e78a000/i2c at 380/fan-controller at 52/fan at 1: failed to match any schema with compatible: ['pmbus-fan'] > > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus at 1e78a000/i2c at 380/fan-controller at 52/fan at 2: failed to match any schema with compatible: ['pmbus-fan'] > > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus at 1e78a000/i2c at 380/fan-controller at 52/fan at 3: failed to match any schema with compatible: ['pmbus-fan'] > > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus at 1e78a000/i2c at 380/fan-controller at 52/fan at 4: failed to match any schema with compatible: ['pmbus-fan'] > > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus at 1e78a000/i2c at 380/fan-controller at 54/fan at 0: failed to match any schema with compatible: ['pmbus-fan'] > > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus at 1e78a000/i2c at 380/fan-controller at 54/fan at 1: failed to match any schema with compatible: ['pmbus-fan'] > > > I have seen these warning. I am not sure how to fix them as it is > already documented in the following file. > > Documentation/devicetree/bindings/hwmon/pmbus/max31785.txt $ git log -- Documentation/devicetree/bindings/hwmon/pmbus/max31785.txt (END) Did you mean Documentation/devicetree/bindings/hwmon/max31785.txt? If so, there is no 'pmbus-fan' in it or anywhere else except aspeed dts files. The state of the aspeed dts files is really quite sad. This is a count of warnings. The first number is total warnings. The second number is unique warnings (stripping the dtb name so we don't have N warnings for N boards). The 2nd list is undocumented (by schema) compatibles. The number of warnings vs. Linus's tree is higher already and we just started the cycle. That's the wrong direction and in general I see very little if any effort reducing the warnings for aspeed. This is all run daily here: https://gitlab.com/robherring/linux-dt/-/jobs arch/arm/boot/dts/aspeed:3682:635 184 sbefifo at 2400: '#address-cells', '#size-cells' do not match any of the regexes: 'pinctrl-[0-9]+' 160 i2cr at 20: #address-cells: 1 was expected 88 eeprom at 0: 'size' is a required property 88 eeprom at 0: 'pagesize' is a required property 88 eeprom at 0: 'address-width' is a required property 70 lpc at 1e789000: reg-io-width: 4 is not of type 'object' 62 kcs at 2c: 'clocks' does not match any of the regexes: 'pinctrl-[0-9]+' 62 kcs at 28: 'clocks' does not match any of the regexes: 'pinctrl-[0-9]+' 62 kcs at 24: 'clocks' does not match any of the regexes: 'pinctrl-[0-9]+' 62 kcs at 114: 'clocks' does not match any of the regexes: 'pinctrl-[0-9]+' 45 sram at 1e720000: 'ranges' is a required property 45 sram at 1e720000: '#size-cells' is a required property 45 sram at 1e720000: '#address-cells' is a required property 45 lpc at 1e789000: lpc-snoop at 90: 'clocks' does not match any of the regexes: 'pinctrl-[0-9]+' 25 timer: 'clocks' does not match any of the regexes: 'pinctrl-[0-9]+' 25 syscon at 1e6e2000: 'smp-memram at 180' does not match any of the regexes: '^interrupt-controller@[0-9a-f]+$', '^p2a-control@[0-9a-f]+$', '^pinctrl(@[0-9a-f]+)?$', '^silicon-id@[0-9a-f]+$', 'pinctrl-[0-9]+' 25 lpc at 1e789000: lpc-snoop at 80: 'clocks' does not match any of the regexes: 'pinctrl-[0-9]+' 25 ftgmac at 1e690000: $nodename:0: 'ftgmac at 1e690000' does not match '^ethernet(@.*)?$' 25 ftgmac at 1e680000: $nodename:0: 'ftgmac at 1e680000' does not match '^ethernet(@.*)?$' 25 ftgmac at 1e670000: $nodename:0: 'ftgmac at 1e670000' does not match '^ethernet(@.*)?$' 25 ftgmac at 1e660000: $nodename:0: 'ftgmac at 1e660000' does not match '^ethernet(@.*)?$' 25 fsi at 1e79b100: compatible: ['aspeed,ast2600-fsi-master', 'fsi-master'] is too long 25 fsi at 1e79b000: compatible: ['aspeed,ast2600-fsi-master', 'fsi-master'] is too long 25 crypto at 1e6fa000: 'aspeed,ahbc' does not match any of the regexes: 'pinctrl-[0-9]+' 25 bus at 1e600000: compatible: ['aspeed,ast2600-ahbc', 'syscon'] is too long 24 sdc at 1e740000: sdhci at 1e740200:compatible: ['aspeed,ast2600-sdhci', 'sdhci'] is too long 24 sdc at 1e740000: sdhci at 1e740100:compatible: ['aspeed,ast2600-sdhci', 'sdhci'] is too long 22 spi at 60: 'eeprom at 0' does not match any of the regexes: '^spi@[0-9a-f]+$', 'pinctrl-[0-9]+' 22 spi at 40: 'eeprom at 0' does not match any of the regexes: '^spi@[0-9a-f]+$', 'pinctrl-[0-9]+' 22 sbefifo at 2400: occ: 'occ-hwmon', 'reg' do not match any of the regexes: 'pinctrl-[0-9]+' arch/arm/boot/dts/aspeed:68 ['adi, adm1272'] ['adm1272'] ['adm1275'] ['aspeed,ast2400-cf-fsi-master', 'fsi-master'] ['aspeed,ast2400-cvic', 'aspeed-cvic'] ['aspeed,ast2400-i2c-ic'] ['aspeed,ast2400-ibt-bmc'] ['aspeed,ast2400-lhc'] ['aspeed,ast2400-p2a-ctrl'] ['aspeed,ast2400-pwm-tacho'] ['aspeed,ast2400-timer'] ['aspeed,ast2400-vic'] ['aspeed,ast2400-video-engine'] ['aspeed,ast2500-cf-fsi-master', 'fsi-master'] ['aspeed,ast2500-cvic', 'aspeed-cvic'] ['aspeed,ast2500-gfx', 'syscon'] ['aspeed,ast2500-i2c-ic'] ['aspeed,ast2500-ibt-bmc'] ['aspeed,ast2500-lhc'] ['aspeed,ast2500-p2a-ctrl'] ['aspeed,ast2500-pwm-tacho'] ['aspeed,ast2500-scu-ic'] ['aspeed,ast2500-sdram-edac'] ['aspeed,ast2500-video-engine'] ['aspeed,ast2500-xdma'] ['aspeed,ast2600-fsi-master', 'fsi-master'] ['aspeed,ast2600-gfx', 'syscon'] ['aspeed,ast2600-ibt-bmc'] ['aspeed,ast2600-lhc'] ['aspeed,ast2600-scu-ic0'] ['aspeed,ast2600-scu-ic1'] ['aspeed,ast2600-sdhci', 'sdhci'] ['aspeed,ast2600-sdram-edac', 'syscon'] ['aspeed,ast2600-smpmem'] ['aspeed,ast2600-timer'] ['aspeed,ast2600-udma'] ['aspeed,ast2600-video-engine'] ['aspeed,ast2600-xdma'] ['atmel,at30tse004a'] ['dallas,ds3231'] ['delta,dps800'] ['dps650ab'] ['fsi-master-gpio', 'fsi-master'] ['fsi-master-hub'] ['ibm,bonnell-bmc', 'aspeed,ast2600'] ['ibm,fsi-i2c-master'] ['ibm,fsi-master-hub'] ['ibm,fsi2pib'] ['ibm,pca9552'] ['infineon,ir35221'] ['infineon,pxe1610'] ['ipmb-dev'] ['isil,isl69260'] ['isil,raa229004'] ['lm75'] ['max31790'] ['maxim,max31785a'] ['microchip,emc1413'] ['mps,mp5023'] ['nuvoton,w83795g'] ['nvt210'] ['nxp,pca9551'] ['nxp,pca9552'] ['nxp,pca9641'] ['pmbus'] ['pmbus-fan'] ['si,si7021a20'] ['ti,ucd90160'] ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v1 0/1] ARM: dts: aspeed: system1: IBM System1 BMC update @ 2024-08-01 17:36 ` Rob Herring 0 siblings, 0 replies; 18+ messages in thread From: Rob Herring @ 2024-08-01 17:36 UTC (permalink / raw) To: Ninad Palsule; +Cc: andrew, eajames, joel, devicetree, linux-aspeed On Thu, Aug 1, 2024 at 10:25 AM Ninad Palsule <ninad@linux.ibm.com> wrote: > > Hi Rob, > > On 8/1/24 10:29, Rob Herring (Arm) wrote: > > On Wed, 31 Jul 2024 16:47:28 -0500, Ninad Palsule wrote: > >> Hello, > >> > >> This patch only applies to openbmc/linux. The max31785 pmbus driver > >> changes are not available in the upstream yet. I will try to send those > >> changes upstream. > >> > >> Ninad Palsule (1): > >> ARM: dts: aspeed: system1: IBM System1 BMC update > >> > >> .../dts/aspeed/aspeed-bmc-ibm-system1.dts | 96 +++++++++++++++++++ > >> 1 file changed, 96 insertions(+) > >> > >> -- > >> 2.43.0 > >> > >> > >> > > > > 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-ibm-system1.dtb' for 20240731214737.986010-1-ninad@linux.ibm.com: > > > > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus@1e78a000/i2c@380/fan-controller@52/fan@0: failed to match any schema with compatible: ['pmbus-fan'] > > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus@1e78a000/i2c@380/fan-controller@52/fan@1: failed to match any schema with compatible: ['pmbus-fan'] > > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus@1e78a000/i2c@380/fan-controller@52/fan@2: failed to match any schema with compatible: ['pmbus-fan'] > > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus@1e78a000/i2c@380/fan-controller@52/fan@3: failed to match any schema with compatible: ['pmbus-fan'] > > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus@1e78a000/i2c@380/fan-controller@52/fan@4: failed to match any schema with compatible: ['pmbus-fan'] > > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus@1e78a000/i2c@380/fan-controller@54/fan@0: failed to match any schema with compatible: ['pmbus-fan'] > > arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dtb: /ahb/apb/bus@1e78a000/i2c@380/fan-controller@54/fan@1: failed to match any schema with compatible: ['pmbus-fan'] > > > I have seen these warning. I am not sure how to fix them as it is > already documented in the following file. > > Documentation/devicetree/bindings/hwmon/pmbus/max31785.txt $ git log -- Documentation/devicetree/bindings/hwmon/pmbus/max31785.txt (END) Did you mean Documentation/devicetree/bindings/hwmon/max31785.txt? If so, there is no 'pmbus-fan' in it or anywhere else except aspeed dts files. The state of the aspeed dts files is really quite sad. This is a count of warnings. The first number is total warnings. The second number is unique warnings (stripping the dtb name so we don't have N warnings for N boards). The 2nd list is undocumented (by schema) compatibles. The number of warnings vs. Linus's tree is higher already and we just started the cycle. That's the wrong direction and in general I see very little if any effort reducing the warnings for aspeed. This is all run daily here: https://gitlab.com/robherring/linux-dt/-/jobs arch/arm/boot/dts/aspeed:3682:635 184 sbefifo@2400: '#address-cells', '#size-cells' do not match any of the regexes: 'pinctrl-[0-9]+' 160 i2cr@20: #address-cells: 1 was expected 88 eeprom@0: 'size' is a required property 88 eeprom@0: 'pagesize' is a required property 88 eeprom@0: 'address-width' is a required property 70 lpc@1e789000: reg-io-width: 4 is not of type 'object' 62 kcs@2c: 'clocks' does not match any of the regexes: 'pinctrl-[0-9]+' 62 kcs@28: 'clocks' does not match any of the regexes: 'pinctrl-[0-9]+' 62 kcs@24: 'clocks' does not match any of the regexes: 'pinctrl-[0-9]+' 62 kcs@114: 'clocks' does not match any of the regexes: 'pinctrl-[0-9]+' 45 sram@1e720000: 'ranges' is a required property 45 sram@1e720000: '#size-cells' is a required property 45 sram@1e720000: '#address-cells' is a required property 45 lpc@1e789000: lpc-snoop@90: 'clocks' does not match any of the regexes: 'pinctrl-[0-9]+' 25 timer: 'clocks' does not match any of the regexes: 'pinctrl-[0-9]+' 25 syscon@1e6e2000: 'smp-memram@180' does not match any of the regexes: '^interrupt-controller@[0-9a-f]+$', '^p2a-control@[0-9a-f]+$', '^pinctrl(@[0-9a-f]+)?$', '^silicon-id@[0-9a-f]+$', 'pinctrl-[0-9]+' 25 lpc@1e789000: lpc-snoop@80: 'clocks' does not match any of the regexes: 'pinctrl-[0-9]+' 25 ftgmac@1e690000: $nodename:0: 'ftgmac@1e690000' does not match '^ethernet(@.*)?$' 25 ftgmac@1e680000: $nodename:0: 'ftgmac@1e680000' does not match '^ethernet(@.*)?$' 25 ftgmac@1e670000: $nodename:0: 'ftgmac@1e670000' does not match '^ethernet(@.*)?$' 25 ftgmac@1e660000: $nodename:0: 'ftgmac@1e660000' does not match '^ethernet(@.*)?$' 25 fsi@1e79b100: compatible: ['aspeed,ast2600-fsi-master', 'fsi-master'] is too long 25 fsi@1e79b000: compatible: ['aspeed,ast2600-fsi-master', 'fsi-master'] is too long 25 crypto@1e6fa000: 'aspeed,ahbc' does not match any of the regexes: 'pinctrl-[0-9]+' 25 bus@1e600000: compatible: ['aspeed,ast2600-ahbc', 'syscon'] is too long 24 sdc@1e740000: sdhci@1e740200:compatible: ['aspeed,ast2600-sdhci', 'sdhci'] is too long 24 sdc@1e740000: sdhci@1e740100:compatible: ['aspeed,ast2600-sdhci', 'sdhci'] is too long 22 spi@60: 'eeprom@0' does not match any of the regexes: '^spi@[0-9a-f]+$', 'pinctrl-[0-9]+' 22 spi@40: 'eeprom@0' does not match any of the regexes: '^spi@[0-9a-f]+$', 'pinctrl-[0-9]+' 22 sbefifo@2400: occ: 'occ-hwmon', 'reg' do not match any of the regexes: 'pinctrl-[0-9]+' arch/arm/boot/dts/aspeed:68 ['adi, adm1272'] ['adm1272'] ['adm1275'] ['aspeed,ast2400-cf-fsi-master', 'fsi-master'] ['aspeed,ast2400-cvic', 'aspeed-cvic'] ['aspeed,ast2400-i2c-ic'] ['aspeed,ast2400-ibt-bmc'] ['aspeed,ast2400-lhc'] ['aspeed,ast2400-p2a-ctrl'] ['aspeed,ast2400-pwm-tacho'] ['aspeed,ast2400-timer'] ['aspeed,ast2400-vic'] ['aspeed,ast2400-video-engine'] ['aspeed,ast2500-cf-fsi-master', 'fsi-master'] ['aspeed,ast2500-cvic', 'aspeed-cvic'] ['aspeed,ast2500-gfx', 'syscon'] ['aspeed,ast2500-i2c-ic'] ['aspeed,ast2500-ibt-bmc'] ['aspeed,ast2500-lhc'] ['aspeed,ast2500-p2a-ctrl'] ['aspeed,ast2500-pwm-tacho'] ['aspeed,ast2500-scu-ic'] ['aspeed,ast2500-sdram-edac'] ['aspeed,ast2500-video-engine'] ['aspeed,ast2500-xdma'] ['aspeed,ast2600-fsi-master', 'fsi-master'] ['aspeed,ast2600-gfx', 'syscon'] ['aspeed,ast2600-ibt-bmc'] ['aspeed,ast2600-lhc'] ['aspeed,ast2600-scu-ic0'] ['aspeed,ast2600-scu-ic1'] ['aspeed,ast2600-sdhci', 'sdhci'] ['aspeed,ast2600-sdram-edac', 'syscon'] ['aspeed,ast2600-smpmem'] ['aspeed,ast2600-timer'] ['aspeed,ast2600-udma'] ['aspeed,ast2600-video-engine'] ['aspeed,ast2600-xdma'] ['atmel,at30tse004a'] ['dallas,ds3231'] ['delta,dps800'] ['dps650ab'] ['fsi-master-gpio', 'fsi-master'] ['fsi-master-hub'] ['ibm,bonnell-bmc', 'aspeed,ast2600'] ['ibm,fsi-i2c-master'] ['ibm,fsi-master-hub'] ['ibm,fsi2pib'] ['ibm,pca9552'] ['infineon,ir35221'] ['infineon,pxe1610'] ['ipmb-dev'] ['isil,isl69260'] ['isil,raa229004'] ['lm75'] ['max31790'] ['maxim,max31785a'] ['microchip,emc1413'] ['mps,mp5023'] ['nuvoton,w83795g'] ['nvt210'] ['nxp,pca9551'] ['nxp,pca9552'] ['nxp,pca9641'] ['pmbus'] ['pmbus-fan'] ['si,si7021a20'] ['ti,ucd90160'] ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH v1 0/1] ARM: dts: aspeed: system1: IBM System1 BMC update 2024-08-01 17:36 ` Rob Herring @ 2024-08-01 18:50 ` Ninad Palsule -1 siblings, 0 replies; 18+ messages in thread From: Ninad Palsule @ 2024-08-01 18:50 UTC (permalink / raw) To: linux-aspeed Hi Rob, On 8/1/24 12:36, Rob Herring wrote: > On Thu, Aug 1, 2024 at 10:25?AM Ninad Palsule <ninad@linux.ibm.com> wrote: >> I have seen these warning. I am not sure how to fix them as it is >> already documented in the following file. >> >> Documentation/devicetree/bindings/hwmon/pmbus/max31785.txt > $ git log -- Documentation/devicetree/bindings/hwmon/pmbus/max31785.txt > (END) Yes, This file only exist in the www.github.com/openbmc/linux. I will try to upstream it along with driver changes later. Sorry I was not suppose to send this patch to upstream. So please ignore the patch. Thanks & Regards, Ninad Palsule > > Did you mean Documentation/devicetree/bindings/hwmon/max31785.txt? If > so, there is no 'pmbus-fan' in it or anywhere else except aspeed dts > files. > > The state of the aspeed dts files is really quite sad. This is a count > of warnings. The first number is total warnings. The second number is > unique warnings (stripping the dtb name so we don't have N warnings > for N boards). The 2nd list is undocumented (by schema) compatibles. > > The number of warnings vs. Linus's tree is higher already and we just > started the cycle. That's the wrong direction and in general I see > very little if any effort reducing the warnings for aspeed. > > This is all run daily here: https://gitlab.com/robherring/linux-dt/-/jobs > > arch/arm/boot/dts/aspeed:3682:635 > 184 sbefifo at 2400: '#address-cells', '#size-cells' do not match > any of the regexes: 'pinctrl-[0-9]+' > 160 i2cr at 20: #address-cells: 1 was expected > 88 eeprom at 0: 'size' is a required property > 88 eeprom at 0: 'pagesize' is a required property > 88 eeprom at 0: 'address-width' is a required property > 70 lpc at 1e789000: reg-io-width: 4 is not of type 'object' > 62 kcs at 2c: 'clocks' does not match any of the regexes: 'pinctrl-[0-9]+' > 62 kcs at 28: 'clocks' does not match any of the regexes: 'pinctrl-[0-9]+' > 62 kcs at 24: 'clocks' does not match any of the regexes: 'pinctrl-[0-9]+' > 62 kcs at 114: 'clocks' does not match any of the regexes: 'pinctrl-[0-9]+' > 45 sram at 1e720000: 'ranges' is a required property > 45 sram at 1e720000: '#size-cells' is a required property > 45 sram at 1e720000: '#address-cells' is a required property > 45 lpc at 1e789000: lpc-snoop at 90: 'clocks' does not match any of > the regexes: 'pinctrl-[0-9]+' > 25 timer: 'clocks' does not match any of the regexes: 'pinctrl-[0-9]+' > 25 syscon at 1e6e2000: 'smp-memram at 180' does not match any of the > regexes: '^interrupt-controller@[0-9a-f]+$', > '^p2a-control@[0-9a-f]+$', '^pinctrl(@[0-9a-f]+)?$', > '^silicon-id@[0-9a-f]+$', 'pinctrl-[0-9]+' > 25 lpc at 1e789000: lpc-snoop at 80: 'clocks' does not match any of > the regexes: 'pinctrl-[0-9]+' > 25 ftgmac at 1e690000: $nodename:0: 'ftgmac at 1e690000' does not > match '^ethernet(@.*)?$' > 25 ftgmac at 1e680000: $nodename:0: 'ftgmac at 1e680000' does not > match '^ethernet(@.*)?$' > 25 ftgmac at 1e670000: $nodename:0: 'ftgmac at 1e670000' does not > match '^ethernet(@.*)?$' > 25 ftgmac at 1e660000: $nodename:0: 'ftgmac at 1e660000' does not > match '^ethernet(@.*)?$' > 25 fsi at 1e79b100: compatible: ['aspeed,ast2600-fsi-master', > 'fsi-master'] is too long > 25 fsi at 1e79b000: compatible: ['aspeed,ast2600-fsi-master', > 'fsi-master'] is too long > 25 crypto at 1e6fa000: 'aspeed,ahbc' does not match any of the > regexes: 'pinctrl-[0-9]+' > 25 bus at 1e600000: compatible: ['aspeed,ast2600-ahbc', 'syscon'] is too long > 24 sdc at 1e740000: sdhci at 1e740200:compatible: > ['aspeed,ast2600-sdhci', 'sdhci'] is too long > 24 sdc at 1e740000: sdhci at 1e740100:compatible: > ['aspeed,ast2600-sdhci', 'sdhci'] is too long > 22 spi at 60: 'eeprom at 0' does not match any of the regexes: > '^spi@[0-9a-f]+$', 'pinctrl-[0-9]+' > 22 spi at 40: 'eeprom at 0' does not match any of the regexes: > '^spi@[0-9a-f]+$', 'pinctrl-[0-9]+' > 22 sbefifo at 2400: occ: 'occ-hwmon', 'reg' do not match any of the > regexes: 'pinctrl-[0-9]+' > > arch/arm/boot/dts/aspeed:68 > ['adi, adm1272'] > ['adm1272'] > ['adm1275'] > ['aspeed,ast2400-cf-fsi-master', 'fsi-master'] > ['aspeed,ast2400-cvic', 'aspeed-cvic'] > ['aspeed,ast2400-i2c-ic'] > ['aspeed,ast2400-ibt-bmc'] > ['aspeed,ast2400-lhc'] > ['aspeed,ast2400-p2a-ctrl'] > ['aspeed,ast2400-pwm-tacho'] > ['aspeed,ast2400-timer'] > ['aspeed,ast2400-vic'] > ['aspeed,ast2400-video-engine'] > ['aspeed,ast2500-cf-fsi-master', 'fsi-master'] > ['aspeed,ast2500-cvic', 'aspeed-cvic'] > ['aspeed,ast2500-gfx', 'syscon'] > ['aspeed,ast2500-i2c-ic'] > ['aspeed,ast2500-ibt-bmc'] > ['aspeed,ast2500-lhc'] > ['aspeed,ast2500-p2a-ctrl'] > ['aspeed,ast2500-pwm-tacho'] > ['aspeed,ast2500-scu-ic'] > ['aspeed,ast2500-sdram-edac'] > ['aspeed,ast2500-video-engine'] > ['aspeed,ast2500-xdma'] > ['aspeed,ast2600-fsi-master', 'fsi-master'] > ['aspeed,ast2600-gfx', 'syscon'] > ['aspeed,ast2600-ibt-bmc'] > ['aspeed,ast2600-lhc'] > ['aspeed,ast2600-scu-ic0'] > ['aspeed,ast2600-scu-ic1'] > ['aspeed,ast2600-sdhci', 'sdhci'] > ['aspeed,ast2600-sdram-edac', 'syscon'] > ['aspeed,ast2600-smpmem'] > ['aspeed,ast2600-timer'] > ['aspeed,ast2600-udma'] > ['aspeed,ast2600-video-engine'] > ['aspeed,ast2600-xdma'] > ['atmel,at30tse004a'] > ['dallas,ds3231'] > ['delta,dps800'] > ['dps650ab'] > ['fsi-master-gpio', 'fsi-master'] > ['fsi-master-hub'] > ['ibm,bonnell-bmc', 'aspeed,ast2600'] > ['ibm,fsi-i2c-master'] > ['ibm,fsi-master-hub'] > ['ibm,fsi2pib'] > ['ibm,pca9552'] > ['infineon,ir35221'] > ['infineon,pxe1610'] > ['ipmb-dev'] > ['isil,isl69260'] > ['isil,raa229004'] > ['lm75'] > ['max31790'] > ['maxim,max31785a'] > ['microchip,emc1413'] > ['mps,mp5023'] > ['nuvoton,w83795g'] > ['nvt210'] > ['nxp,pca9551'] > ['nxp,pca9552'] > ['nxp,pca9641'] > ['pmbus'] > ['pmbus-fan'] > ['si,si7021a20'] > ['ti,ucd90160'] ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v1 0/1] ARM: dts: aspeed: system1: IBM System1 BMC update @ 2024-08-01 18:50 ` Ninad Palsule 0 siblings, 0 replies; 18+ messages in thread From: Ninad Palsule @ 2024-08-01 18:50 UTC (permalink / raw) To: Rob Herring; +Cc: andrew, eajames, joel, devicetree, linux-aspeed Hi Rob, On 8/1/24 12:36, Rob Herring wrote: > On Thu, Aug 1, 2024 at 10:25 AM Ninad Palsule <ninad@linux.ibm.com> wrote: >> I have seen these warning. I am not sure how to fix them as it is >> already documented in the following file. >> >> Documentation/devicetree/bindings/hwmon/pmbus/max31785.txt > $ git log -- Documentation/devicetree/bindings/hwmon/pmbus/max31785.txt > (END) Yes, This file only exist in the www.github.com/openbmc/linux. I will try to upstream it along with driver changes later. Sorry I was not suppose to send this patch to upstream. So please ignore the patch. Thanks & Regards, Ninad Palsule > > Did you mean Documentation/devicetree/bindings/hwmon/max31785.txt? If > so, there is no 'pmbus-fan' in it or anywhere else except aspeed dts > files. > > The state of the aspeed dts files is really quite sad. This is a count > of warnings. The first number is total warnings. The second number is > unique warnings (stripping the dtb name so we don't have N warnings > for N boards). The 2nd list is undocumented (by schema) compatibles. > > The number of warnings vs. Linus's tree is higher already and we just > started the cycle. That's the wrong direction and in general I see > very little if any effort reducing the warnings for aspeed. > > This is all run daily here: https://gitlab.com/robherring/linux-dt/-/jobs > > arch/arm/boot/dts/aspeed:3682:635 > 184 sbefifo@2400: '#address-cells', '#size-cells' do not match > any of the regexes: 'pinctrl-[0-9]+' > 160 i2cr@20: #address-cells: 1 was expected > 88 eeprom@0: 'size' is a required property > 88 eeprom@0: 'pagesize' is a required property > 88 eeprom@0: 'address-width' is a required property > 70 lpc@1e789000: reg-io-width: 4 is not of type 'object' > 62 kcs@2c: 'clocks' does not match any of the regexes: 'pinctrl-[0-9]+' > 62 kcs@28: 'clocks' does not match any of the regexes: 'pinctrl-[0-9]+' > 62 kcs@24: 'clocks' does not match any of the regexes: 'pinctrl-[0-9]+' > 62 kcs@114: 'clocks' does not match any of the regexes: 'pinctrl-[0-9]+' > 45 sram@1e720000: 'ranges' is a required property > 45 sram@1e720000: '#size-cells' is a required property > 45 sram@1e720000: '#address-cells' is a required property > 45 lpc@1e789000: lpc-snoop@90: 'clocks' does not match any of > the regexes: 'pinctrl-[0-9]+' > 25 timer: 'clocks' does not match any of the regexes: 'pinctrl-[0-9]+' > 25 syscon@1e6e2000: 'smp-memram@180' does not match any of the > regexes: '^interrupt-controller@[0-9a-f]+$', > '^p2a-control@[0-9a-f]+$', '^pinctrl(@[0-9a-f]+)?$', > '^silicon-id@[0-9a-f]+$', 'pinctrl-[0-9]+' > 25 lpc@1e789000: lpc-snoop@80: 'clocks' does not match any of > the regexes: 'pinctrl-[0-9]+' > 25 ftgmac@1e690000: $nodename:0: 'ftgmac@1e690000' does not > match '^ethernet(@.*)?$' > 25 ftgmac@1e680000: $nodename:0: 'ftgmac@1e680000' does not > match '^ethernet(@.*)?$' > 25 ftgmac@1e670000: $nodename:0: 'ftgmac@1e670000' does not > match '^ethernet(@.*)?$' > 25 ftgmac@1e660000: $nodename:0: 'ftgmac@1e660000' does not > match '^ethernet(@.*)?$' > 25 fsi@1e79b100: compatible: ['aspeed,ast2600-fsi-master', > 'fsi-master'] is too long > 25 fsi@1e79b000: compatible: ['aspeed,ast2600-fsi-master', > 'fsi-master'] is too long > 25 crypto@1e6fa000: 'aspeed,ahbc' does not match any of the > regexes: 'pinctrl-[0-9]+' > 25 bus@1e600000: compatible: ['aspeed,ast2600-ahbc', 'syscon'] is too long > 24 sdc@1e740000: sdhci@1e740200:compatible: > ['aspeed,ast2600-sdhci', 'sdhci'] is too long > 24 sdc@1e740000: sdhci@1e740100:compatible: > ['aspeed,ast2600-sdhci', 'sdhci'] is too long > 22 spi@60: 'eeprom@0' does not match any of the regexes: > '^spi@[0-9a-f]+$', 'pinctrl-[0-9]+' > 22 spi@40: 'eeprom@0' does not match any of the regexes: > '^spi@[0-9a-f]+$', 'pinctrl-[0-9]+' > 22 sbefifo@2400: occ: 'occ-hwmon', 'reg' do not match any of the > regexes: 'pinctrl-[0-9]+' > > arch/arm/boot/dts/aspeed:68 > ['adi, adm1272'] > ['adm1272'] > ['adm1275'] > ['aspeed,ast2400-cf-fsi-master', 'fsi-master'] > ['aspeed,ast2400-cvic', 'aspeed-cvic'] > ['aspeed,ast2400-i2c-ic'] > ['aspeed,ast2400-ibt-bmc'] > ['aspeed,ast2400-lhc'] > ['aspeed,ast2400-p2a-ctrl'] > ['aspeed,ast2400-pwm-tacho'] > ['aspeed,ast2400-timer'] > ['aspeed,ast2400-vic'] > ['aspeed,ast2400-video-engine'] > ['aspeed,ast2500-cf-fsi-master', 'fsi-master'] > ['aspeed,ast2500-cvic', 'aspeed-cvic'] > ['aspeed,ast2500-gfx', 'syscon'] > ['aspeed,ast2500-i2c-ic'] > ['aspeed,ast2500-ibt-bmc'] > ['aspeed,ast2500-lhc'] > ['aspeed,ast2500-p2a-ctrl'] > ['aspeed,ast2500-pwm-tacho'] > ['aspeed,ast2500-scu-ic'] > ['aspeed,ast2500-sdram-edac'] > ['aspeed,ast2500-video-engine'] > ['aspeed,ast2500-xdma'] > ['aspeed,ast2600-fsi-master', 'fsi-master'] > ['aspeed,ast2600-gfx', 'syscon'] > ['aspeed,ast2600-ibt-bmc'] > ['aspeed,ast2600-lhc'] > ['aspeed,ast2600-scu-ic0'] > ['aspeed,ast2600-scu-ic1'] > ['aspeed,ast2600-sdhci', 'sdhci'] > ['aspeed,ast2600-sdram-edac', 'syscon'] > ['aspeed,ast2600-smpmem'] > ['aspeed,ast2600-timer'] > ['aspeed,ast2600-udma'] > ['aspeed,ast2600-video-engine'] > ['aspeed,ast2600-xdma'] > ['atmel,at30tse004a'] > ['dallas,ds3231'] > ['delta,dps800'] > ['dps650ab'] > ['fsi-master-gpio', 'fsi-master'] > ['fsi-master-hub'] > ['ibm,bonnell-bmc', 'aspeed,ast2600'] > ['ibm,fsi-i2c-master'] > ['ibm,fsi-master-hub'] > ['ibm,fsi2pib'] > ['ibm,pca9552'] > ['infineon,ir35221'] > ['infineon,pxe1610'] > ['ipmb-dev'] > ['isil,isl69260'] > ['isil,raa229004'] > ['lm75'] > ['max31790'] > ['maxim,max31785a'] > ['microchip,emc1413'] > ['mps,mp5023'] > ['nuvoton,w83795g'] > ['nvt210'] > ['nxp,pca9551'] > ['nxp,pca9552'] > ['nxp,pca9641'] > ['pmbus'] > ['pmbus-fan'] > ['si,si7021a20'] > ['ti,ucd90160'] ^ permalink raw reply [flat|nested] 18+ messages in thread
end of thread, other threads:[~2024-08-01 18:50 UTC | newest] Thread overview: 18+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2024-07-31 21:47 [PATCH v1 0/1] ARM: dts: aspeed: system1: IBM System1 BMC update Ninad Palsule 2024-07-31 21:47 ` Ninad Palsule 2024-07-31 21:47 ` [PATCH v1 1/1] " Ninad Palsule 2024-07-31 21:47 ` Ninad Palsule 2024-07-31 21:57 ` Eddie James 2024-07-31 21:57 ` Eddie James 2024-08-01 14:32 ` Ninad Palsule 2024-08-01 1:37 ` Andrew Jeffery 2024-08-01 14:33 ` Ninad Palsule 2024-08-01 1:30 ` [PATCH v1 0/1] " Andrew Jeffery 2024-08-01 15:29 ` Rob Herring 2024-08-01 15:29 ` Rob Herring (Arm) 2024-08-01 16:24 ` Ninad Palsule 2024-08-01 16:24 ` Ninad Palsule 2024-08-01 17:36 ` Rob Herring 2024-08-01 17:36 ` Rob Herring 2024-08-01 18:50 ` Ninad Palsule 2024-08-01 18:50 ` Ninad Palsule
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.