Re: [U-Boot] [PATCH v2 1/2] Add README for the "Falcon" mode

2012-11-14 Thread Otavio Salvador
On Wed, Nov 14, 2012 at 10:19 AM, Stefano Babic wrote: > On 14/11/2012 11:29, Otavio Salvador wrote: > > > > Hi Otavio, > > > > > In the text you have the offset to save the image onto a NAND offset so > > I fail to see how it'd be used for SD-Card. > > > > Can you elaborate it a bit? > > No, I c

Re: [U-Boot] [PATCH v2 1/2] Add README for the "Falcon" mode

2012-11-14 Thread Stefano Babic
On 14/11/2012 11:29, Otavio Salvador wrote: > > Hi Otavio, > > In the text you have the offset to save the image onto a NAND offset so > I fail to see how it'd be used for SD-Card. > > Can you elaborate it a bit? No, I can't without introducing errors. Current code supports Falcon only booti

Re: [U-Boot] [PATCH v2 1/2] Add README for the "Falcon" mode

2012-11-14 Thread Otavio Salvador
On Tue, Nov 13, 2012 at 9:11 AM, Stefano Babic wrote: > Simple howto to add support to a board > for booting the kernel from SPL ("Falcon" mode). > > Signed-off-by: Stefano Babic > --- > Changes in v2: > - spelling, language fixes (Andreas Biessman) > - rewrite some unclear sentences > - drop CO

Re: [U-Boot] [PATCH v2 1/2] Add README for the "Falcon" mode

2012-11-13 Thread Stefano Babic
On 13/11/2012 16:55, Andreas Bießmann wrote: >> +CONFIG_SPL_OS_BOOT Activate Falcon mode. >> +A board should implement the following functions: > > I think reordering this list to have the required functions directly > after the colon would be helpful. Alternatively add a poi

Re: [U-Boot] [PATCH v2 1/2] Add README for the "Falcon" mode

2012-11-13 Thread Andreas Bießmann
Dear Stefano Babic, On 13.11.2012 12: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 > --- > Changes in v2: > - spelling, language fixes (Andreas Biessman) > - rewrite some unclear sentences > -

Re: [U-Boot] [PATCH v2 1/2] Add README for the "Falcon" mode

2012-11-13 Thread Thomas Weber
Hello Stefano, there are some inconsistency for the writing of the Falcon mode: "Falcon" mode, Falcon mode, Falcon, falcon, "Falcon mode" On 11/13/2012 12:11 PM, Stefano Babic wrote: > Simple howto to add support to a board > for booting the kernel from SPL ("Falcon" mode). > > Signed-off-by: Ste

[U-Boot] [PATCH v2 1/2] Add README for the "Falcon" mode

2012-11-13 Thread Stefano Babic
Simple howto to add support to a board for booting the kernel from SPL ("Falcon" mode). Signed-off-by: Stefano Babic --- Changes in v2: - spelling, language fixes (Andreas Biessman) - rewrite some unclear sentences - drop CONFIG_SPL_OS_BOOT_KEY - make example with twister more exhaustive doc/RE