[coreboot-gerrit] New patch to review for coreboot: 69a1e54 sconfig: don't "const" structs twice

Patrick Georgi (patrick@georgi-clan.de) gerrit at coreboot.org
Sat Jan 18 18:29:45 CET 2014


Patrick Georgi (patrick at georgi-clan.de) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/4713

-gerrit

commit 69a1e540a8e36ddea73267d68d846c685b75a99d
Author: Patrick Georgi <patrick at georgi-clan.de>
Date:   Sat Jan 18 16:23:32 2014 +0100

    sconfig: don't "const" structs twice
    
    It's useless and makes clang unhappy.
    
    Change-Id: If256b99aebabd87df30a3a078c5804330b82989b
    Signed-off-by: Patrick Georgi <patrick at georgi-clan.de>
---
 util/sconfig/main.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/util/sconfig/main.c b/util/sconfig/main.c
index 03ba1ef..74193e6 100644
--- a/util/sconfig/main.c
+++ b/util/sconfig/main.c
@@ -500,7 +500,7 @@ static void pass1(FILE *fil, struct device *ptr)
 	}
 	if ((ptr->type == chip) && (ptr->chiph_exists)) {
 		if (ptr->reg) {
-			fprintf(fil, "ROMSTAGE_CONST struct %s_config ROMSTAGE_CONST %s_info_%d = {\n",
+			fprintf(fil, "ROMSTAGE_CONST struct %s_config %s_info_%d = {\n",
 				ptr->name_underscore, ptr->name_underscore,
 				ptr->id);
 			struct reg *r = ptr->reg;
@@ -510,7 +510,7 @@ static void pass1(FILE *fil, struct device *ptr)
 			}
 			fprintf(fil, "};\n\n");
 		} else {
-			fprintf(fil, "ROMSTAGE_CONST struct %s_config ROMSTAGE_CONST %s_info_%d = { };\n",
+			fprintf(fil, "ROMSTAGE_CONST struct %s_config %s_info_%d = { };\n",
 				ptr->name_underscore, ptr->name_underscore, ptr->id);
 		}
 	}



More information about the coreboot-gerrit mailing list