On Fri, 15 Jun 2012 08:24:13 -0700, Guenter Roeck wrote: > Convert to use devm_ functions to reduce code size and simplify the code. > > Signed-off-by: Guenter Roeck <linux@xxxxxxxxxxxx> > --- > drivers/hwmon/smsc47m1.c | 40 ++++++++-------------------------------- > 1 file changed, 8 insertions(+), 32 deletions(-) > > diff --git a/drivers/hwmon/smsc47m1.c b/drivers/hwmon/smsc47m1.c > index b5aa38d..b3a38af 100644 > --- a/drivers/hwmon/smsc47m1.c > +++ b/drivers/hwmon/smsc47m1.c > @@ -584,13 +584,11 @@ static void smsc47m1_restore(const struct smsc47m1_sio_data *sio_data) > > #define CHECK 1 > #define REQUEST 2 > -#define RELEASE 3 > > /* > * This function can be used to: > * - test for resource conflicts with ACPI > * - request the resources > - * - release the resources > * We only allocate the I/O ports we really need, to minimize the risk of > * conflicts with ACPI or with other drivers. > */ > @@ -642,21 +640,13 @@ static int smsc47m1_handle_resources(unsigned short address, enum chips type, This function can now be marked __init. Other than this, I have no issue with the patch. Acked-by: Jean Delvare <khali@xxxxxxxxxxxx> -- Jean Delvare _______________________________________________ lm-sensors mailing list lm-sensors@xxxxxxxxxxxxxx http://lists.lm-sensors.org/mailman/listinfo/lm-sensors