[PATCH] Initial driver for the MAX31785 intelligent fan controller

2016-09-18 Thread Timothy Pearson
Add a basic driver for the MAX31785, focusing on the fan control features but ignoring the temperature and voltage monitoring features of the device. This driver supports all fan control modes and tachometer / PWM readback where applicable. Signed-off-by: Timothy Pearson --- Documentation

Re: [PATCH] Initial driver for the MAX31785 intelligent fan controller

2016-09-19 Thread Timothy Pearson
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 09/18/2016 08:41 PM, Guenter Roeck wrote: > On Sun, Sep 18, 2016 at 07:50:55PM -0500, Timothy Pearson wrote: >> Add a basic driver for the MAX31785, focusing on the fan control >> features but ignoring the temperature and vol

Re: [PATCH] Initial driver for the MAX31785 intelligent fan controller

2016-09-19 Thread Timothy Pearson
y enable outputs that are not connector or may possibly be incorrectly connected depending on hardware design. - -- Timothy Pearson Raptor Engineering +1 (415) 727-8645 (direct line) +1 (512) 690-0200 (switchboard) https://www.raptorengineering.com -BEGIN PGP SIGNATURE- Version: GnuPG v1

Re: [PATCH] Initial driver for the MAX31785 intelligent fan controller

2016-09-19 Thread Timothy Pearson
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 09/19/2016 03:04 PM, Guenter Roeck wrote: > On Mon, Sep 19, 2016 at 01:40:36PM -0500, Timothy Pearson wrote: > Not sure I understand. pwm[1-6]_enable = 0 means no fan speed control. > Do you mean the (full speed) comment ? "no fan

[PATCH V2] Initial driver for the MAX31785 intelligent fan controller

2016-09-19 Thread Timothy Pearson
Add a basic driver for the MAX31785, focusing on the fan control features but ignoring the temperature and voltage monitoring features of the device. This driver supports all fan control modes and tachometer / PWM readback where applicable. Signed-off-by: Timothy Pearson --- Documentation

Re: [PATCH] Initial driver for the MAX31785 intelligent fan controller

2016-09-19 Thread Timothy Pearson
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 09/19/2016 07:54 PM, Guenter Roeck wrote: > On Mon, Sep 19, 2016 at 03:31:33PM -0500, Timothy Pearson wrote: >> On 09/19/2016 03:04 PM, Guenter Roeck wrote: > And then you are using all pmbus commands ? Seems odd. > > I guess I&#

Re: [PATCH] Initial driver for the MAX31785 intelligent fan controller

2016-09-20 Thread Timothy Pearson
Bus drivers do not support configuring the fan control parameters, only monitoring the fan status and speed. Is this correct, and if not where would I find the correct documentation? Thank you, - -- Timothy Pearson Raptor Engineering +1 (415) 727-8645 (direct line) +1 (512) 690-0200 (switchbo

Re: [PATCH] Initial driver for the MAX31785 intelligent fan controller

2016-09-20 Thread Timothy Pearson
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 09/20/2016 02:59 PM, Guenter Roeck wrote: > On Tue, Sep 20, 2016 at 02:41:56PM -0500, Timothy Pearson wrote: >> On 09/19/2016 07:54 PM, Guenter Roeck wrote: >>>> On 09/19/2016 03:04 PM, Guenter Roeck wrote: >>>>

Re: [PATCH] Initial driver for the MAX31785 intelligent fan controller

2016-09-20 Thread Timothy Pearson
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 09/20/2016 10:04 PM, Guenter Roeck wrote: > On 09/20/2016 01:01 PM, Timothy Pearson wrote: >> -BEGIN PGP SIGNED MESSAGE- >> Hash: SHA1 >> >> On 09/20/2016 02:59 PM, Guenter Roeck wrote: >> OK, sounds good

Re: [PATCH] Initial driver for the MAX31785 intelligent fan controller

2016-10-11 Thread Timothy Pearson
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 09/20/2016 10:04 PM, Guenter Roeck wrote: > On 09/20/2016 01:01 PM, Timothy Pearson wrote: >> On 09/20/2016 02:59 PM, Guenter Roeck wrote: >> >> OK, sounds good. I looked back in the notes for this project and we had >>