[LinuxBIOS] A8N5X port
Uwe Hermann
uwe at hermann-uwe.de
Thu Sep 27 19:14:08 CEST 2007
[Sorry for being late, I'm a bit, um... Internet-challenged at the moment.
No PGP signature for the same reason.]
On Wed, Sep 26, 2007 at 11:50:52AM +0200, Robert Millan wrote:
> On Tue, Sep 25, 2007 at 03:53:21PM -0700, ron minnich wrote:
> > Robert, I understand the reason for the include of the .c files, but
> > go ahead and just copy the code. We've tried both ways, over the
> > years, and it's better to clone the code, since the vendors have a
> > habit of changing things on mainboards, and code that includes other
> > code can cause trouble in both directions.
I strongly disagree with this approach. Duplicated code is bad, bad,
bad and should be avoided whereever possible.
Now, in order to not only complain, here's a demo patch which completely
avoids copying a single file. One code base for the
- ASUS A8N-E
- ASUS A8N5X
- ASUS A8NE-FM/S (I'll send an extra patch for that later)
No duplicated code whatsoever. Sure, this may not be too elegant
(there's some room for improvements, though), but it's definately a
_lot_ better than duplicating all those files.
This is (manually) build-tested for the A8N-E and A8N5X and worked for me.
I did not test the produced ROMs, though.
In addition to the patch, you need to create a symlink
src/mainboard/asus/a8n_e -> src/mainboard/asus/a8n5x
in order to make the build system happy.
Uwe.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: v2_a8n5x.patch
Type: text/x-patch
Size: 7495 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20070927/a86b268d/attachment.patch>
More information about the coreboot
mailing list