[U-Boot] [PATCH] Makefile: fix HAVE_VENDOR_COMMON_LIB

[U-Boot] [PATCH] Makefile: fix HAVE_VENDOR_COMMON_LIB

[Scott Wood]

Commit 8b5a02640adf77301f943e8754992c50df004e8a ("Makefile: cosmetic:
optimize usage of LIBS-y") broke the build of boards that have a board
vendor "common" directory, by introducing a space between "LIBS-" and
"y".

Signed-off-by: Scott Wood <scottwood@freescale.com>
---
 Makefile |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Makefile b/Makefile
index 5ce5cc3..947f3ff 100644
--- a/Makefile
+++ b/Makefile
@@ -225,7 +225,7 @@ endif
 
 OBJS := $(addprefix $(obj),$(OBJS))
 
-HAVE_VENDOR_COMMON_LIB = $(if $(wildcard board/$(VENDOR)/common/Makefile), y, n)
+HAVE_VENDOR_COMMON_LIB = $(if $(wildcard board/$(VENDOR)/common/Makefile),y,n)
 
 LIBS-y += lib/libgeneric.o
 LIBS-y += lib/lzma/liblzma.o
-- 
1.7.9.5

[U-Boot] [PATCH v2] Makefile: fix HAVE_VENDOR_COMMON_LIB

[Daniel Schwierzeck]

From: Scott Wood <scottwood@freescale.com>

Commit 8b5a02640adf77301f943e8754992c50df004e8a ("Makefile: cosmetic:
optimize usage of LIBS-y") broke the build of boards that have a board
vendor "common" directory, by introducing a space between "LIBS-" and
"y".

Signed-off-by: Scott Wood <scottwood@freescale.com>
Signed-off-by: Daniel Schwierzeck <daniel.schwierzeck@gmail.com>
---
Changes vor v2:
- fix the wrong spaces also in spl/Makefile

Tested with:
MAKEALL spear600
MAKEALL -s omap3
MAKEALL -s omap4
---
 Makefile     | 2 +-
 spl/Makefile | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/Makefile b/Makefile
index 5ce5cc3..947f3ff 100644
--- a/Makefile
+++ b/Makefile
@@ -225,7 +225,7 @@ endif

 OBJS := $(addprefix $(obj),$(OBJS))

-HAVE_VENDOR_COMMON_LIB = $(if $(wildcard board/$(VENDOR)/common/Makefile), y, n)
+HAVE_VENDOR_COMMON_LIB = $(if $(wildcard board/$(VENDOR)/common/Makefile),y,n)

 LIBS-y += lib/libgeneric.o
 LIBS-y += lib/lzma/liblzma.o
diff --git a/spl/Makefile b/spl/Makefile
index e9ecb9b..2cf7bda 100644
--- a/spl/Makefile
+++ b/spl/Makefile
@@ -23,7 +23,7 @@ include $(TOPDIR)/config.mk
 # We want the final binaries in this directory
 obj := $(OBJTREE)/spl/

-HAVE_VENDOR_COMMON_LIB = $(if $(wildcard $(SRCTREE)/board/$(VENDOR)/common/Makefile), y, n)
+HAVE_VENDOR_COMMON_LIB = $(if $(wildcard $(SRCTREE)/board/$(VENDOR)/common/Makefile),y,n)

 ifdef	CONFIG_SPL_START_S_PATH
 START_PATH := $(subst ",,$(CONFIG_SPL_START_S_PATH))
--
1.7.11.3

[U-Boot] [PATCH v2] Makefile: fix HAVE_VENDOR_COMMON_LIB

[Kim Phillips]

On Tue, 14 Aug 2012 13:44:29 +0200
Daniel Schwierzeck <daniel.schwierzeck@gmail.com> wrote:

> From: Scott Wood <scottwood@freescale.com>
> 
> Commit 8b5a02640adf77301f943e8754992c50df004e8a ("Makefile: cosmetic:
> optimize usage of LIBS-y") broke the build of boards that have a board
> vendor "common" directory, by introducing a space between "LIBS-" and
> "y".
> 
> Signed-off-by: Scott Wood <scottwood@freescale.com>
> Signed-off-by: Daniel Schwierzeck <daniel.schwierzeck@gmail.com>
> ---
> Changes vor v2:
> - fix the wrong spaces also in spl/Makefile
> 
> Tested with:
> MAKEALL spear600
> MAKEALL -s omap3
> MAKEALL -s omap4
> ---

this fixes newly broken mpc83xx builds, e.g.:

Configuring for MPC832XEMDS_ATM - Board: MPC832XEMDS, Options: PQ_MDS_PIB=1,PQ_MDS_PIB_ATM=1
make: *** [u-boot] Error 1
powerpc-fsl-linux-size: './u-boot': No such file
board/freescale/mpc832xemds/libmpc832xemds.o: In function `board_early_init_r':
/home/r1aaha/git/u-boot/board/freescale/mpc832xemds/mpc832xemds.c:90: undefined reference to `pib_init'
make: *** [u-boot] Error 1

So,

Acked-by: Kim Phillips <kim.phillips@freescale.com>

Kim

[U-Boot] [PATCH v2] Makefile: fix HAVE_VENDOR_COMMON_LIB

[Detlev Zundel]

Hi,

> On Tue, 14 Aug 2012 13:44:29 +0200
> Daniel Schwierzeck <daniel.schwierzeck@gmail.com> wrote:
>
>> From: Scott Wood <scottwood@freescale.com>
>> 
>> Commit 8b5a02640adf77301f943e8754992c50df004e8a ("Makefile: cosmetic:
>> optimize usage of LIBS-y") broke the build of boards that have a board
>> vendor "common" directory, by introducing a space between "LIBS-" and
>> "y".
>> 
>> Signed-off-by: Scott Wood <scottwood@freescale.com>
>> Signed-off-by: Daniel Schwierzeck <daniel.schwierzeck@gmail.com>
>> ---
>> Changes vor v2:
>> - fix the wrong spaces also in spl/Makefile
>> 
>> Tested with:
>> MAKEALL spear600
>> MAKEALL -s omap3
>> MAKEALL -s omap4
>> ---
>
> this fixes newly broken mpc83xx builds, e.g.:
>
> Configuring for MPC832XEMDS_ATM - Board: MPC832XEMDS, Options:
> PQ_MDS_PIB=1,PQ_MDS_PIB_ATM=1
> make: *** [u-boot] Error 1
> powerpc-fsl-linux-size: './u-boot': No such file
> board/freescale/mpc832xemds/libmpc832xemds.o: In function
> board_early_init_r':
> /home/r1aaha/git/u-boot/board/freescale/mpc832xemds/mpc832xemds.c:90:
> undefined reference to `pib_init'
> make: *** [u-boot] Error 1
>
> So,
>
> Acked-by: Kim Phillips <kim.phillips@freescale.com>

Applied, thanks.

Cheers
  Detlev

-- 
Science is a way of thinking
much more than it is a body of knowledge.
             -- Carl Sagan
--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: dzu at denx.de