diff options
| author | Edward O'Callaghan <eocallaghan@alterapraxis.com> | 2014-06-29 02:27:19 +1000 |
|---|---|---|
| committer | Edward O'Callaghan <eocallaghan@alterapraxis.com> | 2014-06-29 15:04:48 +0200 |
| commit | d2a7523cb18e804e71eee16656deab364dbf6603 (patch) | |
| tree | 97c8e9adef462689c7c74f9efbdcefdd87ff8371 /src | |
| parent | ec79d7a66ec18874f3f0ac6a8e6e84f4d23be1d9 (diff) | |
| download | coreboot-d2a7523cb18e804e71eee16656deab364dbf6603.tar.xz | |
northbridge/amd/gx2/raminit.c Halt func needs noreturn attrib
Missing "__attribute__((noreturn))" on halt function. This sync's the
implementation to be the same as that of amd/lx thereby avoiding
compiler warnings.
Change-Id: Iead16125805eb36ff875fba767cf8d4e5aa86715
Signed-off-by: Edward O'Callaghan <eocallaghan@alterapraxis.com>
Reviewed-on: http://review.coreboot.org/6157
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <patrick@georgi-clan.de>
Diffstat (limited to 'src')
| -rw-r--r-- | src/northbridge/amd/gx2/raminit.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/northbridge/amd/gx2/raminit.c b/src/northbridge/amd/gx2/raminit.c index 71d0a16999..d9af16108f 100644 --- a/src/northbridge/amd/gx2/raminit.c +++ b/src/northbridge/amd/gx2/raminit.c @@ -26,7 +26,7 @@ static const unsigned char NumColAddr[] = { 0x08, 0x09, 0x0A, 0x0B, 0x0C, 0x0D, 0x0E, 0x0F, }; -static void hcf(void) +static void __attribute__((noreturn)) hcf(void) { printk(BIOS_EMERG, "DIE\n"); /* this guarantees we flush the UART fifos (if any) and also |