On Wed, Aug 04, 2010 at 01:36:22PM +0200, Hans de Goede wrote: > On 08/01/2010 03:30 PM, Giel van Schijndel wrote: >> Allow device probing to recognise the Fintek F71808E. >> >> Sysfs interface: >> * Fan/pwm control is the same as for F71889FG > > My datasheet strongly disagrees with this ... I just noticed this patch was applied to mainline anyway. Regardless, I will (try to) address these issues you raised. Right now however, I am prioritising personal stuff above this driver---bachelor's thesis and graduation presentation. When finished with that (september) I'll allocate time to these issues again. PS Those datasheets are written very poorly, although I have seen worse. And as a reader I tend to become more "lazy" when I discover the writer was "lazy" (not exactly by choice, more out of habit). Unfortunately that doesn't work very well as reading style with technical documentation, so this probably explains some of the errors in this current patch. PPS I do have a patch ready that addresses some of the issues you raised. Those are only the cosmetic changes though (e.g. change naming of chip, comment updates, etc.). Would you suggest me to send it right now already or wait until I have time to address the other issues as well? -- Met vriendelijke groet, With kind regards, Giel van Schijndel
Attachment:
signature.asc
Description: Digital signature