GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

northbridge/amd/gx2,lx: Treat MSR constant as unsigned long 

Clang complains that a signed shift result (0x210000000)
requires 35 bits to represent, but 'int' only has 32 bits.
However, we write the high bits separately and so this is
a spurious warning.

Change-Id: I3e1c57334077feb50004d7b39abff4bd84ca095b
Signed-off-by: Edward O'Callaghan <eocallaghan@alterapraxis.com>
Reviewed-on: http://review.coreboot.org/7673
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Alexandru Gagniuc <mr.nuke.me@gmail.com>
Tested-by: build bot (Jenkins)
This commit is contained in:
Edward O'Callaghan 2014-12-07 03:03:08 +11:00
parent ddb7a9d4a1
commit 2d072d415b
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/northbridge/amd/gx2/northbridgeinit.c
View File

@ -548,7 +548,7 @@ static void rom_shadow_settings(void)
* SYSRC(7:0) = 00h ; writeback, can set to 08h to make writethrough * SYSRC(7:0) = 00h ; writeback, can set to 08h to make writethrough
*/ */
#define SYSMEM_RCONF_WRITETHROUGH 8 #define SYSMEM_RCONF_WRITETHROUGH 8
#define DEVRC_RCONF_DEFAULT 0x21 #define DEVRC_RCONF_DEFAULT 0x21ul
#define ROMBASE_RCONF_DEFAULT 0xFFFC0000 #define ROMBASE_RCONF_DEFAULT 0xFFFC0000
#define ROMRC_RCONF_DEFAULT 0x25 #define ROMRC_RCONF_DEFAULT 0x25

2
src/northbridge/amd/lx/northbridgeinit.c
View File

@ -602,7 +602,7 @@ static void rom_shadow_settings(void)
* *
***************************************************************************/ ***************************************************************************/
#define SYSMEM_RCONF_WRITETHROUGH 8 #define SYSMEM_RCONF_WRITETHROUGH 8
#define DEVRC_RCONF_DEFAULT 0x21 #define DEVRC_RCONF_DEFAULT 0x21ul
#define ROMBASE_RCONF_DEFAULT 0xFFFC0000 #define ROMBASE_RCONF_DEFAULT 0xFFFC0000
#define ROMRC_RCONF_DEFAULT 0x25 #define ROMRC_RCONF_DEFAULT 0x25