Re: [U-Boot] [PATCH V2 08/21] mx6: add plugin file for use with imximage.cfg

2012-09-25 Thread Stefano Babic
On 25/09/2012 00:23, Troy Kisky wrote: > The advantages of a plugin over spl are > 1. smaller code 0x190 bytes, plus the table. > relying on the ROM to do the heavy lifting. This is true. However, as you are writing in the next line, the bootROM boots from devices having much more space as some by

Re: [U-Boot] [PATCH V2 08/21] mx6: add plugin file for use with imximage.cfg

2012-09-25 Thread Stefano Babic
On 24/09/2012 22:46, Troy Kisky wrote: > On 9/23/2012 3:17 AM, Stefano Babic wrote: >> On 22/09/2012 04:39, Troy Kisky wrote: >>> The "plugin" command of mkimage can take this >>> file as an argument. >>> >>> Signed-off-by: Troy Kisky >>> --- Hi Troy, >> >> I agree with Vikram that a better expl

Re: [U-Boot] [PATCH V2 08/21] mx6: add plugin file for use with imximage.cfg

2012-09-24 Thread Troy Kisky
On 9/23/2012 10:08 AM, Stefano Babic wrote: On 23/09/2012 18:23, Eric Nelson wrote: On 09/23/2012 03:17 AM, Stefano Babic wrote: On 22/09/2012 04:39, Troy Kisky wrote: The "plugin" command of mkimage can take this file as an argument. Signed-off-by: Troy Kisky --- Hi Troy, I agree with Vikr

Re: [U-Boot] [PATCH V2 08/21] mx6: add plugin file for use with imximage.cfg

2012-09-24 Thread Troy Kisky
On 9/23/2012 3:17 AM, Stefano Babic wrote: On 22/09/2012 04:39, Troy Kisky wrote: The "plugin" command of mkimage can take this file as an argument. Signed-off-by: Troy Kisky --- Hi Troy, I agree with Vikram that a better explanation of what a plugin is can help to understand without reading

Re: [U-Boot] [PATCH V2 08/21] mx6: add plugin file for use with imximage.cfg

2012-09-24 Thread Stefano Babic
On 24/09/2012 15:48, Eric Nelson wrote: > Hi Stefano, > Hi Eric, >> But I do not get the connection with the imximage and the need for the >> plugin. If the SOC boots with "Serial Downloader", it polls the USB OTG >> waiting for command as specified in the protocol. With your tool you set >> the

Re: [U-Boot] [PATCH V2 08/21] mx6: add plugin file for use with imximage.cfg

2012-09-24 Thread Eric Nelson
Hi Stefano, On 09/24/2012 12:22 AM, Stefano Babic wrote: On 24/09/2012 01:29, Eric Nelson wrote: This is the part that gets interesting. You're right that SPL **can** do the job, but only if it supports the boot media. For the most part, it's reasonable to expect the code to be written in U-B

Re: [U-Boot] [PATCH V2 08/21] mx6: add plugin file for use with imximage.cfg

2012-09-24 Thread Stefano Babic
On 24/09/2012 01:29, Eric Nelson wrote: > > This is the part that gets interesting. > > You're right that SPL **can** do the job, but only if it supports > the boot media. For the most part, it's reasonable to expect the > code to be written in U-Boot for that, since the boot media may > also be

Re: [U-Boot] [PATCH V2 08/21] mx6: add plugin file for use with imximage.cfg

2012-09-23 Thread Eric Nelson
On 09/23/2012 10:08 AM, Stefano Babic wrote: On 23/09/2012 18:23, Eric Nelson wrote: On 09/23/2012 03:17 AM, Stefano Babic wrote: On 22/09/2012 04:39, Troy Kisky wrote: The "plugin" command of mkimage can take this file as an argument. Signed-off-by: Troy Kisky --- Hi Troy, I agree with Vi

Re: [U-Boot] [PATCH V2 08/21] mx6: add plugin file for use with imximage.cfg

2012-09-23 Thread Stefano Babic
On 23/09/2012 18:23, Eric Nelson wrote: > On 09/23/2012 03:17 AM, Stefano Babic wrote: >> On 22/09/2012 04:39, Troy Kisky wrote: >>> The "plugin" command of mkimage can take this >>> file as an argument. >>> >>> Signed-off-by: Troy Kisky >>> --- >> >> Hi Troy, >> >> I agree with Vikram that a bette

Re: [U-Boot] [PATCH V2 08/21] mx6: add plugin file for use with imximage.cfg

2012-09-23 Thread Eric Nelson
On 09/23/2012 03:17 AM, Stefano Babic wrote: On 22/09/2012 04:39, Troy Kisky wrote: The "plugin" command of mkimage can take this file as an argument. Signed-off-by: Troy Kisky --- Hi Troy, I agree with Vikram that a better explanation of what a plugin is can help to understand without readi

Re: [U-Boot] [PATCH V2 08/21] mx6: add plugin file for use with imximage.cfg

2012-09-23 Thread Stefano Babic
On 22/09/2012 04:39, Troy Kisky wrote: > The "plugin" command of mkimage can take this > file as an argument. > > Signed-off-by: Troy Kisky > --- Hi Troy, I agree with Vikram that a better explanation of what a plugin is can help to understand without reading deeply into the i.MX6 manual. So a

Re: [U-Boot] [PATCH V2 08/21] mx6: add plugin file for use with imximage.cfg

2012-09-21 Thread Vikram Narayanan
On 9/22/2012 8:09 AM, Troy Kisky wrote: The "plugin" command of mkimage can take this file as an argument. An explanation of what is "plugin" and what the file plugin.S does should be better. Not in the subject of the patch, you can place it as a README. Signed-off-by: Troy Kisky --- ar

[U-Boot] [PATCH V2 08/21] mx6: add plugin file for use with imximage.cfg

2012-09-21 Thread Troy Kisky
The "plugin" command of mkimage can take this file as an argument. Signed-off-by: Troy Kisky --- arch/arm/cpu/armv7/mx6/Makefile |5 +- arch/arm/cpu/armv7/mx6/plugin.S | 164 ++ arch/arm/include/asm/arch-mx6/imx-regs.h |1 + 3 files changed,