From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hendrik Brueckner Date: Thu, 02 Aug 2018 09:57:26 +0000 Subject: Re: [PATCH 4/5] zipl: Move macro definition to Makefile Message-Id: <20180802095726.GD5361@linux.ibm.com> In-Reply-To: <31847bcc-32d6-b656-c285-bfb1fe87da23@redhat.com> References: <31847bcc-32d6-b656-c285-bfb1fe87da23@redhat.com> To: linux-s390@vger.kernel.org List-ID: 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 > --- > 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