[coreboot-gerrit] Change in coreboot[master]: mb/asus/p2b: list all unused Super I/O resources

[Nico Huber (Code Review)]

Attention is currently required from: Keith Hui.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41459 )

Change subject: mb/asus/p2b: list all unused Super I/O resources
......................................................................

Patch Set 8:

(1 comment)

File src/mainboard/asus/p2b/devicetree.cb:

https://review.coreboot.org/c/coreboot/+/41459/comment/3bb629b7_dcbaad6a 
PS8, Line 38:             io 0x60 = 0
            :             io 0x62 = 0
> I think it's just to silence the warnings about "resource not set". […]

It just sounds like something that happens to work because our infrastructure
is not pedantic about it. But future changes to the allocator, for instance, 
could break it.

I can think of two solutions: a) add syntax to disable a resource; the chip
driver would have to write an explicit 0 to the hardware then. b) the chip
driver would tag a resource as 0-is-special or 0-is-off. Hmmm, or maybe a
third, but I don't like it (just a bad feeling), c) we always treat 0 as disabled.