[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 09/11] hw/gpio/pl061: Convert to 3-phase reset and assert GPI
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH 09/11] hw/gpio/pl061: Convert to 3-phase reset and assert GPIO lines correctly on reset |
Date: |
Fri, 2 Jul 2021 12:56:56 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.11.0 |
On 7/2/21 12:40 PM, Peter Maydell wrote:
> The PL061 comes out of reset with all its lines configured as input,
> which means they might need to be pulled to 0 or 1 depending on the
> 'pullups' and 'pulldowns' properties. Currently we do not assert
> these lines on reset; they will only be set whenever the guest first
> touches a register that triggers a call to pl061_update().
>
> Convert the device to three-phase reset so we have a place where we
> can safely call qemu_set_irq() to set the floating lines to their
> correct values.
>
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ---
> hw/gpio/pl061.c | 29 +++++++++++++++++++++++++----
> hw/gpio/trace-events | 1 +
> 2 files changed, 26 insertions(+), 4 deletions(-)
Nice.
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
- [PATCH 03/11] hw/gpio/pl061: Clean up read/write offset handling logic, (continued)
Re: [PATCH 03/11] hw/gpio/pl061: Clean up read/write offset handling logic, Richard Henderson, 2021/07/06
[PATCH 07/11] hw/gpio/pl061: Make pullup/pulldown of outputs configurable, Peter Maydell, 2021/07/02
[PATCH 09/11] hw/gpio/pl061: Convert to 3-phase reset and assert GPIO lines correctly on reset, Peter Maydell, 2021/07/02
[PATCH 06/11] hw/gpio/pl061: Honour Luminary PL061 PUR and PDR registers, Peter Maydell, 2021/07/02
[PATCH 11/11] hw/arm/stellaris: Expand comment about handling of OLED chipselect, Peter Maydell, 2021/07/02
[PATCH 05/11] hw/gpio/pl061: Document the interface of this device, Peter Maydell, 2021/07/02
[PATCH 10/11] hw/gpio/pl061: Document a shortcoming in our implementation, Peter Maydell, 2021/07/02
[PATCH 08/11] hw/arm/virt: Make PL061 GPIO lines pulled low, not high, Peter Maydell, 2021/07/02