Hi Jean, On Tue, Sep 11, 2012 at 09:56:40AM +0200, Jean Delvare wrote: > Hi Guenter, > > On Mon, 10 Sep 2012 21:09:24 -0700, Guenter Roeck wrote: > > MAX1110 is similar to MAX1111, with 8 instead of 4 channels. MAX1112 and MAX1113 > > are similar to MAX1110 and MAX1111, with 4.096V reference voltage instead of > > 2.048V. > > > > Signed-off-by: Guenter Roeck <linux@xxxxxxxxxxxx> [ ... ] > > > > @@ -93,12 +100,15 @@ EXPORT_SYMBOL(max1111_read_channel); > > static ssize_t show_name(struct device *dev, > > struct device_attribute *attr, char *buf) > > { > > - return sprintf(buf, "max1111\n"); > > + struct max1111_data *data = dev_get_drvdata(dev); > > + > > + return sprintf(buf, "%s\n", data->name); > > Can't you use dev_name(dev) instead? > Would be nice, but dev_name(dev) is the name of the SPI device, eg spi0.0. If I use dev_name, sensors output is: spi0.0-spi-0-0 Adapter: SPI adapter in0: +0.00 V in1: +0.00 V in2: +0.00 V in3: +0.00 V Other drivers all use the data->name approach as far as I can see. Thanks, Guenter _______________________________________________ lm-sensors mailing list lm-sensors@xxxxxxxxxxxxxx http://lists.lm-sensors.org/mailman/listinfo/lm-sensors