RE: [PATCH 2/2] platform/mellanox: nvsw-sn2201: Fix bus number in adapter error message

From: Vadim Pasternak
Date: Wed Jun 25 2025 - 14:13:11 EST




> -----Original Message-----
> From: Alok Tiwari <alok.a.tiwari@xxxxxxxxxx>
> Sent: Sunday, 22 June 2025 10:29
> To: hansg@xxxxxxxxxx; ilpo.jarvinen@xxxxxxxxxxxxxxx; David Thompson
> <davthompson@xxxxxxxxxx>; Michael Shych <michaelsh@xxxxxxxxxx>;
> Vadim Pasternak <vadimp@xxxxxxxxxx>; platform-driver-x86@xxxxxxxxxxxxxxx
> Cc: alok.a.tiwari@xxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx
> Subject: [PATCH 2/2] platform/mellanox: nvsw-sn2201: Fix bus number in
> adapter error message
>
> change error log to use correct bus number from main_mux_devs instead of
> cpld_devs.
>
> Signed-off-by: Alok Tiwari <alok.a.tiwari@xxxxxxxxxx>

Reviewed-by: Vadim Pasternak <vadimp@xxxxxxxxxx>

> ---
> drivers/platform/mellanox/nvsw-sn2201.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/platform/mellanox/nvsw-sn2201.c
> b/drivers/platform/mellanox/nvsw-sn2201.c
> index db31c8bf22553..51504113c17ea 100644
> --- a/drivers/platform/mellanox/nvsw-sn2201.c
> +++ b/drivers/platform/mellanox/nvsw-sn2201.c
> @@ -1181,7 +1181,7 @@ static int
> nvsw_sn2201_i2c_completion_notify(void *handle, int id)
> if (!nvsw_sn2201->main_mux_devs->adapter) {
> err = -ENODEV;
> dev_err(nvsw_sn2201->dev, "Failed to get adapter for bus
> %d\n",
> - nvsw_sn2201->cpld_devs->nr);
> + nvsw_sn2201->main_mux_devs->nr);
> goto i2c_get_adapter_main_fail;
> }
>
> --
> 2.46.0