From: Juergen Beisert <jbe@pengutronix.de>
To: barebox@lists.infradead.org
Subject: SD card experts wanted
Date: Wed, 29 Aug 2012 23:17:03 +0200 [thread overview]
Message-ID: <201208292317.03322.jbe@pengutronix.de> (raw)
Hi there,
any SD/MMC card experts here? I have trouble with the current SD/MMC detection
code in Barebox.
The function sd_change_freq() in the file 'drivers/mci/mci-core.c' tries to
detect the max. transfer frequency the attached card can handle. Nothing
special here, but the following line in this routine fails on my system:
if ((__be32_to_cpu(switch_status[4]) & 0x0f000000) == 0x01000000)
It checks the clock speed capabilities returned from the card. As far as I
understand the "simplified SD card spec" the "0x01000000" means SDR25 speed,
which means still "single transfer per clock at 25 MHz clock rate". But the
sd_change_freq() routine sets the MMC_MODE_HS bit for this case in the card's
capabilities variable. The remaining routines in 'drivers/mci/mci-core.c'
then use 50 MHz for the transfer speed.
This behaviour became an issue here, as my SDHC supports 50 MHz, but the
attached card only up to 25 MHz. After switching to 50 MHz, the card wasn't
able to respond anymore.
For SDHCs which supports only up to 25 MHz this misinterpretation is no issue,
as the routines still use the 25 MHz, and the SD card can still respond.
"simplified SD card spec" talks about the bit's meaning: seems the regular
speed was SDR12 which means 12.5 MHz. Then SDR25 cards appear, which are also
called "high speed card". Could the MMC_MODE_HS just mean this "high speed"
and using it at 50 MHz is just some kind of typo? Or do MMCs act differently
and the code mixes SD and MMC features?
Any pointers would help.
jbe
--
Pengutronix e.K. | Juergen Beisert |
Linux Solutions for Science and Industry | http://www.pengutronix.de/ |
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
next reply other threads:[~2012-08-29 21:18 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-08-29 21:17 Juergen Beisert [this message]
2012-08-30 3:27 ` Jean-Christophe PLAGNIOL-VILLARD
2012-08-30 10:04 ` Johannes Stezenbach
2012-08-30 13:08 ` Juergen Beisert
2012-08-30 16:47 ` Johannes Stezenbach
2012-08-30 18:05 ` Juergen Beisert
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201208292317.03322.jbe@pengutronix.de \
--to=jbe@pengutronix.de \
--cc=barebox@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox