> -----Original Message-----
> From: Hans de Goede <hdegoede@xxxxxxxxxx>
> Sent: Thursday, May 12, 2022 4:35 PM
> To: Michael Shych <michaelsh@xxxxxxxxxx>
> Cc: platform-driver-x86@xxxxxxxxxxxxxxx; Vadim Pasternak
> <vadimp@xxxxxxxxxx>
> Subject: Re: [PATCH 1/1] platform/mellanox: Remove unused variable.
>
> Hi,
>
> On 5/11/22 21:50, michaelsh@xxxxxxxxxx wrote:
> > From: Michael Shych <michaelsh@xxxxxxxxxx>
> >
> > Fix problem of set but unused variable.
> >
> > Fixes: 9e267f050444f ("platform/mellanox: Add support for new SN2201
> > system")
> > Reported-by: kernel test robot <lkp@xxxxxxxxx>
> > Signed-off-by: Michael Shych <michaelsh@xxxxxxxxxx>
>
> Thanks, note since I needed to rebase my review-hans branch for another
> issue anyways I've squashed this into the original:
>
> "platform/mellanox: Add support for new SN2201 system"
>
> commit which introduced this problem.
>
> Regards,
>
> Hans
Ack.
Thanks,
Michael.
>
>
> > ---
> > drivers/platform/mellanox/nvsw-sn2201.c | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/platform/mellanox/nvsw-sn2201.c
> > b/drivers/platform/mellanox/nvsw-sn2201.c
> > index 6165e0154424..0bcdc7c75007 100644
> > --- a/drivers/platform/mellanox/nvsw-sn2201.c
> > +++ b/drivers/platform/mellanox/nvsw-sn2201.c
> > @@ -939,7 +939,7 @@ static int nvsw_sn2201_config_post_init(struct
> nvsw_sn2201 *nvsw_sn2201)
> > struct mlxreg_hotplug_device *sn2201_dev;
> > struct i2c_adapter *adap;
> > struct device *dev;
> > - int i, j, err;
> > + int i, err;
> >
> > dev = nvsw_sn2201->dev;
> > adap = i2c_get_adapter(nvsw_sn2201->main_mux_deferred_nr);
> > @@ -952,7 +952,7 @@ static int nvsw_sn2201_config_post_init(struct
> > nvsw_sn2201 *nvsw_sn2201)
> >
> > /* Update board info. */
> > sn2201_dev = nvsw_sn2201->sn2201_devs;
> > - for (i = 0, j = 0; i < nvsw_sn2201->sn2201_devs_num; i++,
> sn2201_dev++) {
> > + for (i = 0; i < nvsw_sn2201->sn2201_devs_num; i++, sn2201_dev++) {
> > sn2201_dev->adapter = i2c_get_adapter(sn2201_dev->nr);
> > if (!sn2201_dev->adapter)
> > return -ENODEV;
