[coreboot-gerrit] New patch to review for coreboot: 887a241 ASRock E350M1: mptable.c: Include `cpu/amd/amdfam14.h` for `get_bus_conf`

[Paul Menzel]

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

-gerrit

commit 887a24177ad2c3bb97652b8b04269dff84196aff
Author: Paul Menzel 
Date:   Fri Mar 29 20:20:56 2013 +0100

ASRock E350M1: mptable.c: Include `cpu/amd/amdfam14.h` for `get_bus_conf`

When building the ASRock E350M1, the following warning is shown.

$ make # on Jenkins (build server)
        […]
            CC         mainboard/asrock/e350m1/mptable.ramstage.o
        src/mainboard/asrock/e350m1/mptable.c: In function 'smp_write_config_table':
        src/mainboard/asrock/e350m1/mptable.c:58:3: warning: implicit declaration of function 'get_bus_conf' [-Wimplicit-function-declaration]
        […]

Including the header file `cpu/amd/amdfam14.h` declaring the
    function addresses this warning.

The same change was done in the following commit for the
    AMD Persimmon board.

commit d7a696d0f229abccc95ff411f28d91b9b796ab74
        Author: efdesign98 
        Date:   Thu Sep 15 15:24:26 2011 -0600

Persimmon updates for AMD F14 rev C0

Reviewed-on: http://review.coreboot.org/137

Change-Id: I7912571fa57f6512b10fc9b5845427fcb6eb50c0
    Signed-off-by: Paul Menzel 
---
 src/mainboard/asrock/e350m1/mptable.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/mainboard/asrock/e350m1/mptable.c b/src/mainboard/asrock/e350m1/mptable.c
index 360c5c0..1552877 100644
--- a/src/mainboard/asrock/e350m1/mptable.c
+++ b/src/mainboard/asrock/e350m1/mptable.c
@@ -24,6 +24,7 @@
 #include <arch/io.h>
 #include <string.h>
 #include <stdint.h>
+#include <cpu/amd/amdfam14.h>
 #include <SBPLATFORM.h>

extern u8 bus_sb800[2];