Martin L Roth has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/83857?usp=email )
Change subject: util/marvell/doimage_mv: Update CFLAGS to CFLAGS_doimage ......................................................................
util/marvell/doimage_mv: Update CFLAGS to CFLAGS_doimage
Because we import the doimage_mv/Makefile.mk directly into the main coreboot makefile, this was setting a global CFLAGS variable. That's not a huge issue since coreboot doesn't use the global CFLAGS variable, but it should be updated anyway.
Signed-off-by: Martin Roth gaumless@gmail.com Change-Id: Iebe281ca891745c1f993df274ef75b1440059c85 --- M util/marvell/doimage_mv/Makefile.mk 1 file changed, 3 insertions(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/57/83857/1
diff --git a/util/marvell/doimage_mv/Makefile.mk b/util/marvell/doimage_mv/Makefile.mk index 3f4f528..6f3a401 100644 --- a/util/marvell/doimage_mv/Makefile.mk +++ b/util/marvell/doimage_mv/Makefile.mk @@ -3,13 +3,13 @@ DOIMAGE_FOLDER = marvell/doimage_mv DOIMAGE_BINARY = doimage
-CFLAGS = -g -O1 -I./../inc -DMV_CPU_LE -I $(TOP)/src/commonlib/bsd/include +CFLAGS_doimage = -g -O1 -I./../inc -DMV_CPU_LE -I $(TOP)/src/commonlib/bsd/include
$(objutil)/$(DOIMAGE_FOLDER): mkdir -p $@
$(objutil)/$(DOIMAGE_FOLDER)/doimage.o: $(top)/util/marvell/doimage_mv/doimage.c $(objutil)/$(DOIMAGE_FOLDER) - $(HOSTCC) $(CFLAGS) -c $< -o $@ + $(HOSTCC) $(CFLAGS_doimage) -c $< -o $@
$(objutil)/$(DOIMAGE_FOLDER)/$(DOIMAGE_BINARY): $(objutil)/$(DOIMAGE_FOLDER)/doimage.o - $(HOSTCC) $(CFLAGS) -o $@ $^ + $(HOSTCC) $(CFLAGS_doimage) -o $@ $^