From mboxrd@z Thu Jan 1 00:00:00 1970 From: Patrice CHOTARD Date: Thu, 2 Jul 2020 08:02:53 +0000 Subject: [PATCH v2 11/11] pinctrl: stm32: add information on pin configuration In-Reply-To: <20200604143022.v2.11.I080300e7ec59426020b1d175aa207029ecb5e0cb@changeid> References: <20200604123033.25499-1-patrick.delaunay@st.com> <20200604143022.v2.11.I080300e7ec59426020b1d175aa207029ecb5e0cb@changeid> Message-ID: <59146252-c3b1-2d32-00b2-5b1546f6b976@st.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Patrick On 6/4/20 2:30 PM, Patrick Delaunay wrote: > Add information on pin configuration used for pinmux command: > - bias configuration for output (disable, pull up, pull down) > - otype for input (open drain or push pull) > > Signed-off-by: Patrick Delaunay > --- > > Changes in v2: None > > drivers/pinctrl/pinctrl_stm32.c | 27 ++++++++++++++++++++++++--- > 1 file changed, 24 insertions(+), 3 deletions(-) > > diff --git a/drivers/pinctrl/pinctrl_stm32.c b/drivers/pinctrl/pinctrl_stm32.c > index fc241fdcde..71fa29a389 100644 > --- a/drivers/pinctrl/pinctrl_stm32.c > +++ b/drivers/pinctrl/pinctrl_stm32.c > @@ -45,6 +45,17 @@ static const char * const pinmux_mode[PINMUX_MODE_COUNT] = { > "alt function", > }; > > +static const char * const pinmux_output[] = { > + [STM32_GPIO_PUPD_NO] = "bias-disable", > + [STM32_GPIO_PUPD_UP] = "bias-pull-up", > + [STM32_GPIO_PUPD_DOWN] = "bias-pull-down", > +}; > + > +static const char * const pinmux_input[] = { > + [STM32_GPIO_OTYPE_PP] = "drive-push-pull", > + [STM32_GPIO_OTYPE_OD] = "drive-open-drain", > +}; > + > static int stm32_pinctrl_get_af(struct udevice *dev, unsigned int offset) > { > struct stm32_gpio_priv *priv = dev_get_priv(dev); > @@ -182,10 +193,12 @@ static int stm32_pinctrl_get_pin_muxing(struct udevice *dev, > int size) > { > struct udevice *gpio_dev; > + struct stm32_gpio_priv *priv; > const char *label; > int mode; > int af_num; > unsigned int gpio_idx; > + u32 pupd, otype; > > /* look up for the bank which owns the requested pin */ > gpio_dev = stm32_pinctrl_get_gpio_dev(dev, selector, &gpio_idx); > @@ -194,9 +207,9 @@ static int stm32_pinctrl_get_pin_muxing(struct udevice *dev, > return -ENODEV; > > mode = gpio_get_raw_function(gpio_dev, gpio_idx, &label); > - > dev_dbg(dev, "selector = %d gpio_idx = %d mode = %d\n", > selector, gpio_idx, mode); > + priv = dev_get_priv(gpio_dev); > > > switch (mode) { > @@ -211,9 +224,17 @@ static int stm32_pinctrl_get_pin_muxing(struct udevice *dev, > snprintf(buf, size, "%s %d", pinmux_mode[mode], af_num); > break; > case GPIOF_OUTPUT: > + pupd = (readl(&priv->regs->pupdr) >> (gpio_idx * 2)) & > + PUPD_MASK; > + snprintf(buf, size, "%s %s %s", > + pinmux_mode[mode], pinmux_output[pupd], > + label ? label : ""); > + break; > case GPIOF_INPUT: > - snprintf(buf, size, "%s %s", > - pinmux_mode[mode], label ? label : ""); > + otype = (readl(&priv->regs->otyper) >> gpio_idx) & OTYPE_MSK; > + snprintf(buf, size, "%s %s %s", > + pinmux_mode[mode], pinmux_input[otype], > + label ? label : ""); > break; > } > Reviewed-by: Patrice Chotard Thanks