[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 3/3] hw/gpio: Add in AST2600 specific impleme
From: |
Cédric Le Goater |
Subject: |
Re: [Qemu-devel] [PATCH v5 3/3] hw/gpio: Add in AST2600 specific implementation |
Date: |
Fri, 16 Aug 2019 18:01:42 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.8.0 |
On 16/08/2019 09:32, Rashmica Gupta wrote:
> The AST2600 has the same sets of 3.6v gpios as the AST2400 plus an
> addtional two sets of 1.8V gpios.
>
> Signed-off-by: Rashmica Gupta <address@hidden>
One minor comment below, but this is minor.
Reviewed-by: Cédric Le Goater <address@hidden>
Thanks,
C.
> ---
> hw/gpio/aspeed_gpio.c | 142 ++++++++++++++++++++++++++++++++++++++++--
> 1 file changed, 137 insertions(+), 5 deletions(-)
>
> diff --git a/hw/gpio/aspeed_gpio.c b/hw/gpio/aspeed_gpio.c
> index d6a4c0c757..a3306a8f32 100644
> --- a/hw/gpio/aspeed_gpio.c
> +++ b/hw/gpio/aspeed_gpio.c
> @@ -167,6 +167,48 @@
> #define GPIO_3_6V_MEM_SIZE 0x1F0
> #define GPIO_3_6V_REG_ARRAY_SIZE (GPIO_3_6V_MEM_SIZE >> 2)
>
> +/* AST2600 only - 1.8V gpios */
> +/*
> + * The AST2600 has same 3.6V gpios as the AST2400 (memory offsets 0x0-0x198)
> + * and addtional 1.8V gpios (memory offsets 0x800-0x9D4).
> + */
> +#define GPIO_1_8V_REG_OFFSET 0x800
> +#define GPIO_1_8V_ABCD_DATA_VALUE ((0x800 - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_ABCD_DIRECTION ((0x804 - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_ABCD_INT_ENABLE ((0x808 - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_ABCD_INT_SENS_0 ((0x80C - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_ABCD_INT_SENS_1 ((0x810 - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_ABCD_INT_SENS_2 ((0x814 - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_ABCD_INT_STATUS ((0x818 - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_ABCD_RESET_TOLERANT ((0x81C - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_E_DATA_VALUE ((0x820 - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_E_DIRECTION ((0x824 - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_E_INT_ENABLE ((0x828 - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_E_INT_SENS_0 ((0x82C - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_E_INT_SENS_1 ((0x830 - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_E_INT_SENS_2 ((0x834 - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_E_INT_STATUS ((0x838 - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_E_RESET_TOLERANT ((0x83C - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_ABCD_DEBOUNCE_1 ((0x840 - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_ABCD_DEBOUNCE_2 ((0x844 - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_E_DEBOUNCE_1 ((0x848 - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_E_DEBOUNCE_2 ((0x84C - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_DEBOUNCE_TIME_1 ((0x850 - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_DEBOUNCE_TIME_2 ((0x854 - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_DEBOUNCE_TIME_3 ((0x858 - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_ABCD_COMMAND_SRC_0 ((0x860 - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_ABCD_COMMAND_SRC_1 ((0x864 - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_E_COMMAND_SRC_0 ((0x868 - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_E_COMMAND_SRC_1 ((0x86C - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_ABCD_DATA_READ ((0x8C0 - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_E_DATA_READ ((0x8C4 - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_ABCD_INPUT_MASK ((0x9D0 - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_E_INPUT_MASK ((0x9D4 - GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_1_8V_MEM_SIZE 0x9D8
> +#define GPIO_1_8V_REG_ARRAY_SIZE ((GPIO_1_8V_MEM_SIZE - \
> + GPIO_1_8V_REG_OFFSET) >> 2)
> +#define GPIO_MAX_MEM_SIZE MAX(GPIO_3_6V_MEM_SIZE,
> GPIO_1_8V_MEM_SIZE)
> +
> static int aspeed_evaluate_irq(GPIOSets *regs, int gpio_prev_high, int gpio)
> {
> uint32_t falling_edge = 0, rising_edge = 0;
> @@ -461,6 +503,39 @@ static const AspeedGPIOReg
> aspeed_3_6v_gpios[GPIO_3_6V_REG_ARRAY_SIZE] = {
> [GPIO_AC_INPUT_MASK] = { 7, gpio_reg_input_mask },
> };
>
> +static const AspeedGPIOReg aspeed_1_8v_gpios[GPIO_1_8V_REG_ARRAY_SIZE] = {
> + /* 1.8V Set ABCD */
> + [GPIO_1_8V_ABCD_DATA_VALUE] = {0, gpio_reg_data_value},
> + [GPIO_1_8V_ABCD_DIRECTION] = {0, gpio_reg_direction},
> + [GPIO_1_8V_ABCD_INT_ENABLE] = {0, gpio_reg_int_enable},
> + [GPIO_1_8V_ABCD_INT_SENS_0] = {0, gpio_reg_int_sens_0},
> + [GPIO_1_8V_ABCD_INT_SENS_1] = {0, gpio_reg_int_sens_1},
> + [GPIO_1_8V_ABCD_INT_SENS_2] = {0, gpio_reg_int_sens_2},
> + [GPIO_1_8V_ABCD_INT_STATUS] = {0, gpio_reg_int_status},
> + [GPIO_1_8V_ABCD_RESET_TOLERANT] = {0, gpio_reg_reset_tolerant},
> + [GPIO_1_8V_ABCD_DEBOUNCE_1] = {0, gpio_reg_debounce_1},
> + [GPIO_1_8V_ABCD_DEBOUNCE_2] = {0, gpio_reg_debounce_2},
> + [GPIO_1_8V_ABCD_COMMAND_SRC_0] = {0, gpio_reg_cmd_source_0},
> + [GPIO_1_8V_ABCD_COMMAND_SRC_1] = {0, gpio_reg_cmd_source_1},
> + [GPIO_1_8V_ABCD_DATA_READ] = {0, gpio_reg_data_read},
> + [GPIO_1_8V_ABCD_INPUT_MASK] = {0, gpio_reg_input_mask},
> + /* 1.8V Set E */
> + [GPIO_1_8V_E_DATA_VALUE] = {1, gpio_reg_data_value},
> + [GPIO_1_8V_E_DIRECTION] = {1, gpio_reg_direction},
> + [GPIO_1_8V_E_INT_ENABLE] = {1, gpio_reg_int_enable},
> + [GPIO_1_8V_E_INT_SENS_0] = {1, gpio_reg_int_sens_0},
> + [GPIO_1_8V_E_INT_SENS_1] = {1, gpio_reg_int_sens_1},
> + [GPIO_1_8V_E_INT_SENS_2] = {1, gpio_reg_int_sens_2},
> + [GPIO_1_8V_E_INT_STATUS] = {1, gpio_reg_int_status},
> + [GPIO_1_8V_E_RESET_TOLERANT] = {1, gpio_reg_reset_tolerant},
> + [GPIO_1_8V_E_DEBOUNCE_1] = {1, gpio_reg_debounce_1},
> + [GPIO_1_8V_E_DEBOUNCE_2] = {1, gpio_reg_debounce_2},
> + [GPIO_1_8V_E_COMMAND_SRC_0] = {1, gpio_reg_cmd_source_0},
> + [GPIO_1_8V_E_COMMAND_SRC_1] = {1, gpio_reg_cmd_source_1},
> + [GPIO_1_8V_E_DATA_READ] = {1, gpio_reg_data_read},
> + [GPIO_1_8V_E_INPUT_MASK] = {1, gpio_reg_input_mask},
> +};
> +
> static uint64_t aspeed_gpio_read(void *opaque, hwaddr offset, uint32_t size)
> {
> AspeedGPIOState *s = ASPEED_GPIO(opaque);
> @@ -659,8 +734,11 @@ static void aspeed_gpio_get_pin(Object *obj, Visitor *v,
> const char *name,
> int set_idx, group_idx = 0;
>
> if (sscanf(name, "gpio%2[A-Z]%1d", group, &pin) != 2) {
> - error_setg(errp, "%s: error reading %s", __func__, name);
> - return;
> + /* 1.8V gpio */
> + if (sscanf(name, "gpio%3s%1d", group, &pin) != 2) {
> + error_setg(errp, "%s: error reading %s", __func__, name);
> + return;
> + }
We could extend the GPIO class with this pattern instead.
> }
> set_idx = get_set_idx(s, group, &group_idx);
> if (set_idx == -1) {
> @@ -683,8 +761,11 @@ static void aspeed_gpio_set_pin(Object *obj, Visitor *v,
> const char *name,
>
> visit_type_bool(v, name, &level, &local_err);
> if (sscanf(name, "gpio%2[A-Z]%1d", group, &pin) != 2) {
> - error_setg(errp, "%s: error reading %s", __func__, name);
> - return;
> + /* 1.8V gpio */
> + if (sscanf(name, "gpio%3s%1d", group, &pin) != 2) {
> + error_setg(errp, "%s: error reading %s", __func__, name);
> + return;
> + }
> }
> set_idx = get_set_idx(s, group, &group_idx);
> if (set_idx == -1) {
> @@ -716,6 +797,21 @@ static const GPIOSetProperties ast2500_set_props[] = {
> [7] = {0x000000ff, 0x000000ff, {"AC"} },
> };
>
> +static GPIOSetProperties ast2600_3_6v_set_props[] = {
> + [0] = {0xffffffff, 0xffffffff, {"A", "B", "C", "D"} },
> + [1] = {0xffffffff, 0xffffffff, {"E", "F", "G", "H"} },
> + [2] = {0xffffffff, 0xffffffff, {"I", "J", "K", "L"} },
> + [3] = {0xffffffff, 0xffffffff, {"M", "N", "O", "P"} },
> + [4] = {0xffffffff, 0xffffffff, {"Q", "R", "S", "T"} },
> + [5] = {0xffffffff, 0x0000ffff, {"U", "V", "W", "X"} },
> + [6] = {0xffff0000, 0x0fff0000, {"Y", "Z", "", ""} },
> +};
> +
> +static GPIOSetProperties ast2600_1_8v_set_props[] = {
> + [0] = {0xffffffff, 0xffffffff, {"18A", "18B", "18C", "18D"} },
> + [1] = {0x0000000f, 0x0000000f, {"18E"} },
> +};
> +
> static const MemoryRegionOps aspeed_gpio_ops = {
> .read = aspeed_gpio_read,
> .write = aspeed_gpio_write,
> @@ -748,7 +844,7 @@ static void aspeed_gpio_realize(DeviceState *dev, Error
> **errp)
> }
>
> memory_region_init_io(&s->iomem, OBJECT(s), &aspeed_gpio_ops, s,
> - TYPE_ASPEED_GPIO, GPIO_3_6V_MEM_SIZE);
> + TYPE_ASPEED_GPIO, GPIO_MAX_MEM_SIZE);
>
> sysbus_init_mmio(sbd, &s->iomem);
> }
> @@ -841,6 +937,26 @@ static void aspeed_gpio_2500_class_init(ObjectClass
> *klass, void *data)
> agc->reg_table = aspeed_3_6v_gpios;
> }
>
> +static void aspeed_gpio_ast2600_3_6v_class_init(ObjectClass *klass, void
> *data)
> +{
> + AspeedGPIOClass *agc = ASPEED_GPIO_CLASS(klass);
> +
> + agc->props = ast2600_3_6v_set_props;
> + agc->nr_gpio_pins = 208;
> + agc->nr_gpio_sets = 7;
> + agc->reg_table = aspeed_3_6v_gpios;
> +}
> +
> +static void aspeed_gpio_ast2600_1_8v_class_init(ObjectClass *klass, void
> *data)
> +{
> + AspeedGPIOClass *agc = ASPEED_GPIO_CLASS(klass);
> +
> + agc->props = ast2600_1_8v_set_props;
> + agc->nr_gpio_pins = 36;
> + agc->nr_gpio_sets = 2;
> + agc->reg_table = aspeed_1_8v_gpios;
> +}
> +
> static const TypeInfo aspeed_gpio_info = {
> .name = TYPE_ASPEED_GPIO,
> .parent = TYPE_SYS_BUS_DEVICE,
> @@ -864,11 +980,27 @@ static const TypeInfo aspeed_gpio_ast2500_info = {
> .instance_init = aspeed_gpio_init,
> };
>
> +static const TypeInfo aspeed_gpio_ast2600_3_6v_info = {
> + .name = TYPE_ASPEED_GPIO "0-ast2600",
> + .parent = TYPE_ASPEED_GPIO,
> + .class_init = aspeed_gpio_ast2600_3_6v_class_init,
> + .instance_init = aspeed_gpio_init,
> +};
> +
> +static const TypeInfo aspeed_gpio_ast2600_1_8v_info = {
> + .name = TYPE_ASPEED_GPIO "1-ast2600",
> + .parent = TYPE_ASPEED_GPIO,
> + .class_init = aspeed_gpio_ast2600_1_8v_class_init,
> + .instance_init = aspeed_gpio_init,
> +};
> +
> static void aspeed_gpio_register_types(void)
> {
> type_register_static(&aspeed_gpio_info);
> type_register_static(&aspeed_gpio_ast2400_info);
> type_register_static(&aspeed_gpio_ast2500_info);
> + type_register_static(&aspeed_gpio_ast2600_3_6v_info);
> + type_register_static(&aspeed_gpio_ast2600_1_8v_info);
> }
>
> type_init(aspeed_gpio_register_types);
>
- Re: [Qemu-devel] [PATCH v4 1/3] hw/gpio: Add basic Aspeed GPIO model for AST2400 and AST2500, (continued)
- Re: [Qemu-devel] [PATCH v4 0/3] Add Aspeed GPIO controller model, no-reply, 2019/08/14
- [Qemu-devel] [PATCH v4 0/3] Add Aspeed GPIO controller model, Rashmica Gupta, 2019/08/16
- [Qemu-devel] [PATCH v5 1/3] hw/gpio: Add basic Aspeed GPIO model for AST2400 and AST2500, Rashmica Gupta, 2019/08/16
- [Qemu-devel] [PATCH v5 3/3] hw/gpio: Add in AST2600 specific implementation, Rashmica Gupta, 2019/08/16
- Re: [Qemu-devel] [PATCH v5 3/3] hw/gpio: Add in AST2600 specific implementation,
Cédric Le Goater <=
- Re: [Qemu-devel] [PATCH v4 0/3] Add Aspeed GPIO controller model, Cédric Le Goater, 2019/08/16