[PATCH v2 08/10] gpio: rockchip: Add request() ops

Kever Yang kever.yang at rock-chips.com
Fri Aug 9 11:38:00 CEST 2024


On 2024/8/3 06:56, Jonas Karlman wrote:
> Add a request() ops that call pinctrl_gpio_request() when the required
> gpio-ranges prop has been defined to signal pinctrl driver to use gpio
> pinmux.
>
> Signed-off-by: Jonas Karlman <jonas at kwiboo.se>
Reviewed-by: Kever Yang <kever.yang at rock-chips.com>

Thanks,
- Kever
> ---
> v2: New patch from "rockchip: Add gpio request() ops" series
> ---
>   drivers/gpio/rk_gpio.c | 10 ++++++++++
>   1 file changed, 10 insertions(+)
>
> diff --git a/drivers/gpio/rk_gpio.c b/drivers/gpio/rk_gpio.c
> index 5972f7f8612d..65811dbc78d6 100644
> --- a/drivers/gpio/rk_gpio.c
> +++ b/drivers/gpio/rk_gpio.c
> @@ -126,6 +126,15 @@ static int rockchip_gpio_get_function(struct udevice *dev, unsigned offset)
>   	return (data & mask) ? GPIOF_OUTPUT : GPIOF_INPUT;
>   }
>   
> +static int rockchip_gpio_request(struct udevice *dev, unsigned offset,
> +				 const char *label)
> +{
> +	if (CONFIG_IS_ENABLED(PINCTRL) && dev_read_bool(dev, "gpio-ranges"))
> +		return pinctrl_gpio_request(dev, offset, label);
> +
> +	return 0;
> +}
> +
>   /* Simple SPL interface to GPIOs */
>   #ifdef CONFIG_SPL_BUILD
>   
> @@ -229,6 +238,7 @@ static int rockchip_gpio_probe(struct udevice *dev)
>   }
>   
>   static const struct dm_gpio_ops gpio_rockchip_ops = {
> +	.request		= rockchip_gpio_request,
>   	.direction_input	= rockchip_gpio_direction_input,
>   	.direction_output	= rockchip_gpio_direction_output,
>   	.get_value		= rockchip_gpio_get_value,


More information about the U-Boot mailing list