2018-03-07 05:12:33

by <Hariprasath Elango>

Subject: [PATCH] staging: iio: adc: Remove reduntant __func__ from debug print

From: HariPrasath Elango <[email protected]>

dev_dbg includes the function name & line number by default when dynamic
debugging is enabled. Hence__func__ is reduntant here and removed.

Signed-off-by: HariPrasath Elango <[email protected]>
---
drivers/staging/iio/meter/ade7758_trigger.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/iio/meter/ade7758_trigger.c b/drivers/staging/iio/meter/ade7758_trigger.c
index 1f0d1a0..da489ae 100644
--- a/drivers/staging/iio/meter/ade7758_trigger.c
+++ b/drivers/staging/iio/meter/ade7758_trigger.c
@@ -34,7 +34,7 @@ static int ade7758_data_rdy_trigger_set_state(struct iio_trigger *trig,
{
struct iio_dev *indio_dev = iio_trigger_get_drvdata(trig);

- dev_dbg(&indio_dev->dev, "%s (%d)\n", __func__, state);
+ dev_dbg(&indio_dev->dev, "(%d)\n", state);
return ade7758_set_irq(&indio_dev->dev, state);
}

--
2.10.0.GIT

2018-03-07 05:18:34

by <Hariprasath Elango>

Subject: Re: [PATCH] staging: iio: adc: Remove reduntant __func__ from debug print

On Wed, Mar 07, 2018 at 10:40:05AM +0530, [email protected] wrote:
> From: HariPrasath Elango <[email protected]>
>
> dev_dbg includes the function name & line number by default when dynamic
> debugging is enabled. Hence__func__ is reduntant here and removed.
>
> Signed-off-by: HariPrasath Elango <[email protected]>
> ---
> drivers/staging/iio/meter/ade7758_trigger.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/iio/meter/ade7758_trigger.c b/drivers/staging/iio/meter/ade7758_trigger.c
> index 1f0d1a0..da489ae 100644
> --- a/drivers/staging/iio/meter/ade7758_trigger.c
> +++ b/drivers/staging/iio/meter/ade7758_trigger.c
> @@ -34,7 +34,7 @@ static int ade7758_data_rdy_trigger_set_state(struct iio_trigger *trig,
> {
> struct iio_dev *indio_dev = iio_trigger_get_drvdata(trig);
>
> - dev_dbg(&indio_dev->dev, "%s (%d)\n", __func__, state);
> + dev_dbg(&indio_dev->dev, "(%d)\n", state);
> return ade7758_set_irq(&indio_dev->dev, state);
> }
>
> --
> 2.10.0.GIT
>

Please ignore this patch as the subject line is wrong. It should be
'meter' and not 'adc.