New patch to review for coreboot: beca17b amdfam10: add phenom II as known cpu

[Bernhard Urban]

Bernhard Urban ([e-mail]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/864

-gerrit

commit beca17b9dc211fa2c8424083724c28c5f37d52bf
Author: Bernhard Urban 
Date:   Thu Apr 5 17:13:27 2012 +0200

amdfam10: add phenom II as known cpu

Change-Id: I84a0f9e8e7a15c0aac8dc380de3ddf70b1decbd7
    Signed-off-by: Bernhard Urban 
---
 src/northbridge/amd/amdfam10/raminit_amdmct.c |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/src/northbridge/amd/amdfam10/raminit_amdmct.c b/src/northbridge/amd/amdfam10/raminit_amdmct.c
index 0c01cf1..973384b 100644
--- a/src/northbridge/amd/amdfam10/raminit_amdmct.c
+++ b/src/northbridge/amd/amdfam10/raminit_amdmct.c
@@ -211,6 +211,9 @@ u32 mctGetLogicalCPUID(u32 Node)
    case 0x10081:
    	ret = AMD_HY_D1;
    	break;
+	case 0x100a0:
+		ret = AMD_PH_E0;
+		break;
    default:
    	/* FIXME: mabe we should die() here. */
    	print_err("FIXME! CPU Version unknown or not supported! \n");

New patch to review for coreboot: beca17b amdfam10: add phenom II as known cpu

[Paul Menzel]

Dear Bernhard,

Am Donnerstag, den 05.04.2012, 17:13 +0200 schrieb Bernhard Urban:
> Bernhard Urban ([e-mail]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/864
>
> -gerrit
>
> commit beca17b9dc211fa2c8424083724c28c5f37d52bf
> Author: Bernhard Urban 
> Date:   Thu Apr 5 17:13:27 2012 +0200
> amdfam10: add phenom II as known cpu
>
> Change-Id: I84a0f9e8e7a15c0aac8dc380de3ddf70b1decbd7
> Signed-off-by: Bernhard Urban <[e-mail]>
>
> src/northbridge/amd/amdfam10/raminit_amdmct.c |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)

[…]

thanks a lot for your patch.

Just for the record, could you update the commit message with what board
you have tested this (latest master I guess) and if everything works as
expected.

Thanks,

Paul

New patch to review for coreboot: beca17b amdfam10: add phenom II as known cpu

[Bernhard Urban]

On Thu, Apr 5, 2012 at 10:31 PM, Paul Menzel
 wrote:
> Just for the record, could you update the commit message with what board
> you have tested this (latest master I guess) and if everything works as
> expected.

Good point!

Well, the board ("asus m5a99x evo") I've tested with isn't upstream yet.
Some weeks ago I've posted a progress report:
http://www.coreboot.org/pipermail/coreboot/2012-February/068464.html
Since it reaches SeaBIOS, I think this patch should be fine.

Unfortunately it's already merged, so I can't change the commit message :/

Thanks anyway,
Bernhard