gpio: mpc8xxx: Slightly update the code for better readability
authorAxel Lin <axel.lin@ingics.com>
Mon, 22 Feb 2016 07:24:54 +0000 (15:24 +0800)
committerLinus Walleij <linus.walleij@linaro.org>
Wed, 9 Mar 2016 03:44:13 +0000 (10:44 +0700)
Setting gc->direction_output to gc->direction_output looks strange.
I think this change makes the intention more clear.

Signed-off-by: Axel Lin <axel.lin@ingics.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/gpio/gpio-mpc8xxx.c

index bc042ad62cfb4b4272ef922b077c4b166ee4c114..425501c39527038509a0c3af752598d0c53ee049 100644 (file)
@@ -339,8 +339,11 @@ static int mpc8xxx_probe(struct platform_device *pdev)
         */
        mpc8xxx_irq_chip.irq_set_type = devtype->irq_set_type;
 
-       gc->direction_output = devtype->gpio_dir_out ?: gc->direction_output;
-       gc->get = devtype->gpio_get ?: gc->get;
+       if (devtype->gpio_dir_out)
+               gc->direction_output = devtype->gpio_dir_out;
+       if (devtype->gpio_get)
+               gc->get = devtype->gpio_get;
+
        gc->to_irq = mpc8xxx_gpio_to_irq;
 
        ret = gpiochip_add_data(gc, mpc8xxx_gc);