scsi: ufs: renesas: Fix private allocation

shimoday · martinkpetersen · commit b6d128f89a85 · 2023-08-07T21:34:08.000-04:00
Should use devm_kzalloc() for struct ufs_renesas_priv because the .initialized should be false as default. Fixes: d695202 ("scsi: ufs: ufs-renesas: Add support for Renesas R-Car UFS controller") Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com> Link: https://lore.kernel.org/r/20230803081812.1446282-1-yoshihiro.shimoda.uh@renesas.com Reviewed-by: Bart Van Assche <bvanassche@acm.org> Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
diff --git a/drivers/ufs/host/ufs-renesas.c b/drivers/ufs/host/ufs-renesas.c
@@ -359,7 +359,7 @@ static int ufs_renesas_init(struct ufs_hba *hba)
 {
 	struct ufs_renesas_priv *priv;
 
-	priv = devm_kmalloc(hba->dev, sizeof(*priv), GFP_KERNEL);
+	priv = devm_kzalloc(hba->dev, sizeof(*priv), GFP_KERNEL);
 	if (!priv)
 		return -ENOMEM;
 	ufshcd_set_variant(hba, priv);

Original file line number	Diff line number	Diff line change
`@@ -359,7 +359,7 @@ static int ufs_renesas_init(struct ufs_hba *hba)`
`359`	`359`	`{`
`360`	`360`	`struct ufs_renesas_priv *priv;`
`361`	`361`
`362`		`- priv = devm_kmalloc(hba->dev, sizeof(*priv), GFP_KERNEL);`
	`362`	`+ priv = devm_kzalloc(hba->dev, sizeof(*priv), GFP_KERNEL);`
`363`	`363`	`if (!priv)`
`364`	`364`	`return -ENOMEM;`
`365`	`365`	`ufshcd_set_variant(hba, priv);`