U-Boot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] arm: fdt_control: fix a build error with CONFIG_OF_EMBED=y
@ 2014-04-25 12:51 Masahiro Yamada
  2014-06-01 16:36 ` Simon Glass
  2014-06-05 22:45 ` [U-Boot] " Tom Rini
  0 siblings, 2 replies; 3+ messages in thread
From: Masahiro Yamada @ 2014-04-25 12:51 UTC (permalink / raw)
  To: u-boot

The build fails if a non-generic ARM board is compiled
with CONFIG_OF_EMBED=y.

The correct symbol name for embedded FDT is not __dtb_db_begin,
but __dtb_dt_begin. (A typo introduced by commit 6ab6b2af)

Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
---
 arch/arm/lib/board.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/lib/board.c b/arch/arm/lib/board.c
index 92e85c4..d5fabe4 100644
--- a/arch/arm/lib/board.c
+++ b/arch/arm/lib/board.c
@@ -277,7 +277,7 @@ void board_init_f(ulong bootflag)
 	gd->mon_len = (ulong)&__bss_end - (ulong)_start;
 #ifdef CONFIG_OF_EMBED
 	/* Get a pointer to the FDT */
-	gd->fdt_blob = __dtb_db_begin;
+	gd->fdt_blob = __dtb_dt_begin;
 #elif defined CONFIG_OF_SEPARATE
 	/* FDT is at end of image */
 	gd->fdt_blob = &_end;
-- 
1.8.3.2

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

* [U-Boot] [PATCH] arm: fdt_control: fix a build error with CONFIG_OF_EMBED=y
  2014-04-25 12:51 [U-Boot] [PATCH] arm: fdt_control: fix a build error with CONFIG_OF_EMBED=y Masahiro Yamada
@ 2014-06-01 16:36 ` Simon Glass
  2014-06-05 22:45 ` [U-Boot] " Tom Rini
  1 sibling, 0 replies; 3+ messages in thread
From: Simon Glass @ 2014-06-01 16:36 UTC (permalink / raw)
  To: u-boot

Hi Masahiro,

On 25 April 2014 06:51, Masahiro Yamada <yamada.m@jp.panasonic.com> wrote:
> The build fails if a non-generic ARM board is compiled
> with CONFIG_OF_EMBED=y.
>
> The correct symbol name for embedded FDT is not __dtb_db_begin,
> but __dtb_dt_begin. (A typo introduced by commit 6ab6b2af)
>
> Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>

This is a bug-fix.

Acked-by: Simon Glass <sjg@chromium.org>

This should be copied to the ARM maintainer.

Regards,
Simon

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

* [U-Boot] arm: fdt_control: fix a build error with CONFIG_OF_EMBED=y
  2014-04-25 12:51 [U-Boot] [PATCH] arm: fdt_control: fix a build error with CONFIG_OF_EMBED=y Masahiro Yamada
  2014-06-01 16:36 ` Simon Glass
@ 2014-06-05 22:45 ` Tom Rini
  1 sibling, 0 replies; 3+ messages in thread
From: Tom Rini @ 2014-06-05 22:45 UTC (permalink / raw)
  To: u-boot

On Fri, Apr 25, 2014 at 09:51:09PM +0900, Masahiro Yamada wrote:

> The build fails if a non-generic ARM board is compiled
> with CONFIG_OF_EMBED=y.
> 
> The correct symbol name for embedded FDT is not __dtb_db_begin,
> but __dtb_dt_begin. (A typo introduced by commit 6ab6b2af)
> 
> Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
> Acked-by: Simon Glass <sjg@chromium.org>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140605/d691d2b4/attachment.pgp>

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

end of thread, other threads:[~2014-06-05 22:45 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-04-25 12:51 [U-Boot] [PATCH] arm: fdt_control: fix a build error with CONFIG_OF_EMBED=y Masahiro Yamada
2014-06-01 16:36 ` Simon Glass
2014-06-05 22:45 ` [U-Boot] " Tom Rini

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