From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux)) id 1d8jdR-0005yL-Tt for barebox@lists.infradead.org; Thu, 11 May 2017 08:39:47 +0000 Date: Thu, 11 May 2017 10:39:24 +0200 From: Sascha Hauer Message-ID: <20170511083924.n7lxxmpjkl65k4xh@pengutronix.de> References: <20170510174949.18631-1-rockford@yandex.ru> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20170510174949.18631-1-rockford@yandex.ru> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "barebox" Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: [PATCH] Fix linking with new ld, based on u-boot To: Andrey Panov Cc: barebox@lists.infradead.org Hi Andrey, The patch looks fine, but lacks a signed-off-by. Could you resend? Sascha On Wed, May 10, 2017 at 08:49:49PM +0300, Andrey Panov wrote: > --- > Makefile | 3 +++ > images/Makefile | 3 +++ > scripts/Kbuild.include | 2 +- > 3 files changed, 7 insertions(+), 1 deletion(-) > > diff --git a/Makefile b/Makefile > index afa4bf802..c78f66f73 100644 > --- a/Makefile > +++ b/Makefile > @@ -306,6 +306,9 @@ AFLAGS := -D__ASSEMBLY__ > > LDFLAGS_barebox := -Map barebox.map > > +# Avoid 'Not enough room for program headers' error on binutils 2.28 onwards. > +LDFLAGS_barebox += $(call ld-option, --no-dynamic-linker) > + > # Read KERNELRELEASE from include/config/kernel.release (if it exists) > KERNELRELEASE = $(shell cat include/config/kernel.release 2> /dev/null) > KERNELVERSION = $(VERSION).$(PATCHLEVEL).$(SUBLEVEL)$(EXTRAVERSION) > diff --git a/images/Makefile b/images/Makefile > index adf950aa9..8c5dac4bf 100644 > --- a/images/Makefile > +++ b/images/Makefile > @@ -53,6 +53,9 @@ extra-y += $(pbl-lds) > $(pbl-lds): $(obj)/../arch/$(ARCH)/lib/pbl.lds.S FORCE > $(call if_changed_dep,cpp_lds_S) > > +# Avoid 'Not enough room for program headers' error on binutils 2.28 onwards. > +LDFLAGS += $(call ld-option, --no-dynamic-linker) > + > quiet_cmd_elf__ ?= LD $@ > cmd_elf__ ?= $(LD) $(LDFLAGS) --gc-sections -pie \ > -e $(2) -Map $@.map $(LDFLAGS_$(@F)) -o $@ \ > diff --git a/scripts/Kbuild.include b/scripts/Kbuild.include > index c7faf67a2..310dcdc94 100644 > --- a/scripts/Kbuild.include > +++ b/scripts/Kbuild.include > @@ -148,7 +148,7 @@ cc-ldoption = $(call try-run,\ > # ld-option > # Usage: LDFLAGS += $(call ld-option, -X) > ld-option = $(call try-run,\ > - $(CC) /dev/null -c -o "$$TMPO" ; $(LD) $(1) "$$TMPO" -o "$$TMP",$(1),$(2)) > + $(CC) -x c /dev/null -c -o "$$TMPO" ; $(LD) $(1) "$$TMPO" -o "$$TMP",$(1),$(2)) > > # ar-option > # Usage: KBUILD_ARFLAGS := $(call ar-option,D) > -- > 2.11.0 > > > _______________________________________________ > barebox mailing list > barebox@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/barebox > -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox