public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] arm: Tegra2: Fix out of tree builds
@ 2011-04-05 21:33 Anton Staaf
  2011-04-14 21:29 ` Albert ARIBAUD
  0 siblings, 1 reply; 3+ messages in thread
From: Anton Staaf @ 2011-04-05 21:33 UTC (permalink / raw)
  To: u-boot

Create the board/nvidia/common directory in the out of tree build
directory.

Without this patch building out of tree results in a failure to create
the board.o file in the build directory tree.

Signed-off-by: Anton Staaf <robotboy@chromium.org>
Cc: Tom Warren <twarren@nvidia.com>
Cc: Albert ARIBAUD <albert.aribaud@free.fr>
---

An alternative solution would be to create a Makefile in the common directory
that creates an object/library that each board depends on.  I'm not sure which
is better.

 board/nvidia/harmony/Makefile  |    2 ++
 board/nvidia/seaboard/Makefile |    2 ++
 2 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/board/nvidia/harmony/Makefile b/board/nvidia/harmony/Makefile
index 3a146cb..4f4373b 100644
--- a/board/nvidia/harmony/Makefile
+++ b/board/nvidia/harmony/Makefile
@@ -24,6 +24,8 @@

 include $(TOPDIR)/config.mk

+$(shell mkdir -p $(obj)../common)
+
 LIB    = $(obj)lib$(BOARD).o

 COBJS  += ../common/board.o
diff --git a/board/nvidia/seaboard/Makefile b/board/nvidia/seaboard/Makefile
index 3a146cb..4f4373b 100644
--- a/board/nvidia/seaboard/Makefile
+++ b/board/nvidia/seaboard/Makefile
@@ -24,6 +24,8 @@

 include $(TOPDIR)/config.mk

+$(shell mkdir -p $(obj)../common)
+
 LIB    = $(obj)lib$(BOARD).o

 COBJS  += ../common/board.o
-- 
1.7.3.1

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [U-Boot] [PATCH] arm: Tegra2: Fix out of tree builds
  2011-04-05 21:33 [U-Boot] [PATCH] arm: Tegra2: Fix out of tree builds Anton Staaf
@ 2011-04-14 21:29 ` Albert ARIBAUD
  2011-04-14 22:33   ` Anton Staaf
  0 siblings, 1 reply; 3+ messages in thread
From: Albert ARIBAUD @ 2011-04-14 21:29 UTC (permalink / raw)
  To: u-boot

Hi Anton,

Sorry for the dela in replying.

Le 05/04/2011 23:33, Anton Staaf a ?crit :
> Create the board/nvidia/common directory in the out of tree build
> directory.
>
> Without this patch building out of tree results in a failure to create
> the board.o file in the build directory tree.
>
> Signed-off-by: Anton Staaf<robotboy@chromium.org>
> Cc: Tom Warren<twarren@nvidia.com>
> Cc: Albert ARIBAUD<albert.aribaud@free.fr>
> ---
>
> An alternative solution would be to create a Makefile in the common directory
> that creates an object/library that each board depends on.  I'm not sure which
> is better.

I'd prefer the alternative solution -- the issue is with a common 
directory, so it seems logical that the solution is in the common part 
of the code rather than spread over the non-common variants.

Amicalement,
-- 
Albert.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [U-Boot] [PATCH] arm: Tegra2: Fix out of tree builds
  2011-04-14 21:29 ` Albert ARIBAUD
@ 2011-04-14 22:33   ` Anton Staaf
  0 siblings, 0 replies; 3+ messages in thread
From: Anton Staaf @ 2011-04-14 22:33 UTC (permalink / raw)
  To: u-boot

No worries.

On Thu, Apr 14, 2011 at 2:29 PM, Albert ARIBAUD
<albert.u.boot@aribaud.net> wrote:
>>
>> An alternative solution would be to create a Makefile in the common
>> directory
>> that creates an object/library that each board depends on. ?I'm not sure
>> which
>> is better.
>
> I'd prefer the alternative solution -- the issue is with a common directory,
> so it seems logical that the solution is in the common part of the code
> rather than spread over the non-common variants.
>

Yes, that makes perfect sense.  I'll make the change.

Thanks,
    Anton

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2011-04-14 22:33 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-04-05 21:33 [U-Boot] [PATCH] arm: Tegra2: Fix out of tree builds Anton Staaf
2011-04-14 21:29 ` Albert ARIBAUD
2011-04-14 22:33   ` Anton Staaf

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox