[coreboot-gerrit] Change in coreboot[master]: drivers/i2c: Add chip driver for GPIO based I2C multiplexer

[Evan Green (Code Review)]

Evan Green has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45911 )

Change subject: drivers/i2c: Add chip driver for GPIO based I2C multiplexer
......................................................................

Patch Set 6:

(1 comment)

https://review.coreboot.org/c/coreboot/+/45911/6/src/drivers/i2c/gpiomux/mux... 
File src/drivers/i2c/gpiomux/mux/mux.c:

https://review.coreboot.org/c/coreboot/+/45911/6/src/drivers/i2c/gpiomux/mux... 
PS6, Line 37: "_HID", ACPI_DT_NAMESPACE_HID)
Wait but if you do this and use PRP0001, aren't you deferring into the DT world entirely, including its bindings? We're not following the DT-based bindings exactly (around i2c-parent). Don't we need an ACPI-based _HID so we can say "this is the ACPI version"?