Skip to content

Commit

Permalink
hso: Use static attribute groups for sysfs entry
Browse files Browse the repository at this point in the history
Pass the static attribute groups and the driver data via
tty_port_register_device_attr() instead of manual device_create_file()
and device_remove_file() calls.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Takashi Iwai authored and David S. Miller committed Feb 5, 2015
1 parent b0ebfae commit 4134069
Showing 1 changed file with 10 additions and 7 deletions.
17 changes: 10 additions & 7 deletions drivers/net/usb/hso.c
Original file line number Diff line number Diff line change
Expand Up @@ -531,6 +531,13 @@ static ssize_t hso_sysfs_show_porttype(struct device *dev,
}
static DEVICE_ATTR(hsotype, S_IRUGO, hso_sysfs_show_porttype, NULL);

static struct attribute *hso_serial_dev_attrs[] = {
&dev_attr_hsotype.attr,
NULL
};

ATTRIBUTE_GROUPS(hso_serial_dev);

static int hso_urb_to_index(struct hso_serial *serial, struct urb *urb)
{
int idx;
Expand Down Expand Up @@ -2236,9 +2243,6 @@ static int hso_stop_serial_device(struct hso_device *hso_dev)

static void hso_serial_tty_unregister(struct hso_serial *serial)
{
if (serial->parent->dev)
device_remove_file(serial->parent->dev, &dev_attr_hsotype);

tty_unregister_device(tty_drv, serial->minor);
}

Expand Down Expand Up @@ -2274,11 +2278,10 @@ static int hso_serial_common_create(struct hso_serial *serial, int num_urbs,
goto exit;

/* register our minor number */
serial->parent->dev = tty_port_register_device(&serial->port, tty_drv,
minor, &serial->parent->interface->dev);
serial->parent->dev = tty_port_register_device_attr(&serial->port,
tty_drv, minor, &serial->parent->interface->dev,
serial->parent, hso_serial_dev_groups);
dev = serial->parent->dev;
dev_set_drvdata(dev, serial->parent);
i = device_create_file(dev, &dev_attr_hsotype);

/* fill in specific data for later use */
serial->minor = minor;
Expand Down

0 comments on commit 4134069

Please sign in to comment.