Re: core: negotiate ocr during resume
> -Original Message-
> From: linux-mmc-ow...@vger.kernel.org
> [mailto:linux-mmc-ow...@vger.kernel.org] On Behalf Of Prasanna NAVARATNA
> Sent: Friday, April 05, 2013 6:50 PM
> To: linux-mmc@vger.kernel.org
> Subject: Re: core: negotiate ocr during resume
>
> Kevin Liu gmail.com> writes:
>
> Please find my updated patch and comments under _PNN_ below :-
>
>> I think there are several issues here:
>> 1. with this patch the vmmc voltage should still be 3.3v after resume
>> back since the vmmc voltage is updated in set_ios which depend on
>> mmc_select_voltage. But mmc_select_voltage didn't notice your ocr
>> update.
> _PNN_: Yes, good point. I agree. Accordingly updated the patch.
>
>> 2. you didn't handle the special case that the card is changed to a
>> new card during suspended. So the ocr may be different here.
> _PNN_ : This case doesn't holds good because even when new card is inserted
> druing suspend, cd interrupt is generated and mmc_detect_change will be
> called and mmc_rescan will be scheduled to begin initialization from the
> begining, where ocr is freshly read from new card (mmc_attach_***) and my
> patch will take care to update new ocr in card->ocr.
> Although consider an hypothetical situtaion where within the debounce time
> of cd, suppose resume gets called, it may fail. But fair enough after 200ms,
> fresh initialization will begin and card re-inits. So no need to take
> special consideration here. Don't you agree?
>
But if the card detect can't wakeup system then the cd interrupt may
be discarded if it occured during system suspended. For example, if
the card detect used slot-gpio and the slot power is off during system
suspended or card used polling for card detect.
So there must be a solution to verify whether a same card after system
resumed. In fact, there have been such method in current code:
in mmc_sd_init_card:
if (oldcard) {
if (memcmp(cid, oldcard->raw_cid, sizeof(cid)) != 0)
return -ENOENT;
card = oldcard;
So in my patch, I moved mmc_select_voltage after this verification.
Please consider this case.
>> 3. mmc.c and sdio.c should also be updated besides sd.c.
>>
> _PNN_ : Thanks for pointing this out. Patch updated.
>
>> I have another old version of this patch attached below for your
> reference.
> _PNN_ : I do agree with your patch, but i would suggest to consider my below
> patch rather than yours for following reasons:
> a. Very simple & less code changes.
> b. Similar to rca, cid, csa, better to have "ocr" structure member in
> "struct mmc_card" which is card specific.
> c. mmc_select_voltage is localised in core.c rather than moving it accross
> sd.c sdio.c and mmc.c
> d. ocr field might be helpful in future too. So better to have it rather
> than "ocr_bak" in struct mmc_host.
>
> From ad70dbd00db355d1e8ca08e7ad12e73cb41df960 Mon Sep 17 00:00:00 2001
> From: Prasanna NAVARATNA
> Date: Thu, 4 Apr 2013 19:55:19 +0530
> Subject: [PATCH] mmc: core: proper ocr negotiation during resume
>
> Save the card ocr into struct mmc_card when read from card
> during initialization of sd/mmc/sdio.
> Druing mmc_resume_host, supply saved card's ocr to
> mmc_select_voltage so as to negotiate voltage appropriately.
> ---
> drivers/mmc/core/core.c |2 +-
> drivers/mmc/core/mmc.c |3 +++
> drivers/mmc/core/sd.c|3 +++
> drivers/mmc/core/sdio.c |3 +++
> include/linux/mmc/card.h |1 +
> 5 files changed, 11 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/mmc/core/core.c b/drivers/mmc/core/core.c
> index ad7decc..aad511a 100644
> --- a/drivers/mmc/core/core.c
> +++ b/drivers/mmc/core/core.c
> @@ -2680,7 +2680,7 @@ int mmc_resume_host(struct mmc_host *host)
> if (host->bus_ops && !host->bus_dead) {
> if (!mmc_card_keep_power(host)) {
> mmc_power_up(host);
> - mmc_select_voltage(host, host->ocr);
> + mmc_select_voltage(host, host->card->ocr);
> /*
> * Tell runtime PM core we just powered up the card,
> * since it still believes the card is powered off.
> diff --git a/drivers/mmc/core/mmc.c b/drivers/mmc/core/mmc.c
> index d584f7c..923eb53 100644
> --- a/drivers/mmc/core/mmc.c
> +++ b/drivers/mmc/core/mmc.c
> @@ -1566,6 +1566,9 @@ int mmc_attach_mmc(struct mmc_host *host)
> ocr &= ~0x7F;
> }
>
> + /* Save the card OCR */
> + host->card->ocr = ocr;
> +
> host->ocr = mmc_select_voltage(host, ocr);
>
> /*
> diff --git a/drivers/mmc/core/sd.c b/drivers/mmc/core/sd.c
> index 9e645e1..965504b 100644
> --- a/drivers/mmc/core/sd.c
> +++ b/drivers/mmc/core/sd.c
> @@ -1185,6 +1185,9 @@ int mmc_attach_sd(struct mmc_host *host)
> ocr &= ~MMC_VDD_165_195;
> }
>
> + /* Save the card OCR */
> + host->card->ocr = ocr;
> +
> host->ocr = mmc_
Re: core: negotiate ocr during resume
Kevin Liu gmail.com> writes:
Please find my updated patch and comments under _PNN_ below :-
> I think there are several issues here:
> 1. with this patch the vmmc voltage should still be 3.3v after resume
> back since the vmmc voltage is updated in set_ios which depend on
> mmc_select_voltage. But mmc_select_voltage didn't notice your ocr
> update.
_PNN_: Yes, good point. I agree. Accordingly updated the patch.
> 2. you didn't handle the special case that the card is changed to a
> new card during suspended. So the ocr may be different here.
_PNN_ : This case doesn't holds good because even when new card is inserted
druing suspend, cd interrupt is generated and mmc_detect_change will be
called and mmc_rescan will be scheduled to begin initialization from the
begining, where ocr is freshly read from new card (mmc_attach_***) and my
patch will take care to update new ocr in card->ocr.
Although consider an hypothetical situtaion where within the debounce time
of cd, suppose resume gets called, it may fail. But fair enough after 200ms,
fresh initialization will begin and card re-inits. So no need to take
special consideration here. Don't you agree?
> 3. mmc.c and sdio.c should also be updated besides sd.c.
>
_PNN_ : Thanks for pointing this out. Patch updated.
> I have another old version of this patch attached below for your
reference.
_PNN_ : I do agree with your patch, but i would suggest to consider my below
patch rather than yours for following reasons:
a. Very simple & less code changes.
b. Similar to rca, cid, csa, better to have "ocr" structure member in
"struct mmc_card" which is card specific.
c. mmc_select_voltage is localised in core.c rather than moving it accross
sd.c sdio.c and mmc.c
d. ocr field might be helpful in future too. So better to have it rather
than "ocr_bak" in struct mmc_host.
>From ad70dbd00db355d1e8ca08e7ad12e73cb41df960 Mon Sep 17 00:00:00 2001
From: Prasanna NAVARATNA
Date: Thu, 4 Apr 2013 19:55:19 +0530
Subject: [PATCH] mmc: core: proper ocr negotiation during resume
Save the card ocr into struct mmc_card when read from card
during initialization of sd/mmc/sdio.
Druing mmc_resume_host, supply saved card's ocr to
mmc_select_voltage so as to negotiate voltage appropriately.
---
drivers/mmc/core/core.c |2 +-
drivers/mmc/core/mmc.c |3 +++
drivers/mmc/core/sd.c|3 +++
drivers/mmc/core/sdio.c |3 +++
include/linux/mmc/card.h |1 +
5 files changed, 11 insertions(+), 1 deletions(-)
diff --git a/drivers/mmc/core/core.c b/drivers/mmc/core/core.c
index ad7decc..aad511a 100644
--- a/drivers/mmc/core/core.c
+++ b/drivers/mmc/core/core.c
@@ -2680,7 +2680,7 @@ int mmc_resume_host(struct mmc_host *host)
if (host->bus_ops && !host->bus_dead) {
if (!mmc_card_keep_power(host)) {
mmc_power_up(host);
- mmc_select_voltage(host, host->ocr);
+ mmc_select_voltage(host, host->card->ocr);
/*
* Tell runtime PM core we just powered up the card,
* since it still believes the card is powered off.
diff --git a/drivers/mmc/core/mmc.c b/drivers/mmc/core/mmc.c
index d584f7c..923eb53 100644
--- a/drivers/mmc/core/mmc.c
+++ b/drivers/mmc/core/mmc.c
@@ -1566,6 +1566,9 @@ int mmc_attach_mmc(struct mmc_host *host)
ocr &= ~0x7F;
}
+ /* Save the card OCR */
+ host->card->ocr = ocr;
+
host->ocr = mmc_select_voltage(host, ocr);
/*
diff --git a/drivers/mmc/core/sd.c b/drivers/mmc/core/sd.c
index 9e645e1..965504b 100644
--- a/drivers/mmc/core/sd.c
+++ b/drivers/mmc/core/sd.c
@@ -1185,6 +1185,9 @@ int mmc_attach_sd(struct mmc_host *host)
ocr &= ~MMC_VDD_165_195;
}
+ /* Save the card OCR */
+ host->card->ocr = ocr;
+
host->ocr = mmc_select_voltage(host, ocr);
/*
diff --git a/drivers/mmc/core/sdio.c b/drivers/mmc/core/sdio.c
index aa0719a..3f9e08d 100644
--- a/drivers/mmc/core/sdio.c
+++ b/drivers/mmc/core/sdio.c
@@ -1090,6 +1090,9 @@ int mmc_attach_sdio(struct mmc_host *host)
ocr &= ~0x7F;
}
+ /* Save the card OCR */
+ host->card->ocr = ocr;
+
host->ocr = mmc_select_voltage(host, ocr);
/*
diff --git a/include/linux/mmc/card.h b/include/linux/mmc/card.h
index f31725b..1bbec2f 100644
--- a/include/linux/mmc/card.h
+++ b/include/linux/mmc/card.h
@@ -273,6 +273,7 @@ struct mmc_card {
u32 raw_cid[4]; /* raw card CID */
u32 raw_csd[4]; /* raw card CSD */
u32 raw_scr[2]; /* raw card SCR */
+ u32 ocr;/* card OCR */
struct mmc_cid cid;/* card identification */
struct mmc_csd csd;/* card specific */
struct mmc_ext_csd ext_csd;/* mmc v4 exten
Re: core: negotiate ocr during resume
Please find my updated patch and comments under _PNN_ below :-
I think there are several issues here:
1. with this patch the vmmc voltage should still be 3.3v after resume
back since the vmmc voltage is updated in set_ios which depend on
mmc_select_voltage. But mmc_select_voltage didn't notice your ocr
update.
_PNN_: Yes, good point. I agree. Accordingly updated the patch.
2. you didn't handle the special case that the card is changed to a
new card during suspended. So the ocr may be different here.
_PNN_ : This case doesn't holds good because even when new card is inserted
druing suspend, cd interrupt is generated and mmc_detect_change will be
called and mmc_rescan will be scheduled to begin initialization from the
begining, where ocr is freshly read from new card (mmc_attach_***) and my
patch will take care to update new ocr in card->ocr.
Although consider an hypothetical situtaion where within the debounce time
of cd, suppose resume gets called, it may fail. But fair enough after 200ms,
fresh initialization will begin and card re-inits. So no need to take
special consideration here. Don't you agree?
3. mmc.c and sdio.c should also be updated besides sd.c.
_PNN_ : Thanks for pointing this out. Patch updated.
I have another old version of this patch attached below for your reference.
_PNN_ : I do agree with your patch, but i would suggest to consider my below
patch rather than yours for following reasons:
a. Obviously simple & less code changes.
b. Similar to rca, cid, csa, better to have "ocr" structure member in
"struct mmc_card" which is card specific.
c. mmc_select_voltage is localised in core.c rather than moving it accross
sd.c sdio.c and mmc.c
d. ocr field might be helpful in future too. So better to have it rather
than "ocr_bak" in struct mmc_host.
>From ad70dbd00db355d1e8ca08e7ad12e73cb41df960 Mon Sep 17 00:00:00 2001
From: Prasanna NAVARATNA
Date: Thu, 4 Apr 2013 19:55:19 +0530
Subject: [PATCH] mmc: core: proper ocr negotiation during resume
Save the card ocr into struct mmc_card when read from card
during initialization of sd/mmc/sdio.
Druing mmc_resume_host, supply saved card's ocr to
mmc_select_voltage so as to negotiate voltage appropriately.
---
drivers/mmc/core/core.c |2 +-
drivers/mmc/core/mmc.c |3 +++
drivers/mmc/core/sd.c|3 +++
drivers/mmc/core/sdio.c |3 +++
include/linux/mmc/card.h |1 +
5 files changed, 11 insertions(+), 1 deletions(-)
diff --git a/drivers/mmc/core/core.c b/drivers/mmc/core/core.c
index ad7decc..aad511a 100644
--- a/drivers/mmc/core/core.c
+++ b/drivers/mmc/core/core.c
@@ -2680,7 +2680,7 @@ int mmc_resume_host(struct mmc_host *host)
if (host->bus_ops && !host->bus_dead) {
if (!mmc_card_keep_power(host)) {
mmc_power_up(host);
- mmc_select_voltage(host, host->ocr);
+ mmc_select_voltage(host, host->card->ocr);
/*
* Tell runtime PM core we just powered up the card,
* since it still believes the card is powered off.
diff --git a/drivers/mmc/core/mmc.c b/drivers/mmc/core/mmc.c
index d584f7c..923eb53 100644
--- a/drivers/mmc/core/mmc.c
+++ b/drivers/mmc/core/mmc.c
@@ -1566,6 +1566,9 @@ int mmc_attach_mmc(struct mmc_host *host)
ocr &= ~0x7F;
}
+ /* Save the card OCR */
+ host->card->ocr = ocr;
+
host->ocr = mmc_select_voltage(host, ocr);
/*
diff --git a/drivers/mmc/core/sd.c b/drivers/mmc/core/sd.c
index 9e645e1..965504b 100644
--- a/drivers/mmc/core/sd.c
+++ b/drivers/mmc/core/sd.c
@@ -1185,6 +1185,9 @@ int mmc_attach_sd(struct mmc_host *host)
ocr &= ~MMC_VDD_165_195;
}
+ /* Save the card OCR */
+ host->card->ocr = ocr;
+
host->ocr = mmc_select_voltage(host, ocr);
/*
diff --git a/drivers/mmc/core/sdio.c b/drivers/mmc/core/sdio.c
index aa0719a..3f9e08d 100644
--- a/drivers/mmc/core/sdio.c
+++ b/drivers/mmc/core/sdio.c
@@ -1090,6 +1090,9 @@ int mmc_attach_sdio(struct mmc_host *host)
ocr &= ~0x7F;
}
+ /* Save the card OCR */
+ host->card->ocr = ocr;
+
host->ocr = mmc_select_voltage(host, ocr);
/*
diff --git a/include/linux/mmc/card.h b/include/linux/mmc/card.h
index f31725b..1bbec2f 100644
--- a/include/linux/mmc/card.h
+++ b/include/linux/mmc/card.h
@@ -273,6 +273,7 @@ struct mmc_card {
u32 raw_cid[4]; /* raw card CID */
u32 raw_csd[4]; /* raw card CSD */
u32 raw_scr[2]; /* raw card SCR */
+ u32 ocr;/* card OCR */
struct mmc_cid cid;/* card identification */
struct mmc_csd csd;/* card specific */
struct mmc_ext_csd ext_csd;/* mmc v4 extended card
specific */
--
1.7.6
Note : mmc_sel
Re: core: negotiate ocr during resume
Please find my updated patch and comments under _PNN_ below :-
I think there are several issues here:
1. with this patch the vmmc voltage should still be 3.3v after resume
back since the vmmc voltage is updated in set_ios which depend on
mmc_select_voltage. But mmc_select_voltage didn't notice your ocr
update.
_PNN_: Yes, good point. I agree. Accordingly updated the patch.
2. you didn't handle the special case that the card is changed to a
new card during suspended. So the ocr may be different here.
_PNN_ : This case doesn't holds good because even when new card is inserted
druing suspend, cd interrupt is generated and mmc_detect_change will be
called and mmc_rescan will be scheduled to begin initialization from the
begining, where ocr is freshly read from new card (mmc_attach_***) and my
patch will take care to update new ocr in card->ocr.
Although consider an hypothetical situtaion where within the debounce time
of cd, suppose resume gets called, it may fail. But fair enough after 200ms,
fresh initialization will begin and card re-inits. So no need to take
special consideration here. Don't you agree?
3. mmc.c and sdio.c should also be updated besides sd.c.
_PNN_ : Thanks for pointing this out. Patch updated.
I have another old version of this patch attached below for your reference.
_PNN_ : I do agree with your patch, but i would suggest to consider my below
patch rather than yours for following reasons:
a. Obviously simple & less code changes.
b. Similar to rca, cid, csa, better to have "ocr" structure member in
"struct mmc_card" which is card specific.
c. mmc_select_voltage is localised in core.c rather than moving it accross
sd.c sdio.c and mmc.c
d. ocr field might be helpful in future too. So better to have it rather
than "ocr_bak" in struct mmc_host.
>From ad70dbd00db355d1e8ca08e7ad12e73cb41df960 Mon Sep 17 00:00:00 2001
From: Prasanna NAVARATNA
Date: Thu, 4 Apr 2013 19:55:19 +0530
Subject: [PATCH] mmc: core: proper ocr negotiation during resume
Save the card ocr into struct mmc_card when read from card
during initialization of sd/mmc/sdio.
Druing mmc_resume_host, supply saved card's ocr to
mmc_select_voltage so as to negotiate voltage appropriately.
---
drivers/mmc/core/core.c |2 +-
drivers/mmc/core/mmc.c |3 +++
drivers/mmc/core/sd.c|3 +++
drivers/mmc/core/sdio.c |3 +++
include/linux/mmc/card.h |1 +
5 files changed, 11 insertions(+), 1 deletions(-)
diff --git a/drivers/mmc/core/core.c b/drivers/mmc/core/core.c
index ad7decc..aad511a 100644
--- a/drivers/mmc/core/core.c
+++ b/drivers/mmc/core/core.c
@@ -2680,7 +2680,7 @@ int mmc_resume_host(struct mmc_host *host)
if (host->bus_ops && !host->bus_dead) {
if (!mmc_card_keep_power(host)) {
mmc_power_up(host);
- mmc_select_voltage(host, host->ocr);
+ mmc_select_voltage(host, host->card->ocr);
/*
* Tell runtime PM core we just powered up the card,
* since it still believes the card is powered off.
diff --git a/drivers/mmc/core/mmc.c b/drivers/mmc/core/mmc.c
index d584f7c..923eb53 100644
--- a/drivers/mmc/core/mmc.c
+++ b/drivers/mmc/core/mmc.c
@@ -1566,6 +1566,9 @@ int mmc_attach_mmc(struct mmc_host *host)
ocr &= ~0x7F;
}
+ /* Save the card OCR */
+ host->card->ocr = ocr;
+
host->ocr = mmc_select_voltage(host, ocr);
/*
diff --git a/drivers/mmc/core/sd.c b/drivers/mmc/core/sd.c
index 9e645e1..965504b 100644
--- a/drivers/mmc/core/sd.c
+++ b/drivers/mmc/core/sd.c
@@ -1185,6 +1185,9 @@ int mmc_attach_sd(struct mmc_host *host)
ocr &= ~MMC_VDD_165_195;
}
+ /* Save the card OCR */
+ host->card->ocr = ocr;
+
host->ocr = mmc_select_voltage(host, ocr);
/*
diff --git a/drivers/mmc/core/sdio.c b/drivers/mmc/core/sdio.c
index aa0719a..3f9e08d 100644
--- a/drivers/mmc/core/sdio.c
+++ b/drivers/mmc/core/sdio.c
@@ -1090,6 +1090,9 @@ int mmc_attach_sdio(struct mmc_host *host)
ocr &= ~0x7F;
}
+ /* Save the card OCR */
+ host->card->ocr = ocr;
+
host->ocr = mmc_select_voltage(host, ocr);
/*
diff --git a/include/linux/mmc/card.h b/include/linux/mmc/card.h
index f31725b..1bbec2f 100644
--- a/include/linux/mmc/card.h
+++ b/include/linux/mmc/card.h
@@ -273,6 +273,7 @@ struct mmc_card {
u32 raw_cid[4]; /* raw card CID */
u32 raw_csd[4]; /* raw card CSD */
u32 raw_scr[2]; /* raw card SCR */
+ u32 ocr;/* card OCR */
struct mmc_cid cid;/* card identification */
struct mmc_csd csd;/* card specific */
struct mmc_ext_csd ext_csd;/* mmc v4 extended card
specific */
--
1.7.6
Note : mmc_sel
[PATCH] mmc: core: proper ocr negotiation during resume
>From ad70dbd00db355d1e8ca08e7ad12e73cb41df960 Mon Sep 17 00:00:00 2001
From: Prasanna NAVARATNA
Date: Thu, 4 Apr 2013 19:55:19 +0530
Subject: [PATCH] mmc: core: proper ocr negotiation during resume
Save the card ocr into struct mmc_card when read from card
during initialization of sd/mmc/sdio.
Druing mmc_resume_host, supply saved card's ocr to
mmc_select_voltage so as to negotiate voltage appropriately.
---
drivers/mmc/core/core.c |2 +-
drivers/mmc/core/mmc.c |3 +++
drivers/mmc/core/sd.c|3 +++
drivers/mmc/core/sdio.c |3 +++
include/linux/mmc/card.h |1 +
5 files changed, 11 insertions(+), 1 deletions(-)
diff --git a/drivers/mmc/core/core.c b/drivers/mmc/core/core.c
index ad7decc..aad511a 100644
--- a/drivers/mmc/core/core.c
+++ b/drivers/mmc/core/core.c
@@ -2680,7 +2680,7 @@ int mmc_resume_host(struct mmc_host *host)
if (host->bus_ops && !host->bus_dead) {
if (!mmc_card_keep_power(host)) {
mmc_power_up(host);
- mmc_select_voltage(host, host->ocr);
+ mmc_select_voltage(host, host->card->ocr);
/*
* Tell runtime PM core we just powered up the card,
* since it still believes the card is powered off.
diff --git a/drivers/mmc/core/mmc.c b/drivers/mmc/core/mmc.c
index d584f7c..923eb53 100644
--- a/drivers/mmc/core/mmc.c
+++ b/drivers/mmc/core/mmc.c
@@ -1566,6 +1566,9 @@ int mmc_attach_mmc(struct mmc_host *host)
ocr &= ~0x7F;
}
+ /* Save the card OCR */
+ host->card->ocr = ocr;
+
host->ocr = mmc_select_voltage(host, ocr);
/*
diff --git a/drivers/mmc/core/sd.c b/drivers/mmc/core/sd.c
index 9e645e1..965504b 100644
--- a/drivers/mmc/core/sd.c
+++ b/drivers/mmc/core/sd.c
@@ -1185,6 +1185,9 @@ int mmc_attach_sd(struct mmc_host *host)
ocr &= ~MMC_VDD_165_195;
}
+ /* Save the card OCR */
+ host->card->ocr = ocr;
+
host->ocr = mmc_select_voltage(host, ocr);
/*
diff --git a/drivers/mmc/core/sdio.c b/drivers/mmc/core/sdio.c
index aa0719a..3f9e08d 100644
--- a/drivers/mmc/core/sdio.c
+++ b/drivers/mmc/core/sdio.c
@@ -1090,6 +1090,9 @@ int mmc_attach_sdio(struct mmc_host *host)
ocr &= ~0x7F;
}
+ /* Save the card OCR */
+ host->card->ocr = ocr;
+
host->ocr = mmc_select_voltage(host, ocr);
/*
diff --git a/include/linux/mmc/card.h b/include/linux/mmc/card.h
index f31725b..1bbec2f 100644
--- a/include/linux/mmc/card.h
+++ b/include/linux/mmc/card.h
@@ -273,6 +273,7 @@ struct mmc_card {
u32 raw_cid[4]; /* raw card CID */
u32 raw_csd[4]; /* raw card CSD */
u32 raw_scr[2]; /* raw card SCR */
+ u32 ocr;/* card OCR */
struct mmc_cid cid;/* card identification */
struct mmc_csd csd;/* card specific */
struct mmc_ext_csd ext_csd;/* mmc v4 extended card
specific */
--
1.7.6
Note :-
1. mmc_select_voltage(host, host->card->ocr);
Here host->card->ocr is directly dereferenced because during resume, we will
always have host->card !
2. Similar to rca, cid, csa, better to have "ocr" structure member in
"struct mmc_card" which is card specific and hopefully ocr field might be
helpful in future too.
Regards,
Prasanna NAVARATNA
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] mmc: mxs-mmc: Add broken-cd devicetree property
Dear Alexandre Pereira da Silva, > On Sat, Apr 6, 2013 at 10:58 AM, Marc Kleine-Budde > wrote: > > According to the documentation broken card detect is something different > > > > than non-removable: > > > - broken-cd: There is no card detection available; polling must be > > > used. - non-removable: non-removable slot (like eMMC); assume always > > > present. > > > > But mxs-mmc set MMC_CAP_NEEDS_POLL unconditionally. > > I will work on a better way to fix this. NEEDS_POLL looks nice and seems to fit this case. Best regards, Marek Vasut -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] mmc: mxs-mmc: Add broken-cd devicetree property
On Sat, Apr 6, 2013 at 10:58 AM, Marc Kleine-Budde wrote: > According to the documentation broken card detect is something different > than non-removable: > > > - broken-cd: There is no card detection available; polling must be used. > > - non-removable: non-removable slot (like eMMC); assume always present. > > But mxs-mmc set MMC_CAP_NEEDS_POLL unconditionally. I will work on a better way to fix this. -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] mmc: mxs-mmc: Add broken-cd devicetree property
On 04/06/2013 03:43 PM, Alexandre Pereira da Silva wrote: > In case of broken card detect support, assume the card is non-removable According to the documentation broken card detect is something different than non-removable: > - broken-cd: There is no card detection available; polling must be used. > - non-removable: non-removable slot (like eMMC); assume always present. But mxs-mmc set MMC_CAP_NEEDS_POLL unconditionally. http://lxr.free-electrons.com/source/drivers/mmc/host/mxs-mmc.c#L682 Marc -- Pengutronix e.K. | Marc Kleine-Budde | Industrial Linux Solutions| Phone: +49-231-2826-924 | Vertretung West/Dortmund | Fax: +49-5121-206917- | Amtsgericht Hildesheim, HRA 2686 | http://www.pengutronix.de | signature.asc Description: OpenPGP digital signature
[PATCH] mmc: mxs-mmc: Add broken-cd devicetree property
In case of broken card detect support, assume the card is non-removable Signed-off-by: Alexandre Pereira da Silva --- This patch depends on Marc Kleine-Budde series adding support for non-removable. drivers/mmc/host/mxs-mmc.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/mmc/host/mxs-mmc.c b/drivers/mmc/host/mxs-mmc.c index c231881..8418928 100644 --- a/drivers/mmc/host/mxs-mmc.c +++ b/drivers/mmc/host/mxs-mmc.c @@ -689,7 +689,8 @@ static int mxs_mmc_probe(struct platform_device *pdev) mmc->caps |= MMC_CAP_4_BIT_DATA; else if (bus_width == 8) mmc->caps |= MMC_CAP_4_BIT_DATA | MMC_CAP_8_BIT_DATA; - host->non_removable = of_property_read_bool(np, "non-removable"); + host->non_removable = of_property_read_bool(np, "non-removable") | + of_property_read_bool(np, "broken-cd"); if (host->non_removable) mmc->caps |= MMC_CAP_NONREMOVABLE; host->wp_gpio = of_get_named_gpio_flags(np, "wp-gpios", 0, &flags); -- 1.7.10 -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
