[coreboot-gerrit] New patch to review for coreboot: d1308fb nb/intel/i945/raminit.c: duplicate 'const' declaration specifier

Edward O'Callaghan (eocallaghan@alterapraxis.com) gerrit at coreboot.org
Sat May 24 22:38:47 CEST 2014


Edward O'Callaghan (eocallaghan at alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5843

-gerrit

commit d1308fb41f0974e5c8e30d4ed415a933bcf04966
Author: Edward O'Callaghan <eocallaghan at alterapraxis.com>
Date:   Sun May 25 06:24:39 2014 +1000

    nb/intel/i945/raminit.c: duplicate 'const' declaration specifier
    
    Spotted by Clang
    
    Change-Id: I7e91f3edfa773560131e267a7776d8bf1ff7e295
    Signed-off-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
---
 src/northbridge/intel/i945/raminit.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/northbridge/intel/i945/raminit.c b/src/northbridge/intel/i945/raminit.c
index 6943fd9..885b49d 100644
--- a/src/northbridge/intel/i945/raminit.c
+++ b/src/northbridge/intel/i945/raminit.c
@@ -1735,14 +1735,14 @@ static void sdram_set_timing_and_control(struct sys_info *sysinfo)
 	u32 temp_drt;
 	int i, page_size;
 
-	static const u8 const drt0_table[] = {
+	static const u8 drt0_table[] = {
 	  /* CL 3, 4, 5 */
 		3, 4, 5, 	/* FSB533/400, DDR533/400 */
 		4, 5, 6,	/* FSB667, DDR533/400 */
 		4, 5, 6,	/* FSB667, DDR667 */
 	};
 
-	static const u8 const cas_table[] = {
+	static const u8 cas_table[] = {
 		2, 1, 0, 3
 	};
 



More information about the coreboot-gerrit mailing list