[PATCH] bsps/arm: Improve GICv3 support

[Sebastian Huber]

On 16.04.24 16:51, Kinsey Moore wrote:
> This adds warnings for arm_interrupt_enable_interrupts and 
> arm_interrupt_restore_interrupts. I suspect a missing header. They also 
> generate a link error on the a53_lp64_qemu bsp. I also dislike the 
> while(true), but I don't think we officially have anything against it 
> and there are existing examples in the codebase.

Did you test this with the latest master? Actually, I committed a 
similar patch for the GICv2 and then noticed that this broke the GICv3 
support. This is fixed by the current patch.

What would be your alternative to this while (true) loop?

void bsp_interrupt_dispatch(void)
{
   while (true) {
     uint32_t icciar = READ_SR(ICC_IAR1);
     rtems_vector_number vector = GIC_CPUIF_ICCIAR_ACKINTID_GET(icciar);
     uint32_t status;

     if (!bsp_interrupt_is_valid_vector(vector)) {
       break;
     }

     status = arm_interrupt_enable_interrupts();
     bsp_interrupt_handler_dispatch_unchecked(vector);
     arm_interrupt_restore_interrupts(status);

     WRITE_SR(ICC_EOIR1, icciar);
   }
}

-- 
embedded brains GmbH & Co. KG
Herr Sebastian HUBER
Dornierstr. 4
82178 Puchheim
Germany
email: sebastian.huber at embedded-brains.de
phone: +49-89-18 94 741 - 16
fax:   +49-89-18 94 741 - 08

Registergericht: Amtsgericht München
Registernummer: HRB 157899
Vertretungsberechtigte Geschäftsführer: Peter Rasmussen, Thomas Dörfler
Unsere Datenschutzerklärung finden Sie hier:
https://embedded-brains.de/datenschutzerklaerung/