[PATCH 2/2 RESEND 2] hwmon: add vt1211 documentation

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

 



> > > +different thermal input but - and here's the weird part - only one set of
> > > +thermal thresholds exist that control both PWMs output duty cycles (??).
> >
> > Why these question marks?
> 
> Just to show my confusion why they implemented it that way. I'll take
> them out...

Yeah, every chip designer seems to be proud to add a new confusion
source. My experience is that, each time a chip tries to be smart and
spares registers by sharing values amongst different channels, it ends
up being a nightmare for us to implement support, and even more for the
end user to configure the chip to do what he/she wants.My advice to
chip designers, if it matters, would be: keep things simple, don't try
to be smart.

-- 
Jean Delvare




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

  Powered by Linux