[S] Change in coreboot[master]: cpu/x86/mtrr: Print apic id in when set up MTRRs for BSP/APs

[Kane Chen (Code Review)]

Attention is currently required from: Subrata Banik, Kane Chen, Tim Wawrzynczak, Arthur Heymans.

Kane Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/64467 )

Change subject: cpu/x86/mtrr: Print apic id in when set up MTRRs for BSP/APs
......................................................................

Patch Set 4:

(2 comments)

File src/cpu/x86/mtrr/mtrr.c:

https://review.coreboot.org/c/coreboot/+/64467/comment/4435e0df_28a2fa42 
PS3, Line 339: lapicid
> Use a variable instead of calling this over and over...

Done

https://review.coreboot.org/c/coreboot/+/64467/comment/220efbc9_925780a9 
PS3, Line 358: lapicid
> why not calling into `cpu_index()` to know the CPU ID instead APIC ID, which would be difficult to k […]

i checked cpu_index function, there is loop inside to compare apic id. i feel it's too much.  
The apic_id here is to indicate what cpu is running the task, so i guess apic_id should be good enough?
thanks