Skip to content

Commit

Permalink
USB: typec: Re-use DEFINE_SHOW_ATTRIBUTE() macro
Browse files Browse the repository at this point in the history
...instead of open coding file operations followed by custom ->open()
callbacks per each attribute.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
Reviewed-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
  • Loading branch information
Andy Shevchenko authored and Greg Kroah-Hartman committed Mar 9, 2018
1 parent e23500d commit 9e902c5
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 28 deletions.
17 changes: 3 additions & 14 deletions drivers/usb/typec/fusb302/fusb302.c
Original file line number Diff line number Diff line change
Expand Up @@ -199,7 +199,7 @@ static void fusb302_log(struct fusb302_chip *chip, const char *fmt, ...)
va_end(args);
}

static int fusb302_seq_show(struct seq_file *s, void *v)
static int fusb302_debug_show(struct seq_file *s, void *v)
{
struct fusb302_chip *chip = (struct fusb302_chip *)s->private;
int tail;
Expand All @@ -216,18 +216,7 @@ static int fusb302_seq_show(struct seq_file *s, void *v)

return 0;
}

static int fusb302_debug_open(struct inode *inode, struct file *file)
{
return single_open(file, fusb302_seq_show, inode->i_private);
}

static const struct file_operations fusb302_debug_operations = {
.open = fusb302_debug_open,
.llseek = seq_lseek,
.read = seq_read,
.release = single_release,
};
DEFINE_SHOW_ATTRIBUTE(fusb302_debug);

static struct dentry *rootdir;

Expand All @@ -242,7 +231,7 @@ static int fusb302_debugfs_init(struct fusb302_chip *chip)

chip->dentry = debugfs_create_file(dev_name(chip->dev),
S_IFREG | 0444, rootdir,
chip, &fusb302_debug_operations);
chip, &fusb302_debug_fops);

return 0;
}
Expand Down
17 changes: 3 additions & 14 deletions drivers/usb/typec/tcpm.c
Original file line number Diff line number Diff line change
Expand Up @@ -506,7 +506,7 @@ static void tcpm_log_source_caps(struct tcpm_port *port)
}
}

static int tcpm_seq_show(struct seq_file *s, void *v)
static int tcpm_debug_show(struct seq_file *s, void *v)
{
struct tcpm_port *port = (struct tcpm_port *)s->private;
int tail;
Expand All @@ -523,18 +523,7 @@ static int tcpm_seq_show(struct seq_file *s, void *v)

return 0;
}

static int tcpm_debug_open(struct inode *inode, struct file *file)
{
return single_open(file, tcpm_seq_show, inode->i_private);
}

static const struct file_operations tcpm_debug_operations = {
.open = tcpm_debug_open,
.llseek = seq_lseek,
.read = seq_read,
.release = single_release,
};
DEFINE_SHOW_ATTRIBUTE(tcpm_debug);

static struct dentry *rootdir;

Expand All @@ -550,7 +539,7 @@ static int tcpm_debugfs_init(struct tcpm_port *port)

port->dentry = debugfs_create_file(dev_name(port->dev),
S_IFREG | 0444, rootdir,
port, &tcpm_debug_operations);
port, &tcpm_debug_fops);

return 0;
}
Expand Down

0 comments on commit 9e902c5

Please sign in to comment.