Re: [PATCH 2.6.19] mmc: Add support for SDHC cards (Take 5)

2007-01-04 Thread Pierre Ossman
Philip Langdale wrote: > Thanks to the generous donation of an SDHC card by John Gilmore, and the > surprisingly enlightened decision by the SD Card Association to publish > useful specs, I've been able to bash out support for SDHC. The changes > are not too profound: > > Looks good. I'll

[PATCH 2.6.19] mmc: Add support for SDHC cards (Take 5)

2007-01-04 Thread Philip Langdale
Thanks to the generous donation of an SDHC card by John Gilmore, and the surprisingly enlightened decision by the SD Card Association to publish useful specs, I've been able to bash out support for SDHC. The changes are not too profound: i) Add a card flag indicating the card uses block level

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards (Take 4)

2007-01-04 Thread Pierre Ossman
Philip Langdale wrote: > In the process of developing this patch, we realised that the R6 response > definition > was incorrect and that it should have been identical to R1 (and so should R7). > Correcting this mistake revealed problems in a couple of host controller > drivers that > relied on

[PATCH 2.6.19] mmc: Add support for SDHC cards (Take 4)

2007-01-04 Thread Philip Langdale
Thanks to the generous donation of an SDHC card by John Gilmore, and the surprisingly enlightened decision by the SD Card Association to publish useful specs, I've been able to bash out support for SDHC. The changes are not too profound: i) Add a card flag indicating the card uses block level

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards (Take 2)

2007-01-04 Thread Alex Dubov
I think the trivial fix will do (after all, there's nothing that should matter to the controller in the R6 response; I don't know about R7). I don't have any SDHC cards so I can't test this. --- tifm_sd.c.orig 2006-12-11 01:39:28.0 +1100 +++ tifm_sd.c 2007-01-04

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards (Take 2)

2007-01-04 Thread Alex Dubov
I think the trivial fix will do (after all, there's nothing that should matter to the controller in the R6 response; I don't know about R7). I don't have any SDHC cards so I can't test this. --- tifm_sd.c.orig 2006-12-11 01:39:28.0 +1100 +++ tifm_sd.c 2007-01-04

[PATCH 2.6.19] mmc: Add support for SDHC cards (Take 4)

2007-01-04 Thread Philip Langdale
Thanks to the generous donation of an SDHC card by John Gilmore, and the surprisingly enlightened decision by the SD Card Association to publish useful specs, I've been able to bash out support for SDHC. The changes are not too profound: i) Add a card flag indicating the card uses block level

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards (Take 4)

2007-01-04 Thread Pierre Ossman
Philip Langdale wrote: In the process of developing this patch, we realised that the R6 response definition was incorrect and that it should have been identical to R1 (and so should R7). Correcting this mistake revealed problems in a couple of host controller drivers that relied on the

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards (Take 5)

2007-01-04 Thread Pierre Ossman
Philip Langdale wrote: Thanks to the generous donation of an SDHC card by John Gilmore, and the surprisingly enlightened decision by the SD Card Association to publish useful specs, I've been able to bash out support for SDHC. The changes are not too profound: Looks good. I'll queue up

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards (Take 2)

2007-01-03 Thread Pierre Ossman
Philip Langdale wrote: > Pierre Ossman wrote: > >> Amen to that. All hw vendors that implement this particular form of >> brain damage should be dragged out and shot. >> >> I'll fix a patch for this later on. >> > > See my updated Take 3 patch. I've implemented a uniqueness fix by > adding

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards (Take 2)

2007-01-03 Thread Philip Langdale
Pierre Ossman wrote: > > Amen to that. All hw vendors that implement this particular form of > brain damage should be dragged out and shot. > > I'll fix a patch for this later on. See my updated Take 3 patch. I've implemented a uniqueness fix by adding additional RSP flags do make R6 and R7

[PATCH 2.6.19] mmc: Add support for SDHC cards (Take 3)

2007-01-03 Thread Philip Langdale
Thanks to the generous donation of an SDHC card by John Gilmore, and the surprisingly enlightened decision by the SD Card Association to publish useful specs, I've been able to bash out support for SDHC. The changes are not too profound: i) Add a card flag indicating the card uses block level

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards (Take 2)

2007-01-03 Thread Pierre Ossman
Philip Langdale wrote: > This is a bug. The MMC_RSP_R? #defines do not fully characterise the > responses (specically, the way that the response is parsed is not > characterised) and consequently there is no guarantee of uniqueness. > Given this reality - the way that the tifm_sd driver works is

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards (Take 2)

2007-01-03 Thread Philip Langdale
Andrew Morton wrote: > On Mon, 01 Jan 2007 07:29:55 -0800 > Philip Langdale <[EMAIL PROTECTED]> wrote: > >> #define MMC_RSP_R1B >> (MMC_RSP_PRESENT|MMC_RSP_CRC|MMC_RSP_OPCODE|MMC_RSP_BUSY) >> #define MMC_RSP_R2 (MMC_RSP_PRESENT|MMC_RSP_136|MMC_RSP_CRC) >> #define MMC_RSP_R3

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards (Take 2)

2007-01-03 Thread Andrew Morton
On Mon, 01 Jan 2007 07:29:55 -0800 Philip Langdale <[EMAIL PROTECTED]> wrote: > #define MMC_RSP_R1B > (MMC_RSP_PRESENT|MMC_RSP_CRC|MMC_RSP_OPCODE|MMC_RSP_BUSY) > #define MMC_RSP_R2 (MMC_RSP_PRESENT|MMC_RSP_136|MMC_RSP_CRC) > #define MMC_RSP_R3 (MMC_RSP_PRESENT) > -#define MMC_RSP_R6

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards (Take 2)

2007-01-03 Thread Pierre Ossman
Philip Langdale wrote: > @@ -1386,10 +1420,37 @@ >* all get the idea that they should be ready for CMD2. >* (My SanDisk card seems to need this.) >*/ > - if (host->mode == MMC_MODE_SD) > - mmc_send_app_op_cond(host, host->ocr, NULL); > - else > + if

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards

2007-01-03 Thread Pierre Ossman
Philip Langdale wrote: > Heh. I forget that you don't want to manually alter the email. Will do. > > I'm rather reluctant to change anything once someone has put a signed-off-by line to it. Different people have different limits as to what they regard a trivial modification. > Done. I thought

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards

2007-01-03 Thread Pierre Ossman
Philip Langdale wrote: Heh. I forget that you don't want to manually alter the email. Will do. I'm rather reluctant to change anything once someone has put a signed-off-by line to it. Different people have different limits as to what they regard a trivial modification. Done. I thought we

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards (Take 2)

2007-01-03 Thread Pierre Ossman
Philip Langdale wrote: @@ -1386,10 +1420,37 @@ * all get the idea that they should be ready for CMD2. * (My SanDisk card seems to need this.) */ - if (host-mode == MMC_MODE_SD) - mmc_send_app_op_cond(host, host-ocr, NULL); - else + if (host-mode

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards (Take 2)

2007-01-03 Thread Andrew Morton
On Mon, 01 Jan 2007 07:29:55 -0800 Philip Langdale [EMAIL PROTECTED] wrote: #define MMC_RSP_R1B (MMC_RSP_PRESENT|MMC_RSP_CRC|MMC_RSP_OPCODE|MMC_RSP_BUSY) #define MMC_RSP_R2 (MMC_RSP_PRESENT|MMC_RSP_136|MMC_RSP_CRC) #define MMC_RSP_R3 (MMC_RSP_PRESENT) -#define MMC_RSP_R6

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards (Take 2)

2007-01-03 Thread Philip Langdale
Andrew Morton wrote: On Mon, 01 Jan 2007 07:29:55 -0800 Philip Langdale [EMAIL PROTECTED] wrote: #define MMC_RSP_R1B (MMC_RSP_PRESENT|MMC_RSP_CRC|MMC_RSP_OPCODE|MMC_RSP_BUSY) #define MMC_RSP_R2 (MMC_RSP_PRESENT|MMC_RSP_136|MMC_RSP_CRC) #define MMC_RSP_R3 (MMC_RSP_PRESENT) -#define

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards (Take 2)

2007-01-03 Thread Pierre Ossman
Philip Langdale wrote: This is a bug. The MMC_RSP_R? #defines do not fully characterise the responses (specically, the way that the response is parsed is not characterised) and consequently there is no guarantee of uniqueness. Given this reality - the way that the tifm_sd driver works is

[PATCH 2.6.19] mmc: Add support for SDHC cards (Take 3)

2007-01-03 Thread Philip Langdale
Thanks to the generous donation of an SDHC card by John Gilmore, and the surprisingly enlightened decision by the SD Card Association to publish useful specs, I've been able to bash out support for SDHC. The changes are not too profound: i) Add a card flag indicating the card uses block level

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards

2007-01-01 Thread Pierre Ossman
Pavel Machek wrote: > Would you describe what SDHC is? I know SD flash cards, and IIRC SDIO > cards exist, with functionality such as bluetooth...? But SDHC? > > SDHC is short for "Secure Digital High Capacity". It's simply SD flash cards than conform to a new version of the protocol, a

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards

2007-01-01 Thread Pavel Machek
Hi! > > Hi all, > > > > Thanks to the generous donation of an SDHC card by John Gilmore, and the > > surprisingly enlightened decision by the SD Card Association to publish > > useful specs, I've been able to bash out support for SDHC. The changes > > are not too profound: > > So I sent that

[PATCH 2.6.19] mmc: Add support for SDHC cards (Take 2)

2007-01-01 Thread Philip Langdale
Thanks to the generous donation of an SDHC card by John Gilmore, and the surprisingly enlightened decision by the SD Card Association to publish useful specs, I've been able to bash out support for SDHC. The changes are not too profound: i) Add a card flag indicating the card uses block level

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards

2007-01-01 Thread Philip Langdale
I will post the updated diff as a separate follow up. Pierre Ossman wrote: > > When you have a commit message larger than the patch, you know there is > something wrong. ;) > > Please skip the part about MMC at least. Heh. I forget that you don't want to manually alter the email. Will do. >

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards

2007-01-01 Thread Philip Langdale
I will post the updated diff as a separate follow up. Pierre Ossman wrote: When you have a commit message larger than the patch, you know there is something wrong. ;) Please skip the part about MMC at least. Heh. I forget that you don't want to manually alter the email. Will do.

[PATCH 2.6.19] mmc: Add support for SDHC cards (Take 2)

2007-01-01 Thread Philip Langdale
Thanks to the generous donation of an SDHC card by John Gilmore, and the surprisingly enlightened decision by the SD Card Association to publish useful specs, I've been able to bash out support for SDHC. The changes are not too profound: i) Add a card flag indicating the card uses block level

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards

2007-01-01 Thread Pavel Machek
Hi! Hi all, Thanks to the generous donation of an SDHC card by John Gilmore, and the surprisingly enlightened decision by the SD Card Association to publish useful specs, I've been able to bash out support for SDHC. The changes are not too profound: So I sent that with the wrong

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards

2007-01-01 Thread Pierre Ossman
Pavel Machek wrote: Would you describe what SDHC is? I know SD flash cards, and IIRC SDIO cards exist, with functionality such as bluetooth...? But SDHC? SDHC is short for Secure Digital High Capacity. It's simply SD flash cards than conform to a new version of the protocol, a version

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards

2006-12-31 Thread Pierre Ossman
Philip Langdale wrote: > Hi all, > > > *snip* > Signed-off-by: Philipl Langdale <[EMAIL PROTECTED]> > When you have a commit message larger than the patch, you know there is something wrong. ;) Please skip the part about MMC at least. > @@ -588,12 +591,15 @@ > > if

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards

2006-12-31 Thread Pierre Ossman
Philip Langdale wrote: Hi all, *snip* Signed-off-by: Philipl Langdale [EMAIL PROTECTED] When you have a commit message larger than the patch, you know there is something wrong. ;) Please skip the part about MMC at least. @@ -588,12 +591,15 @@ if (mmc_card_sd(card)) {

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards

2006-12-22 Thread Philip Langdale
Philip Langdale wrote: > Hi all, > > Thanks to the generous donation of an SDHC card by John Gilmore, and the > surprisingly enlightened decision by the SD Card Association to publish > useful specs, I've been able to bash out support for SDHC. The changes > are not too profound: So I sent that

[PATCH 2.6.19] mmc: Add support for SDHC cards

2006-12-22 Thread Philip Langdale
Hi all, Thanks to the generous donation of an SDHC card by John Gilmore, and the surprisingly enlightened decision by the SD Card Association to publish useful specs, I've been able to bash out support for SDHC. The changes are not too profound: i) Add a card flag indicating the card uses block

[PATCH 2.6.19] mmc: Add support for SDHC cards

2006-12-22 Thread Philip Langdale
Hi all, Thanks to the generous donation of an SDHC card by John Gilmore, and the surprisingly enlightened decision by the SD Card Association to publish useful specs, I've been able to bash out support for SDHC. The changes are not too profound: i) Add a card flag indicating the card uses block

Re: [PATCH 2.6.19] mmc: Add support for SDHC cards

2006-12-22 Thread Philip Langdale
Philip Langdale wrote: Hi all, Thanks to the generous donation of an SDHC card by John Gilmore, and the surprisingly enlightened decision by the SD Card Association to publish useful specs, I've been able to bash out support for SDHC. The changes are not too profound: So I sent that with