[PATCH v4] hwmon: Versatile Express hwmon driver

Pawel Moll pawel.moll at arm.com
Mon Sep 24 08:03:02 EDT 2012


On Fri, 2012-09-21 at 19:18 +0100, Guenter Roeck wrote:
> On Fri, Sep 21, 2012 at 05:56:31PM +0100, Pawel Moll wrote:
> > From: Pawel Moll <pawel.moll at arm.com>
> > 
> > hwmon framework driver for Versatile Express sensors, providing
> > information about board level voltage (only when regulator driver
> > is not configured), currents, temperature and power/energy usage.
> > Labels for the values can be defined as DT properties.
> > 
> > Signed-off-by: Pawel Moll <pawel.moll at arm.com>
> > ---
> >  .../devicetree/bindings/hwmon/vexpress.txt         |   23 ++
> >  Documentation/hwmon/vexpress                       |   34 +++
> >  drivers/hwmon/Kconfig                              |    8 +
> >  drivers/hwmon/Makefile                             |    1 +
> >  drivers/hwmon/vexpress.c                           |  229 ++++++++++++++++++++
> >  5 files changed, 295 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/hwmon/vexpress.txt
> >  create mode 100644 Documentation/hwmon/vexpress
> >  create mode 100644 drivers/hwmon/vexpress.c
> > 
> [ ... ]
> 
> > +
> > +	err = sysfs_create_group(&pdev->dev.kobj, match->data);
> > +	if (err)
> > +		goto error;
> 
> You'll need a second label for that. Since the group was not created, you can
> not delete it.

Actually I think I can... The sysfs_remove_group() effectively is a
wrapper for sysfs_hash_and_remove() which acts like "rm -f" - does
nothing if the file doesn't exist. Even drivers/hwmon/lm83.c (which, by
pure coincidence, I was looking at when re-working this driver) does
that in exit_remove_files...

Anyway, it's nothing to argue about, so I'll change it and send
(hopefully ;-) final version in a jiffy.

Thanks!

Paweł





More information about the linux-arm-kernel mailing list