diff --git a/util/bincfg/bincfg.l b/util/bincfg/bincfg.l
index ca93547648..e55d6c609a 100644
--- a/util/bincfg/bincfg.l
+++ b/util/bincfg/bincfg.l
@@ -11,7 +11,7 @@ extern struct blob binary;
 
 unsigned int parsehex (char *s)
 {
-	unsigned int i, nib, val = 0;
+	unsigned int i, nib, retval = 0;
 	unsigned int nibs = strlen(s) - 2;
 
 	for (i = 2; i < nibs + 2; i++) {
@@ -24,9 +24,9 @@ unsigned int parsehex (char *s)
 		} else {
 			return 0;
 		}
-		val |= nib << (((nibs - 1) - (i - 2)) * 4);
+		retval |= nib << (((nibs - 1) - (i - 2)) * 4);
 	}
-	return val;
+	return retval;
 }
 
 char* stripquotes (char *string)
diff --git a/util/bincfg/bincfg.y b/util/bincfg/bincfg.y
index 0aaf13af28..43b3a00e0d 100644
--- a/util/bincfg/bincfg.y
+++ b/util/bincfg/bincfg.y
@@ -223,7 +223,7 @@ static void generate_setter_bitfields(FILE* fp, unsigned char *bin)
 
 static void generate_binary_with_gbe_checksum(FILE* fp)
 {
-	int i;
+	unsigned int i;
 	unsigned short checksum;
 
 	/* traverse spec, push to blob and add up for checksum */
@@ -407,6 +407,7 @@ setpair:
 /* Called by yyparse on error.  */
 static void yyerror (FILE* fp, char const *s)
 {
+	(void)fp;
 	fprintf (stderr, "yyerror: %s\n", s);
 }
 
@@ -448,8 +449,6 @@ int main (int argc, char *argv[])
 {
 	unsigned int lenspec;
 	unsigned char *parsestring;
-	unsigned char c;
-	unsigned int pos = 0;
 	int ret = 0;
 	FILE* fp;