On Wed, Jul 25, 2018 at 01:36:14PM +0200, Philipp Rudo wrote: > In the zipl Makefile macros are defined with the sole purpose to define > other macros in zipl.h. Remove this unnecessary indirection. > > Signed-off-by: Philipp Rudo <prudo@xxxxxxxxxxxxx> > --- > zipl/include/zipl.h | 5 ----- > zipl/src/Makefile | 5 ++--- > 2 files changed, 2 insertions(+), 8 deletions(-) > > diff --git a/zipl/include/zipl.h b/zipl/include/zipl.h > index fb68ae88..770801e5 100644 > --- a/zipl/include/zipl.h > +++ b/zipl/include/zipl.h > @@ -48,11 +48,6 @@ > #define MENU_DEFAULT_PROMPT 0 > #define MENU_DEFAULT_TIMEOUT 0 > > -#define ZFCPDUMP_IMAGE STRINGIFY(ZFCPDUMP_DIR) "/" \ > - STRINGIFY(ZFCPDUMP_PART_IMAGE) > -#define ZFCPDUMP_INITRD STRINGIFY(ZFCPDUMP_DIR) "/" \ > - STRINGIFY(ZFCPDUMP_PART_RD) > - > #define MAX_DUMP_VOLUMES 32 > > /* Internal component load address type */ > diff --git a/zipl/src/Makefile b/zipl/src/Makefile > index b9018b4b..be14fce5 100644 > --- a/zipl/src/Makefile > +++ b/zipl/src/Makefile > @@ -2,9 +2,8 @@ > include ../../common.mak > > ALL_CPPFLAGS += -I../include -I../boot \ > - -DZFCPDUMP_DIR=$(ZFCPDUMP_DIR) \ > - -DZFCPDUMP_PART_IMAGE=$(ZFCPDUMP_IMAGE) \ > - -DZFCPDUMP_PART_RD=$(ZFCPDUMP_INITRD) \ > + -DZFCPDUMP_IMAGE="STRINGIFY($(ZFCPDUMP_DIR)/$(ZFCPDUMP_IMAGE))" \ > + -DZFCPDUMP_INITRD="STRINGIFY($(ZFCPDUMP_DIR)/$(ZFCPDUMP_INITRD))" \ > -D_FILE_OFFSET_BITS=64 $(NO_PIE_CFLAGS) > ALL_LDFLAGS += -Wl,-z,noexecstack $(NO_PIE_LDFLAGS) > Reviewed-by: Hendrik Brueckner <brueckner@xxxxxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-s390" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html