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
> ---
> 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;
