Attached patch fixes the LPC decode ranges of SB700. We enable early only Serial...
authorRudolf Marek <r.marek@assembler.cz>
Fri, 28 Jan 2011 20:57:48 +0000 (20:57 +0000)
committerRudolf Marek <r.marek@assembler.cz>
Fri, 28 Jan 2011 20:57:48 +0000 (20:57 +0000)
commit86bd99aba2cec223df48c2a00bdea0450f6346e0
tree0b8aecef2fe7ce84aafa08ca2dc4e3ca1e10d79c
parent16ce01b0d8414a7250fcf142a966ff22c153e85f
Attached patch fixes the LPC decode ranges of SB700. We enable early only Serial/SIO/RTC. Everything else needs to be done by lpc.c Problem was that early settings survived, because the lpc.c is doing ORs only...
Hence we decode quite a lot and even strange ranges like IO port 0x4600 etc...

Also, if some port which does not fit to predefined set is requested, like 0x290 for Hardware monitor, the wide port is done, but in our case it has range 512 bytes which means we decode in fact 0x290 - 0x490. And if we hit GPU in the 0x3bx range I receive MCE exception if I do isadump -f 0x300 which is bad.
Therefore If I detect that the requested range is small (16 bytes) I additionally set the small wide io region so only 16 bytes is decoded.

While at it, I fix spelling typos and I init the regs so we don't write random garbage to regs even if we don't enable them later.

Signed-off-by: Rudolf Marek <r.marek@assembler.cz>
Acked-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
git-svn-id: svn://svn.coreboot.org/coreboot/trunk@6316 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
src/southbridge/amd/sb700/early_setup.c
src/southbridge/amd/sb700/lpc.c