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

[Nico Huber (Code Review)]

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

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

Patch Set 1:

(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.
If there is any device behind a device, the latter is a bridge by definition.
The question is, do we need such a topology at all? For instance, do we need
superio/aspeed/ast2400 devices behind the superio/common device? or could
we have something like

chip superio/common
            device generic 2e.0 on end
    end
    chip superio/aspeed/ast2400
            device pnp 2e.2 on
                    ...