[M] Change in coreboot[main]: soc/intel/xeon_sp: Add ACPI names

[Shuo Liu (Code Review)]

Attention is currently required from: Arthur Heymans, Christian Walter, Johnny Lin, Lean Sheng Tan, Patrick Rudolph, Tim Chu.

Shuo Liu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/80794?usp=email )

Change subject: soc/intel/xeon_sp: Add ACPI names
......................................................................

Patch Set 2:

(1 comment)

File src/soc/intel/xeon_sp/acpi.c:

https://review.coreboot.org/c/coreboot/+/80794/comment/f750f375_b636f5d1 :
PS2, Line 144: char *name = xmalloc(ACPI_NAME_BUFFER_SIZE);
> I could squash this with Ic4cc81d198fb88300394055682a3954bf22db570 and generate the proper ACPI name […]

From a reader point of view, handling among multiple names, a.k.a. device->name, and acpi_name ..., are somewhat confusing. Maybe we can add a new field 'tag' to struct device. This field could be defined for device specific usage (just like the resource->index).

For IIO domains devices, we can use the device->tag to hold the domain type. While for device->name, we can use it for acpi names. We can xmalloc at the 1st reference but read the values for subsequent references after the name is created.

What about this?