[coreboot] Two fixes for your patch

Myles Watson mylesgw at gmail.com
Mon Aug 24 21:40:38 CEST 2009


>> I changed the prototype for hard_reset in
>> src/northbridge/amd/amdk8/amdk8_f.h to:
>> #include <part/hard_reset.h>
> Does this mean that you added this line to the file?
> If yes, the compiler error stays here the same.
Interesting.  Our trees must have diverged somewhere.

> But as mentioned right now on the irc channel I wonder why there is no "CC
> mcp55_reset.o" in the log, and also such a file doesn't exist in the build/
> directory after a "failing" make.
I think that's because it fails before it gets there.  As I looked
further, I think the root of the problem is that HAVE_HARD_RESET isn't
set correctly.

If you grep -Ir HAVE_HARD build, it comes up as 0.  Maybe that's
because you have it as a bool with 1 for the default?

Changing it to y fixes it for me.  HAVE_HIGH_TABLES has the same problem.

I think the correct thing is to #include <part/hard_reset.h> and
remove the prototype, but I think the correct value for
HAVE_HARD_RESET will also fix the problem.

Good luck,
Myles




More information about the coreboot mailing list