public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH v5 0/5] hwmon: PMBus device driver
@ 2011-02-26 17:50 Guenter Roeck
  2011-02-26 17:50 ` [PATCH v5 1/5] " Guenter Roeck
                   ` (4 more replies)
  0 siblings, 5 replies; 8+ messages in thread
From: Guenter Roeck @ 2011-02-26 17:50 UTC (permalink / raw)
  To: Jean Delvare
  Cc: Jonathan Cameron, Randy Dunlap, Greg Schnorr, lm-sensors,
	linux-doc, linux-kernel, Guenter Roeck

This patchset adds support for hardware monitoring features of PMBus devices.

---
Post-commit: Check if pmbus_find_attributes() can be rewritten to use tables.
[ I looked into it, but concluded it would be too risky to do that now. So I'll
  try that later and submit as cleanup patch if it works out. ]

v5 changes:
- Added a note to driver documentation stating that fan support will be added
  later.
- max34440: Added comment describing why R=3 instead of 0. Formatting fix.
- Core driver:
  - Added function to check status register
  - Reduced number of times clear status function is called.
  - Added explanation describing when get_status and identify functions are
    needed.
  - Describe pmbus_get_boolean() functionality in more detail.
  - Declared some static arrays as const.
  - No longer export pmbus_read_byte_data(), since it is not used outside
    the core driver.
- Generic pmbus driver:
  - Removed pmbus_entry list. Replaced with API in core driver to retrieve
    pointer to struct pmbus_driver_info from client data.

v4 changes:
- Fixed PMBUS entry in Kconfig

v3 changes:
  Tested with real hardware (bmr453, bmr454, ltc2978, max8688, max16064,
  max34441).
  Changes are too numerous to count. Lots of bug fixes. Code was completely
  rearranged to permit addition of chips without having to modify the core
  pmbus driver. Split main driver into pmbus_core.c for core support and
  pmbus.c as front-end driver to support generic PMBus chips.
  Support for additional chips can now be added by writing small front-end
  drivers which basically describe chip capabilities.

v2 changes:
- Calculate maximum number of attributes / sensors / booleans / labels instead
  of using defines
- Increased maximum number of supported pages to the PMBus maximum of 32.
- Replaced most fixed size arrays with dynamically allocated arrays/structures.
- Added comments and clarifications
- Renamed local defines to start with PB_ or PMBUS_ prefix
- Removed pages module parameter. Replaced with code to auto-detect the number
  of supported pages.
- Split probe code into several functions to make it better readable
- Replaced _fault attributes with _lcrit_alarm and _crit_alarm attributes
- Several other minor changes to address review feedback
- PMBus emulator: Fixed fault status detection code. Also use more reasonable
  power limits (eg 200W instead of 2kW).

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2011-02-26 18:32 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-02-26 17:50 [PATCH v5 0/5] hwmon: PMBus device driver Guenter Roeck
2011-02-26 17:50 ` [PATCH v5 1/5] " Guenter Roeck
2011-02-26 18:25   ` Jonathan Cameron
2011-02-26 18:31     ` Guenter Roeck
2011-02-26 17:50 ` [PATCH v5 2/5] hwmon: (pmbus) Add support for Maxim MAX8688 Guenter Roeck
2011-02-26 17:50 ` [PATCH v5 3/5] hwmon: (pmbus) Add support for Maxim MAX16064 Guenter Roeck
2011-02-26 17:50 ` [PATCH v5 4/5] hwmon: (pmbus) Add support for Maxim MAX34440/MAX34441 Guenter Roeck
2011-02-26 17:50 ` [PATCH v5 5/5] hwmon: pmbus driver documentation Guenter Roeck

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox