pinctrl: a3700: Fix the issue that gpio controller is registered with wrong node id
authorKen Ma <make@marvell.com>
Thu, 22 Jun 2017 09:13:36 +0000 (17:13 +0800)
committerStefan Roese <sr@denx.de>
Fri, 23 Jun 2017 05:09:40 +0000 (07:09 +0200)
In armada_37xx_gpiochip_register, the return value of fdtdec_get_bool
should be true when gpio-controller is found; current codes makes a
wrong inverse return value judgement, this patch fixes it.

Signed-off-by: Ken Ma <make@marvell.com>
Cc: Stefan Roese <sr@denx.de>
Cc: Kostya Porotchkin <kostap@marvell.com>
Cc: Gregory CLEMENT <gregory.clement@free-electrons.com>
Cc: Nadav Haklai <nadavh@marvell.com>
Cc: Wilson Ding <dingwei@marvell.com>
Reviewed-by: Stefan Roese <sr@denx.de>
Signed-off-by: Stefan Roese <sr@denx.de>
drivers/pinctrl/mvebu/pinctrl-armada-37xx.c

index 2ac66ec325c8991917c144b8d648acc7d4ce8b3d..27165b0007739b04785976e3d25d969076436ca9 100644 (file)
@@ -539,7 +539,7 @@ static int armada_37xx_gpiochip_register(struct udevice *parent,
        }
 
        fdt_for_each_subnode(subnode, blob, node) {
-               if (!fdtdec_get_bool(blob, subnode, "gpio-controller")) {
+               if (fdtdec_get_bool(blob, subnode, "gpio-controller")) {
                        ret = 0;
                        break;
                }