Message ID | 20210111140347.5754-2-patrice.chotard@foss.st.com |
---|---|
State | Not Applicable |
Delegated to: | Patrick Delaunay |
Headers | show |
Series | Fix pinmux status display with long pin's name | expand |
Hi This series is abandoned and will be replaced by a new one. Patrice On 1/11/21 3:03 PM, Patrice Chotard wrote: > MAX_PIN_NAME_LEN is set to 7 whereas stmfx pin name prefix "stmfx_gpio" > is 10 char long. So "pinmux status" output looks like: > > STM32MP> pinmux status -a > -------------------------- > stmfx@42: > stmfx_ : input > stmfx_ : input > stmfx_ : input > stmfx_ : input > stmfx_ : input > ..... > > Set MAX_PIN_NAME_LEN to 13 to get a correct pinmux command output. > > Fixes: e27e96aa804e("pinctrl: stmfx: update pin name") > > Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com> > > --- > > drivers/pinctrl/pinctrl-stmfx.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/pinctrl/pinctrl-stmfx.c b/drivers/pinctrl/pinctrl-stmfx.c > index a62be44d2d..d23ad862f2 100644 > --- a/drivers/pinctrl/pinctrl-stmfx.c > +++ b/drivers/pinctrl/pinctrl-stmfx.c > @@ -346,7 +346,7 @@ static int stmfx_pinctrl_get_pins_count(struct udevice *dev) > * STMFX pins[15:0] are called "stmfx_gpio[15:0]" > * and STMFX pins[23:16] are called "stmfx_agpio[7:0]" > */ > -#define MAX_PIN_NAME_LEN 7 > +#define MAX_PIN_NAME_LEN 13 > static char pin_name[MAX_PIN_NAME_LEN]; > static const char *stmfx_pinctrl_get_pin_name(struct udevice *dev, > unsigned int selector) >
diff --git a/drivers/pinctrl/pinctrl-stmfx.c b/drivers/pinctrl/pinctrl-stmfx.c index a62be44d2d..d23ad862f2 100644 --- a/drivers/pinctrl/pinctrl-stmfx.c +++ b/drivers/pinctrl/pinctrl-stmfx.c @@ -346,7 +346,7 @@ static int stmfx_pinctrl_get_pins_count(struct udevice *dev) * STMFX pins[15:0] are called "stmfx_gpio[15:0]" * and STMFX pins[23:16] are called "stmfx_agpio[7:0]" */ -#define MAX_PIN_NAME_LEN 7 +#define MAX_PIN_NAME_LEN 13 static char pin_name[MAX_PIN_NAME_LEN]; static const char *stmfx_pinctrl_get_pin_name(struct udevice *dev, unsigned int selector)
MAX_PIN_NAME_LEN is set to 7 whereas stmfx pin name prefix "stmfx_gpio" is 10 char long. So "pinmux status" output looks like: STM32MP> pinmux status -a -------------------------- stmfx@42: stmfx_ : input stmfx_ : input stmfx_ : input stmfx_ : input stmfx_ : input ..... Set MAX_PIN_NAME_LEN to 13 to get a correct pinmux command output. Fixes: e27e96aa804e("pinctrl: stmfx: update pin name") Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com> --- drivers/pinctrl/pinctrl-stmfx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)