[M] Change in coreboot[main]: acpi: Add IORT helper functions

[Arthur Heymans (Code Review)]

Attention is currently required from: Angel Pons, Cliff Huang, Felix Held, Felix Singer, Lance Zhao, Naresh Solanki, Patrick Rudolph, Tim Wawrzynczak.

Arthur Heymans has posted comments on this change by Naresh Solanki. ( https://review.coreboot.org/c/coreboot/+/79404?usp=email )

Change subject: acpi: Add IORT helper functions
......................................................................

Patch Set 6:

(4 comments)

File src/acpi/acpi_iort.c:

https://review.coreboot.org/c/coreboot/+/79404/comment/f348810c_23be7915?usp... :
PS6, Line 7: acpi_iort_node_t **its,
Does it make sense to return this? There are 0 mappings?

https://review.coreboot.org/c/coreboot/+/79404/comment/8e009e7a_247a250f?usp... :
PS6, Line 13: *its = (acpi_iort_node_t *)current;
it might be a bit easier to read to have a local variable too:

acpi_iort_node_t *its_entry = (acpi_iort_node_t *)current;

I suspect the compiler generates the same code.  (*its)->xxx becomes its_enry->

https://review.coreboot.org/c/coreboot/+/79404/comment/793f5e6b_591da878?usp... :
PS6, Line 46: (*smmu_v3)->mapping_count = 0;
already done in memset.

https://review.coreboot.org/c/coreboot/+/79404/comment/e3349eb3_36375439?usp... :
PS6, Line 82: memset(nc_node_data->device_name, 0, 16);
            : 	strncpy(nc_node_data->device_name, device_name, device_name_len);
            : 
            : 	current += device_name_len + 2;
            : 	current = ALIGN_UP(current, 4);
Should this not be a null terminated string?