In "struct mt6370_priv", the 'reg_cfgs' field is unused.
Moreover the "struct reg_cfg" is defined nowhere. Neither in this file, nor
in a global .h file, so it is completely pointless.
Remove it.
Found with cppcheck, unusedStructMember.
So, remove it.
Signed-off-by: Christophe JAILLET <[email protected]>
---
Compile tested only.
---
drivers/leds/rgb/leds-mt6370-rgb.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/leds/rgb/leds-mt6370-rgb.c b/drivers/leds/rgb/leds-mt6370-rgb.c
index 448d0da11848..359ef00498b4 100644
--- a/drivers/leds/rgb/leds-mt6370-rgb.c
+++ b/drivers/leds/rgb/leds-mt6370-rgb.c
@@ -149,7 +149,6 @@ struct mt6370_priv {
struct regmap_field *fields[F_MAX_FIELDS];
const struct reg_field *reg_fields;
const struct linear_range *ranges;
- struct reg_cfg *reg_cfgs;
const struct mt6370_pdata *pdata;
unsigned int leds_count;
unsigned int leds_active;
--
2.44.0
Il 28/04/24 20:27, Christophe JAILLET ha scritto:
> In "struct mt6370_priv", the 'reg_cfgs' field is unused.
>
> Moreover the "struct reg_cfg" is defined nowhere. Neither in this file, nor
> in a global .h file, so it is completely pointless.
>
> Remove it.
Sure
>
> Found with cppcheck, unusedStructMember.
>
> So, remove it.
Again?! :-P
>
> Signed-off-by: Christophe JAILLET <[email protected]>
Anyway, this commit misses a Fixes tag; please add the relevant one, after which,
Reviewed-by: AngeloGioacchino Del Regno <[email protected]>
> ---
> Compile tested only.
> ---
> drivers/leds/rgb/leds-mt6370-rgb.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/leds/rgb/leds-mt6370-rgb.c b/drivers/leds/rgb/leds-mt6370-rgb.c
> index 448d0da11848..359ef00498b4 100644
> --- a/drivers/leds/rgb/leds-mt6370-rgb.c
> +++ b/drivers/leds/rgb/leds-mt6370-rgb.c
> @@ -149,7 +149,6 @@ struct mt6370_priv {
> struct regmap_field *fields[F_MAX_FIELDS];
> const struct reg_field *reg_fields;
> const struct linear_range *ranges;
> - struct reg_cfg *reg_cfgs;
> const struct mt6370_pdata *pdata;
> unsigned int leds_count;
> unsigned int leds_active;
Le 29/04/2024 à 10:13, AngeloGioacchino Del Regno a écrit :
> Il 28/04/24 20:27, Christophe JAILLET ha scritto:
>> In "struct mt6370_priv", the 'reg_cfgs' field is unused.
>>
>> Moreover the "struct reg_cfg" is defined nowhere. Neither in this
>> file, nor
>> in a global .h file, so it is completely pointless.
>>
>> Remove it.
>
> Sure
>
>>
>> Found with cppcheck, unusedStructMember.
>>
>> So, remove it.
>
> Again?! :-P
Yes. This way we safe twice the memory space :).
>
>>
>> Signed-off-by: Christophe JAILLET <[email protected]>
>
>
> Anyway, this commit misses a Fixes tag; please add the relevant one,
> after which,
Will do, but this patch does not fix anything.
It is just a clean-up.
CJ
>
> Reviewed-by: AngeloGioacchino Del Regno
> <[email protected]>
On Mon, 29 Apr 2024, Christophe JAILLET wrote:
> Le 29/04/2024 à 10:13, AngeloGioacchino Del Regno a écrit :
> > Il 28/04/24 20:27, Christophe JAILLET ha scritto:
> > > In "struct mt6370_priv", the 'reg_cfgs' field is unused.
> > >
> > > Moreover the "struct reg_cfg" is defined nowhere. Neither in this
> > > file, nor
> > > in a global .h file, so it is completely pointless.
> > >
> > > Remove it.
> >
> > Sure
> >
> > >
> > > Found with cppcheck, unusedStructMember.
> > >
> > > So, remove it.
> >
> > Again?! :-P
>
> Yes. This way we safe twice the memory space :).
I changed all of the commit messages in this disjointed set to flow a
little better. No need for a Fixes tag either I feel.
--
Lee Jones [李琼斯]
On Sun, 28 Apr 2024 20:27:31 +0200, Christophe JAILLET wrote:
> In "struct mt6370_priv", the 'reg_cfgs' field is unused.
>
> Moreover the "struct reg_cfg" is defined nowhere. Neither in this file, nor
> in a global .h file, so it is completely pointless.
>
> Remove it.
>
> [...]
Applied, thanks!
[1/1] leds: mt6370: Remove an unused field in struct mt6370_priv
commit: f2994f5341e03b8680a88abc5f1dee950033c3a9
--
Lee Jones [李琼斯]