[coreboot-gerrit] Change in coreboot[master]: src/mb: Get rid of wrong _ADR objects

[Martin Roth (Code Review)]

Martin Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36317 )

Change subject: src/mb: Get rid of wrong _ADR objects
......................................................................

Patch Set 6:

(1 comment)

https://review.coreboot.org/c/coreboot/+/36317/6/src/mainboard/advansus/a785... 
File src/mainboard/advansus/a785e-i/dsdt.asl:

https://review.coreboot.org/c/coreboot/+/36317/6/src/mainboard/advansus/a785... 
PS6, Line 1128: 			Name(_ADR, 0x00180000)	/* Dev# = BSP Dev#, Func# = 0 */
Why did you choose to get rid of the _ADR objects instead of the _HID?  I'm not saying it's incorrect, but it seems like _ADR is a more specific and useful description of the device than the Plug and Play Hardware ID.

Top level question moved into the source to prevent accidental merging before it's looked at.