[LinuxBIOS] r2893 - in trunk/LinuxBIOSv2: src/console src/southbridge/intel/i82801xx util/options

svn at openbios.org svn at openbios.org
Wed Oct 24 13:12:15 CEST 2007


Author: stepan
Date: 2007-10-24 13:12:15 +0200 (Wed, 24 Oct 2007)
New Revision: 2893

Modified:
   trunk/LinuxBIOSv2/src/console/vtxprintf.c
   trunk/LinuxBIOSv2/src/southbridge/intel/i82801xx/i82801xx_pci.c
   trunk/LinuxBIOSv2/util/options/build_opt_tbl.c
Log:
smaller changes to silence build warnings. (trivial)

Signed-off-by: Stefan Reinauer <stepan at coresystems.de>
Acked-by: Stefan Reinauer <stepan at coresystems.de>



Modified: trunk/LinuxBIOSv2/src/console/vtxprintf.c
===================================================================
--- trunk/LinuxBIOSv2/src/console/vtxprintf.c	2007-10-24 11:10:21 UTC (rev 2892)
+++ trunk/LinuxBIOSv2/src/console/vtxprintf.c	2007-10-24 11:12:15 UTC (rev 2893)
@@ -13,6 +13,11 @@
 #define is_digit isdigit
 #define isxdigit(c)	(((c) >= '0' && (c) <= '9') || ((c) >= 'a' && (c) <= 'f') || ((c) >= 'A' && (c) <= 'F'))
 
+#if 0
+/* We are using number() instead. So this code is obsoleted and should
+ * probably go away.
+ */
+
 static unsigned long simple_strtoul(const char *cp,char **endp,unsigned int base)
 {
 	unsigned long result = 0,value;
@@ -44,8 +49,8 @@
 		return -simple_strtoul(cp+1,endp,base);
 	return simple_strtoul(cp,endp,base);
 }
+#endif
 
-
 static int skip_atoi(const char **s)
 {
 	int i=0;

Modified: trunk/LinuxBIOSv2/src/southbridge/intel/i82801xx/i82801xx_pci.c
===================================================================
--- trunk/LinuxBIOSv2/src/southbridge/intel/i82801xx/i82801xx_pci.c	2007-10-24 11:10:21 UTC (rev 2892)
+++ trunk/LinuxBIOSv2/src/southbridge/intel/i82801xx/i82801xx_pci.c	2007-10-24 11:12:15 UTC (rev 2893)
@@ -26,7 +26,6 @@
 
 static void pci_init(struct device *dev)
 {
-	uint32_t reg32;
 	uint16_t reg16;
 
 	/* Clear system errors */

Modified: trunk/LinuxBIOSv2/util/options/build_opt_tbl.c
===================================================================
--- trunk/LinuxBIOSv2/util/options/build_opt_tbl.c	2007-10-24 11:10:21 UTC (rev 2892)
+++ trunk/LinuxBIOSv2/util/options/build_opt_tbl.c	2007-10-24 11:12:15 UTC (rev 2893)
@@ -217,7 +217,7 @@
 	struct cmos_entries *ce;
 	struct cmos_enums *c_enums, *c_enums_start;
 	struct cmos_checksum *cs;
-	unsigned char line[INPUT_LINE_MAX];
+	char line[INPUT_LINE_MAX];
 	unsigned char uc;
 	int entry_mode=0;
 	int enum_mode=0;
@@ -229,7 +229,7 @@
 	int entries_length;
 	int enum_length;
 	int len;
-	unsigned char buf[16];
+	char buf[16];
 
         for(i=1;i<argc;i++) {
                 if(argv[i][0]!='-') {
@@ -332,7 +332,7 @@
 			fprintf(stderr, "Error - Length is to long in line \n%s\n",line);
 			exit(1);
 		}
-		if (!is_ident(ce->name)) {
+		if (!is_ident((char *)ce->name)) {
 			fprintf(stderr, 
 				"Error - Name %s is an invalid identifier in line\n %s\n", 
 				ce->name, line);
@@ -341,7 +341,7 @@
 		/* put in the record type */
 		ce->tag=LB_TAG_OPTION;
 		/* calculate and save the record length */
-		len=strlen(ce->name)+1;
+		len=strlen((char *)ce->name)+1;
 		/* make the record int aligned */
 		if(len%4)
 			len+=(4-(len%4));
@@ -541,7 +541,7 @@
 			if (ce->config == 'r') {
 				continue;
 			}
-			if (!is_ident(ce->name)) {
+			if (!is_ident((char *)ce->name)) {
 				fprintf(stderr, "Invalid identifier: %s\n",
 					ce->name);
 				exit(1);





More information about the coreboot mailing list