Change in coreboot[master]: util/bucts: Clean up Makefile to match others

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/50847 )

Change subject: util/bucts: Clean up Makefile to match others
......................................................................

util/bucts: Clean up Makefile to match others

- Add a TARGET variable
- Enable optimization and additional warnings
- Add distclean target
- Add help target

Signed-off-by: Martin Roth 
Change-Id: I8eb190abd1ab20da7dd1ae43ef0358ba91df000e
Reviewed-on: https://review.coreboot.org/c/coreboot/+/50847
Tested-by: build bot (Jenkins) 
Reviewed-by: Patrick Georgi 
Reviewed-by: Angel Pons 
---
M util/bucts/Makefile
1 file changed, 15 insertions(+), 4 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Patrick Georgi: Looks good to me, approved
  Angel Pons: Looks good to me, but someone else must approve

diff --git a/util/bucts/Makefile b/util/bucts/Makefile
index 3bc73ee..98f7314 100644
--- a/util/bucts/Makefile
+++ b/util/bucts/Makefile
@@ -1,21 +1,32 @@
 CC:=gcc
 OBJ:=bucts.o
+TARGET=bucts
 VERSION:=$(shell git describe)
-CFLAGS+=-Wall
+WERROR=-Werror
+CFLAGS=-O2 -Wall -Wextra -Wshadow ${WERROR}

ifeq ($(shell uname), FreeBSD)
    CFLAGS += -I/usr/local/include
    LDFLAGS += -L/usr/local/lib
 endif

-all: bucts
+all: $(TARGET)

-bucts: $(OBJ)
+$(TARGET): $(OBJ)
    $(CC) -o $@ $(OBJ) $(LDFLAGS) -lpci

%.o: %.c
    $(CC) $(CFLAGS) -DVERSION='"$(VERSION)"' -c $<

-.PHONY: clean
 clean:
    rm -f bucts $(OBJ)
+
+distclean: clean
+
+help:
+	@echo "${TARGET}: tool to manipulate the BUC.TS bit on Intel targets."
+	@echo "Targets: all, clean, distclean, help"
+	@echo "To disable warnings as errors, run make as:"
+	@echo "  make all WERROR="""
+
+.PHONY: all clean distclean help