EDAC/versalnet: Return the correct error in mc_probe()

Dan Carpenter · bp3tk0v · commit c2fcb2e79d6f · 2025-09-18T14:29:17.000+02:00
Return -ENOMEM if memory allocation in mc_probe() fails. [ bp: Massage commit message. ] Fixes: d5fe2fe ("EDAC: Add a driver for the AMD Versal NET DDR controller") Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org> Signed-off-by: Borislav Petkov (AMD) <bp@alien8.de>
diff --git a/drivers/edac/versalnet_edac.c b/drivers/edac/versalnet_edac.c
@@ -888,8 +888,10 @@ static int mc_probe(struct platform_device *pdev)
 	}
 
 	priv = devm_kzalloc(&pdev->dev, sizeof(*priv), GFP_KERNEL);
-	if (!priv)
+	if (!priv) {
+		rc = -ENOMEM;
 		goto err_alloc;
+	}
 
 	amd_rpmsg_id_table[0].driver_data = (kernel_ulong_t)priv;
 

Original file line number	Diff line number	Diff line change
`@@ -888,8 +888,10 @@ static int mc_probe(struct platform_device *pdev)`
`888`	`888`	`}`
`889`	`889`
`890`	`890`	`priv = devm_kzalloc(&pdev->dev, sizeof(*priv), GFP_KERNEL);`
`891`		`- if (!priv)`
	`891`	`+ if (!priv) {`
	`892`	`+ rc = -ENOMEM;`
`892`	`893`	`goto err_alloc;`
	`894`	`+ }`
`893`	`895`
`894`	`896`	`amd_rpmsg_id_table[0].driver_data = (kernel_ulong_t)priv;`
`895`	`897`