Hi Robert, hi Guenter,
Sorry for joining the game a little late...
On Wed, 8 May 2013 22:45:54 -0700, Robert Coulson wrote:
> +static umode_t ds1621_attribute_visible(struct kobject *kobj,
> + struct attribute *attr, int index)
> +{
> + struct device *dev = container_of(kobj, struct device, kobj);
> + struct i2c_client *client = to_i2c_client(dev);
> + struct ds1621_data *data = i2c_get_clientdata(client);
> + struct device_attribute *devattr;
> +
> + devattr = container_of(attr, struct device_attribute, attr);
> + if (attr == &dev_attr_update_interval.attr)
> + if (data->kind != ds1721)
> + /* shhh, we're hiding update_interval */
> + return 0;
> + return attr->mode;
> +}
"make W=1" complains that devattr is defined but never used. And this is
true, and should be fixed.
Thanks,
--
Jean Delvare
_______________________________________________
lm-sensors mailing list
lm-sensors@xxxxxxxxxxxxxx
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
