* [PATCH 1/1]mmc: set timeout for SDHCI host before sending busy cmd @ 2011-02-24 11:19 Chuanxiao Dong 2011-02-24 14:18 ` Jae hoon Chung 0 siblings, 1 reply; 8+ messages in thread From: Chuanxiao Dong @ 2011-02-24 11:19 UTC (permalink / raw) To: linux-mmc, cjb; +Cc: linux-kernel, arnd Set the timeout control register for SDHCI host when it needs to send some commands which need busy signal. Use the maximum timeout value will be safe. Signed-off-by: Chuanxiao Dong <chuanxiao.dong@intel.com> --- drivers/mmc/host/sdhci.c | 14 ++++++++++---- 1 files changed, 10 insertions(+), 4 deletions(-) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index 9e15f41..32b7475 100644 --- a/drivers/mmc/host/sdhci.c +++ b/drivers/mmc/host/sdhci.c @@ -40,7 +40,6 @@ static unsigned int debug_quirks = 0; -static void sdhci_prepare_data(struct sdhci_host *, struct mmc_data *); static void sdhci_finish_data(struct sdhci_host *); static void sdhci_send_command(struct sdhci_host *, struct mmc_command *); @@ -651,16 +650,23 @@ static void sdhci_set_transfer_irqs(struct sdhci_host *host) sdhci_clear_set_irqs(host, dma_irqs, pio_irqs); } -static void sdhci_prepare_data(struct sdhci_host *host, struct mmc_data *data) +static void sdhci_prepare_data(struct sdhci_host *host, struct mmc_command *cmd) { u8 count; u8 ctrl; + struct mmc_data *data = cmd->data; int ret; WARN_ON(host->data); - if (data == NULL) + if (data == NULL) { + /* + * set timeout to be maximum value for command with busy signal. + */ + if (cmd->flags & MMC_RSP_BUSY) + sdhci_writeb(host, 0xE, SDHCI_TIMEOUT_CONTROL); return; + } /* Sanity checks */ BUG_ON(data->blksz * data->blocks > 524288); @@ -920,7 +926,7 @@ static void sdhci_send_command(struct sdhci_host *host, struct mmc_command *cmd) host->cmd = cmd; - sdhci_prepare_data(host, cmd->data); + sdhci_prepare_data(host, cmd); sdhci_writel(host, cmd->arg, SDHCI_ARGUMENT); -- 1.6.6.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH 1/1]mmc: set timeout for SDHCI host before sending busy cmd 2011-02-24 11:19 [PATCH 1/1]mmc: set timeout for SDHCI host before sending busy cmd Chuanxiao Dong @ 2011-02-24 14:18 ` Jae hoon Chung 2011-02-24 14:54 ` Dong, Chuanxiao 0 siblings, 1 reply; 8+ messages in thread From: Jae hoon Chung @ 2011-02-24 14:18 UTC (permalink / raw) To: Chuanxiao Dong; +Cc: linux-mmc, cjb, linux-kernel, arnd, Kyungmin Park Hi This patch looks like this. I sent to the RFC patch.. http://marc.info/?l=linux-mmc&m=129109794815028&w=3 Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com> Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com> --- drivers/mmc/host/sdhci.c | 6 +++++- include/linux/mmc/sdhci.h | 3 ++- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index 782c0ee..3b93d97 100644 --- a/drivers/mmc/host/sdhci.c +++ b/drivers/mmc/host/sdhci.c @@ -655,8 +655,12 @@ static void sdhci_prepare_data(struct sdhci_host *host, struct mmc_data *data) WARN_ON(host->data); - if (data == NULL) + if (data == NULL) { + if ((host->quirks & SDHCI_QUIRK_SET_DATA_TIMEOUT_VAL) && + (host->cmd->flags & MMC_RSP_BUSY)) + sdhci_writel(host, 0xE, SDHCI_TIMEOUT_CONTROL); return; + } /* Sanity checks */ BUG_ON(data->blksz * data->blocks > 524288); diff --git a/include/linux/mmc/sdhci.h b/include/linux/mmc/sdhci.h index 1fdc673..315ff49 100644 --- a/include/linux/mmc/sdhci.h +++ b/include/linux/mmc/sdhci.h @@ -83,7 +83,8 @@ struct sdhci_host { #define SDHCI_QUIRK_MULTIBLOCK_READ_ACMD12 (1<<28) /* Controller doesn't have HISPD bit field in HI-SPEED SD card */ #define SDHCI_QUIRK_NO_HISPD_BIT (1<<29) - +/* Controller need set data timeout value when card is busy */ +#define SDHCI_QUIRK_SET_DATA_TIMEOUT_VAL (1<<30) int irq; /* Device IRQ */ void __iomem *ioaddr; /* Mapped address */ -- 1.6.0.4 2011/2/24 Chuanxiao Dong <chuanxiao.dong@intel.com>: > Set the timeout control register for SDHCI host when it needs to send some > commands which need busy signal. Use the maximum timeout value will be safe. > > Signed-off-by: Chuanxiao Dong <chuanxiao.dong@intel.com> > --- > drivers/mmc/host/sdhci.c | 14 ++++++++++---- > 1 files changed, 10 insertions(+), 4 deletions(-) > > diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c > index 9e15f41..32b7475 100644 > --- a/drivers/mmc/host/sdhci.c > +++ b/drivers/mmc/host/sdhci.c > @@ -40,7 +40,6 @@ > > static unsigned int debug_quirks = 0; > > -static void sdhci_prepare_data(struct sdhci_host *, struct mmc_data *); > static void sdhci_finish_data(struct sdhci_host *); > > static void sdhci_send_command(struct sdhci_host *, struct mmc_command *); > @@ -651,16 +650,23 @@ static void sdhci_set_transfer_irqs(struct sdhci_host *host) > sdhci_clear_set_irqs(host, dma_irqs, pio_irqs); > } > > -static void sdhci_prepare_data(struct sdhci_host *host, struct mmc_data *data) > +static void sdhci_prepare_data(struct sdhci_host *host, struct mmc_command *cmd) > { > u8 count; > u8 ctrl; > + struct mmc_data *data = cmd->data; > int ret; > > WARN_ON(host->data); > > - if (data == NULL) > + if (data == NULL) { > + /* > + * set timeout to be maximum value for command with busy signal. > + */ > + if (cmd->flags & MMC_RSP_BUSY) > + sdhci_writeb(host, 0xE, SDHCI_TIMEOUT_CONTROL); > return; > + } > > /* Sanity checks */ > BUG_ON(data->blksz * data->blocks > 524288); > @@ -920,7 +926,7 @@ static void sdhci_send_command(struct sdhci_host *host, struct mmc_command *cmd) > > host->cmd = cmd; > > - sdhci_prepare_data(host, cmd->data); > + sdhci_prepare_data(host, cmd); > > sdhci_writel(host, cmd->arg, SDHCI_ARGUMENT); > > -- > 1.6.6.1 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-mmc" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > ^ permalink raw reply related [flat|nested] 8+ messages in thread
* RE: [PATCH 1/1]mmc: set timeout for SDHCI host before sending busy cmd 2011-02-24 14:18 ` Jae hoon Chung @ 2011-02-24 14:54 ` Dong, Chuanxiao 2011-02-24 18:32 ` Philip Rakity 0 siblings, 1 reply; 8+ messages in thread From: Dong, Chuanxiao @ 2011-02-24 14:54 UTC (permalink / raw) To: Jae hoon Chung; +Cc: linux-mmc, cjb, linux-kernel, arnd, Kyungmin Park Great. Maybe we wouldn't need to add this quirk since give a maximum timeout value is safe for all host controller I think. We would better save the quirk for other really critical silicon bugs. > -----Original Message----- > From: Jae hoon Chung [mailto:jh80.chung@gmail.com] > Sent: Thursday, February 24, 2011 10:18 PM > To: Dong, Chuanxiao > Cc: linux-mmc@vger.kernel.org; cjb@laptop.org; linux-kernel@vger.kernel.org; > arnd@arndb.de; Kyungmin Park > Subject: Re: [PATCH 1/1]mmc: set timeout for SDHCI host before sending busy cmd > > Hi > > This patch looks like this. I sent to the RFC patch.. > > http://marc.info/?l=linux-mmc&m=129109794815028&w=3 > > Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com> > Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com> > > --- > drivers/mmc/host/sdhci.c | 6 +++++- > include/linux/mmc/sdhci.h | 3 ++- > 2 files changed, 7 insertions(+), 2 deletions(-) > > diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c > index 782c0ee..3b93d97 100644 > --- a/drivers/mmc/host/sdhci.c > +++ b/drivers/mmc/host/sdhci.c > @@ -655,8 +655,12 @@ static void sdhci_prepare_data(struct sdhci_host > *host, struct mmc_data *data) > > WARN_ON(host->data); > > - if (data == NULL) > + if (data == NULL) { > + if ((host->quirks & SDHCI_QUIRK_SET_DATA_TIMEOUT_VAL) > && > + (host->cmd->flags & MMC_RSP_BUSY)) > + sdhci_writel(host, 0xE, SDHCI_TIMEOUT_CONTROL); > return; > + } > > /* Sanity checks */ > BUG_ON(data->blksz * data->blocks > 524288); > diff --git a/include/linux/mmc/sdhci.h b/include/linux/mmc/sdhci.h > index 1fdc673..315ff49 100644 > --- a/include/linux/mmc/sdhci.h > +++ b/include/linux/mmc/sdhci.h > @@ -83,7 +83,8 @@ struct sdhci_host { > #define SDHCI_QUIRK_MULTIBLOCK_READ_ACMD12 (1<<28) > /* Controller doesn't have HISPD bit field in HI-SPEED SD card */ > #define SDHCI_QUIRK_NO_HISPD_BIT (1<<29) > - > +/* Controller need set data timeout value when card is busy */ > +#define SDHCI_QUIRK_SET_DATA_TIMEOUT_VAL (1<<30) > int irq; /* Device IRQ */ > void __iomem *ioaddr; /* Mapped address */ > > -- > 1.6.0.4 > > 2011/2/24 Chuanxiao Dong <chuanxiao.dong@intel.com>: > > Set the timeout control register for SDHCI host when it needs to send some > > commands which need busy signal. Use the maximum timeout value will be safe. > > > > Signed-off-by: Chuanxiao Dong <chuanxiao.dong@intel.com> > > --- > > drivers/mmc/host/sdhci.c | 14 ++++++++++---- > > 1 files changed, 10 insertions(+), 4 deletions(-) > > > > diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c > > index 9e15f41..32b7475 100644 > > --- a/drivers/mmc/host/sdhci.c > > +++ b/drivers/mmc/host/sdhci.c > > @@ -40,7 +40,6 @@ > > > > static unsigned int debug_quirks = 0; > > > > -static void sdhci_prepare_data(struct sdhci_host *, struct mmc_data *); > > static void sdhci_finish_data(struct sdhci_host *); > > > > static void sdhci_send_command(struct sdhci_host *, struct mmc_command > *); > > @@ -651,16 +650,23 @@ static void sdhci_set_transfer_irqs(struct sdhci_host > *host) > > sdhci_clear_set_irqs(host, dma_irqs, pio_irqs); > > } > > > > -static void sdhci_prepare_data(struct sdhci_host *host, struct mmc_data *data) > > +static void sdhci_prepare_data(struct sdhci_host *host, struct mmc_command > *cmd) > > { > > u8 count; > > u8 ctrl; > > + struct mmc_data *data = cmd->data; > > int ret; > > > > WARN_ON(host->data); > > > > - if (data == NULL) > > + if (data == NULL) { > > + /* > > + * set timeout to be maximum value for command with busy > signal. > > + */ > > + if (cmd->flags & MMC_RSP_BUSY) > > + sdhci_writeb(host, 0xE, > SDHCI_TIMEOUT_CONTROL); > > return; > > + } > > > > /* Sanity checks */ > > BUG_ON(data->blksz * data->blocks > 524288); > > @@ -920,7 +926,7 @@ static void sdhci_send_command(struct sdhci_host *host, > struct mmc_command *cmd) > > > > host->cmd = cmd; > > > > - sdhci_prepare_data(host, cmd->data); > > + sdhci_prepare_data(host, cmd); > > > > sdhci_writel(host, cmd->arg, SDHCI_ARGUMENT); > > > > -- > > 1.6.6.1 > > > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-mmc" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/1]mmc: set timeout for SDHCI host before sending busy cmd 2011-02-24 14:54 ` Dong, Chuanxiao @ 2011-02-24 18:32 ` Philip Rakity 2011-02-24 20:15 ` Chris Ball 0 siblings, 1 reply; 8+ messages in thread From: Philip Rakity @ 2011-02-24 18:32 UTC (permalink / raw) To: Dong, Chuanxiao Cc: Jae hoon Chung, linux-mmc, cjb, linux-kernel, arnd, Kyungmin Park On Feb 24, 2011, at 6:54 AM, Dong, Chuanxiao wrote: > Great. Maybe we wouldn't need to add this quirk since give a maximum timeout value is safe for all host controller I think. We would better save the quirk for other really critical silicon bugs. proposed this a while ago and strongly support just removing the quirk for broken timeout and setting the timeout value to maximum of 0xE. This also handles the case of the sd device having a timeout value too low. In my testing I have come across SD cards that do not provide the correct value. We force our pxa168, pxa910, and mmp2 controllers to have 0xE. Philip > >> -----Original Message----- >> From: Jae hoon Chung [mailto:jh80.chung@gmail.com] >> Sent: Thursday, February 24, 2011 10:18 PM >> To: Dong, Chuanxiao >> Cc: linux-mmc@vger.kernel.org; cjb@laptop.org; linux-kernel@vger.kernel.org; >> arnd@arndb.de; Kyungmin Park >> Subject: Re: [PATCH 1/1]mmc: set timeout for SDHCI host before sending busy cmd >> >> Hi >> >> This patch looks like this. I sent to the RFC patch.. >> >> http://marc.info/?l=linux-mmc&m=129109794815028&w=3 >> >> Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com> >> Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com> >> >> --- >> drivers/mmc/host/sdhci.c | 6 +++++- >> include/linux/mmc/sdhci.h | 3 ++- >> 2 files changed, 7 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c >> index 782c0ee..3b93d97 100644 >> --- a/drivers/mmc/host/sdhci.c >> +++ b/drivers/mmc/host/sdhci.c >> @@ -655,8 +655,12 @@ static void sdhci_prepare_data(struct sdhci_host >> *host, struct mmc_data *data) >> >> WARN_ON(host->data); >> >> - if (data == NULL) >> + if (data == NULL) { >> + if ((host->quirks & SDHCI_QUIRK_SET_DATA_TIMEOUT_VAL) >> && >> + (host->cmd->flags & MMC_RSP_BUSY)) >> + sdhci_writel(host, 0xE, SDHCI_TIMEOUT_CONTROL); >> return; >> + } >> >> /* Sanity checks */ >> BUG_ON(data->blksz * data->blocks > 524288); >> diff --git a/include/linux/mmc/sdhci.h b/include/linux/mmc/sdhci.h >> index 1fdc673..315ff49 100644 >> --- a/include/linux/mmc/sdhci.h >> +++ b/include/linux/mmc/sdhci.h >> @@ -83,7 +83,8 @@ struct sdhci_host { >> #define SDHCI_QUIRK_MULTIBLOCK_READ_ACMD12 (1<<28) >> /* Controller doesn't have HISPD bit field in HI-SPEED SD card */ >> #define SDHCI_QUIRK_NO_HISPD_BIT (1<<29) >> - >> +/* Controller need set data timeout value when card is busy */ >> +#define SDHCI_QUIRK_SET_DATA_TIMEOUT_VAL (1<<30) >> int irq; /* Device IRQ */ >> void __iomem *ioaddr; /* Mapped address */ >> >> -- >> 1.6.0.4 >> >> 2011/2/24 Chuanxiao Dong <chuanxiao.dong@intel.com>: >>> Set the timeout control register for SDHCI host when it needs to send some >>> commands which need busy signal. Use the maximum timeout value will be safe. >>> >>> Signed-off-by: Chuanxiao Dong <chuanxiao.dong@intel.com> >>> --- >>> drivers/mmc/host/sdhci.c | 14 ++++++++++---- >>> 1 files changed, 10 insertions(+), 4 deletions(-) >>> >>> diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c >>> index 9e15f41..32b7475 100644 >>> --- a/drivers/mmc/host/sdhci.c >>> +++ b/drivers/mmc/host/sdhci.c >>> @@ -40,7 +40,6 @@ >>> >>> static unsigned int debug_quirks = 0; >>> >>> -static void sdhci_prepare_data(struct sdhci_host *, struct mmc_data *); >>> static void sdhci_finish_data(struct sdhci_host *); >>> >>> static void sdhci_send_command(struct sdhci_host *, struct mmc_command >> *); >>> @@ -651,16 +650,23 @@ static void sdhci_set_transfer_irqs(struct sdhci_host >> *host) >>> sdhci_clear_set_irqs(host, dma_irqs, pio_irqs); >>> } >>> >>> -static void sdhci_prepare_data(struct sdhci_host *host, struct mmc_data *data) >>> +static void sdhci_prepare_data(struct sdhci_host *host, struct mmc_command >> *cmd) >>> { >>> u8 count; >>> u8 ctrl; >>> + struct mmc_data *data = cmd->data; >>> int ret; >>> >>> WARN_ON(host->data); >>> >>> - if (data == NULL) >>> + if (data == NULL) { >>> + /* >>> + * set timeout to be maximum value for command with busy >> signal. >>> + */ >>> + if (cmd->flags & MMC_RSP_BUSY) >>> + sdhci_writeb(host, 0xE, >> SDHCI_TIMEOUT_CONTROL); >>> return; >>> + } >>> >>> /* Sanity checks */ >>> BUG_ON(data->blksz * data->blocks > 524288); >>> @@ -920,7 +926,7 @@ static void sdhci_send_command(struct sdhci_host *host, >> struct mmc_command *cmd) >>> >>> host->cmd = cmd; >>> >>> - sdhci_prepare_data(host, cmd->data); >>> + sdhci_prepare_data(host, cmd); >>> >>> sdhci_writel(host, cmd->arg, SDHCI_ARGUMENT); >>> >>> -- >>> 1.6.6.1 >>> >>> -- >>> To unsubscribe from this list: send the line "unsubscribe linux-mmc" in >>> the body of a message to majordomo@vger.kernel.org >>> More majordomo info at http://vger.kernel.org/majordomo-info.html >>> > -- > To unsubscribe from this list: send the line "unsubscribe linux-mmc" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/1]mmc: set timeout for SDHCI host before sending busy cmd 2011-02-24 18:32 ` Philip Rakity @ 2011-02-24 20:15 ` Chris Ball 2011-02-24 20:43 ` Wolfram Sang 2011-02-25 1:25 ` Philip Rakity 0 siblings, 2 replies; 8+ messages in thread From: Chris Ball @ 2011-02-24 20:15 UTC (permalink / raw) To: Philip Rakity Cc: Dong, Chuanxiao, Jae hoon Chung, linux-mmc, linux-kernel, arnd, Kyungmin Park On Thu, Feb 24, 2011 at 10:32:30AM -0800, Philip Rakity wrote: > proposed this a while ago and strongly support just removing the quirk for broken timeout and setting the timeout value to maximum of 0xE. > > This also handles the case of the sd device having a timeout value too low. In my testing I have come across SD cards that do not provide the correct value. > We force our pxa168, pxa910, and mmp2 controllers to have 0xE. Yeah, OLPC's CaFe controller -- which might be the same hardware as yours, actually -- has the same problem. Does anyone know of a reason (beyond strict spec-compliance, I suppose) for honoring the timeout value rather than using 0xE everywhere? If not, I'm willing to try out Philip's suggestion. Thanks, -- Chris Ball <cjb@laptop.org> <http://printf.net/> One Laptop Per Child ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/1]mmc: set timeout for SDHCI host before sending busy cmd 2011-02-24 20:15 ` Chris Ball @ 2011-02-24 20:43 ` Wolfram Sang 2011-02-24 21:24 ` Chris Ball 2011-02-25 1:25 ` Philip Rakity 1 sibling, 1 reply; 8+ messages in thread From: Wolfram Sang @ 2011-02-24 20:43 UTC (permalink / raw) To: Chris Ball Cc: Philip Rakity, Dong, Chuanxiao, Jae hoon Chung, linux-mmc, linux-kernel, arnd, Kyungmin Park [-- Attachment #1: Type: text/plain, Size: 1127 bytes --] On Thu, Feb 24, 2011 at 08:15:04PM +0000, Chris Ball wrote: > On Thu, Feb 24, 2011 at 10:32:30AM -0800, Philip Rakity wrote: > > proposed this a while ago and strongly support just removing the quirk for broken timeout and setting the timeout value to maximum of 0xE. > > > > This also handles the case of the sd device having a timeout value too low. In my testing I have come across SD cards that do not provide the correct value. > > We force our pxa168, pxa910, and mmp2 controllers to have 0xE. > > Yeah, OLPC's CaFe controller -- which might be the same hardware as yours, > actually -- has the same problem. > > Does anyone know of a reason (beyond strict spec-compliance, I suppose) > for honoring the timeout value rather than using 0xE everywhere? If not, > I'm willing to try out Philip's suggestion. +1. A full cycle in linux-next might an idea to be on the safe side? That would be 2.6.40-material then. Or too slow? Wolfram -- Pengutronix e.K. | Wolfram Sang | Industrial Linux Solutions | http://www.pengutronix.de/ | [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 197 bytes --] ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/1]mmc: set timeout for SDHCI host before sending busy cmd 2011-02-24 20:43 ` Wolfram Sang @ 2011-02-24 21:24 ` Chris Ball 0 siblings, 0 replies; 8+ messages in thread From: Chris Ball @ 2011-02-24 21:24 UTC (permalink / raw) To: Wolfram Sang Cc: Philip Rakity, Dong, Chuanxiao, Jae hoon Chung, linux-mmc, linux-kernel, arnd, Kyungmin Park Hi, On Thu, Feb 24, 2011 at 09:43:52PM +0100, Wolfram Sang wrote: > +1. A full cycle in linux-next might an idea to be on the safe side? That > would be 2.6.40-material then. Or too slow? I think it could be okay to merge for .39, but that's mainly because I don't think we start getting testing with a lot of cards until patches appear in an -rc1 -- so for something that requires broad testing, I'd rather merge it for -rc1 with a plan to revert it if we find anything unexpected. But I don't have a strong opinion, so if anyone thinks there's a reason to wait (for example, an existence-proof of a card that misbehaves when configured with a max timeout value) then I'm happy to do so. Thanks, -- Chris Ball <cjb@laptop.org> <http://printf.net/> One Laptop Per Child ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/1]mmc: set timeout for SDHCI host before sending busy cmd 2011-02-24 20:15 ` Chris Ball 2011-02-24 20:43 ` Wolfram Sang @ 2011-02-25 1:25 ` Philip Rakity 1 sibling, 0 replies; 8+ messages in thread From: Philip Rakity @ 2011-02-25 1:25 UTC (permalink / raw) To: Chris Ball Cc: Dong, Chuanxiao, Jae hoon Chung, linux-mmc, linux-kernel, arnd, Kyungmin Park On Feb 24, 2011, at 12:15 PM, Chris Ball wrote: > On Thu, Feb 24, 2011 at 10:32:30AM -0800, Philip Rakity wrote: >> proposed this a while ago and strongly support just removing the quirk for broken timeout and setting the timeout value to maximum of 0xE. >> >> This also handles the case of the sd device having a timeout value too low. In my testing I have come across SD cards that do not provide the correct value. >> We force our pxa168, pxa910, and mmp2 controllers to have 0xE. > > Yeah, OLPC's CaFe controller -- which might be the same hardware as yours, > actually -- has the same problem. different controller not same as on mmp2, pxa168, pxa910 > > Does anyone know of a reason (beyond strict spec-compliance, I suppose) > for honoring the timeout value rather than using 0xE everywhere? If not, > I'm willing to try out Philip's suggestion. > > Thanks, > > -- > Chris Ball <cjb@laptop.org> <http://printf.net/> > One Laptop Per Child ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2011-02-25 1:25 UTC | newest] Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2011-02-24 11:19 [PATCH 1/1]mmc: set timeout for SDHCI host before sending busy cmd Chuanxiao Dong 2011-02-24 14:18 ` Jae hoon Chung 2011-02-24 14:54 ` Dong, Chuanxiao 2011-02-24 18:32 ` Philip Rakity 2011-02-24 20:15 ` Chris Ball 2011-02-24 20:43 ` Wolfram Sang 2011-02-24 21:24 ` Chris Ball 2011-02-25 1:25 ` Philip Rakity
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.