On Sun, 21 Jan 2018 13:14:53 +0530
Venkat Prashanth B U <venkat.prashanth2498@xxxxxxxxx> wrote:
> On Sun, Jan 14, 2018 at 4:26 PM, Jonathan Cameron <jic23@xxxxxxxxxx> wrote:
>
> > On Mon, 8 Jan 2018 11:11:17 +0530
> > venkat.prashanth2498@xxxxxxxxx wrote:
> >
> > > From: Venkat Prashanth B U <venkat.prashanth2498@xxxxxxxxx>
> > >
> > > Use DEFINE_DEBUGFS_ATTRIBUTE rather than DEFINE_SIMPLE_ATTRIBUTE
> > > for debugfs files.
> > >
> > > Semantic patch information:
> > > Rationale: DEFINE_SIMPLE_ATTRIBUTE + debugfs_create_file()
> > > imposes some significant overhead as compared to
> > > DEFINE_DEBUGFS_ATTRIBUTE + debugfs_create_file_unsafe().
> > >
> > > Generated by: scripts/coccinelle/api/debugfs/debugfs_simple_attr.cocci
> > >
> > > Signed-off-by: Venkat Prashanth B U <venkat.prashanth2498@xxxxxxxxx>
> >
> > Patch should have been [PATCH v2] If you don't do that it is far
> > too easy for me to accidentally pick up out of date versions.
> >
> > Anyhow, applied to the togreg branch of iio.git and pushed out as
> > testing for the autobuilders to play with it.
> >
>
> Thank you. But I cannot find it in aforementioned branches.
> Is it publicly pushed?
>
> Regards,
> Venkat
Sorry, got distracted and forgot to push out.
Should be there now.
Jonathan
>
>
> > Thanks,
> >
> > Jonathan
> > > ---
> > > Change Log v1:-
> > > - Fixed indentation issue.
> > > - changed the subject line appropriately.
> > > ---
> > > drivers/iio/imu/adis16400_core.c | 13 +++++++------
> > > 1 file changed, 7 insertions(+), 6 deletions(-)
> > >
> > > diff --git a/drivers/iio/imu/adis16400_core.c
> > b/drivers/iio/imu/adis16400_core.c
> > > index 46a5690..b199c73 100644
> > > --- a/drivers/iio/imu/adis16400_core.c
> > > +++ b/drivers/iio/imu/adis16400_core.c
> > > @@ -85,7 +85,7 @@ static int adis16400_show_product_id(void *arg, u64
> > *val)
> > >
> > > return 0;
> > > }
> > > -DEFINE_SIMPLE_ATTRIBUTE(adis16400_product_id_fops,
> > > +DEFINE_DEBUGFS_ATTRIBUTE(adis16400_product_id_fops,
> > > adis16400_show_product_id, NULL, "%lld\n");
> > >
> > > static int adis16400_show_flash_count(void *arg, u64 *val)
> > > @@ -102,7 +102,7 @@ static int adis16400_show_flash_count(void *arg,
> > u64 *val)
> > >
> > > return 0;
> > > }
> > > -DEFINE_SIMPLE_ATTRIBUTE(adis16400_flash_count_fops,
> > > +DEFINE_DEBUGFS_ATTRIBUTE(adis16400_flash_count_fops,
> > > adis16400_show_flash_count, NULL, "%lld\n");
> > >
> > > static int adis16400_debugfs_init(struct iio_dev *indio_dev)
> > > @@ -110,15 +110,16 @@ static int adis16400_debugfs_init(struct iio_dev
> > *indio_dev)
> > > struct adis16400_state *st = iio_priv(indio_dev);
> > >
> > > if (st->variant->flags & ADIS16400_HAS_SERIAL_NUMBER)
> > > - debugfs_create_file("serial_number", 0400,
> > > + debugfs_create_file_unsafe("serial_number", 0400,
> > > indio_dev->debugfs_dentry, st,
> > > &adis16400_serial_number_fops);
> > > if (st->variant->flags & ADIS16400_HAS_PROD_ID)
> > > - debugfs_create_file("product_id", 0400,
> > > + debugfs_create_file_unsafe("product_id", 0400,
> > > indio_dev->debugfs_dentry, st,
> > > &adis16400_product_id_fops);
> > > - debugfs_create_file("flash_count", 0400,
> > indio_dev->debugfs_dentry,
> > > - st, &adis16400_flash_count_fops);
> > > + debugfs_create_file_unsafe("flash_count", 0400,
> > > + indio_dev->debugfs_dentry, st,
> > > + &adis16400_flash_count_fops);
> > >
> > > return 0;
> > > }
> > > --
> > > 1.9.1
> > > :wq
> > >
> >
> >
--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
