mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Sascha Hauer <s.hauer@pengutronix.de>
To: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
Cc: barebox@lists.infradead.org
Subject: Re: [PATCH 2/4] ARM: OMAP: register OMAP specific barebox bootm handler
Date: Fri, 27 Sep 2013 10:37:22 +0200	[thread overview]
Message-ID: <20130927083722.GM30088@pengutronix.de> (raw)
In-Reply-To: <20130927082537.GD32444@ns203013.ovh.net>

On Fri, Sep 27, 2013 at 10:25:37AM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote:
> On 09:17 Fri 27 Sep     , Sascha Hauer wrote:
> > The OMAP ROM code passes the boot information via r0 to the
> > bootloader. Add an OMAP specific barebox handler to pass this
> > information to the next stage. This allows us to chainload
> > bootloaders without loosing the information where we booted from.
> 
> as you are working on the xload can test the patch I send to switch to the
> generic bootstap  code

I could test them on the beaglebone, yes.

Sascha

> 
> Best Regards,
> J.
> > 
> > Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
> > ---
> >  arch/arm/mach-omap/include/mach/generic.h |  1 +
> >  arch/arm/mach-omap/omap_generic.c         | 52 +++++++++++++++++++++++++++++++
> >  arch/arm/mach-omap/xload.c                | 10 ++----
> >  3 files changed, 55 insertions(+), 8 deletions(-)
> > 
> > diff --git a/arch/arm/mach-omap/include/mach/generic.h b/arch/arm/mach-omap/include/mach/generic.h
> > index ece8c2b..1cd7f04 100644
> > --- a/arch/arm/mach-omap/include/mach/generic.h
> > +++ b/arch/arm/mach-omap/include/mach/generic.h
> > @@ -56,6 +56,7 @@ static inline int omap_set_mmc_dev(const char *mmcdev)
> >  
> >  extern uint32_t omap_bootinfo[3];
> >  void omap_save_bootinfo(void *data);
> > +void __noreturn omap_start_barebox(void *barebox);
> >  
> >  void omap_set_bootmmc_devname(const char *devname);
> >  const char *omap_get_bootmmc_devname(void);
> > diff --git a/arch/arm/mach-omap/omap_generic.c b/arch/arm/mach-omap/omap_generic.c
> > index f024de6..595ecbe 100644
> > --- a/arch/arm/mach-omap/omap_generic.c
> > +++ b/arch/arm/mach-omap/omap_generic.c
> > @@ -15,12 +15,64 @@
> >  #include <common.h>
> >  #include <bootsource.h>
> >  #include <envfs.h>
> > +#include <boot.h>
> >  #include <init.h>
> >  #include <io.h>
> >  #include <fs.h>
> >  #include <malloc.h>
> >  #include <linux/stat.h>
> >  #include <mach/generic.h>
> > +#include <mach/am33xx-silicon.h>
> > +#include <mach/omap3-silicon.h>
> > +#include <mach/omap4-silicon.h>
> > +
> > +static void *omap_sram_start(void)
> > +{
> > +	if (cpu_is_am33xx())
> > +		return (void *)AM33XX_SRAM0_START;
> > +	if (cpu_is_omap34xx())
> > +		return (void *)OMAP3_SRAM_BASE;
> > +	if (cpu_is_omap4xxx())
> > +		return (void *)OMAP44XX_SRAM_BASE;
> > +}
> > +
> > +void __noreturn omap_start_barebox(void *barebox)
> > +{
> > +	int (*func)(void *) = barebox;
> > +	uint32_t *arg;
> > +	void *sramadr = omap_sram_start();
> > +
> > +	arg = (uint32_t *)&omap_bootinfo;
> > +
> > +	memcpy(sramadr, &omap_bootinfo, sizeof(uint32_t) * 3);
> > +
> > +	shutdown_barebox();
> > +	func(sramadr);
> > +	hang();
> > +}
> > +
> > +static int do_bootm_omap_barebox(struct image_data *data)
> > +{
> > +	void (*barebox)(uint32_t);
> > +
> > +	barebox = read_file(data->os_file, NULL);
> > +	if (!barebox)
> > +		return -EINVAL;
> > +
> > +	omap_start_barebox(barebox);
> > +}
> > +
> > +static struct image_handler omap_barebox_handler = {
> > +	.name = "OMAP barebox",
> > +	.bootm = do_bootm_omap_barebox,
> > +	.filetype = filetype_arm_barebox,
> > +};
> > +
> > +static int omap_bootm_barebox(void)
> > +{
> > +	return register_image_handler(&omap_barebox_handler);
> > +}
> > +device_initcall(omap_bootm_barebox);
> >  
> >  const static char *omap_bootmmc_dev;
> >  
> > diff --git a/arch/arm/mach-omap/xload.c b/arch/arm/mach-omap/xload.c
> > index b4d1b55..69e3e42 100644
> > --- a/arch/arm/mach-omap/xload.c
> > +++ b/arch/arm/mach-omap/xload.c
> > @@ -188,8 +188,7 @@ static void *omap4_xload_boot_usb(void){
> >   */
> >  static __noreturn int omap_xload(void)
> >  {
> > -	int (*func)(void *) = NULL;
> > -	uint32_t *arg;
> > +	void *func;
> >  
> >  	if (!barebox_part)
> >  		barebox_part = &default_part;
> > @@ -230,12 +229,7 @@ static __noreturn int omap_xload(void)
> >  		while (1);
> >  	}
> >  
> > -	arg = (uint32_t *)&omap_bootinfo;
> > -
> > -	shutdown_barebox();
> > -	func(arg);
> > -
> > -	while (1);
> > +	omap_start_barebox(func);
> >  }
> >  
> >  int omap_set_barebox_part(struct omap_barebox_part *part)
> > -- 
> > 1.8.4.rc3
> > 
> > 
> > _______________________________________________
> > barebox mailing list
> > barebox@lists.infradead.org
> > http://lists.infradead.org/mailman/listinfo/barebox
> 

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

  reply	other threads:[~2013-09-27  8:37 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-27  7:17 OMAP patches Sascha Hauer
2013-09-27  7:17 ` [PATCH 1/4] ARM: OMAP: Add missing include Sascha Hauer
2013-09-27  7:17 ` [PATCH 2/4] ARM: OMAP: register OMAP specific barebox bootm handler Sascha Hauer
2013-09-27  8:25   ` Jean-Christophe PLAGNIOL-VILLARD
2013-09-27  8:37     ` Sascha Hauer [this message]
2013-09-27  7:17 ` [PATCH 3/4] ARM: OMAP: unconditionally use /boot/barebox.env Sascha Hauer
2013-09-27  7:17 ` [PATCH 4/4] i2c: omap: remove unncessary printf Sascha Hauer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20130927083722.GM30088@pengutronix.de \
    --to=s.hauer@pengutronix.de \
    --cc=barebox@lists.infradead.org \
    --cc=plagnioj@jcrosoft.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox