Re: [PATCH] hwmon: f71882fg: Add support for the Fintek F71808E

From: Hans de Goede
Date: Fri Aug 13 2010 - 05:57:27 EST


Hi,

On 08/10/2010 09:11 PM, Giel van Schijndel wrote:
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.


I've send a mail directly to akpm asking for this to be removed, hopefully
this will help.

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?

My goal for 2.6.36 is to pull the patch, and then we can wait til all
issues are fixed properly before re-introducing it.

Regards,

Hans
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/