[coreboot-gerrit] Change in coreboot[master]: superio/common/generic: Assign resources behind device

[Patrick Rudolph (Code Review)]

Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37607 )

Change subject: superio/common/generic: Assign resources behind device
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/37607/1//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/37607/1//COMMIT_MSG@10 
PS1, Line 10: need to scan the complete 'bus' even though it is not a bridge.
> Can you elaborate why it has to be the parent? i.e. why can't we do the same […]

1. There can be multiple siblings, we would need to reference one somehow (either in C or in ACPI)
2. Disabled devices do not generate SSDTs

With the parent/child approach that's not needed as there's only one parent, it's enabled by default and the ACPI path is known.