[coreboot-gerrit] Change in coreboot[master]: soc/intel/tigerlake: Fix ACPI path for GPIO

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39344 )

Change subject: soc/intel/tigerlake: Fix ACPI path for GPIO
......................................................................

Patch Set 5:

(1 comment)

https://review.coreboot.org/c/coreboot/+/39344/5/src/soc/intel/tigerlake/gpi... 
File src/soc/intel/tigerlake/gpio_tgl.c:

https://review.coreboot.org/c/coreboot/+/39344/5/src/soc/intel/tigerlake/gpi... 
PS5, Line 128: No GCM2 device and group2 is not visible to kernel
Then, why have an incorrect ACPI path? Shouldn't it be left as NULL or ""? I actually think adding NULL and ensuring there is a check in soc/intel/common/block/gpio which catches if acpi path returned is NULL would be helpful.