[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 04/10] hw/arm/bcm2835_peripherals: Use the SYS_timer
From: |
Alistair Francis |
Subject: |
Re: [PATCH v2 04/10] hw/arm/bcm2835_peripherals: Use the SYS_timer |
Date: |
Thu, 17 Oct 2019 16:59:50 -0700 |
On Thu, Oct 17, 2019 at 4:05 PM Philippe Mathieu-Daudé <address@hidden> wrote:
>
> Connect the recently added SYS_timer.
> Now U-Boot does not hang anymore polling a free running counter
> stuck at 0.
> This timer is also used by the Linux kernel thermal subsystem.
>
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Reviewed-by: Alistair Francis <address@hidden>
Alistair
> ---
> v2: Remove spurious error check (Alex)
> ---
> hw/arm/bcm2835_peripherals.c | 17 ++++++++++++++++-
> include/hw/arm/bcm2835_peripherals.h | 3 ++-
> 2 files changed, 18 insertions(+), 2 deletions(-)
>
> diff --git a/hw/arm/bcm2835_peripherals.c b/hw/arm/bcm2835_peripherals.c
> index 70bf927a02..17207ae07e 100644
> --- a/hw/arm/bcm2835_peripherals.c
> +++ b/hw/arm/bcm2835_peripherals.c
> @@ -58,6 +58,10 @@ static void bcm2835_peripherals_init(Object *obj)
> /* Interrupt Controller */
> sysbus_init_child_obj(obj, "ic", &s->ic, sizeof(s->ic), TYPE_BCM2835_IC);
>
> + /* SYS Timer */
> + sysbus_init_child_obj(obj, "systimer", &s->systmr, sizeof(s->systmr),
> + TYPE_BCM2835_SYSTIMER);
> +
> /* UART0 */
> sysbus_init_child_obj(obj, "uart0", &s->uart0, sizeof(s->uart0),
> TYPE_PL011);
> @@ -171,6 +175,18 @@ static void bcm2835_peripherals_realize(DeviceState
> *dev, Error **errp)
> sysbus_mmio_get_region(SYS_BUS_DEVICE(&s->ic), 0));
> sysbus_pass_irq(SYS_BUS_DEVICE(s), SYS_BUS_DEVICE(&s->ic));
>
> + /* Sys Timer */
> + object_property_set_bool(OBJECT(&s->systmr), true, "realized", &err);
> + if (err) {
> + error_propagate(errp, err);
> + return;
> + }
> + memory_region_add_subregion(&s->peri_mr, ST_OFFSET,
> + sysbus_mmio_get_region(SYS_BUS_DEVICE(&s->systmr), 0));
> + sysbus_connect_irq(SYS_BUS_DEVICE(&s->systmr), 0,
> + qdev_get_gpio_in_named(DEVICE(&s->ic), BCM2835_IC_ARM_IRQ,
> + INTERRUPT_ARM_TIMER));
> +
> /* UART0 */
> qdev_prop_set_chr(DEVICE(&s->uart0), "chardev", serial_hd(0));
> object_property_set_bool(OBJECT(&s->uart0), true, "realized", &err);
> @@ -352,7 +368,6 @@ static void bcm2835_peripherals_realize(DeviceState *dev,
> Error **errp)
> }
>
> create_unimp(s, &s->armtmr, "bcm2835-sp804", ARMCTRL_TIMER0_1_OFFSET,
> 0x40);
> - create_unimp(s, &s->systmr, "bcm2835-systimer", ST_OFFSET, 0x20);
> create_unimp(s, &s->cprman, "bcm2835-cprman", CPRMAN_OFFSET, 0x1000);
> create_unimp(s, &s->a2w, "bcm2835-a2w", A2W_OFFSET, 0x1000);
> create_unimp(s, &s->i2s, "bcm2835-i2s", I2S_OFFSET, 0x100);
> diff --git a/include/hw/arm/bcm2835_peripherals.h
> b/include/hw/arm/bcm2835_peripherals.h
> index be7ad9b499..7859281e11 100644
> --- a/include/hw/arm/bcm2835_peripherals.h
> +++ b/include/hw/arm/bcm2835_peripherals.h
> @@ -24,6 +24,7 @@
> #include "hw/sd/sdhci.h"
> #include "hw/sd/bcm2835_sdhost.h"
> #include "hw/gpio/bcm2835_gpio.h"
> +#include "hw/timer/bcm2835_systmr.h"
> #include "hw/misc/unimp.h"
>
> #define TYPE_BCM2835_PERIPHERALS "bcm2835-peripherals"
> @@ -39,7 +40,7 @@ typedef struct BCM2835PeripheralState {
> MemoryRegion ram_alias[4];
> qemu_irq irq, fiq;
>
> - UnimplementedDeviceState systmr;
> + BCM2835SystemTimerState systmr;
> UnimplementedDeviceState armtmr;
> UnimplementedDeviceState cprman;
> UnimplementedDeviceState a2w;
> --
> 2.21.0
>
>
- [PATCH v2 00/10] hw/arm/raspi: Add thermal/timer, improve multicore address space, Philippe Mathieu-Daudé, 2019/10/17
- [PATCH v2 01/10] hw/misc/bcm2835_thermal: Add a dummy BCM2835 thermal sensor, Philippe Mathieu-Daudé, 2019/10/17
- [PATCH v2 02/10] hw/arm/bcm2835_peripherals: Use the thermal sensor block, Philippe Mathieu-Daudé, 2019/10/17
- [PATCH v2 03/10] hw/timer/bcm2835: Add the BCM2835 SYS_timer, Philippe Mathieu-Daudé, 2019/10/17
- [PATCH v2 04/10] hw/arm/bcm2835_peripherals: Use the SYS_timer, Philippe Mathieu-Daudé, 2019/10/17
- Re: [PATCH v2 04/10] hw/arm/bcm2835_peripherals: Use the SYS_timer,
Alistair Francis <=
- [PATCH v2 05/10] hw/arm/bcm2836: Make the SoC code modular, Philippe Mathieu-Daudé, 2019/10/17
- [PATCH v2 06/10] hw/arm/bcm2836: Create VideoCore address space in the SoC, Philippe Mathieu-Daudé, 2019/10/17
- [PATCH v2 08/10] hw/arm/raspi: Use AddressSpace when using arm_boot::write_secondary_boot, Philippe Mathieu-Daudé, 2019/10/17
- [PATCH v2 07/10] hw/arm/bcm2836: Use per CPU address spaces, Philippe Mathieu-Daudé, 2019/10/17
- [PATCH v2 09/10] hw/arm/raspi: Make the board code modular, Philippe Mathieu-Daudé, 2019/10/17
- [PATCH v2 10/10] hw/arm/highbank: Use AddressSpace when using write_secondary_boot(), Philippe Mathieu-Daudé, 2019/10/17
- Re: [PATCH v2 00/10] hw/arm/raspi: Add thermal/timer, improve multicore address space, no-reply, 2019/10/18