hw/arm/aspeed_ast27x0: minor IRQ number cleanup

[PATCH 0/2] hw/arm/aspeed_ast27x0: minor IRQ number cleanup

Posted by Peter Maydell 3 weeks, 1 day ago

In the course of a conversation on IRC with somebody who was using
the ast27x0 code as a model for a new board, I noticed that the
code currently defines a local ARCH_GIC_MAINT_IRQ with a different
value from the constant of the same name that we define in the
include/hw/arm/bsa.h header. This patchset cleans that up and
also another minor awkwardness that I spotted in the process.

thanks
-- PMM

Peter Maydell (2):
  hw/arm/aspeed_ast27x0: Use bsa.h for PPI definitions
  hw/arm/aspeed_ast27x0: Avoid hardcoded '256' in IRQ calculation

 hw/arm/aspeed_ast27x0.c | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

-- 
2.34.1

Re: [PATCH 0/2] hw/arm/aspeed_ast27x0: minor IRQ number cleanup

Posted by Cédric Le Goater 3 weeks ago

On 11/1/24 17:11, Peter Maydell wrote:
> In the course of a conversation on IRC with somebody who was using
> the ast27x0 code as a model for a new board, I noticed that the
> code currently defines a local ARCH_GIC_MAINT_IRQ with a different
> value from the constant of the same name that we define in the
> include/hw/arm/bsa.h header. This patchset cleans that up and
> also another minor awkwardness that I spotted in the process.
> 
> thanks
> -- PMM
> 
> Peter Maydell (2):
>    hw/arm/aspeed_ast27x0: Use bsa.h for PPI definitions
>    hw/arm/aspeed_ast27x0: Avoid hardcoded '256' in IRQ calculation
> 
>   hw/arm/aspeed_ast27x0.c | 22 +++++++++++-----------
>   1 file changed, 11 insertions(+), 11 deletions(-)
> 

Applied to aspeed-next.

Thanks,

C.