All of lore.kernel.org
 help / color / mirror / Atom feed
* [RFC 2/5] arm: boot: allow zboot with appended DTB
@ 2013-07-30 17:57 Ulrich Hecht
  0 siblings, 0 replies; 2+ messages in thread
From: Ulrich Hecht @ 2013-07-30 17:57 UTC (permalink / raw)
  To: linux-sh

I don't see any reason why this should not be allowed.
---
 arch/arm/Kconfig |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index 0ac9be6..54ac595 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -1938,7 +1938,7 @@ endchoice
 
 config ARM_APPENDED_DTB
 	bool "Use appended device tree blob to zImage (EXPERIMENTAL)"
-	depends on OF && !ZBOOT_ROM
+	depends on OF
 	help
 	  With this option, the boot code will look for a device tree binary
 	  (DTB) appended to zImage
-- 
1.7.10.4


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

* Re: [RFC 2/5] arm: boot: allow zboot with appended DTB
@ 2013-08-01  9:39 Magnus Damm
  0 siblings, 0 replies; 2+ messages in thread
From: Magnus Damm @ 2013-08-01  9:39 UTC (permalink / raw)
  To: linux-sh

Hi Ulrich,

On Wed, Jul 31, 2013 at 2:57 AM, Ulrich Hecht <ulrich.hecht@gmail.com> wrote:
> I don't see any reason why this should not be allowed.
> ---
>  arch/arm/Kconfig |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> index 0ac9be6..54ac595 100644
> --- a/arch/arm/Kconfig
> +++ b/arch/arm/Kconfig
> @@ -1938,7 +1938,7 @@ endchoice
>
>  config ARM_APPENDED_DTB
>         bool "Use appended device tree blob to zImage (EXPERIMENTAL)"
> -       depends on OF && !ZBOOT_ROM
> +       depends on OF
>         help
>           With this option, the boot code will look for a device tree binary
>           (DTB) appended to zImage
> --
> 1.7.10.4

Thanks for your patches. I'm not sure about the reason behind this
!ZBOOT_ROM - perhaps the kernel should run from ROM initially somehow
and shouldn't be overwritten?

Regardless I believe the code handles DTB automatically in
arch/arm/boot/compressed/head-shmobile.S - at least in recent
renesas.git (I checked renesas-devel-20130801).

So based on that I don't think you need this patch. At least I've been
able to boot a zImage via USB Mask ROM on Mackerel with appended DTB
without setting ARM_APPENDED_DTB. The reason to special case our DTB
handling is that the MERAM used for DTB storage while booting from USB
Mask ROM can't really be used to pass the DTB to the kernel - so
because of that we need to relocate our DTB to RAM early on.

Cheers,

/ magnus

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

end of thread, other threads:[~2013-08-01  9:39 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-07-30 17:57 [RFC 2/5] arm: boot: allow zboot with appended DTB Ulrich Hecht
2013-08-01  9:39 Magnus Damm

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.