Re: [PATCH v2 3/4] hwmon: (f75375s) Properly map the F75387 automatic modes to pwm_enable

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 2012-03-02 at 16:26 -0500, Nikolaus Schulz wrote:
> 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.
> 
Yes.

Guenter



_______________________________________________
lm-sensors mailing list
lm-sensors@xxxxxxxxxxxxxx
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors


[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux