From: "Andreas Bießmann" <andreas.devel@googlemail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3 1/3] Add README for the "Falcon" mode
Date: Mon, 19 Nov 2012 11:21:43 +0100 [thread overview]
Message-ID: <50AA0837.2000002@gmail.com> (raw)
In-Reply-To: <1353316265-24430-1-git-send-email-sbabic@denx.de>
Dear Stefano Babic,
On 19.11.2012 10:11, Stefano Babic wrote:
> Simple howto to add support to a board
> for booting the kernel from SPL ("Falcon" mode).
>
> Signed-off-by: Stefano Babic <sbabic@denx.de>
despite a small change
Acked-by: Andreas Bie?mann <andreas.devel@googlemail.com>
> ---
> Changes in v3:
> - parameter initrd_addr was removed in V2 (Andreas Biessmann)
> - added patch to fix help usage for spl export (Andreas Biessmann)
> - Added empty lines (Otavio Salvador)
> - add a more exhaustive description explaining that
> spl export does not save into media (Lukasz Majewski).
>
> Changes in v2:
> - spelling, language fixes (Andreas Biessman)
> - rewrite some unclear sentences
> - drop CONFIG_SPL_OS_BOOT_KEY
> - make example with twister more exhaustive
>
<snip>
> +Configuration
> +----------------------------
> +CONFIG_CMD_SPL Enable the "spl export" command.
> + The command "spl export" is then available in U-Boot
> + mode
> +CONFIG_SPL_OS_BOOT Activate Falcon Mode.
> + A board should implement the following functions:
> +
> +CONFIG_SYS_SPL_ARGS_ADDR Address in RAM where the parameters must be
> + copied by SPL.
> + In most cases, it is <start_of_ram> + 0x100
> +
> +CONFIG_SYS_NAND_SPL_KERNEL_OFFS Offset in NAND where the kernel is stored
> +
> +CONFIG_CMD_SPL_NAND_OFS Offset in NAND where the parameters area was saved.
> +
> +CONFIG_CMD_SPL_WRITE_SIZE Size of the parameters area to be copied
> +
> +Function that a board must implement
> +------------------------------------
> +
> +void spl_board_prepare_for_linux(void) : optional
> + Called from SPL before starting the kernel
> +
> +spl_start_uboot() : required
> + Returns "0" if SPL starts the kernel, "1" if U-Boot
> + must be started.
> +
> +
shouldn't we reorder that thing here (move CONFIG_SPL_OS_BOOT down to
the functions)?
Best regards
Andreas Bie?mann
next prev parent reply other threads:[~2012-11-19 10:21 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-11-12 10:59 [U-Boot] [PATCH] Add README for the "Falcon" mode Stefano Babic
2012-11-12 11:35 ` Andreas Bießmann
2012-11-12 13:02 ` Stefano Babic
2012-11-12 13:33 ` Andreas Bießmann
2012-11-13 11:11 ` [U-Boot] [PATCH v2 1/2] " Stefano Babic
2012-11-13 11:11 ` [U-Boot] [PATCH v2 2/2] OMAP3: drop CONFIG_SPL_OS_BOOT_KEY and use local define Stefano Babic
2012-11-13 13:54 ` Thomas Weber
2012-11-13 13:13 ` [U-Boot] [PATCH v2 1/2] Add README for the "Falcon" mode Thomas Weber
2012-11-13 15:55 ` Andreas Bießmann
2012-11-13 17:08 ` Stefano Babic
2012-11-14 10:29 ` Otavio Salvador
2012-11-14 12:19 ` Stefano Babic
2012-11-14 12:22 ` Otavio Salvador
2012-11-19 9:11 ` [U-Boot] [PATCH v3 1/3] " Stefano Babic
2012-11-19 9:11 ` [U-Boot] [PATCH v3 2/3] OMAP3: drop CONFIG_SPL_OS_BOOT_KEY and use local define Stefano Babic
2012-11-19 9:11 ` [U-Boot] [PATCH v3 3/3] SPL: Change description for spl command Stefano Babic
2012-11-19 10:17 ` Andreas Bießmann
2012-11-19 10:14 ` [U-Boot] [PATCH v3 1/3] Add README for the "Falcon" mode Otavio Salvador
2012-11-19 10:36 ` Stefano Babic
2012-11-19 10:21 ` Andreas Bießmann [this message]
2012-11-19 10:37 ` Stefano Babic
2012-11-23 15:31 ` [U-Boot] [PATCH v4 " Stefano Babic
2012-11-23 15:31 ` [U-Boot] [PATCH v4 2/3] OMAP3: drop CONFIG_SPL_OS_BOOT_KEY and use local define Stefano Babic
2012-11-23 15:31 ` [U-Boot] [PATCH v4 3/3] SPL: Change description for spl command Stefano Babic
2012-11-23 18:10 ` [U-Boot] [PATCH v4 1/3] Add README for the "Falcon" mode Vikram Narayanan
2012-11-23 21:59 ` Andreas Bießmann
2012-11-25 3:47 ` Vikram Narayanan
2012-11-24 14:18 ` Stefano Babic
2013-02-11 21:12 ` [U-Boot] [PATCH] " Otavio Salvador
2013-02-12 8:23 ` Stefano Babic
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=50AA0837.2000002@gmail.com \
--to=andreas.devel@googlemail.com \
--cc=u-boot@lists.denx.de \
/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 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.