[coreboot-gerrit] Change in coreboot[master]: drivers/generic/max98357a: Allow custom _HID from config

[Tim Wawrzynczak (Code Review)]

Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39286 )

Change subject: drivers/generic/max98357a: Allow custom _HID from config
......................................................................

Patch Set 11:

(1 comment)

https://review.coreboot.org/c/coreboot/+/39286/11/src/drivers/generic/max983... 
File src/drivers/generic/max98357a/max98357a.c:

https://review.coreboot.org/c/coreboot/+/39286/11/src/drivers/generic/max983... 
PS11, Line 46: if (!config->hid)
             : 		acpigen_write_name_string("_HID", MAX98357A_ACPI_HID);
             : 	else
             : 		acpigen_write_name_string("_HID", config->hid);
I believe Paul meant to just use:
acpigen_write_name_string("_HID", config->hid);
here, and then update all of the devicetrees that use this device to add register "hid" = "MAX98357A_ACPI_HID" (or other as appropriate).