From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guenter Roeck Date: Tue, 16 Nov 2010 15:03:05 +0000 Subject: Re: [lm-sensors] [PATCH] hwmon: (lm95241) Rewritten without using Message-Id: <20101116150305.GA4442@ericsson.com> List-Id: References: In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: lm-sensors@vger.kernel.org On Thu, Nov 11, 2010 at 01:18:56PM -0500, Davide Rizzo wrote: > > Hi Davide, > > > > this looks much better, and gives me a chance for a more complete > > review. Please see below. > > > > Unfortunately, I got hit with the MS Exchange server bug, meaning I can > > not apply your patch. So I might still miss something. Next time please > > copy lkml - that will give me a chance to retrieve the patch from > > patchwork.kernel.org. > > > > Thanks, > > Guenter > Hi Guenther, here is the driver again with requested modifies. > What do you mean with "copy lkml" ? > Regards > Davide > > From: Davide Rizzo > > > Rewriting of driver/hwmon/lm95241.c to avoid using macros > Now it uses SENSOR_DEVICE_ATTR to distinguish similar attr. > Signed-off-by: Davide Rizzo > > --- > --- linux-2.6.37-rc1/drivers/hwmon/lm95241.c 2010-11-01 12:54:12.000000000 +0100 > +++ linux-2.6.37-rc1.elpa/drivers/hwmon/lm95241.c 2010-11-11 19:12:28.392515642 +0100 > @@ -1,13 +1,9 @@ > /* > - * lm95241.c - Part of lm_sensors, Linux kernel modules for hardware > - * monitoring > - * Copyright (C) 2008 Davide Rizzo > > + * Copyright (C) 2008, 2010 Davide Rizzo > > * > - * Based on the max1619 driver. The LM95241 is a sensor chip made by National > - * Semiconductors. > - * It reports up to three temperatures (its own plus up to > - * two external ones). Complete datasheet can be > - * obtained from National's website at: > + * The LM95241 is a sensor chip made by National Semiconductors. > + * It reports up to three temperatures (its own plus up to two external ones). > + * Complete datasheet can be obtained from National's website at: > * http://www.national.com/ds.cgi/LM/LM95241.pdf > * > * This program is free software; you can redistribute it and/or modify > @@ -25,16 +21,13 @@ > * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. > */ > > -#include > -#include > #include > -#include > #include > #include > #include > #include > -#include > -#include Removing the above includes violates SubmitChecklist rule #1. Also, please reparent to Linus' latest tree. Other than that, there are a only few formatting issues, but I can take care of those myself. So please re-submit with the above changes, and we should be ready to go. Thanks, Guenter _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors