diff mbox

[U-Boot,1/5] mx53loco: Allow booting a zImage kernel

Message ID 1351107870-5255-1-git-send-email-festevam@gmail.com
State Awaiting Upstream
Delegated to: Stefano Babic
Headers show

Commit Message

Fabio Estevam Oct. 24, 2012, 7:44 p.m. UTC
From: Fabio Estevam <fabio.estevam@freescale.com>

Allow booting a zImage kernel.

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
 include/configs/mx53loco.h |    1 +
 1 file changed, 1 insertion(+)

Comments

Robert Nelson Oct. 25, 2012, 2:07 a.m. UTC | #1
On Wed, Oct 24, 2012 at 2:44 PM, Fabio Estevam <festevam@gmail.com> wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
>
> Allow booting a zImage kernel.
>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
>  include/configs/mx53loco.h |    1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/include/configs/mx53loco.h b/include/configs/mx53loco.h
> index 0658dd3..25672c0 100644
> --- a/include/configs/mx53loco.h
> +++ b/include/configs/mx53loco.h
> @@ -103,6 +103,7 @@
>
>  /* Command definition */
>  #include <config_cmd_default.h>
> +#define CONFIG_CMD_BOOTZ

Fabio,

Any thoughts on also enabling "define CONFIG_SUPPORT_RAW_INITRD" at
the same time, otherwise you'd still have to run mkimage on the other
half "initrd.img-$(uname -r)" when using booting an initramfs with
your main kernel image..

Regards,
Fabio Estevam Oct. 25, 2012, 2:15 a.m. UTC | #2
Hi Robert,

On Thu, Oct 25, 2012 at 12:07 AM, Robert Nelson <robertcnelson@gmail.com> wrote:

> Fabio,
>
> Any thoughts on also enabling "define CONFIG_SUPPORT_RAW_INITRD" at
> the same time, otherwise you'd still have to run mkimage on the other
> half "initrd.img-$(uname -r)" when using booting an initramfs with
> your main kernel image..

Sure, no problem. We can enable CONFIG_SUPPORT_RAW_INITRD as well.

Stefano,

Should I enable  CONFIG_SUPPORT_RAW_INITRD as part of the same patch
series or on a separate one?

Thanks,

Fabio Estevam
Stefano Babic Oct. 25, 2012, 7 a.m. UTC | #3
Am 25/10/2012 04:15, schrieb Fabio Estevam:
> Hi Robert,
> 
> On Thu, Oct 25, 2012 at 12:07 AM, Robert Nelson <robertcnelson@gmail.com> wrote:
> 
>> Fabio,
>>
>> Any thoughts on also enabling "define CONFIG_SUPPORT_RAW_INITRD" at
>> the same time, otherwise you'd still have to run mkimage on the other
>> half "initrd.img-$(uname -r)" when using booting an initramfs with
>> your main kernel image..
> 
> Sure, no problem. We can enable CONFIG_SUPPORT_RAW_INITRD as well.
> 
> Stefano,
> 
> Should I enable  CONFIG_SUPPORT_RAW_INITRD as part of the same patch
> series or on a separate one?

Your decision, both are fine with me.

Regards,

Stefano Babic
Fabio Estevam Oct. 25, 2012, 11:16 a.m. UTC | #4
On Thu, Oct 25, 2012 at 5:00 AM, Stefano Babic <sbabic@denx.de> wrote:

>> Should I enable  CONFIG_SUPPORT_RAW_INITRD as part of the same patch
>> series or on a separate one?
>
> Your decision, both are fine with me.

Ok, then I prefer to keep this series as is.

I will send a separate one for adding CONFIG_SUPPORT_RAW_INITRD  only
for mx53loco as requested by Robert.

Regards,

Fabio Estevam
Stefano Babic Oct. 25, 2012, 11:25 a.m. UTC | #5
Am 25/10/2012 13:16, schrieb Fabio Estevam:
> On Thu, Oct 25, 2012 at 5:00 AM, Stefano Babic <sbabic@denx.de> wrote:
> 
>>> Should I enable  CONFIG_SUPPORT_RAW_INITRD as part of the same patch
>>> series or on a separate one?
>>
>> Your decision, both are fine with me.
> 
> Ok, then I prefer to keep this series as is.
> 
> I will send a separate one for adding CONFIG_SUPPORT_RAW_INITRD  only
> for mx53loco as requested by Robert.

Ok, fine. This series is then ready to be merged.

Regards,
Stefano Babic
Stefano Babic Oct. 28, 2012, 11:53 a.m. UTC | #6
On 24/10/2012 21:44, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
> 
> Allow booting a zImage kernel.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
>  include/configs/mx53loco.h |    1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/include/configs/mx53loco.h b/include/configs/mx53loco.h
> index 0658dd3..25672c0 100644
> --- a/include/configs/mx53loco.h
> +++ b/include/configs/mx53loco.h
> @@ -103,6 +103,7 @@
>  
>  /* Command definition */
>  #include <config_cmd_default.h>
> +#define CONFIG_CMD_BOOTZ
>  
>  #undef CONFIG_CMD_IMLS
>  
> 

Applied to u-boot-imx, thanks.

Best regards,
Stefano Babic
diff mbox

Patch

diff --git a/include/configs/mx53loco.h b/include/configs/mx53loco.h
index 0658dd3..25672c0 100644
--- a/include/configs/mx53loco.h
+++ b/include/configs/mx53loco.h
@@ -103,6 +103,7 @@ 
 
 /* Command definition */
 #include <config_cmd_default.h>
+#define CONFIG_CMD_BOOTZ
 
 #undef CONFIG_CMD_IMLS