Skip to contentSkip to conversation

staging: mt29f_spinand: Replacing pr_info with dev_info after the call to devm_kzalloc

PATCH New G Pooja Shamili 1 participant 1 message Mar 11, 2016

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH v3 1/1 staging: mt29f_spinand: Replacing pr_info with dev_info after the call to devm_kzalloc

The function devm_kzalloc has a first argument of type struct device *. This is the type of argument required by printing functions such as dev_info, dev_err, etc. Thus, functions like pr_info should not normally be used after a call to devm_kzalloc. Thus, all pr_info occurances are replaced with dev_info function calls.

This was done using Coccinelle, the patch being:
@@
expression E1,E2;
expression list args;
@@

E1 = devm_kzalloc(E2, ...);
<...
- pr_info(
+ dev_info(E2,
args);
...>

---
Changes from Version 2:
* Changed the title from pr_err to pr_info and dev_err to
dev_info
Changes from Version 1:
* Changed the name in the From: and Signed-off-by: to full
name.

 drivers/staging/mt29f_spinand/mt29f_spinand.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
drivers/staging/mt29f_spinand/mt29f_spinand.c +2 -1
index aa0170e..9d47c5d 100644
@@ -893,7 +893,8 @@ static int spinand_probe(struct spi_device *spi_nand)
 #else
 	chip->ecc.mode	= NAND_ECC_SOFT;
 	if (spinand_disable_ecc(spi_nand) < 0)
-		pr_info("%s: disable ecc failed!\n", __func__);
+		dev_info(&spi_nand->dev, "%s: disable ecc failed!\n",
+			 __func__);
 #endif
 
 	nand_set_flash_node(chip, spi_nand->dev.of_node);
Signature · 1 line
2.1.4