staging: frontier: use better DEVICE_ATTR_ macros
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 24 Aug 2013 17:30:28 +0000 (10:30 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 24 Aug 2013 17:30:28 +0000 (10:30 -0700)
Use the DEVICE_ATTR_RO/RW macros to better describe the permissions on
the file, and make them easier to audit.

Cc: David Täht <d@teklibre.com>
Cc: Sara Bird <sara.bird.iar@gmail.com>
Cc: Masanari Iida <standby24x7@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/frontier/tranzport.c

index 6cbf9c7..074b0e5 100644 (file)
@@ -177,24 +177,24 @@ static void usb_tranzport_abort_transfers(struct usb_tranzport *dev)
 }
 
 #define show_int(value)        \
-       static ssize_t show_##value(struct device *dev, \
+       static ssize_t value##_show(struct device *dev, \
                              struct device_attribute *attr, char *buf) \
        {       \
                struct usb_interface *intf = to_usb_interface(dev);     \
                struct usb_tranzport *t = usb_get_intfdata(intf);       \
                return sprintf(buf, "%d\n", t->value);  \
        }       \
-       static DEVICE_ATTR(value, S_IRUGO, show_##value, NULL);
+       static DEVICE_ATTR_RO(value)
 
 #define show_set_int(value)    \
-       static ssize_t show_##value(struct device *dev, \
+       static ssize_t value##_show(struct device *dev, \
                              struct device_attribute *attr, char *buf) \
        {       \
                struct usb_interface *intf = to_usb_interface(dev);     \
                struct usb_tranzport *t = usb_get_intfdata(intf);       \
                return sprintf(buf, "%d\n", t->value);  \
        }       \
-       static ssize_t set_##value(struct device *dev,  \
+       static ssize_t value##_store(struct device *dev,        \
                             struct device_attribute *attr,             \
                             const char *buf, size_t count)             \
        {       \
@@ -206,7 +206,7 @@ static void usb_tranzport_abort_transfers(struct usb_tranzport *dev)
                t->value = temp;        \
                return count;   \
        }       \
-       static DEVICE_ATTR(value, S_IWUSR | S_IRUGO, show_##value, set_##value);
+       static DEVICE_ATTR_RW(value)
 
 show_int(enable);
 show_int(offline);