On Thu, Sep 22, 2022 at 10:25:05PM +0800, Liu Shixin wrote:
> Use DEFINE_SHOW_ATTRIBUTE helper macro to simplify the code.
> No functional change.
>
> Signed-off-by: Liu Shixin <liushixin2@xxxxxxxxxx>
Acked-by: Heikki Krogerus <heikki.krogerus@xxxxxxxxxxxxxxx>
> ---
> drivers/usb/common/ulpi.c | 20 +++-----------------
> 1 file changed, 3 insertions(+), 17 deletions(-)
>
> diff --git a/drivers/usb/common/ulpi.c b/drivers/usb/common/ulpi.c
> index 0a4f441aff8f..d7c8461976ce 100644
> --- a/drivers/usb/common/ulpi.c
> +++ b/drivers/usb/common/ulpi.c
> @@ -233,7 +233,7 @@ static int ulpi_read_id(struct ulpi *ulpi)
> return 0;
> }
>
> -static int ulpi_regs_read(struct seq_file *seq, void *data)
> +static int ulpi_regs_show(struct seq_file *seq, void *data)
> {
> struct ulpi *ulpi = seq->private;
>
> @@ -269,21 +269,7 @@ static int ulpi_regs_read(struct seq_file *seq, void *data)
>
> return 0;
> }
> -
> -static int ulpi_regs_open(struct inode *inode, struct file *f)
> -{
> - struct ulpi *ulpi = inode->i_private;
> -
> - return single_open(f, ulpi_regs_read, ulpi);
> -}
> -
> -static const struct file_operations ulpi_regs_ops = {
> - .owner = THIS_MODULE,
> - .open = ulpi_regs_open,
> - .release = single_release,
> - .read = seq_read,
> - .llseek = seq_lseek
> -};
> +DEFINE_SHOW_ATTRIBUTE(ulpi_regs);
>
> #define ULPI_ROOT debugfs_lookup(KBUILD_MODNAME, NULL)
>
> @@ -316,7 +302,7 @@ static int ulpi_register(struct device *dev, struct ulpi *ulpi)
> }
>
> root = debugfs_create_dir(dev_name(dev), ULPI_ROOT);
> - debugfs_create_file("regs", 0444, root, ulpi, &ulpi_regs_ops);
> + debugfs_create_file("regs", 0444, root, ulpi, &ulpi_regs_fops);
>
> dev_dbg(&ulpi->dev, "registered ULPI PHY: vendor %04x, product %04x\n",
> ulpi->id.vendor, ulpi->id.product);
> --
> 2.25.1
--
heikki
