From mboxrd@z Thu Jan  1 00:00:00 1970
From: =?ISO-8859-1?Q?Andreas_Bie=DFmann?= <andreas.devel@googlemail.com>
Date: Thu, 25 Aug 2011 13:15:09 +0200
Subject: [U-Boot] [PATCH V3 8/8] savebp: added Readme
In-Reply-To: <1314261196-23197-9-git-send-email-simonschwarzcor@gmail.com>
References: <1314182363-25732-1-git-send-email-simonschwarzcor@gmail.com>
	<1314261196-23197-1-git-send-email-simonschwarzcor@gmail.com>
	<1314261196-23197-9-git-send-email-simonschwarzcor@gmail.com>
Message-ID: <4E562EBD.1070503@gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Dear Simon,

Am 25.08.2011 10:33, schrieb Simon Schwarz:
> Adds a Readme for the savebp command

This patch should be merged with 'Add savebp command'
> 
> Signed-off-by: Simon Schwarz <simonschwarzcor@gmail.com>
> ---
> 
> V2 changes:
> ADDED in V2
> 
> V3 changes:
> nothing
> ---
>  doc/README.commands.savebp |   28 ++++++++++++++++++++++++++++
>  1 files changed, 28 insertions(+), 0 deletions(-)
>  create mode 100644 doc/README.commands.savebp
> 
> diff --git a/doc/README.commands.savebp b/doc/README.commands.savebp
> new file mode 100644
> index 0000000..dc05ee0
> --- /dev/null
> +++ b/doc/README.commands.savebp
> @@ -0,0 +1,28 @@
> +The savebp (=save boot parameters) is used to save a boot parameter image to
> +non-volatile memory.
> +
> +To execute the command everything has to be in place as if bootm should be
> +used.
> +(kernel image, initrd-image, fdt-image etc.)
> +
> +Call is:
> +savebp [ftd|atags] [nand_offset] [kernel_addr] [initrd_addr] [fdt_addr]
> +
> +Only the first parameter [ftd|atags] is mandatory if the others are left blank
> +standard values are used.
> +
> +e.g:
> +savebp fdt 0x680000 0x82000000 0x80000100 -
> +savebo atags
> +
> +typical call on OMAP3:
> +nandecc hw
> +nand read 0x82000000 0x280000 0x400000 /* Read kernel image from NAND */

why read the kernel image here?

> +tftpboot 0x80000100 devkit8000.dtb /* Read fdt */
> +savebp fdt 0x680000 0x82000000 0x80000100 - /* Save the image */
> +
> +------------Behind the scene---------------
> +Atm the implementation is that we have /common/cmd_savebp.c which implements
> +the command tself and the subcommand calls to bootm.
              ^itself
> +Then the arch specific implementation of do_save_atags or do_savebp_fdt
> +in /arch/arm/lib is called.

(therefore a errormessage should/could be added to common/cmd_savebp.c,
if ARCH != ARM)

regrads

Andreas Bie?mann