[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 06/15] hw/riscv: sifive_gpio: Add a new 'ngpio' property
From: |
Alistair Francis |
Subject: |
Re: [PATCH 06/15] hw/riscv: sifive_gpio: Add a new 'ngpio' property |
Date: |
Mon, 15 Jun 2020 09:16:19 -0700 |
On Mon, Jun 8, 2020 at 7:22 AM Bin Meng <bmeng.cn@gmail.com> wrote:
>
> From: Bin Meng <bin.meng@windriver.com>
>
> Add a new property to represent the number of GPIO pins supported
> by the GPIO controller.
>
> Signed-off-by: Bin Meng <bin.meng@windriver.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Alistair
> ---
>
> hw/riscv/sifive_gpio.c | 30 +++++++++++++++++++-----------
> include/hw/riscv/sifive_gpio.h | 3 +++
> 2 files changed, 22 insertions(+), 11 deletions(-)
>
> diff --git a/hw/riscv/sifive_gpio.c b/hw/riscv/sifive_gpio.c
> index c9cffa2..0d0fd2b 100644
> --- a/hw/riscv/sifive_gpio.c
> +++ b/hw/riscv/sifive_gpio.c
> @@ -14,6 +14,7 @@
> #include "qemu/osdep.h"
> #include "qemu/log.h"
> #include "hw/irq.h"
> +#include "hw/qdev-properties.h"
> #include "hw/riscv/sifive_gpio.h"
> #include "migration/vmstate.h"
> #include "trace.h"
> @@ -28,7 +29,7 @@ static void update_output_irq(SIFIVEGPIOState *s)
> pending |= s->rise_ip & s->rise_ie;
> pending |= s->fall_ip & s->fall_ie;
>
> - for (int i = 0; i < SIFIVE_GPIO_PINS; i++) {
> + for (int i = 0; i < s->ngpio; i++) {
> pin = 1 << i;
> qemu_set_irq(s->irq[i], (pending & pin) != 0);
> trace_sifive_gpio_update_output_irq(i, (pending & pin) != 0);
> @@ -41,7 +42,7 @@ static void update_state(SIFIVEGPIOState *s)
> bool prev_ival, in, in_mask, port, out_xor, pull, output_en, input_en,
> rise_ip, fall_ip, low_ip, high_ip, oval, actual_value, ival;
>
> - for (i = 0; i < SIFIVE_GPIO_PINS; i++) {
> + for (i = 0; i < s->ngpio; i++) {
>
> prev_ival = extract32(s->value, i, 1);
> in = extract32(s->in, i, 1);
> @@ -346,27 +347,35 @@ static const VMStateDescription vmstate_sifive_gpio = {
> }
> };
>
> -static void sifive_gpio_init(Object *obj)
> +static Property sifive_gpio_properties[] = {
> + DEFINE_PROP_UINT32("ngpio", SIFIVEGPIOState, ngpio, SIFIVE_GPIO_PINS),
> + DEFINE_PROP_END_OF_LIST(),
> +};
> +
> +static void sifive_gpio_realize(DeviceState *dev, Error **errp)
> {
> - SIFIVEGPIOState *s = SIFIVE_GPIO(obj);
> + SIFIVEGPIOState *s = SIFIVE_GPIO(dev);
>
> - memory_region_init_io(&s->mmio, obj, &gpio_ops, s,
> + memory_region_init_io(&s->mmio, OBJECT(dev), &gpio_ops, s,
> TYPE_SIFIVE_GPIO, SIFIVE_GPIO_SIZE);
> - sysbus_init_mmio(SYS_BUS_DEVICE(obj), &s->mmio);
>
> - for (int i = 0; i < SIFIVE_GPIO_PINS; i++) {
> - sysbus_init_irq(SYS_BUS_DEVICE(obj), &s->irq[i]);
> + sysbus_init_mmio(SYS_BUS_DEVICE(dev), &s->mmio);
> +
> + for (int i = 0; i < s->ngpio; i++) {
> + sysbus_init_irq(SYS_BUS_DEVICE(dev), &s->irq[i]);
> }
>
> - qdev_init_gpio_in(DEVICE(s), sifive_gpio_set, SIFIVE_GPIO_PINS);
> - qdev_init_gpio_out(DEVICE(s), s->output, SIFIVE_GPIO_PINS);
> + qdev_init_gpio_in(DEVICE(s), sifive_gpio_set, s->ngpio);
> + qdev_init_gpio_out(DEVICE(s), s->output, s->ngpio);
> }
>
> static void sifive_gpio_class_init(ObjectClass *klass, void *data)
> {
> DeviceClass *dc = DEVICE_CLASS(klass);
>
> + device_class_set_props(dc, sifive_gpio_properties);
> dc->vmsd = &vmstate_sifive_gpio;
> + dc->realize = sifive_gpio_realize;
> dc->reset = sifive_gpio_reset;
> dc->desc = "SiFive GPIO";
> }
> @@ -375,7 +384,6 @@ static const TypeInfo sifive_gpio_info = {
> .name = TYPE_SIFIVE_GPIO,
> .parent = TYPE_SYS_BUS_DEVICE,
> .instance_size = sizeof(SIFIVEGPIOState),
> - .instance_init = sifive_gpio_init,
> .class_init = sifive_gpio_class_init
> };
>
> diff --git a/include/hw/riscv/sifive_gpio.h b/include/hw/riscv/sifive_gpio.h
> index ad915b2..cf12fcf 100644
> --- a/include/hw/riscv/sifive_gpio.h
> +++ b/include/hw/riscv/sifive_gpio.h
> @@ -68,6 +68,9 @@ typedef struct SIFIVEGPIOState {
> uint32_t out_xor;
> uint32_t in;
> uint32_t in_mask;
> +
> + /* config */
> + uint32_t ngpio;
> } SIFIVEGPIOState;
>
> #endif /* SIFIVE_GPIO_H */
> --
> 2.7.4
>
>
- Re: [PATCH 01/15] hw/riscv: sifive_e: Remove the riscv_ prefix of the machine* and soc* functions, (continued)
- [PATCH 04/15] hw/riscv: sifive_u: Generate device tree node for OTP, Bin Meng, 2020/06/08
- [PATCH 03/15] hw/riscv: sifive_u: Simplify the GEM IRQ connect code a little bit, Bin Meng, 2020/06/08
- [PATCH 08/15] hw/riscv: sifive_gpio: Do not blindly trigger output IRQs, Bin Meng, 2020/06/08
- [PATCH 05/15] hw/riscv: sifive_gpio: Clean up the codes, Bin Meng, 2020/06/08
- [PATCH 06/15] hw/riscv: sifive_gpio: Add a new 'ngpio' property, Bin Meng, 2020/06/08
- Re: [PATCH 06/15] hw/riscv: sifive_gpio: Add a new 'ngpio' property,
Alistair Francis <=
- [PATCH 07/15] hw/riscv: sifive_u: Hook a GPIO controller, Bin Meng, 2020/06/08
- [PATCH 09/15] hw/riscv: sifive_u: Add reset functionality, Bin Meng, 2020/06/08
- [PATCH 10/15] hw/riscv: sifive_u: Rename serial property get/set functions to a generic name, Bin Meng, 2020/06/08
- [PATCH 13/15] hw/riscv: sifive_u: Support different boot source per MSEL pin state, Bin Meng, 2020/06/08
- [PATCH 11/15] hw/riscv: sifive_u: Add a new property msel for MSEL pin state, Bin Meng, 2020/06/08