[coreboot-gerrit] [S] Change in coreboot[master]: soc/intel/alderlake/acpi.c: Don't look up coreboot CPU index

[Arthur Heymans (Code Review)]

Attention is currently required from: Tarun Tuli, Michał Żygowski, Subrata Banik.

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69510 )

Change subject: soc/intel/alderlake/acpi.c: Don't look up coreboot CPU index
......................................................................

Patch Set 6:

(1 comment)

Patchset:

PS6:
> I mean it doesn't make any difference with and without the patch applied.

Right. Looking at the code:
loop over struct dev
      loop over i 
      if( dev->apic_id == cpu_get_apic_id(i)
             ...

is really the same as
loop over i as filling in the apicid happens based on which CPU gets the index spinlock first.

So this patch is just a cleaner way of writing the same.