Re: [PATCH v1] pinctrl: fix signature of extract pinctrl_generic_to_map() in nop case
From: Frank Li
Date: Thu May 14 2026 - 17:08:40 EST
On Thu, May 14, 2026 at 09:01:27PM +0100, Conor Dooley wrote:
> From: Conor Dooley <conor.dooley@xxxxxxxxxxxxx>
>
> The signature of extract pinctrl_generic_to_map() is incorrect when
> CONFIG_GENERIC_PINCTRL and CONFIG_OF are not enabled. Probably a
> hangover from an earlier revision of the patchset that was not updated
> that went undetected given it depends on OF being disabled.
Thanks, I already posted fix patch at
https://lore.kernel.org/imx/20260512144806.22447-1-Frank.Li@xxxxxxx/T/#u
Frank
>
> Fixes: aaaf31be04260 ("pinctrl: extract pinctrl_generic_to_map() from pinctrl_generic_pins_function_dt_node_to_map()")
> Signed-off-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx>
> ---
> CC: Linus Walleij <linusw@xxxxxxxxxx>
> CC: Frank Li <Frank.Li@xxxxxxx>
> CC: Conor Dooley <conor.dooley@xxxxxxxxxxxxx>
> CC: linux-gpio@xxxxxxxxxxxxxxx
> CC: linux-kernel@xxxxxxxxxxxxxxx
> ---
> drivers/pinctrl/pinconf.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/pinctrl/pinconf.h b/drivers/pinctrl/pinconf.h
> index fa8fb0d290d1d..9711d16c38b62 100644
> --- a/drivers/pinctrl/pinconf.h
> +++ b/drivers/pinctrl/pinconf.h
> @@ -195,7 +195,7 @@ pinctrl_generic_to_map(struct pinctrl_dev *pctldev, struct device_node *parent,
> unsigned int *num_maps, unsigned int *num_reserved_maps,
> const char **group_name, unsigned int ngroups,
> const char **functions, unsigned int *pins,
> - void *function_data)
> + unsigned int npins)
> {
> return -ENOTSUPP;
> }
> --
> 2.53.0
>