On Fri, Oct 15, 2010 at 04:10:26PM +0200, Nils wrote:
As i stated in the original message i tested the patch on my Geode GX2 Wyse S50 board. I didn`t test it on the Rumba and Frontrunner because i don`t have them and people who have them don`t seem to have time/interest to test.
But i don`t think that is very imported because both boards are broken in the current state for years.
I agree. Testing on one of the GX2 boards is fully sufficient, nobody can always test _all_ boards of a given chipset.
We're also routinely doing massive changes on the tree, some without much hardware-testing at all, so your patch -- being boot-tested on hardware -- is already in much better shape than many others we already committed.
I don`t think the Frontrunner board works at the present state either. I think i found a double bug in the cpureginit.c code that exists since the first commit in r2248 ! (The FooGlue setup should only be done for CS5535 boards and uses a wrong address) I am working on a patch for that.
Great, thanks!
I think the GX2 code was never realy finished on any target and was/is a mess.
Yep.
Because exept for my Wyse S50 i don`t think there is a working board. (there is not even a free available working VSA blob for the current GX2 tree, i had to make it myself!)
Can you update any wiki instructions wrt to VSA which may be out of date, and or shall we upload a known-good blob to the wiki?
So i think my patches are a big improvement and it is hard to "break" someones board with it.
Yes, I fully agree.
I am not planning to do a lot of little patches and abuild and boot test all of them separately as that cost me a lot of extra time. As i understand it only a few, for the real user unimportant, white space and comment change patches would get acked and committed and no one dares to ack the real patches.
Nah, we will review and commit this stuff, don't worry. But I agree with Myles that mega-patches containing both whitespace changes and random other functional changes are hard to review. Let us know if you can split the stuff into, say, 2-3 distinct patches, one with whitespace and coding style changes and 1-2 with the actual code changes.
If not, I'll try to separate at least parts of the whitespace stuff and commit it, so the rest becomes more readable. Then we'll review that and commit.
Maybe Ron is right and all of the non functional boards (i think at least 90 percent of coreboot) should be deleted.
No, definately not. I whole-heartedly disagree with any notions of "let's drop this board, it wasn't tested recently" and I will object to any patches in that direction.
Sorry it took so long to review your changes, but don't be discouraged, we will definately merge your changes. Please let us know if you're going to post updated/splitted patches or whether we should try to split your patch.
Thanks, Uwe.