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:
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:
And then you are using all pmbus commands ? Seems odd.
I guess I'l buy an evaluation board if one is available and check if your claim
is correct. I am not inclined to accept a non-pmbus driver for a pmbus
device without good reason.
Guenter
Regarding the PMBus driver, I looked over the documentation available
here: https://www.kernel.org/doc/Documentation/hwmon/pmbus
From what I can tell PMBus 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?
So far that wasn't needed. That doesn't mean it can not be added.
I orderd an evaluation board and will likely spend some time on it myself
after I get it.
Guenter
OK, sounds good. I looked back in the notes for this project and we had
originally considered using a PMBus driver but ran into the kernel
documentation, noted the missing control features, and I subsequently
It is always useful to talk with the maintainer.
misinterpreted the datasheet using the kernel docs as a reference. This
is why the hwmon driver was implemented.
I am still not sure I understand what irked you off in the datasheet.
Is it the "supports a subset of the commands defined in the PMBus
Specfication" ? If so, please keep in mind that every single chip
supporting PMBus will only support a subset of PMBus commands.
Please let me know if I can be of any assistance.
Sure, I'll let you know.
Thanks,
Guenter
--
To unsubscribe from this list: send the line "unsubscribe linux-hwmon" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
