gpio: stm32: correct the bias management
authorPatrick Delaunay <patrick.delaunay@st.com>
Wed, 28 Oct 2020 09:49:08 +0000 (10:49 +0100)
committerPatrick Delaunay <patrick.delaunay@st.com>
Wed, 25 Nov 2020 11:01:46 +0000 (12:01 +0100)
Use the bias configuration for all the GPIO configurations and not
only for input GPIO, as indicated in Reference manual
(Table 81. Port bit configuration table).

Fixes: 43efbb6a3ebf0223f9eab8d45916f602d876319f ("gpio: stm32: add ops get_dir_flags")
Fixes: f13ff88b61c32ac8f0e9068c41328b265ef619eb ("gpio: stm32: add ops set_dir_flags")
Signed-off-by: Patrick Delaunay <patrick.delaunay@st.com>
Reviewed-by: Patrice Chotard <patrice.chotard@st.com>
drivers/gpio/stm32_gpio.c

index b885cfb..51e1efd 100644 (file)
@@ -212,11 +212,11 @@ static int stm32_gpio_set_dir_flags(struct udevice *dev, unsigned int offset,
 
        } else if (flags & GPIOD_IS_IN) {
                stm32_gpio_set_moder(regs, idx, STM32_GPIO_MODE_IN);
-               if (flags & GPIOD_PULL_UP)
-                       stm32_gpio_set_pupd(regs, idx, STM32_GPIO_PUPD_UP);
-               else if (flags & GPIOD_PULL_DOWN)
-                       stm32_gpio_set_pupd(regs, idx, STM32_GPIO_PUPD_DOWN);
        }
+       if (flags & GPIOD_PULL_UP)
+               stm32_gpio_set_pupd(regs, idx, STM32_GPIO_PUPD_UP);
+       else if (flags & GPIOD_PULL_DOWN)
+               stm32_gpio_set_pupd(regs, idx, STM32_GPIO_PUPD_DOWN);
 
        return 0;
 }
@@ -243,16 +243,16 @@ static int stm32_gpio_get_dir_flags(struct udevice *dev, unsigned int offset,
                break;
        case STM32_GPIO_MODE_IN:
                dir_flags |= GPIOD_IS_IN;
-               switch (stm32_gpio_get_pupd(regs, idx)) {
-               case STM32_GPIO_PUPD_UP:
-                       dir_flags |= GPIOD_PULL_UP;
-                       break;
-               case STM32_GPIO_PUPD_DOWN:
-                       dir_flags |= GPIOD_PULL_DOWN;
-                       break;
-               default:
-                       break;
-               }
+               break;
+       default:
+               break;
+       }
+       switch (stm32_gpio_get_pupd(regs, idx)) {
+       case STM32_GPIO_PUPD_UP:
+               dir_flags |= GPIOD_PULL_UP;
+               break;
+       case STM32_GPIO_PUPD_DOWN:
+               dir_flags |= GPIOD_PULL_DOWN;
                break;
        default:
                break;