]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
pinctrl: meson: fix pin input property for a4
authorXianwei Zhao <xianwei.zhao@amlogic.com>
Mon, 3 Mar 2025 07:53:20 +0000 (15:53 +0800)
committerLinus Walleij <linus.walleij@linaro.org>
Tue, 4 Mar 2025 08:37:33 +0000 (09:37 +0100)
The register of pin input attribute means the opposite.
value of reigster meaning is 1 for input and 0 for output.
So fix it.

Fixes: 6e9be3abb78c ("pinctrl: Add driver support for Amlogic SoCs")
Signed-off-by: Xianwei Zhao <xianwei.zhao@amlogic.com>
Link: https://lore.kernel.org/20250303-fix-a4-pinctl-v1-1-6579888b4272@amlogic.com
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/meson/pinctrl-amlogic-a4.c

index a5218e8f1e034afaacf05a944ffa78291efd03d3..35d5540b8eaa2378197d7791ef2470ca6591f693 100644 (file)
@@ -775,7 +775,7 @@ static int aml_gpio_get_direction(struct gpio_chip *chip, unsigned int gpio)
        if (ret)
                return ret;
 
-       return BIT(bit) & val ? GPIO_LINE_DIRECTION_OUT : GPIO_LINE_DIRECTION_IN;
+       return BIT(bit) & val ? GPIO_LINE_DIRECTION_IN : GPIO_LINE_DIRECTION_OUT;
 }
 
 static int aml_gpio_direction_input(struct gpio_chip *chip, unsigned int gpio)
@@ -785,7 +785,7 @@ static int aml_gpio_direction_input(struct gpio_chip *chip, unsigned int gpio)
 
        aml_gpio_calc_reg_and_bit(bank, AML_REG_DIR, gpio, &reg, &bit);
 
-       return regmap_update_bits(bank->reg_gpio, reg, BIT(bit), 0);
+       return regmap_update_bits(bank->reg_gpio, reg, BIT(bit), BIT(bit));
 }
 
 static int aml_gpio_direction_output(struct gpio_chip *chip, unsigned int gpio,
@@ -796,7 +796,7 @@ static int aml_gpio_direction_output(struct gpio_chip *chip, unsigned int gpio,
        int ret;
 
        aml_gpio_calc_reg_and_bit(bank, AML_REG_DIR, gpio, &reg, &bit);
-       ret = regmap_update_bits(bank->reg_gpio, reg, BIT(bit), BIT(bit));
+       ret = regmap_update_bits(bank->reg_gpio, reg, BIT(bit), 0);
        if (ret < 0)
                return ret;