drivers/leds/rgb/leds-mt6370-rgb.c | 1 - 1 file changed, 1 deletion(-)
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 <christophe.jaillet@wanadoo.fr>
---
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
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 [李琼斯]
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 <christophe.jaillet@wanadoo.fr> Anyway, this commit misses a Fixes tag; please add the relevant one, after which, Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> > --- > 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 <christophe.jaillet@wanadoo.fr> > > > 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 > <angelogioacchino.delregno@collabora.com>
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 [李琼斯]
© 2016 - 2024 Red Hat, Inc.