Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38350 )
Change subject: soc/intel/broadwell: Don't use _ADR and _HID ......................................................................
Patch Set 8:
If anyone disagrees with these patches,
I do. It leaves the tree in an inconsistent state. Nothing changed since this was reverted. We have discussed this in depth about three times already. Many alternatives were proposed. IIRC, we even agreed that the lynxpoint/broadwell code around _ADR (sio_acpi_mod = 0) can be removed altogether.
I've also proposed to disable the warning (imho, it doesn't provide any value, just a huge waste of time). Nobody liked that, though.
Ok, So you're saying you agree with my suggested patch then? Or do you care to write something different?
I don't care anymore (as long as it's not leaving dead code). Spent too much time on this already, and there is nothing to gain.