Re: [U-Boot] [U-BOOT] [PATCH] mmc: enable switch partition function

2011-04-13 Thread Andy Fleming
> >> > 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]"? > > Right.  That's what we use on other storage devices, too. Right now, getting rid of all of those is a bit of a chore. Persona

Re: [U-Boot] [U-BOOT] [PATCH] mmc: enable switch partition function

2011-04-12 Thread Lei Wen
Hi Wolfgang, On Wed, Apr 13, 2011 at 1:49 PM, Wolfgang Denk wrote: > Dear Lei Wen, > > In message you wrote: >> >> > 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

Re: [U-Boot] [U-BOOT] [PATCH] mmc: enable switch partition function

2011-04-12 Thread Wolfgang Denk
Dear Lei Wen, In message you wrote: > > > 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 t

Re: [U-Boot] [U-BOOT] [PATCH] mmc: enable switch partition function

2011-04-12 Thread Lei Wen
Hi Wolfgang, On Wed, Apr 13, 2011 at 1:35 PM, Wolfgang Denk wrote: > Dear Lei Wen, > > In message 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" dev

Re: [U-Boot] [U-BOOT] [PATCH] mmc: enable switch partition function

2011-04-12 Thread Wolfgang Denk
Dear Lei Wen, In message 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.

Re: [U-Boot] [U-BOOT] [PATCH] mmc: enable switch partition function

2011-04-12 Thread Lei Wen
Hi Wolfgang, On Wed, Apr 13, 2011 at 3:31 AM, Wolfgang Denk wrote: > Dear Lei Wen, > > In message you > wrote: >> >> Further refine this idea, don't need to call mmc_register multi-times. >> For the purpose is to switch partition when needed, we really do want to one >> real device. So I propos

Re: [U-Boot] [U-BOOT] [PATCH] mmc: enable switch partition function

2011-04-12 Thread Wolfgang Denk
Dear Lei Wen, In message you wrote: > > Further refine this idea, don't need to call mmc_register multi-times. > For the purpose is to switch partition when needed, we really do want to one > real device. So I propose a new solution as: > > Assume there is two sd/mmc controller on the board, na

Re: [U-Boot] [U-BOOT] [PATCH] mmc: enable switch partition function

2011-02-11 Thread Lei Wen
Hi, On Fri, Feb 11, 2011 at 11:05 PM, Lei Wen wrote: >> >> I am thinking now approach for this. >> How about adding addtitional member in the mmc structure, like part. >> During mmc probe, the driver could know whether the device support partition >> switch or not, if it support, we could registe

Re: [U-Boot] [U-BOOT] [PATCH] mmc: enable switch partition function

2011-02-11 Thread Lei Wen
> > I am thinking now approach for this. > How about adding addtitional member in the mmc structure, like part. > During mmc probe, the driver could know whether the device support partition > switch or not, if it support, we could register three other "faked" mmc device > corresponding to two boot

Re: [U-Boot] [U-BOOT] [PATCH] mmc: enable switch partition function

2011-02-08 Thread Lei Wen
Hi Wolfgang, On Tue, Feb 8, 2011 at 11:13 PM, Wolfgang Denk wrote: > Dear Lei Wen, > > In message you > wrote: >> >> EMMC is sightly different witht the other device with partition. > > Maybe.  But does this really require a different iunterface in the > U-Boot context?   I doubt that. > >> For

Re: [U-Boot] [U-BOOT] [PATCH] mmc: enable switch partition function

2011-02-08 Thread Wolfgang Denk
Dear Lei Wen, In message you wrote: > > EMMC is sightly different witht the other device with partition. Maybe. But does this really require a different iunterface in the U-Boot context? I doubt that. > For IDE as example, we could copy file between part A and B without problem. No, we c

Re: [U-Boot] [U-BOOT] [PATCH] mmc: enable switch partition function

2011-02-08 Thread Lei Wen
Hi Wolfgang, On Tue, Feb 8, 2011 at 10:27 PM, Wolfgang Denk wrote: > Dear Lei Wen, > > In message <1296463665-26551-1-git-send-email-lei...@marvell.com> you wrote: >> For emmc, it may has upto 7 partitions: two boot partitions, one >> user partition, one RPMB partition and four general purpose pa

Re: [U-Boot] [U-BOOT] [PATCH] mmc: enable switch partition function

2011-02-08 Thread Wolfgang Denk
Dear Lei Wen, In message <1296463665-26551-1-git-send-email-lei...@marvell.com> you wrote: > For emmc, it may has upto 7 partitions: two boot partitions, one > user partition, one RPMB partition and four general purpose partitions. > (Refer to JESD84-A44.pdf/page 154) > > As bootloader may need t

Re: [U-Boot] [U-BOOT] [PATCH] mmc: enable switch partition function

2011-02-08 Thread Wolfgang Denk
Dear Lei Wen, In message you wrote: > > Does this patch could get the chance to be included in the 2011.03 release? This is not a bug fix, and it was submitted after the merge window was closed. > I just see it haven't gotten any response after eight days... Will reply ASAP. Best regards, W

[U-Boot] [U-BOOT] [PATCH] mmc: enable switch partition function

2011-01-31 Thread Lei Wen
For emmc, it may has upto 7 partitions: two boot partitions, one user partition, one RPMB partition and four general purpose partitions. (Refer to JESD84-A44.pdf/page 154) As bootloader may need to read out or reflashing images on those different partitions, it is better to enable the partition sw