[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