* [PATCH linux dev-5.8] ARM: dts: aspeed: rainier: Set MAX31785 config
@ 2020-11-24 4:49 Matthew Barth
2020-11-24 19:31 ` Eddie James
2020-11-27 4:54 ` Joel Stanley
0 siblings, 2 replies; 4+ messages in thread
From: Matthew Barth @ 2020-11-24 4:49 UTC (permalink / raw)
To: openbmc, joel
Set the MAX31785 device configuration properties
Signed-off-by: Matthew Barth <msbarth@linux.ibm.com>
---
arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts | 54 ++++++++++++++++++++
1 file changed, 54 insertions(+)
diff --git a/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts b/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
index a4b77aec5424..4692c5e8f919 100644
--- a/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
+++ b/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
@@ -732,24 +732,78 @@ 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;
+ };
+
+ fan5: fan@5 {
+ compatible = "pmbus-fan";
+ reg = <5>;
+ 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;
};
};
--
2.28.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH linux dev-5.8] ARM: dts: aspeed: rainier: Set MAX31785 config
2020-11-24 4:49 [PATCH linux dev-5.8] ARM: dts: aspeed: rainier: Set MAX31785 config Matthew Barth
@ 2020-11-24 19:31 ` Eddie James
2020-11-27 4:54 ` Joel Stanley
1 sibling, 0 replies; 4+ messages in thread
From: Eddie James @ 2020-11-24 19:31 UTC (permalink / raw)
To: Matthew Barth, openbmc, joel
On Mon, 2020-11-23 at 22:49 -0600, Matthew Barth wrote:
> Set the MAX31785 device configuration properties
Reviewed-by: Eddie James <eajames@linux.ibm.com>
>
> Signed-off-by: Matthew Barth <msbarth@linux.ibm.com>
> ---
> arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts | 54
> ++++++++++++++++++++
> 1 file changed, 54 insertions(+)
>
> diff --git a/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
> b/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
> index a4b77aec5424..4692c5e8f919 100644
> --- a/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
> +++ b/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
> @@ -732,24 +732,78 @@ 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;
> + };
> +
> + fan5: fan@5 {
> + compatible = "pmbus-fan";
> + reg = <5>;
> + 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;
> };
> };
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH linux dev-5.8] ARM: dts: aspeed: rainier: Set MAX31785 config
2020-11-24 4:49 [PATCH linux dev-5.8] ARM: dts: aspeed: rainier: Set MAX31785 config Matthew Barth
2020-11-24 19:31 ` Eddie James
@ 2020-11-27 4:54 ` Joel Stanley
2020-11-30 23:54 ` Andrew Jeffery
1 sibling, 1 reply; 4+ messages in thread
From: Joel Stanley @ 2020-11-27 4:54 UTC (permalink / raw)
To: Matthew Barth, Andrew Jeffery; +Cc: OpenBMC Maillist
On Tue, 24 Nov 2020 at 04:49, Matthew Barth <msbarth@linux.ibm.com> wrote:
>
> Set the MAX31785 device configuration properties
Andrew,
As I understand it these properties are not upstream and we have no
intention of making them so as the correct fix is to properly program
the EEPROM in the fan controller with these settings. Can you clarify
here?
>
> Signed-off-by: Matthew Barth <msbarth@linux.ibm.com>
> ---
> arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts | 54 ++++++++++++++++++++
> 1 file changed, 54 insertions(+)
>
> diff --git a/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts b/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
> index a4b77aec5424..4692c5e8f919 100644
> --- a/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
> +++ b/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
> @@ -732,24 +732,78 @@ 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;
> + };
> +
> + fan5: fan@5 {
> + compatible = "pmbus-fan";
> + reg = <5>;
> + 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;
> };
> };
>
> --
> 2.28.0
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH linux dev-5.8] ARM: dts: aspeed: rainier: Set MAX31785 config
2020-11-27 4:54 ` Joel Stanley
@ 2020-11-30 23:54 ` Andrew Jeffery
0 siblings, 0 replies; 4+ messages in thread
From: Andrew Jeffery @ 2020-11-30 23:54 UTC (permalink / raw)
To: Joel Stanley, Matthew Barth; +Cc: OpenBMC Maillist
On Fri, 27 Nov 2020, at 15:24, Joel Stanley wrote:
> On Tue, 24 Nov 2020 at 04:49, Matthew Barth <msbarth@linux.ibm.com> wrote:
> >
> > Set the MAX31785 device configuration properties
>
> Andrew,
>
> As I understand it these properties are not upstream
Correct.
> and we have no
> intention of making them so
Well, they were rejected last time I tried. I think there are some things I
could try to fix, but I'm not optimistic that the fixes will get the binding
changes across the line.
> as the correct fix is to properly program
> the EEPROM in the fan controller with these settings. Can you clarify
> here?
Right - I've been considering whether we could use a first-boot systemd service
to program the fan controller settings from userspace using i2c[sg]et.
Conceptually nice, but we may need to come up with a means to reprogram the
chip after the fan card is replaced due to a fault or some such issue.
I haven't put any effort in to prototyping the first-boot approach, so we
should probably take this patch for now.
Andrew
>
> >
> > Signed-off-by: Matthew Barth <msbarth@linux.ibm.com>
> > ---
> > arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts | 54 ++++++++++++++++++++
> > 1 file changed, 54 insertions(+)
> >
> > diff --git a/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts b/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
> > index a4b77aec5424..4692c5e8f919 100644
> > --- a/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
> > +++ b/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
> > @@ -732,24 +732,78 @@ 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;
> > + };
> > +
> > + fan5: fan@5 {
> > + compatible = "pmbus-fan";
> > + reg = <5>;
> > + 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;
> > };
> > };
> >
> > --
> > 2.28.0
> >
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-11-30 23:57 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-24 4:49 [PATCH linux dev-5.8] ARM: dts: aspeed: rainier: Set MAX31785 config Matthew Barth
2020-11-24 19:31 ` Eddie James
2020-11-27 4:54 ` Joel Stanley
2020-11-30 23:54 ` Andrew Jeffery
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).