[lkml]   [2017]   [Sep]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
Subject[PATCH v3 0/3] pmbus: Expand fan support and add MAX31785 driver

These three patches lay the ground work for fan control in the pmbus core and
introduce a driver for the MAX31785 Intelligent Fan Controller that makes use
of the new control features.

Since v2[1] I've addressed Rob's comments on the bindings, integrating the thermal
support and cleaning up some warts. One of the warts was regarding the
maxim,tmp-fans property which was switched to use a list of fan phandles rather
than indexes. A corresponding change is made in the driver. I also did a
comparison with the Aspeed PWM/Tach bindings but I'm not convinced there's much
to take away from it other than those bindings are somewhat broken.

In v3 I've dropped patch 4 from v2, which was a bunch of work-arounds for bad
behaviour I observed in testing. At this stage the bad behaviour appears to be
a product of the larger hardware design, not necessarily a problem with the
Maxim chip. Guenter has performed some quick testing of a cut-down v2 driver
against the MAX31785 evaluation board and couldn't reproduce the results I was
seeing[2][3], which increases the likelyhood that it's not the Maxim chip and
the work-around patch is inappropriate.

Please review!



Andrew Jeffery (3):
dt-bindings: hwmon: pmbus: Add Maxim MAX31785 documentation
hwmon: pmbus: Add fan control support
pmbus: Add driver for Maxim MAX31785 Intelligent Fan Controller

.../devicetree/bindings/hwmon/pmbus/max31785.txt | 158 +++++
drivers/hwmon/pmbus/Kconfig | 10 +
drivers/hwmon/pmbus/Makefile | 1 +
drivers/hwmon/pmbus/max31785.c | 673 +++++++++++++++++++++
drivers/hwmon/pmbus/pmbus.h | 29 +
drivers/hwmon/pmbus/pmbus_core.c | 224 ++++++-
6 files changed, 1080 insertions(+), 15 deletions(-)
create mode 100644 Documentation/devicetree/bindings/hwmon/pmbus/max31785.txt
create mode 100644 drivers/hwmon/pmbus/max31785.c


 \ /
  Last update: 2017-09-08 06:40    [W:0.059 / U:1.888 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site