[lm-sensors] [PATCH v2 3/4] hwmon: (f75375s) Properly map the F75387 automatic modes to pwm_enable
Nikolaus Schulz
mail at microschulz.de
Fri Mar 2 22:26:54 CET 2012
On Fri, Mar 02, 2012 at 12:02:44PM -0800, Guenter Roeck wrote:
> On Fri, 2012-03-02 at 14:47 -0500, Nikolaus Schulz wrote:
> > > That is why I suggested to split the patch into two parts. I don't mind
> > > a fix for pwm_mode==3, limiting its scope. My concern is that pwm_mode=4
> > > adds functionality which I would prefer to delay until 3.4.
> >
> > Well, as I have said, this functionality may already be activated by the
> > BIOS. The driver has to deal with it, right? It's not so much a matter
> > of supporting something, but a matter of not messing it up.
> >
> Hmm - good point. I'll accept that. I'll add your reasoning to the patch
> description and apply it to 3.3.
Thank you. And yes, adding an explicit note about that to the commit
message is a good idea. I should have been more clear there.
Sorry for bugging you some more, do you consider
331255d35d6f517020485aee38dbb8b8dfaa1642 "hwmon: (f75375s) Fix writes to
the pwm* attribute for the F75387" for v3.3? I think it should go in.
Thanks,
Nikolaus
More information about the lm-sensors
mailing list