[U-Boot] [PATCH V3 8/8] savebp: added Readme

[Simon Schwarz]

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<simonschwarzcor at 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?
>

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