Re: [lm-sensors 2/2] hwmon: (ds1621) Add ds1721 update interval sysfs attribute

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On Thu, Jun 6, 2013 at 4:41 AM, Guenter Roeck <linux@xxxxxxxxxxxx> wrote:
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.
>
I'll take care of it.

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

[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux