[PATCH 6/7] pinctrl: qcom: sdm845: add special pin names

[Neil Armstrong]

On 09/04/2024 20:03, Caleb Connolly wrote:
> Adjust sdm845_get_pin_name() to return the correct names for the special
> pins. This fixes a non-fatal -ENOSYS error when probing MMC.
> 
> Signed-off-by: Caleb Connolly <caleb.connolly at linaro.org>
> ---
>   drivers/pinctrl/qcom/pinctrl-sdm845.c | 13 ++++++++++++-
>   1 file changed, 12 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/pinctrl/qcom/pinctrl-sdm845.c b/drivers/pinctrl/qcom/pinctrl-sdm845.c
> index 459a4329ec80..c1e5cc01fded 100644
> --- a/drivers/pinctrl/qcom/pinctrl-sdm845.c
> +++ b/drivers/pinctrl/qcom/pinctrl-sdm845.c
> @@ -65,9 +65,20 @@ static const char *sdm845_get_function_name(struct udevice *dev,
>   
>   static const char *sdm845_get_pin_name(struct udevice *dev,
>   					unsigned int selector)
>   {
> -	snprintf(pin_name, MAX_PIN_NAME_LEN, "gpio%u", selector);
> +	static const char *special_pins_names[] = {
> +		"ufs_reset",
> +		"sdc2_clk",
> +		"sdc2_cmd",
> +		"sdc2_data",
> +	};
> +
> +	if (selector >= 150 && selector <= 154)
> +		snprintf(pin_name, MAX_PIN_NAME_LEN, special_pins_names[selector - 150]);
> +	else
> +		snprintf(pin_name, MAX_PIN_NAME_LEN, "gpio%u", selector);
> +
>   	return pin_name;
>   }
>   
>   static unsigned int sdm845_get_function_mux(__maybe_unused unsigned int pin,
> 

At some point we should add the pinconf settings for SDC and UFS, but for now it's ok!

Reviewed-by: Neil Armstrong <neil.armstrong at linaro.org>