linux-i2c.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH/RFC v2 0/4] hwmon: PMBus device driver
@ 2010-07-05  4:10 Guenter Roeck
       [not found] ` <1278303018-22069-1-git-send-email-guenter.roeck-IzeFyvvaP7pWk0Htik3J/w@public.gmane.org>
  0 siblings, 1 reply; 8+ messages in thread
From: Guenter Roeck @ 2010-07-05  4:10 UTC (permalink / raw)
  To: Jean Delvare, Jonathan Cameron, Randy Dunlap, Andrew Morton
  Cc: lm-sensors-GZX6beZjE8VD60Wz+7aTrA,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	linux-i2c-u79uwXL29TY76Z2rM5mHXA, Guenter Roeck

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

The driver has not yet been tested with actual hardware. It is submitted for
early review and to solicit test coverage.

A PMBus device emulator is provided for reference, to show the level of testing
performed. I don't currently plan to submit this emulator in the final patchset.
If there is interest to have it included, and if someone is willing to review
the code, I will be happy to clean it up and include it.

I will test the driver with real HW for BMR453 and LTC2978 as soon as I get
access to it. For other devices, I will add a remark that support is untested,
unless I get feedback that a specific device was tested and is working. 

---
Open issues/questions:
- The driver adds new power monitoring attributes to the hwmon sysfs API.
  Need to get feedback if those are acceptable.
- Currently no support for chips with multiple phases (PHASE command).
  Adding this will require access to a system with a chip supporting
  multiple phases (e.g., UCD9240/46/48) to determine a feasible implementation.
- Need to add support for COEFFICIENTS command. Will require system with
  real chip for testing, or will not add support at this time.
- Need to add fan support. Will require system with real chip for testing, or
  will not add support at this time.
  Datasheets of chips with fan control support indicate that chip specific
  code may be necessary. See UCD9240 or UCD90124 datasheets.

---
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:[~2010-07-06  7:21 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-07-05  4:10 [PATCH/RFC v2 0/4] hwmon: PMBus device driver Guenter Roeck
     [not found] ` <1278303018-22069-1-git-send-email-guenter.roeck-IzeFyvvaP7pWk0Htik3J/w@public.gmane.org>
2010-07-05  4:10   ` [PATCH/RFC v2 1/4] " Guenter Roeck
2010-07-05  4:10   ` [PATCH/RFC v2 2/4] hwmon: i2c PMBus device emulator Guenter Roeck
2010-07-05  4:10   ` [PATCH/RFC v2 3/4] hwmon: pmbus driver documentation Guenter Roeck
2010-07-05  4:10   ` [PATCH/RFC v2 4/4] hwmon: sysfs API updates Guenter Roeck
     [not found]     ` <1278303018-22069-5-git-send-email-guenter.roeck-IzeFyvvaP7pWk0Htik3J/w@public.gmane.org>
2010-07-05  7:18       ` Jean Delvare
     [not found]         ` <20100705091857.77ce0077-ig7AzVSIIG7kN2dkZ6Wm7A@public.gmane.org>
2010-07-05 21:39           ` Guenter Roeck
     [not found]             ` <20100705213953.GA27095-IzeFyvvaP7pWk0Htik3J/w@public.gmane.org>
2010-07-06  7:21               ` Jean Delvare

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).