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

[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 shortcuts.
>> "bootmmc0" is the same as "resetmode mmc0 && reset"
>>
> Hi Troy,
> 
> As one of the few that made this request:
> 
>     I take it back...
> 
> I was mostly after a single command that also performs 'reset',
> and agree with Stefano that an argument for the mode is a better
> implementation.

+1

> 
>     U-Boot> bootto usb
>     U-Boot> bootto mmc
> 
> I'm also rethinking the request for 'resetmode' because
> of the conflict with 'reset' (i.e. 'res' is no longer a
> unique command-complete).

Yes, there is no reason to have a conflict with "reset"

> The first implementation with
> 'rsmode' seemed to be an odd abbreviation.
> 
> Anybody else have a suggestion?
> 
>     U-Boot> rstmode usb && reset
>     U-Boot> bmode usb && reset

In Freescale documentation this is called "boot mode" - using the same
terminology can reduce confusion. I do not discuss which name is better,
but if someone reads the SOC's manual and then find a command for "boot
mode", it is normal that he automatically knows what is going on. "Boot"
conflicts with other u-boot command, then "bmode" is my first choice.

Best regards,
Stefano Babic


-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================