pinctrl: core: Delete an error message for a failed memory allocation in pinctrl_register_map()

Message ID 31e88aa1-83b1-586b-447b-94ffc122567f@users.sourceforge.net
State New
Headers show

Commit Message

SF Markus Elfring Aug. 26, 2017, 6:20 p.m.
From: Markus Elfring <elfring@users.sourceforge.net>
Date: Sat, 26 Aug 2017 20:15:21 +0200

Omit an extra message for a memory allocation failure in this function.

This issue was detected by using the Coccinelle software.

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
---
 drivers/pinctrl/core.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Linus Walleij Aug. 31, 2017, 1:44 p.m. | #1
On Sat, Aug 26, 2017 at 8:20 PM, SF Markus Elfring
<elfring@users.sourceforge.net> wrote:

> From: Markus Elfring <elfring@users.sourceforge.net>
> Date: Sat, 26 Aug 2017 20:15:21 +0200
>
> Omit an extra message for a memory allocation failure in this function.
>
> This issue was detected by using the Coccinelle software.
>
> Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>

Patch applied.

Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch

diff --git a/drivers/pinctrl/core.c b/drivers/pinctrl/core.c
index c5e2c5705058..350f8962831e 100644
--- a/drivers/pinctrl/core.c
+++ b/drivers/pinctrl/core.c
@@ -1380,7 +1380,6 @@  int pinctrl_register_map(struct pinctrl_map const *maps, unsigned num_maps,
 		maps_node->maps = kmemdup(maps, sizeof(*maps) * num_maps,
 					  GFP_KERNEL);
 		if (!maps_node->maps) {
-			pr_err("failed to duplicate mapping table\n");
 			kfree(maps_node);
 			return -ENOMEM;
 		}