Hi
On selected boards, some hardware initialisation is placed in the bootblock. The source files and directories are currently hard-coded in Kconfigs, which is sort of ugly.
A few months ago I put together changeset [1], which hasn't drawn much review or interest. One benefit of my changeset is that it can be extended to move superio and console initialisation to bootblock. Serial-line IO can then be used to switch between fallback/normal romstage and early POSTs could go to serial too.
Thanks for any comments.
KM
Am 15.03.2012 15:00, schrieb Kyösti Mälkki:
On selected boards, some hardware initialisation is placed in the bootblock. The source files and directories are currently hard-coded in Kconfigs, which is sort of ugly.
A few months ago I put together changeset [1], which hasn't drawn much review or interest.
Thank you for that contribution - I'm really sorry that it fell through the cracks. This was partly due to timing (slow development back then), partly because it's a rather involved patch, doing similar things at once.
It's not a very conscious decision on my part, but I sometimes look at changes, and quickly close them because they overwhelm me, and I don't always have the time and concentration to dive into it very deeply - this one, I looked at several times, with no real results :-(
I hereby pledge to resist that urge in the future and at least give some hints on how I'd like to see things split up to simplify a review.
Things that could be separated out: - removing the AMD no-op bootblocks could be a separate change - that probably would be committed in a day or two.
- renaming the functions - I explicitely added the bootblock_ prefix so the context in which this code is used is clear. It can be argued that the filenames are enough of a clue, so that would probably go through as well, after talking about this aspect.
- the rest. It's a rather involved change. It doesn't turn coreboot upside down, but I think it's much easier to consider the consequences if we don't have to fear some side effect hidden in the "noise" of the two changes above.
One benefit of my changeset is that it can be extended to move superio and console initialisation to bootblock. Serial-line IO can then be used to switch between fallback/normal romstage and early POSTs could go to serial too.
We can consider doing so (It adds complexity to something we tried to keep small, but I think it's worth it). But we should also aim at moving all chipsets to behave that way.
You can't do that alone (of course), but a discussion on that on the list (like what's going on now) goes a long way to make sure we all know what to expect from the change, and to enlist support in making it happen everywhere.
Thanks, Patrick
On Thu, 2012-03-15 at 18:41 +0100, Patrick Georgi wrote:
Am 15.03.2012 15:00, schrieb Kyösti Mälkki:
One benefit of my changeset is that it can be extended to move superio and console initialisation to bootblock. Serial-line IO can then be used to switch between fallback/normal romstage and early POSTs could go to serial too.
We can consider doing so (It adds complexity to something we tried to keep small, but I think it's worth it). But we should also aim at moving all chipsets to behave that way.
Okay. So I won't abandon that one, then. I'll split it once I see some free space appear on my gerrit workspace -- a few merges there waiting to happen. But I do understand GSoC probably takes a lot of Your contributed time in the following few weeks.
Early debug console should be straight-forward. Pick the correct superio/early_serial.c and PNP device number of a serial port from devicetree.cb and enable 0x2e/0x4e and 0x3f8/0x2f8 IO ranges to LPC from southbridge code.
You can't do that alone (of course), but a discussion on that on the list (like what's going on now) goes a long way to make sure we all know what to expect from the change, and to enlist support in making it happen everywhere.
I just tend to push the changes I needed or found useful to gerrit review without much further thought. The times I have asked questions on the list, I often get to keep the pleasure of finding out and learning all to myself :]
KM