[PATCH] gpio/desginware: add Synopsys DesignWare APB GPIO support
Anup Patel
anup at brainfault.org
Fri Mar 3 04:27:14 PST 2023
On Wed, Mar 1, 2023 at 5:55 PM Ben Dooks <ben.dooks at sifive.com> wrote:
>
> Add a driver for the Synopsys DesignWare APB GPIO IP block found in many
> SoCs.
>
> Signed-off-by: Ben Dooks <ben.dooks at sifive.com>
Looks good to me.
Reviewed-by: Anup Patel <anup at brainfault.org>
Regards,
Anup
> ---
> lib/utils/gpio/Kconfig | 4 +
> lib/utils/gpio/fdt_gpio_designware.c | 140 +++++++++++++++++++++++++++
> lib/utils/gpio/objects.mk | 2 +
> platform/generic/configs/defconfig | 1 +
> 4 files changed, 147 insertions(+)
> create mode 100644 lib/utils/gpio/fdt_gpio_designware.c
>
> diff --git a/lib/utils/gpio/Kconfig b/lib/utils/gpio/Kconfig
> index 38a9d75..f229928 100644
> --- a/lib/utils/gpio/Kconfig
> +++ b/lib/utils/gpio/Kconfig
> @@ -14,6 +14,10 @@ config FDT_GPIO_SIFIVE
> bool "SiFive GPIO FDT driver"
> default n
>
> +config FDT_GPIO_DESIGNWARE
> + bool "DesignWare GPIO driver"
> + default n
> +
> endif
>
> config GPIO
> diff --git a/lib/utils/gpio/fdt_gpio_designware.c b/lib/utils/gpio/fdt_gpio_designware.c
> new file mode 100644
> index 0000000..6223d7e
> --- /dev/null
> +++ b/lib/utils/gpio/fdt_gpio_designware.c
> @@ -0,0 +1,140 @@
> +/*
> + * SPDX-License-Identifier: BSD-2-Clause
> + *
> + * Copyright (c) 2022 SiFive
> + *
> + * GPIO driver for Synopsys DesignWare APB GPIO
> + *
> + * Authors:
> + * Ben Dooks <ben.dooks at sifive.com>
> + */
> +
> +#include <libfdt.h>
> +
> +#include <sbi/riscv_io.h>
> +#include <sbi/sbi_error.h>
> +
> +#include <sbi_utils/fdt/fdt_helper.h>
> +#include <sbi_utils/gpio/fdt_gpio.h>
> +
> +#define DW_GPIO_CHIP_MAX 4 /* need 1 per bank in use */
> +#define DW_GPIO_PINS_MAX 32
> +
> +#define DW_GPIO_DDR 0x4
> +#define DW_GPIO_DR 0x0
> +#define DW_GPIO_BIT(_b) (1UL << (_b))
> +
> +struct dw_gpio_chip {
> + void *dr;
> + void *ext;
> + struct gpio_chip chip;
> +};
> +
> +extern struct fdt_gpio fdt_gpio_designware;
> +
> +static unsigned int dw_gpio_chip_count;
> +static struct dw_gpio_chip dw_gpio_chip_array[DW_GPIO_CHIP_MAX];
> +
> +#define pin_to_chip(__p) container_of((__p)->chip, struct dw_gpio_chip, chip);
> +
> +static int dw_gpio_direction_output(struct gpio_pin *gp, int value)
> +{
> + struct dw_gpio_chip *chip = pin_to_chip(gp);
> + unsigned long v;
> +
> + v = readl(chip->dr + DW_GPIO_DR);
> + if (!value)
> + v &= ~DW_GPIO_BIT(gp->offset);
> + else
> + v |= DW_GPIO_BIT(gp->offset);
> + writel(v, chip->dr + DW_GPIO_DR);
> +
> + /* the DR is output only so we can set it then the DDR to set
> + * the data direction, to avoid glitches.
> + */
> + v = readl(chip->dr + DW_GPIO_DDR);
> + v |= DW_GPIO_BIT(gp->offset);
> + writel(v, chip->dr + DW_GPIO_DDR);
> +
> + return 0;
> +}
> +
> +static void dw_gpio_set(struct gpio_pin *gp, int value)
> +{
> + struct dw_gpio_chip *chip = pin_to_chip(gp);
> + unsigned long v;
> +
> + v = readl(chip->dr + DW_GPIO_DR);
> + if (!value)
> + v &= ~DW_GPIO_BIT(gp->offset);
> + else
> + v |= DW_GPIO_BIT(gp->offset);
> + writel(v, chip->dr + DW_GPIO_DR);
> +}
> +
> +/* notes:
> + * each sub node is a bank and has ngpios or snpns,nr-gpios and a reg property
> + * with the compatible `snps,dw-apb-gpio-port`.
> + * bank A is the only one with irq support but we're not using it here
> +*/
> +
> +static int dw_gpio_init_bank(void *fdt, int nodeoff, u32 phandle,
> + const struct fdt_match *match)
> +{
> + struct dw_gpio_chip *chip;
> + const fdt32_t *val;
> + uint64_t addr;
> + int rc, poff, nr_pins, bank, len;
> +
> + if (dw_gpio_chip_count >= DW_GPIO_CHIP_MAX)
> + return SBI_ENOSPC;
> +
> + /* need to get parent for the address property */
> + poff = fdt_parent_offset(fdt, nodeoff);
> + if (poff < 0)
> + return SBI_EINVAL;
> +
> + rc = fdt_get_node_addr_size(fdt, poff, 0, &addr, NULL);
> + if (rc)
> + return rc;
> +
> + val = fdt_getprop(fdt, nodeoff, "reg", &len);
> + if (!val || len <= 0)
> + return SBI_EINVAL;
> + bank = fdt32_to_cpu(*val);
> +
> + val = fdt_getprop(fdt, nodeoff, "snps,nr-gpios", &len);
> + if (!val)
> + val = fdt_getprop(fdt, nodeoff, "ngpios", &len);
> + if (!val || len <= 0)
> + return SBI_EINVAL;
> + nr_pins = fdt32_to_cpu(*val);
> +
> + chip = &dw_gpio_chip_array[dw_gpio_chip_count];
> +
> + chip->dr = (void *)addr + (bank * 0xc);
> + chip->ext = (void *)addr + (bank * 4) + 0x50;
> + chip->chip.driver = &fdt_gpio_designware;
> + chip->chip.id = phandle;
> + chip->chip.ngpio = nr_pins;
> + chip->chip.set = dw_gpio_set;
> + chip->chip.direction_output = dw_gpio_direction_output;
> + rc = gpio_chip_add(&chip->chip);
> + if (rc)
> + return rc;
> +
> + dw_gpio_chip_count++;
> + return 0;
> +}
> +
> +/* since we're only probed when used, match on port not main controller node */
> +static const struct fdt_match dw_gpio_match[] = {
> + { .compatible = "snps,dw-apb-gpio-port" },
> + { },
> +};
> +
> +struct fdt_gpio fdt_gpio_designware = {
> + .match_table = dw_gpio_match,
> + .xlate = fdt_gpio_simple_xlate,
> + .init = dw_gpio_init_bank,
> +};
> diff --git a/lib/utils/gpio/objects.mk b/lib/utils/gpio/objects.mk
> index eedd699..00ea7af 100644
> --- a/lib/utils/gpio/objects.mk
> +++ b/lib/utils/gpio/objects.mk
> @@ -11,6 +11,8 @@ libsbiutils-objs-$(CONFIG_FDT_GPIO) += gpio/fdt_gpio.o
> libsbiutils-objs-$(CONFIG_FDT_GPIO) += gpio/fdt_gpio_drivers.o
>
> carray-fdt_gpio_drivers-$(CONFIG_FDT_GPIO_SIFIVE) += fdt_gpio_sifive
> +carray-fdt_gpio_drivers-$(CONFIG_FDT_GPIO_DESIGNWARE) += fdt_gpio_designware
> libsbiutils-objs-$(CONFIG_FDT_GPIO_SIFIVE) += gpio/fdt_gpio_sifive.o
> +libsbiutils-objs-$(CONFIG_FDT_GPIO_DESIGNWARE) += gpio/fdt_gpio_designware.o
>
> libsbiutils-objs-$(CONFIG_GPIO) += gpio/gpio.o
> diff --git a/platform/generic/configs/defconfig b/platform/generic/configs/defconfig
> index 4b0842e..759caaa 100644
> --- a/platform/generic/configs/defconfig
> +++ b/platform/generic/configs/defconfig
> @@ -5,6 +5,7 @@ CONFIG_PLATFORM_SIFIVE_FU540=y
> CONFIG_PLATFORM_SIFIVE_FU740=y
> CONFIG_PLATFORM_STARFIVE_JH7110=y
> CONFIG_FDT_GPIO=y
> +CONFIG_FDT_GPIO_DESIGNWARE=y
> CONFIG_FDT_GPIO_SIFIVE=y
> CONFIG_FDT_I2C=y
> CONFIG_FDT_I2C_SIFIVE=y
> --
> 2.39.2
>
>
> --
> opensbi mailing list
> opensbi at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/opensbi
More information about the opensbi
mailing list