On Fri, Aug 21, 2009 at 11:09 AM, Myles Watson<[email protected]> wrote: >> Let's first do AM2. Then nvidia south. And so on. >> >> OK, I am going to apply Cristi's patch, and, if it looks good, commit. >> Then we can do nvidia south. > From Cristi's patch: > --- /dev/null > +++ b/src/cpu/amd/microcode/Makefile.inc > @@ -0,0 +1 @@ > +obj-y += microcode.o > > Let's not create a whole file for one obj. Can we just put that in > socket_AM2/Makefile.inc like the intel sockets do? > > obj-y += ../microcode/microcode.o
The reason to create a whole file for one object is that, over time, kif we get more than one object, nobody has to change their makefiles -- we add to microcode/Makefile.inc. I think the subdir usage and extra Makefile.inc are fine, if we're going to have the directory. At same time, I'm not overly concerned either way. I have to go out rest of the day, the AM2 patch is almost there, Myles, can you and Cristi wrap it up? ron -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
