gpu: host1x: Use DEFINE_SHOW_ATTRIBUTE to simplify debugfs code
authorLiu Shixin <liushixin2@huawei.com>
Thu, 22 Sep 2022 14:23:16 +0000 (22:23 +0800)
committerThierry Reding <treding@nvidia.com>
Fri, 25 Nov 2022 15:14:59 +0000 (16:14 +0100)
Use DEFINE_SHOW_ATTRIBUTE helper macro to simplify the debugfs code for
the status and status_all entries. No functional change.

Signed-off-by: Liu Shixin <liushixin2@huawei.com>
Signed-off-by: Thierry Reding <treding@nvidia.com>
drivers/gpu/host1x/debug.c

index 34c2e36d09e9afec39ac0f12e8fc43755155bf27..6649b04b713164a5ed490a090ef9d62233dac624 100644 (file)
@@ -140,7 +140,7 @@ static void show_all(struct host1x *m, struct output *o, bool show_fifo)
        }
 }
 
-static int host1x_debug_show_all(struct seq_file *s, void *unused)
+static int host1x_debug_all_show(struct seq_file *s, void *unused)
 {
        struct output o = {
                .fn = write_to_seqfile,
@@ -151,6 +151,7 @@ static int host1x_debug_show_all(struct seq_file *s, void *unused)
 
        return 0;
 }
+DEFINE_SHOW_ATTRIBUTE(host1x_debug_all);
 
 static int host1x_debug_show(struct seq_file *s, void *unused)
 {
@@ -163,30 +164,7 @@ static int host1x_debug_show(struct seq_file *s, void *unused)
 
        return 0;
 }
-
-static int host1x_debug_open_all(struct inode *inode, struct file *file)
-{
-       return single_open(file, host1x_debug_show_all, inode->i_private);
-}
-
-static const struct file_operations host1x_debug_all_fops = {
-       .open = host1x_debug_open_all,
-       .read = seq_read,
-       .llseek = seq_lseek,
-       .release = single_release,
-};
-
-static int host1x_debug_open(struct inode *inode, struct file *file)
-{
-       return single_open(file, host1x_debug_show, inode->i_private);
-}
-
-static const struct file_operations host1x_debug_fops = {
-       .open = host1x_debug_open,
-       .read = seq_read,
-       .llseek = seq_lseek,
-       .release = single_release,
-};
+DEFINE_SHOW_ATTRIBUTE(host1x_debug);
 
 static void host1x_debugfs_init(struct host1x *host1x)
 {