Re: [PATCH 1/1] pinctrl: st: st_pinconf_dbg_show wrong format string

From: Patrice Chotard
Date: Mon Nov 07 2016 - 04:03:51 EST




On 11/05/2016 03:25 PM, Heinrich Schuchardt wrote:
> function is defined as unsigned int.
> So we need %u to print it.
>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@xxxxxx>
> ---
> drivers/pinctrl/pinctrl-st.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/pinctrl/pinctrl-st.c b/drivers/pinctrl/pinctrl-st.c
> index b7bb371..8308f15 100644
> --- a/drivers/pinctrl/pinctrl-st.c
> +++ b/drivers/pinctrl/pinctrl-st.c
> @@ -1006,7 +1006,7 @@ static void st_pinconf_dbg_show(struct pinctrl_dev *pctldev,
>
> function = st_pctl_get_pin_function(pc, offset);
> if (function)
> - snprintf(f, 10, "Alt Fn %d", function);
> + snprintf(f, 10, "Alt Fn %u", function);
> else
> snprintf(f, 5, "GPIO");
>
>


Hi Heinrich

Acked-by: Patrice Chotard <patrice.chotard@xxxxxx>

Thanks