Patch Set 2:

This is untested and no objective was given why one of _ADR or _HID was preferred over the other. It's unclear if there are OS that requires one of them to be present.

SerialIO devices are always in ACPI mode for these platforms on all boards in the tree, and _HID is required for ACPI mode, so makes sense to drop _ADR here

Yes this is correct these were always just put in ACPI mode on baytrail. This is something where Intel went back and forth on and eventually settled PCI mode in future chipsets.

I tried to support both modes in coreboot (which is why it had both _HID and _ADR) but it may be easier to settle on one path, or put this behind a Kconfig variable.

View Change

To view, visit change 37935. To unsubscribe, or for help writing mail filters, visit settings.