On Thu, Jun 6, 2013 at 4:41 AM, Guenter Roeck <linux@xxxxxxxxxxxx> wrote:
I'll take care of it.On Thu, Jun 06, 2013 at 09:48:11AM +0200, Jean Delvare wrote:
> 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 = ""> > > + 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.
>
Guenter
Thank you for catching it Jean, I appreciate it.
My apologies for missing it Guenter; Now that I am back, I can take care of it this morning.
Rob
_______________________________________________ lm-sensors mailing list lm-sensors@xxxxxxxxxxxxxx http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
