From mboxrd@z Thu Jan 1 00:00:00 1970 From: Simon Schwarz Date: Fri, 26 Aug 2011 12:23:23 +0200 Subject: [U-Boot] [PATCH V3 8/8] savebp: added Readme In-Reply-To: <4E562EBD.1070503@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> <4E562EBD.1070503@gmail.com> Message-ID: <4E57741B.7090202@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 08/25/2011 01:15 PM, Andreas Bie?mann wrote: > 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 >> --- >> >> 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? > Because bootm needs it. >> +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) will add. > > regrads > > Andreas Bie?mann Regards Simon