Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38810 )
Change subject: Makefile.inc: Ignore _HID & _ADR conflicts in Broadwell & Lynxpoint ......................................................................
Patch Set 2: Code-Review+1
(2 comments)
https://review.coreboot.org/c/coreboot/+/38810/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/38810/2//COMMIT_MSG@9 PS2, Line 9: We haven't been able to update IASL in 8 months because of this : conflict. Rather because people were bullied (IIRC, there were even threats to remove boards, as if this bullshit warning would weigh as much as no-CAR-global-migration). If review comments had been honored instead of pushing the same (already reverted) broken patches again and again, we could have updated it months ago.
https://review.coreboot.org/c/coreboot/+/38810/2//COMMIT_MSG@10 PS2, Line 10: Ignoring it doesn't make things any worse than they are now. That seems highly subjective. By disabling it for specific cases, you imply that the warning is reasonable otherwise. Which was never discussed nor has anybody explained its benefits.