From 932bb6a0eb72ead9a983da964f00c1d24f612e7d Mon Sep 17 00:00:00 2001 From: Paul Menzel Date: Sun, 7 May 2017 09:05:55 +0200 Subject: [PATCH] UPSTREAM: intelmetool: Enable warning *set-but-unused-variable* There are no GCC warnings anymore about set but unused variables, and Clang warns about this switch, as it doesnt know it. So remove the switch to use the default set by the switch `Wall`. BUG=none BRANCH=none TEST=none Change-Id: I2b36e2f6e972a95014319d544735403bcf2081c6 Signed-off-by: Patrick Georgi Original-Commit-Id: 10984d1034a5525d590b9501f4d34b44bccc52be Original-Change-Id: Ie9eb26d4f8b298af231b952b547b71d68c649eaf Original-Signed-off-by: Paul Menzel Original-Reviewed-on: https://review.coreboot.org/19613 Original-Tested-by: build bot (Jenkins) Original-Reviewed-by: Philippe Mathieu-Daud Original-Reviewed-by: Philipp Deppenwiese Reviewed-on: https://chromium-review.googlesource.com/506180 Commit-Ready: Patrick Georgi Tested-by: Patrick Georgi Reviewed-by: Patrick Georgi --- util/intelmetool/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/util/intelmetool/Makefile b/util/intelmetool/Makefile index affcf30d90..8857d954cd 100644 --- a/util/intelmetool/Makefile +++ b/util/intelmetool/Makefile @@ -17,7 +17,7 @@ PROGRAM = intelmetool CC ?= gcc INSTALL ?= /usr/bin/install PREFIX ?= /usr/local -CFLAGS ?= -O0 -g -Wall -W -Wno-unused-parameter -Wno-unused-but-set-variable -Wno-sign-compare -Wno-unused-function +CFLAGS ?= -O0 -g -Wall -W -Wno-unused-parameter -Wno-sign-compare -Wno-unused-function LDFLAGS += -lpci -lz OBJS = intelmetool.o me.o me_status.o mmap.o