From: Guenter Roeck <guenter.roeck@ericsson.com>
To: Jean Delvare <khali@linux-fr.org>
Cc: Jonathan Cameron <kernel@jic23.retrosnub.co.uk>,
Randy Dunlap <rdunlap@xenotime.net>,
Greg Schnorr <gschnorr@cisco.com>, <lm-sensors@lm-sensors.org>,
<linux-doc@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
Guenter Roeck <guenter.roeck@ericsson.com>
Subject: [PATCH v4 0/5] hwmon: PMBus device driver
Date: Thu, 17 Feb 2011 11:00:12 -0800 [thread overview]
Message-ID: <1297969217-9564-1-git-send-email-guenter.roeck@ericsson.com> (raw)
This patchset adds support for hardware monitoring features of PMBus devices.
---
I did not get significant feedback from the previous submission, so here it is
again with extended audience. Only change since v3 is a minor fix in Kconfig.
I understand this is a large patch set. Question is if it would make sense
to create its own subdirectory for pmbus devices, and to split pmbus.c and
pmbus_core.c into more files. I'd be more than happy to do that if it helps
for the review process.
There is still some work to be done. Fan support is only rudimentary,
and the PHASE command is still not supported. However, the driver
is now stable enough to be useful, so it is worth going ahead.
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).
next reply other threads:[~2011-02-17 19:02 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-02-17 19:00 Guenter Roeck [this message]
2011-02-17 19:00 ` [PATCH v4 1/5] " Guenter Roeck
2011-02-25 20:23 ` Jonathan Cameron
2011-02-26 2:45 ` Guenter Roeck
2011-02-26 10:41 ` Jonathan Cameron
2011-02-26 15:19 ` Guenter Roeck
2011-02-17 19:00 ` [PATCH v4 2/5] hwmon: (pmbus) Add support for Maxim MAX8688 Guenter Roeck
2011-02-25 20:24 ` Jonathan Cameron
2011-02-25 21:46 ` Guenter Roeck
2011-02-17 19:00 ` [PATCH v4 3/5] hwmon: (pmbus) Add support for Maxim MAX16064 Guenter Roeck
2011-02-25 20:26 ` Jonathan Cameron
2011-02-25 21:42 ` Guenter Roeck
2011-02-17 19:00 ` [PATCH v4 4/5] hwmon: (pmbus) Add support for Maxim MAX34440/MAX34441 Guenter Roeck
2011-02-25 20:42 ` Jonathan Cameron
2011-02-25 21:32 ` Guenter Roeck
2011-02-26 10:43 ` Jonathan Cameron
2011-02-26 15:21 ` Guenter Roeck
2011-02-17 19:00 ` [PATCH v4 5/5] hwmon: pmbus driver documentation Guenter Roeck
2011-02-25 20:45 ` Jonathan Cameron
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1297969217-9564-1-git-send-email-guenter.roeck@ericsson.com \
--to=guenter.roeck@ericsson.com \
--cc=gschnorr@cisco.com \
--cc=kernel@jic23.retrosnub.co.uk \
--cc=khali@linux-fr.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lm-sensors@lm-sensors.org \
--cc=rdunlap@xenotime.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Powered by JetHome