From 0bb62907eb65788768168d735bce466d743ac393 Mon Sep 17 00:00:00 2001 From: Martin Roth Date: Tue, 16 Feb 2021 16:52:46 -0700 Subject: [PATCH] util/ectool: Update Makefile - Add a help target - Add the -Wshadow and -Werror options - Add a way to disable -Werror Signed-off-by: Martin Roth Change-Id: I0d9fe5beb3a2e103a0bf4603712c3a5ed15f93be Reviewed-on: https://review.coreboot.org/c/coreboot/+/50850 Tested-by: build bot (Jenkins) Reviewed-by: Patrick Georgi Reviewed-by: Angel Pons --- util/ectool/Makefile | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/util/ectool/Makefile b/util/ectool/Makefile index 75504c73b6d8..1ef74d25f271 100644 --- a/util/ectool/Makefile +++ b/util/ectool/Makefile @@ -1,7 +1,8 @@ ## SPDX-License-Identifier: GPL-2.0-only CC = gcc -CFLAGS = -O2 -Wall -Wextra +WERROR=-Werror +CFLAGS = -O2 -Wall -Wextra -Wshadow $(WERROR) PROGRAM = ectool INSTALL = /usr/bin/env install PREFIX = /usr/local @@ -31,4 +32,10 @@ distclean: clean %.o: %.c $(CC) $(CFLAGS) -c $^ -I. -o $@ -.PHONY: all install clean distclean +help: + @echo "${PROGRAM}: Dump RAM of Embedded Controller (EC)" + @echo "Targets: all, clean, distclean, help, install" + @echo "To disable warnings as errors, run make as:" + @echo " make all WERROR=\"\"" + +.PHONY: all clean distclean help install