1 comment:
File src/drivers/uart/uart8250io.c:
Patch Set #2, Line 78: static const unsigned int bases[] = { 0x3f8, 0x2f8, 0x3e8, 0x2e8 };
Use any base address. […]
What address would you use that isn't listed? I think that bases[] is the only valid settings. It is structured this way so that menuconfig works. I'm trying to understand the use case for change this runtime and not build time. Also, this doesn't handle the early uart/serial init.
I don't think that a weak function boots the right way. I'd rather a CONFIG option that is more deterministic, but I don't know what the current coreboot policy is on this. It is hard to tell which function is being built, but I guess this is more obvious since it would need to be in the mainboard you are building.
To view, visit change 45405. To unsubscribe, or for help writing mail filters, visit settings.