RE: [PATCH] mmc: fixed wrong SD card detection

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



I didn't see my patch :(
I use the f_min to replace the f_init when f_init is zero and that is used for get_cd callback function.

Thanks and Best Regards
Jerry Huang


> -----Original Message-----
> From: Jaehoon Chung [mailto:jh80.chung@xxxxxxxxxxx]
> Sent: Thursday, May 12, 2011 1:14 PM
> To: Huang Changming-R66093
> Cc: Jaehoon Chung; linux-mmc@xxxxxxxxxxxxxxx; Chris Ball; Kyungmin Park
> Subject: Re: [PATCH] mmc: fixed wrong SD card detection
> 
> Hi Huang.
> 
> Thanks for your comments. but this is not my hardware issue. :) But i
> found out your patch "[PATCH 3/3] SDHC/MMC: add f_min to mmc_power_on"
> 
> I think that related with this problem.
> I will test then share the results
> 
> Best regards,
> Jaehoon Chung
> 
> Huang Changming-R66093 wrote:
> > I don't see this.
> > Maybe that is your hardware issue.
> >
> > Thanks and Best Regards
> > Jerry Huang
> >
> >
> >> -----Original Message-----
> >> From: linux-mmc-owner@xxxxxxxxxxxxxxx [mailto:linux-mmc-
> >> owner@xxxxxxxxxxxxxxx] On Behalf Of Jaehoon Chung
> >> Sent: Friday, May 06, 2011 5:39 PM
> >> To: linux-mmc@xxxxxxxxxxxxxxx
> >> Cc: Chris Ball; Kyungmin Park
> >> Subject: [PATCH] mmc: fixed wrong SD card detection
> >>
> >> This patch added mmc_power_off after mmc_attach_sd()
> >>
> >> i found that didn't detect the second card insert.(using SDHCI) you
> >> can see the below log.
> >>
> >> i want to know that somebody know this issue...
> >>
> >> ->first card inserted (correct card detect)
> >> # mmc1: new SDHC card at address e624
> >> mmcblk1: mmc1:e624 SU04G 3.69 GiB
> >>  mmcblk1: p1
> >> #
> >> # mmc1: card e624 removed
> >> -> second card inserted
> >> mmc1: error -110 whilst initialising SD card
> >> mmc1: Card removed during transfer!
> >> mmc1: Resetting controller.
> >> -> third card inserted
> >> mmc1: new SDHC card at address e624
> >> mmcblk1: mmc1:e624 SU04G 3.69 GiB
> >>  mmcblk1: p1
> >>
> >> I added mmc_power_off() after only mmc_attach_sd().
> >>
> >> Signed-off-by: Jaehoon Chung <jh80.chung@xxxxxxxxxxx>
> >> Signed-off-by: kyungmin Park <kyungmin.park@xxxxxxxxxxx>
> >> ---
> >>  drivers/mmc/core/core.c |    4 +++-
> >>  1 files changed, 3 insertions(+), 1 deletions(-)
> >>
> >> diff --git a/drivers/mmc/core/core.c b/drivers/mmc/core/core.c index
> >> 1dbc185..d5d8ac0 100644
> >> --- a/drivers/mmc/core/core.c
> >> +++ b/drivers/mmc/core/core.c
> >> @@ -1522,8 +1522,10 @@ static int mmc_rescan_try_freq(struct mmc_host
> >> *host, unsigned freq)
> >>  	/* Order's important: probe SDIO, then SD, then MMC */
> >>  	if (!mmc_attach_sdio(host))
> >>  		return 0;
> >> -	if (!mmc_attach_sd(host))
> >> +	if (!mmc_attach_sd(host)) {
> >> +		mmc_power_off(host);
> >>  		return 0;
> >> +	}
> >>  	if (!mmc_attach_mmc(host))
> >>  		return 0;
> >>
> >> --
> >> To unsubscribe from this list: send the line "unsubscribe linux-mmc"
> >> in the body of a message to majordomo@xxxxxxxxxxxxxxx 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@xxxxxxxxxxxxxxx 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@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux