pinctrl: tz1090-pdc: Delete an error message for a failed memory allocation in two...
authorMarkus Elfring <elfring@users.sourceforge.net>
Mon, 25 Dec 2017 17:28:02 +0000 (18:28 +0100)
committerLinus Walleij <linus.walleij@linaro.org>
Wed, 3 Jan 2018 07:46:52 +0000 (08:46 +0100)
Omit an extra message for a memory allocation failure in these functions.

This issue was detected by using the Coccinelle software.

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/pinctrl-tz1090-pdc.c

index 5cfa93cecf73fc1fd1eefe110a9e3382da8857c4..b16d1c96b7eba04a41cf725cbe3f1e7a414e7df0 100644 (file)
@@ -357,10 +357,8 @@ static int add_map_configs(struct device *dev,
 
        dup_configs = kmemdup(configs, num_configs * sizeof(*dup_configs),
                              GFP_KERNEL);
-       if (!dup_configs) {
-               dev_err(dev, "kmemdup(configs) failed\n");
+       if (!dup_configs)
                return -ENOMEM;
-       }
 
        /*
         * We support both pins and pin groups, but we need to figure out which
@@ -931,10 +929,9 @@ static int tz1090_pdc_pinctrl_probe(struct platform_device *pdev)
        struct resource *res;
 
        pmx = devm_kzalloc(&pdev->dev, sizeof(*pmx), GFP_KERNEL);
-       if (!pmx) {
-               dev_err(&pdev->dev, "Can't alloc tz1090_pdc_pmx\n");
+       if (!pmx)
                return -ENOMEM;
-       }
+
        pmx->dev = &pdev->dev;
        spin_lock_init(&pmx->lock);