Commit d56fbfba authored by Chen Ni's avatar Chen Ni Committed by Hans de Goede
Browse files

platform/mellanox: nvsw-sn2201: Add check for platform_device_add_resources 



Add check for the return value of platform_device_add_resources() and
return the error if it fails in order to catch the error.

Signed-off-by: default avatarChen Ni <nichen@iscas.ac.cn>
Link: https://lore.kernel.org/r/20240605032745.2916183-1-nichen@iscas.ac.cn


Reviewed-by: default avatarHans de Goede <hdegoede@redhat.com>
Signed-off-by: default avatarHans de Goede <hdegoede@redhat.com>
parent 77f1972b

drivers/platform/mellanox/nvsw-sn2201.c

+4 −1
Original line number Diff line number Diff line
@@ -1198,6 +1198,7 @@ static int nvsw_sn2201_config_pre_init(struct nvsw_sn2201 *nvsw_sn2201) 
static int nvsw_sn2201_probe(struct platform_device *pdev)

{

	struct nvsw_sn2201 *nvsw_sn2201;

	int ret;



	nvsw_sn2201 = devm_kzalloc(&pdev->dev, sizeof(*nvsw_sn2201), GFP_KERNEL);

	if (!nvsw_sn2201)
@@ -1205,8 +1206,10 @@ static int nvsw_sn2201_probe(struct platform_device *pdev) 


	nvsw_sn2201->dev = &pdev->dev;

	platform_set_drvdata(pdev, nvsw_sn2201);

	platform_device_add_resources(pdev, nvsw_sn2201_lpc_io_resources,

	ret = platform_device_add_resources(pdev, nvsw_sn2201_lpc_io_resources,

				      ARRAY_SIZE(nvsw_sn2201_lpc_io_resources));

	if (ret)

		return ret;



	nvsw_sn2201->main_mux_deferred_nr = NVSW_SN2201_MAIN_MUX_DEFER_NR;

	nvsw_sn2201->main_mux_devs = nvsw_sn2201_main_mux_brdinfo;