Re: [U-Boot] [PATCH V3 1/3] imx-common/cmd_resetmode.c: add imx resetmode command

2012-08-11 Thread Stefano Babic
On 10/08/2012 03:09, Eric Nelson wrote: >> How difficult would it be for someone to implement a general "alias" >> command ? >> ie. 'alias bootto "resetmode $1 && reset"' >> > > It's usually a customer that asks the "How hard would it be?" question, > but you can almost get there like this: > >

Re: [U-Boot] [PATCH V3 1/3] imx-common/cmd_resetmode.c: add imx resetmode command

2012-08-09 Thread Eric Nelson
On 08/09/2012 02:15 PM, Troy Kisky wrote: On 8/9/2012 12:29 PM, Eric Nelson wrote: On 08/09/2012 11:48 AM, Troy Kisky wrote: On 8/9/2012 12:21 AM, Stefano Babic wrote: On 09/08/2012 03:26, Troy Kisky wrote: >>> >>> +int do_bootmmc0(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]

Re: [U-Boot] [PATCH V3 1/3] imx-common/cmd_resetmode.c: add imx resetmode command

2012-08-09 Thread Troy Kisky
On 8/9/2012 2:12 PM, stefano babic wrote: Am 09/08/2012 21:29, schrieb Eric Nelson: Why do you not pass the mode to do_resetmode_cmd() and let this function parse the argument, without these help functions ? Then we do not need to add new functions if we have, for example, mmc2 or mmc3. These

Re: [U-Boot] [PATCH V3 1/3] imx-common/cmd_resetmode.c: add imx resetmode command

2012-08-09 Thread Troy Kisky
On 8/9/2012 12:29 PM, Eric Nelson wrote: On 08/09/2012 11:48 AM, Troy Kisky wrote: On 8/9/2012 12:21 AM, Stefano Babic wrote: On 09/08/2012 03:26, Troy Kisky wrote: >>> >>> +int do_bootmmc0(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) +{ + return do_resetmode_cmd("mmc0"); +}

Re: [U-Boot] [PATCH V3 1/3] imx-common/cmd_resetmode.c: add imx resetmode command

2012-08-09 Thread stefano babic
Am 09/08/2012 21:29, schrieb Eric Nelson: >>> Why do you not pass the mode to do_resetmode_cmd() and let this function >>> parse the argument, without these help functions ? Then we do not need >>> to add new functions if we have, for example, mmc2 or mmc3. >> >> These functions are requested shor

Re: [U-Boot] [PATCH V3 1/3] imx-common/cmd_resetmode.c: add imx resetmode command

2012-08-09 Thread stefano babic
Am 09/08/2012 20:48, schrieb Troy Kisky: >> Maybe I am searching in wrong places. Where can I find the description >> for the feature you have implemented ? > > The only documentations I've found are in the freescale release of u-boot. > cpu/arm_cortexa8/mx53/generic.c > > void do_switch_mfgmode

Re: [U-Boot] [PATCH V3 1/3] imx-common/cmd_resetmode.c: add imx resetmode command

2012-08-09 Thread Eric Nelson
On 08/09/2012 11:48 AM, Troy Kisky wrote: On 8/9/2012 12:21 AM, Stefano Babic wrote: On 09/08/2012 03:26, Troy Kisky wrote: >>> >>> +int do_bootmmc0(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) +{ + return do_resetmode_cmd("mmc0"); +} + +int do_bootmmc1(cmd_tbl_t *cmdtp, int f

Re: [U-Boot] [PATCH V3 1/3] imx-common/cmd_resetmode.c: add imx resetmode command

2012-08-09 Thread Troy Kisky
On 8/9/2012 12:21 AM, Stefano Babic wrote: On 09/08/2012 03:26, Troy Kisky wrote: This is useful for forcing the ROM's usb downloader to activate upon a watchdog reset. Or, you can boot from either SD Card. Hi Troy, Currently, support added for MX53 and MX6Q Signed-off-by: Troy Kisky Note:

Re: [U-Boot] [PATCH V3 1/3] imx-common/cmd_resetmode.c: add imx resetmode command

2012-08-09 Thread Stefano Babic
On 09/08/2012 03:26, Troy Kisky wrote: > This is useful for forcing the ROM's > usb downloader to activate upon a watchdog reset. > Or, you can boot from either SD Card. > Hi Troy, > Currently, support added for MX53 and MX6Q > Signed-off-by: Troy Kisky > > Note: MX53 support untested. It can

[U-Boot] [PATCH V3 1/3] imx-common/cmd_resetmode.c: add imx resetmode command

2012-08-08 Thread Troy Kisky
This is useful for forcing the ROM's usb downloader to activate upon a watchdog reset. Or, you can boot from either SD Card. Currently, support added for MX53 and MX6Q Signed-off-by: Troy Kisky Note: MX53 support untested. --- arch/arm/cpu/armv7/imx-common/Makefile|1 + arch/arm/cpu