[PATCH] rockchip: derive GPIO bank from alias if available

Kever Yang kever.yang at rock-chips.com
Wed Feb 22 09:41:22 CET 2023


Hi John,

     I have pick below patch for gpio bank support, which is the same 
logic in kernel and other SoCs.

https://patchwork.ozlabs.org/project/uboot/patch/20230213222742.135093-2-macroalpha82@gmail.com/


Thanks,

- Kever

On 2023/1/18 02:15, John Keeping wrote:
> Upstream device trees now use standard node names like "gpio at ff..." but
> the rk_gpio driver expects a name like "gpio0 at ff..." (note the index
> before the @).
>
> This is not a change that can be made in a -u-boot.dtsi file, so
> updating to the latest upstream device trees requires updating the
> driver.
>
> Other GPIO drivers already use the sequence number to derive the bank.
> This can be set explicitly using an alias in the device tree (which can
> be added in a -u-boot.dtsi file) but Rockchip GPIO banks are defined in
> order in the device tree anyway so when no aliases are supplied the
> indexes should still be correct.
>
> Note that the bank name is only used by the `gpio` command or
> potentially by board code via dm_gpio_lookup_name() and related
> functions; it is not used by driver code which will lookup GPIOs from
> the device tree by phandle.  No Rockchip platforms make any use of
> dm_gpio_lookup_name() or `gpio` in the boot scripts.
>
> Cc: Quentin Schulz <quentin.schulz at theobroma-systems.com>
> Cc: Johan Jonker <jbx6244 at gmail.com>
> Signed-off-by: John Keeping <john at metanate.com>
> ---
>   drivers/gpio/rk_gpio.c | 6 ++----
>   1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpio/rk_gpio.c b/drivers/gpio/rk_gpio.c
> index 68f30157a9..8568f10cd0 100644
> --- a/drivers/gpio/rk_gpio.c
> +++ b/drivers/gpio/rk_gpio.c
> @@ -142,7 +142,6 @@ static int rockchip_gpio_probe(struct udevice *dev)
>   {
>   	struct gpio_dev_priv *uc_priv = dev_get_uclass_priv(dev);
>   	struct rockchip_gpio_priv *priv = dev_get_priv(dev);
> -	char *end;
>   	int ret;
>   
>   	priv->regs = dev_read_addr_ptr(dev);
> @@ -151,9 +150,8 @@ static int rockchip_gpio_probe(struct udevice *dev)
>   		return ret;
>   
>   	uc_priv->gpio_count = ROCKCHIP_GPIOS_PER_BANK;
> -	end = strrchr(dev->name, '@');
> -	priv->bank = trailing_strtoln(dev->name, end);
> -	priv->name[0] = 'A' + priv->bank;
> +
> +	priv->name[0] = 'A' + dev_seq(dev);
>   	uc_priv->bank_name = priv->name;
>   
>   	return 0;


More information about the U-Boot mailing list