Re: [U-Boot] [U-Boot, 3/9] nand: add a hw specific subcommand to the nand command.

Dear Scott Wood,

In message 4E3AF62B.8080605@freescale.com you wrote:

Again, I think "nand hwfunc" conveys little information about what it's actually going to do, versus something like:

=> nandrbl off

I agree with Heiko that NAND related commands should be implemented as subcommands of the "nand" command.

Havong "nand <subcommand>" and "nandrbl" at the same time is not acceptable to me - that should be changed into "nand rbl".

At least "nand hwfunc type" or "nand hwfunc rbl" would be more informative, but I question the benefit that the verbosity buys us.

I agree that "hwfunc" is an unlucky name.

So now it's "nand hwfunc ctrl type rbl"? Just to avoid grepping for U_BOOT_CMD?

No. I would not like this either.

Best regards,

Wolfgang Denk