diff mbox series

clk: tegra: Change to use DEFINE_SHOW_ATTRIBUTE macro

Message ID 20181123150104.27336-1-tiny.windzz@gmail.com
State Accepted
Headers show
Series clk: tegra: Change to use DEFINE_SHOW_ATTRIBUTE macro | expand

Commit Message

Frank Lee Nov. 23, 2018, 3:01 p.m. UTC
Use macro to simplify the code.

Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
---
 drivers/clk/tegra/clk-dfll.c | 12 +-----------
 1 file changed, 1 insertion(+), 11 deletions(-)

Comments

Thierry Reding Nov. 28, 2018, 9:15 a.m. UTC | #1
On Fri, Nov 23, 2018 at 10:01:04AM -0500, Yangtao Li wrote:
> Use macro to simplify the code.
> 
> Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
> ---
>  drivers/clk/tegra/clk-dfll.c | 12 +-----------
>  1 file changed, 1 insertion(+), 11 deletions(-)

Acked-by: Thierry Reding <treding@nvidia.com>
Stephen Boyd Nov. 28, 2018, 10:08 p.m. UTC | #2
Quoting Yangtao Li (2018-11-23 07:01:04)
> Use macro to simplify the code.
> 
> Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
> ---

Applied to clk-next
diff mbox series

Patch

diff --git a/drivers/clk/tegra/clk-dfll.c b/drivers/clk/tegra/clk-dfll.c
index ebb0e1b6bf01..609e363dabf8 100644
--- a/drivers/clk/tegra/clk-dfll.c
+++ b/drivers/clk/tegra/clk-dfll.c
@@ -1184,17 +1184,7 @@  static int attr_registers_show(struct seq_file *s, void *data)
 	return 0;
 }
 
-static int attr_registers_open(struct inode *inode, struct file *file)
-{
-	return single_open(file, attr_registers_show, inode->i_private);
-}
-
-static const struct file_operations attr_registers_fops = {
-	.open		= attr_registers_open,
-	.read		= seq_read,
-	.llseek		= seq_lseek,
-	.release	= single_release,
-};
+DEFINE_SHOW_ATTRIBUTE(attr_registers);
 
 static void dfll_debug_init(struct tegra_dfll *td)
 {