2024-01-06 15:32:16

by Colin Ian King

[permalink] [raw]
Subject: [PATCH][next] iio: invensense: remove redundant initialization of variable period

The variable period is being initialized with a value that is never
read, it is being re-assigned a new value later on before it is read.
The initialization is redundant and can be removed.

Cleans up clang scan build warning:
Value stored to 'period' during its initialization is never
read [deadcode.DeadStores]

Signed-off-by: Colin Ian King <[email protected]>
---
drivers/iio/common/inv_sensors/inv_sensors_timestamp.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/iio/common/inv_sensors/inv_sensors_timestamp.c b/drivers/iio/common/inv_sensors/inv_sensors_timestamp.c
index 03823ee57f59..3b0f9598a7c7 100644
--- a/drivers/iio/common/inv_sensors/inv_sensors_timestamp.c
+++ b/drivers/iio/common/inv_sensors/inv_sensors_timestamp.c
@@ -126,7 +126,7 @@ void inv_sensors_timestamp_interrupt(struct inv_sensors_timestamp *ts,
struct inv_sensors_timestamp_interval *it;
int64_t delta, interval;
const uint32_t fifo_mult = fifo_period / ts->chip.clock_period;
- uint32_t period = ts->period;
+ uint32_t period;
bool valid = false;

if (fifo_nb == 0)
--
2.39.2



2024-01-07 16:07:56

by Jonathan Cameron

[permalink] [raw]
Subject: Re: [PATCH][next] iio: invensense: remove redundant initialization of variable period

On Sat, 6 Jan 2024 15:32:02 +0000
Colin Ian King <[email protected]> wrote:

> The variable period is being initialized with a value that is never
> read, it is being re-assigned a new value later on before it is read.
> The initialization is redundant and can be removed.
>
> Cleans up clang scan build warning:
> Value stored to 'period' during its initialization is never
> read [deadcode.DeadStores]
>
> Signed-off-by: Colin Ian King <[email protected]>
Hi Colin,

I definitely want input from someone who can test this.
There is direct use of ts->period as well as the local
variable that is indeed overwritten as you've noted.
Feels like naming needs some work and perhaps reduce the scope of
the period local variable so it's obvious it was only intended
for more local use than it currently looks like.

Thanks,

Jonathan

> ---
> drivers/iio/common/inv_sensors/inv_sensors_timestamp.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/iio/common/inv_sensors/inv_sensors_timestamp.c b/drivers/iio/common/inv_sensors/inv_sensors_timestamp.c
> index 03823ee57f59..3b0f9598a7c7 100644
> --- a/drivers/iio/common/inv_sensors/inv_sensors_timestamp.c
> +++ b/drivers/iio/common/inv_sensors/inv_sensors_timestamp.c
> @@ -126,7 +126,7 @@ void inv_sensors_timestamp_interrupt(struct inv_sensors_timestamp *ts,
> struct inv_sensors_timestamp_interval *it;
> int64_t delta, interval;
> const uint32_t fifo_mult = fifo_period / ts->chip.clock_period;
> - uint32_t period = ts->period;
> + uint32_t period;
> bool valid = false;
>
> if (fifo_nb == 0)


2024-01-07 21:28:29

by Jean-Baptiste Maneyrol

[permalink] [raw]
Subject: Re: [PATCH][next] iio: invensense: remove redundant initialization of variable period

Hello Colin & Jonathan,

this is OK for me, thanks for the patch.
The initialization is certainly coming from first development and is not needed.

Acked-by: Jean-Baptiste Maneyrol <[email protected]>

Thanks,
JB

From: Jonathan Cameron <[email protected]>
Sent: Sunday, January 7, 2024 17:07
To: Colin Ian King <[email protected]>
Cc: Lars-Peter Clausen <[email protected]>; Jean-Baptiste Maneyrol <[email protected]>; Andy Shevchenko <[email protected]>; [email protected] <[email protected]>; [email protected] <[email protected]>; [email protected] <[email protected]>
Subject: Re: [PATCH][next] iio: invensense: remove redundant initialization of variable period
 
On Sat, 6 Jan 2024 15: 32: 02 +0000 Colin Ian King <colin. i. king@ gmail. com> wrote: > The variable period is being initialized with a value that is never > read, it is being re-assigned a new value later on before it is read. >
ZjQcmQRYFpfptBannerStart
This Message Is From an External Sender
This message came from outside your organization.
 
ZjQcmQRYFpfptBannerEnd
On Sat, 6 Jan 2024 15:32:02 +0000
Colin Ian King <[email protected]> wrote:

> The variable period is being initialized with a value that is never
> read, it is being re-assigned a new value later on before it is read.
> The initialization is redundant and can be removed.
>
> Cleans up clang scan build warning:
> Value stored to 'period' during its initialization is never
> read [deadcode.DeadStores]
>
> Signed-off-by: Colin Ian King <[email protected]>
Hi Colin,

I definitely want input from someone who can test this.
There is direct use of ts->period as well as the local
variable that is indeed overwritten as you've noted.
Feels like naming needs some work and perhaps reduce the scope of
the period local variable so it's obvious it was only intended
for more local use than it currently looks like.

Thanks,

Jonathan

> ---
> drivers/iio/common/inv_sensors/inv_sensors_timestamp.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/iio/common/inv_sensors/inv_sensors_timestamp.c b/drivers/iio/common/inv_sensors/inv_sensors_timestamp.c
> index 03823ee57f59..3b0f9598a7c7 100644
> --- a/drivers/iio/common/inv_sensors/inv_sensors_timestamp.c
> +++ b/drivers/iio/common/inv_sensors/inv_sensors_timestamp.c
> @@ -126,7 +126,7 @@ void inv_sensors_timestamp_interrupt(struct inv_sensors_timestamp *ts,
> struct inv_sensors_timestamp_interval *it;
> int64_t delta, interval;
> const uint32_t fifo_mult = fifo_period / ts->chip.clock_period;
> - uint32_t period = ts->period;
> + uint32_t period;
> bool valid = false;
>
> if (fifo_nb == 0)


2024-01-13 15:11:29

by Jonathan Cameron

[permalink] [raw]
Subject: Re: [PATCH][next] iio: invensense: remove redundant initialization of variable period

On Sun, 7 Jan 2024 20:18:03 +0000
Jean-Baptiste Maneyrol <[email protected]> wrote:

> Hello Colin & Jonathan,
>
> this is OK for me, thanks for the patch.
> The initialization is certainly coming from first development and is not needed.
>
> Acked-by: Jean-Baptiste Maneyrol <[email protected]>
Applied to the togreg branch of iio.git and pushed out as testing.
I'll be rebasing that tree on rc1 once available.

Thanks,

Jonathan

>
> Thanks,
> JB
>
> From: Jonathan Cameron <[email protected]>
> Sent: Sunday, January 7, 2024 17:07
> To: Colin Ian King <[email protected]>
> Cc: Lars-Peter Clausen <[email protected]>; Jean-Baptiste Maneyrol <[email protected]>; Andy Shevchenko <[email protected]>; [email protected] <[email protected]>; [email protected] <[email protected]>; [email protected] <[email protected]>
> Subject: Re: [PATCH][next] iio: invensense: remove redundant initialization of variable period
>  
> On Sat, 6 Jan 2024 15: 32: 02 +0000 Colin Ian King <colin i. king@ gmail. com> wrote: > The variable period is being initialized with a value that is never > read, it is being re-assigned a new value later on before it is read. >
> ZjQcmQRYFpfptBannerStart
> This Message Is From an External Sender
> This message came from outside your organization.
>  
> ZjQcmQRYFpfptBannerEnd
> On Sat, 6 Jan 2024 15:32:02 +0000
> Colin Ian King <[email protected]> wrote:
>
> > The variable period is being initialized with a value that is never
> > read, it is being re-assigned a new value later on before it is read.
> > The initialization is redundant and can be removed.
> >
> > Cleans up clang scan build warning:
> > Value stored to 'period' during its initialization is never
> > read [deadcode.DeadStores]
> >
> > Signed-off-by: Colin Ian King <[email protected]>
> Hi Colin,
>
> I definitely want input from someone who can test this.
> There is direct use of ts->period as well as the local
> variable that is indeed overwritten as you've noted.
> Feels like naming needs some work and perhaps reduce the scope of
> the period local variable so it's obvious it was only intended
> for more local use than it currently looks like.
>
> Thanks,
>
> Jonathan
>
> > ---
> > drivers/iio/common/inv_sensors/inv_sensors_timestamp.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/iio/common/inv_sensors/inv_sensors_timestamp.c b/drivers/iio/common/inv_sensors/inv_sensors_timestamp.c
> > index 03823ee57f59..3b0f9598a7c7 100644
> > --- a/drivers/iio/common/inv_sensors/inv_sensors_timestamp.c
> > +++ b/drivers/iio/common/inv_sensors/inv_sensors_timestamp.c
> > @@ -126,7 +126,7 @@ void inv_sensors_timestamp_interrupt(struct inv_sensors_timestamp *ts,
> > struct inv_sensors_timestamp_interval *it;
> > int64_t delta, interval;
> > const uint32_t fifo_mult = fifo_period / ts->chip.clock_period;
> > - uint32_t period = ts->period;
> > + uint32_t period;
> > bool valid = false;
> >
> > if (fifo_nb == 0)
>
>