firmware: ti_sci: Change to use DEFINE_SHOW_ATTRIBUTE macro
PATCH 1/1 firmware: ti_sci: Change to use DEFINE_SHOW_ATTRIBUTE macro
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
Signed-off-by Yangtao Li <[email protected]>
--- drivers/firmware/ti_sci.c | 21 ++------------------- 1 file changed, 2 insertions(+), 19 deletions(-)
drivers/firmware/ti_sci.c +2 -19
@@ -146,25 +146,8 @@ static int ti_sci_debug_show(struct seq_file *s, void *unused)return 0 ; } -/** -* ti_sci_debug_open() - debug file open -* @inode: inode pointer -* @file: file pointer -* -* Return: result of single_open -*/ -static int ti_sci_debug_open (struct inode *inode,struct file *file) -{ -return single_open (file, ti_sci_debug_show, inode->i_private); -} - -/* log file operations */ -static const struct file_operations ti_sci_debug_fops = { - .open = ti_sci_debug_open, - .read = seq_read, - .llseek = seq_lseek, - .release = single_release, -}; +/* Provide the log file operations interface*/ +DEFINE_SHOW_ATTRIBUTE (ti_sci_debug);/** * ti_sci_debugfs_create() - Create log debug file
Signature · 1 line
2.17.0
Quoted 3 lines · Yangtao Li
Use
DEFINE_SHOW_ATTRIBUTEmacro to simplify the code.Signed-off-by Yangtao Li <[email protected]>
Thanks for the same and sorry for responding so late.
[...]
Santosh,
could you pick this up? maybe for next rev or so?
Reviewed-by Nishanth Menon <[email protected]>
Signature · 2 lines
Regards, Nishanth Menon
Quoted 5 lines · Nishanth Menon
Quoted 3 lines · Yangtao Li
Use
DEFINE_SHOW_ATTRIBUTEmacro to simplify the code.Signed-off-by Yangtao Li <[email protected]>
Quoted 9 lines
Thanks for the same and sorry for responding so late.
[...]
Santosh,
could you pick this up? maybe for next rev or so?Reviewed-by Nishanth Menon <[email protected]>
Sure !!