Hi Wolfgang,
On Wed, Apr 13, 2011 at 1:35 PM, Wolfgang Denk wd@denx.de wrote:
Dear Lei Wen,
In message BANLkTikWwbbuNyHppQQygZ3JDh4aXUg3NA@mail.gmail.com you wrote:
Why don't you follow the existing examples from other storage devices, where we have a "dev" subcommand that allows to select one of the existing devices as "active" device? =A0Similarly, you could here select an active device/partition pair.
dev subcommand to act as "sw_part" in my first patch?
I'm not really sure I understand what this command was supposed to do. What I want to see is consistency with other drivers, i. e. you should be able to use "mmc dev" to select a device and optionally a partition on it as current device, so the remaining "mmc" commands do not need any device arguments any more.
Like this mmc dev[:part], could it be acceptable?
How about this solution? Like if want to access the dev 0 and hardware partition 1, we could use the command as "mmc read 0:1 ***". If not pass the :[part] parameter to the dev number, like "mmc read 0 ***", that would means the partition is 0 by default. For this case, if that device has no capabilty of multi hardware partition, it would act like normal, or switch to the default 0 before any further action.
I want to get rid of the device argument in all these calls.
So the the mmc read would becomes "mmc read [ram address] [mmc sector start] [sector num]"?
Best regards, Lei