[coreboot] Trying to add Kconfig support for M57SLI - build error

ron minnich rminnich at gmail.com
Fri Aug 21 20:54:35 CEST 2009

On Fri, Aug 21, 2009 at 11:09 AM, Myles Watson<mylesgw at gmail.com> 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?


More information about the coreboot mailing list