On Thu, Sep 17, 2009 at 03:34:50PM +0200, Jean Delvare wrote: > Hi Rodolfo, > > On Fri, 11 Sep 2009 17:07:05 +0200, Rodolfo Giometti wrote: > > The file has been splitted up into two parts: > > Spelling: split (it's an irregular verb.) > > > > > * drivers/mfd/w83627hf-core.c - detects the chip and define proper > > platform devices into mfd support > > > > * drivers/hwmon/w83627hf.c - implements the driver for hwmon > > functionality only > > > > The patch also fixes up some non reentrant code and some C-style issues. > > Sounds wrong. Mixing coding style cleanups with real changes makes > reviewing much harder. You'll have to move these changes to a separate > patch I'm afraid. Ok, I'll repropose a patch. I prefer propose a patch with real changes only, is that correct even if checkpatch.pl says no? :) Thanks, Rodolfo -- GNU/Linux Solutions e-mail: giometti@xxxxxxxxxxxx Linux Device Driver giometti@xxxxxxxx Embedded Systems phone: +39 349 2432127 UNIX programming skype: rodolfo.giometti Freelance ICT Italia - Consulente ICT Italia - www.consulenti-ict.it
Attachment:
signature.asc
Description: Digital signature
_______________________________________________ lm-sensors mailing list lm-sensors@xxxxxxxxxxxxxx http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
