Hi Yang, On Wed, Feb 24, 2021 at 04:43:26PM +0800, Yang Li wrote: > Fix the following coccicheck warning: > ./drivers/input/touchscreen/edt-ft5x06.c:697:0-23: WARNING: > debugfs_mode_fops should be defined with DEFINE_DEBUGFS_ATTRIBUTE
Thank you for the patch, however reading the coccinelle script not only we need to change the attribute definition, but also use debugfs_create_file_unsafe() to register it. I do not see the 2nd part of the change in your patch... > > Reported-by: Abaci Robot <ab...@linux.alibaba.com> > Signed-off-by: Yang Li <yang....@linux.alibaba.com> > --- > drivers/input/touchscreen/edt-ft5x06.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/input/touchscreen/edt-ft5x06.c > b/drivers/input/touchscreen/edt-ft5x06.c > index 2eefbc2..12bbc58 100644 > --- a/drivers/input/touchscreen/edt-ft5x06.c > +++ b/drivers/input/touchscreen/edt-ft5x06.c > @@ -694,7 +694,7 @@ static int edt_ft5x06_debugfs_mode_set(void *data, u64 > mode) > return retval; > }; > > -DEFINE_SIMPLE_ATTRIBUTE(debugfs_mode_fops, edt_ft5x06_debugfs_mode_get, > +DEFINE_DEBUGFS_ATTRIBUTE(debugfs_mode_fops, edt_ft5x06_debugfs_mode_get, > edt_ft5x06_debugfs_mode_set, "%llu\n"); > > static ssize_t edt_ft5x06_debugfs_raw_data_read(struct file *file, > -- > 1.8.3.1 > Thanks. -- Dmitry
