hollalinux
/
linux-sg2042
mirror of https://github.com/sophgo/linux-riscv.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

staging: usbip: use DEVICE_ATTR_RO/RW for sysfs attributes 

It's easier to audit the sysfs files if they use the RO/RW macros, so
convert the usbip code to them.

Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Greg Kroah-Hartman 2013-08-26 12:02:54 -07:00
parent 7322b320d6
commit b1f56acaca
3 changed files with 11 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
drivers/staging/usbip/stub_dev.c
View File

@ -56,8 +56,8 @@ MODULE_DEVICE_TABLE(usb, stub_table);
* usbip_status shows the status of usbip-host as long as this driver is bound * usbip_status shows the status of usbip-host as long as this driver is bound
* to the target device. * to the target device.
*/ */
static ssize_t show_status(struct device *dev, struct device_attribute *attr, static ssize_t usbip_status_show(struct device *dev,
char *buf) struct device_attribute *attr, char *buf)
{ {
struct stub_device *sdev = dev_get_drvdata(dev); struct stub_device *sdev = dev_get_drvdata(dev);
int status; int status;
@ -73,7 +73,7 @@ static ssize_t show_status(struct device *dev, struct device_attribute *attr,
return snprintf(buf, PAGE_SIZE, "%d\n", status); return snprintf(buf, PAGE_SIZE, "%d\n", status);
} }
static DEVICE_ATTR(usbip_status, S_IRUGO, show_status, NULL); static DEVICE_ATTR_RO(usbip_status);
/* /*
* usbip_sockfd gets a socket descriptor of an established TCP connection that * usbip_sockfd gets a socket descriptor of an established TCP connection that

11
drivers/staging/usbip/usbip_common.c
View File

@ -45,19 +45,20 @@ MODULE_PARM_DESC(usbip_debug_flag, "debug flags (defined in usbip_common.h)");
struct device_attribute dev_attr_usbip_debug; struct device_attribute dev_attr_usbip_debug;
EXPORT_SYMBOL_GPL(dev_attr_usbip_debug); EXPORT_SYMBOL_GPL(dev_attr_usbip_debug);
static ssize_t show_flag(struct device *dev, struct device_attribute *attr, static ssize_t usbip_debug_show(struct device *dev,
char *buf) struct device_attribute *attr, char *buf)
{ {
return sprintf(buf, "%lx\n", usbip_debug_flag); return sprintf(buf, "%lx\n", usbip_debug_flag);
} }
static ssize_t store_flag(struct device *dev, struct device_attribute *attr, static ssize_t usbip_debug_store(struct device *dev,
const char *buf, size_t count) struct device_attribute *attr, const char *buf,
size_t count)
{ {
sscanf(buf, "%lx", &usbip_debug_flag); sscanf(buf, "%lx", &usbip_debug_flag);
return count; return count;
} }
DEVICE_ATTR(usbip_debug, (S_IRUGO | S_IWUSR), show_flag, store_flag); DEVICE_ATTR_RW(usbip_debug);
static void usbip_dump_buffer(char *buff, int bufflen) static void usbip_dump_buffer(char *buff, int bufflen)
{ {

4
drivers/staging/usbip/vhci_sysfs.c
View File

@ -27,7 +27,7 @@
/* TODO: refine locking ?*/ /* TODO: refine locking ?*/
/* Sysfs entry to show port status */ /* Sysfs entry to show port status */
static ssize_t show_status(struct device *dev, struct device_attribute *attr, static ssize_t status_show(struct device *dev, struct device_attribute *attr,
char *out) char *out)
{ {
char *s = out; char *s = out;
@ -74,7 +74,7 @@ static ssize_t show_status(struct device *dev, struct device_attribute *attr,
return out - s; return out - s;
} }
static DEVICE_ATTR(status, S_IRUGO, show_status, NULL); static DEVICE_ATTR_RO(status);
/* Sysfs entry to shutdown a virtual connection */ /* Sysfs entry to shutdown a virtual connection */
static int vhci_port_disconnect(__u32 rhport) static int vhci_port_disconnect(__u32 rhport)