Date: Mon, 1 Sep 2008 11:53:28 +0200 From: "Jacques Fourie" <jacques.fourie@gmail.com> To: "Stanislav Sedov" <stas@freebsd.org> Cc: freebsd-arm@freebsd.org Subject: Re: MMC Controller driver for PXA255 Message-ID: <be2f52430809010253wbd5054fn783a3b420296406@mail.gmail.com> In-Reply-To: <be2f52430808300037y79723a2ascf62c33aec7579f3@mail.gmail.com> References: <be2f52430808290920w5e586180q657c842cae07277b@mail.gmail.com> <20080829231522.201a591b.stas@FreeBSD.org> <be2f52430808300037y79723a2ascf62c33aec7579f3@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
------=_Part_7236_33491345.1220262808340 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline >> On Fri, 29 Aug 2008 18:20:16 +0200 >> "Jacques Fourie" <jacques.fourie@gmail.com> mentioned: >> >>> Hi, >>> >>> I've written a driver for the MMC controller found on the Intel Xscale >>> PXA255 (as found on the Gumstix Connex). It seems to work OK - I've >>> tested with a range of SD cards. The driver works in PIO mode (still >>> busy to debug some DMA issues) and on my Gumstix Connex I get around >>> 400kB/s. If anyone wants to review the code for inclusion let me know >>> and I'll be happy to provide it. >>> >> >> Great work! >> Upload it somewhere or post here, so we can take a look. >> >> Thanks! >> -- >> Stanislav Sedov >> ST4096-RIPE >> > Hi, > > I previously sent Warner some mods but forgot to cc the list. Here is > a diff against current (svn revision 182470). I also made some minor > mods to the mmc stack that I'll post as soon as I've cleaned them up - > they contain a lot of extra printf's at the moment. > > Jacques > Here is a diff against mmc.c that contains my mods to the mmc stack. Very minor - I've added support for retrying commands. Some SD cards that I've tested with requires one extra ACMD_SD_SEND_OP_COND so I've included that as well. Jacques ------=_Part_7236_33491345.1220262808340 Content-Type: text/plain; name=mmc_diff.txt Content-Transfer-Encoding: base64 X-Attachment-Id: f_fkkx0tq11 Content-Disposition: attachment; filename=mmc_diff.txt LS0tIGZic2RfY3VycmVudF8yMDA4MDgzMC9zcmMvc3lzL2Rldi9tbWMvbW1jLmMJMjAwOC0wOC0z MCAwODo0NToxMS4wMDAwMDAwMDAgKzAyMDAKKysrIGZic2RfamZfcHJpdi9zcmMvc3lzL2Rldi9t bWMvbW1jLmMJMjAwOC0wOS0wMSAxMTozNTozOS4wMDAwMDAwMDAgKzAyMDAKQEAgLTUxLDcgKzUx LDcgQEAKICAqLwogCiAjaW5jbHVkZSA8c3lzL2NkZWZzLmg+Ci1fX0ZCU0RJRCgiJEZyZWVCU0Q6 IGhlYWQvc3lzL2Rldi9tbWMvbW1jLmMgMTcwMzM3IDIwMDctMDYtMDUgMTc6MDQ6NDRaIGltcCAk Iik7CitfX0ZCU0RJRCgiJEZyZWVCU0QkIik7CiAKICNpbmNsdWRlIDxzeXMvcGFyYW0uaD4KICNp bmNsdWRlIDxzeXMvc3lzdG0uaD4KQEAgLTI1OCwxMyArMjU4LDIyIEBACiBtbWNfd2FrZXVwKHN0 cnVjdCBtbWNfcmVxdWVzdCAqcmVxKQogewogCXN0cnVjdCBtbWNfc29mdGMgKnNjOworCXN0cnVj dCBtbWNfY29tbWFuZCAqY21kID0gcmVxLT5jbWQ7CiAKIC8vCXByaW50ZigiV2FrZXVwIGZvciBy ZXEgJXAgZG9uZV9kYXRhICVwXG4iLCByZXEsIHJlcS0+ZG9uZV9kYXRhKTsKIAlzYyA9IChzdHJ1 Y3QgbW1jX3NvZnRjICopcmVxLT5kb25lX2RhdGE7CiAJTU1DX0xPQ0soc2MpOwotCXJlcS0+Zmxh Z3MgfD0gTU1DX1JFUV9ET05FOwotCXdha2V1cChyZXEpOwotCU1NQ19VTkxPQ0soc2MpOworCWlm IChjbWQtPmVycm9yICYmIGNtZC0+cmV0cmllcykgeworCQljbWQtPnJldHJpZXMtLTsKKwkJY21k LT5lcnJvciA9IDA7CisJCU1NQ19VTkxPQ0soc2MpOworCQlNTUNCUl9SRVFVRVNUKGRldmljZV9n ZXRfcGFyZW50KHNjLT5kZXYpLCBzYy0+ZGV2LCByZXEpOworCX0KKwllbHNlIHsKKwkJcmVxLT5m bGFncyB8PSBNTUNfUkVRX0RPTkU7CisJCXdha2V1cChyZXEpOworCQlNTUNfVU5MT0NLKHNjKTsK Kwl9CiB9CiAKIHN0YXRpYyBpbnQKQEAgLTU1Niw3ICs1NjUsNyBAQAogCWNtZC5vcGNvZGUgPSBN TUNfQUxMX1NFTkRfQ0lEOwogCWNtZC5hcmcgPSAwOwogCWNtZC5mbGFncyA9IE1NQ19SU1BfUjIg fCBNTUNfQ01EX0JDUjsKLQllcnIgPSBtbWNfd2FpdF9mb3JfY21kKHNjLCAmY21kLCAwKTsKKwll cnIgPSBtbWNfd2FpdF9mb3JfY21kKHNjLCAmY21kLCBDTURfUkVUUklFUyk7CiAJbWVtY3B5KHJh d2NpZCwgY21kLnJlc3AsIDQgKiBzaXplb2YodWludDMyX3QpKTsKIAlyZXR1cm4gKGVycik7CiB9 CkBAIC01NzAsNyArNTc5LDcgQEAKIAljbWQub3Bjb2RlID0gTU1DX1NFTkRfQ1NEOwogCWNtZC5h cmcgPSByY2EgPDwgMTY7CiAJY21kLmZsYWdzID0gTU1DX1JTUF9SMiB8IE1NQ19DTURfQkNSOwot CWVyciA9IG1tY193YWl0X2Zvcl9jbWQoc2MsICZjbWQsIDApOworCWVyciA9IG1tY193YWl0X2Zv cl9jbWQoc2MsICZjbWQsIENNRF9SRVRSSUVTKTsKIAltZW1jcHkocmF3Y2lkLCBjbWQucmVzcCwg NCAqIHNpemVvZih1aW50MzJfdCkpOwogCXJldHVybiAoZXJyKTsKIH0KQEAgLTU4NCw3ICs1OTMs NyBAQAogCWNtZC5vcGNvZGUgPSBTRF9TRU5EX1JFTEFUSVZFX0FERFI7CiAJY21kLmFyZyA9IDA7 CiAJY21kLmZsYWdzID0gTU1DX1JTUF9SNiB8IE1NQ19DTURfQkNSOwotCWVyciA9IG1tY193YWl0 X2Zvcl9jbWQoc2MsICZjbWQsIDApOworCWVyciA9IG1tY193YWl0X2Zvcl9jbWQoc2MsICZjbWQs IENNRF9SRVRSSUVTKTsKIAkqcmVzcCA9IGNtZC5yZXNwWzBdOwogCXJldHVybiAoZXJyKTsKIH0K QEAgLTY2NCwxMiArNjczLDE2IEBACiAJLyoKIAkgKiBSZXNlbGVjdCB0aGUgY2FyZHMgYWZ0ZXIg d2UndmUgaWRsZWQgdGhlbSBhYm92ZS4KIAkgKi8KLQlpZiAobW1jYnJfZ2V0X21vZGUoZGV2KSA9 PSBtb2RlX3NkKQotCQltbWNfc2VuZF9hcHBfb3BfY29uZChzYywgbW1jYnJfZ2V0X29jcihkZXYp LCBOVUxMKTsKKwlpZiAobW1jYnJfZ2V0X21vZGUoZGV2KSA9PSBtb2RlX3NkKSB7CisJCWludCBp OworCisJCWZvciAoaSA9IDA7IGkgPCAyOyBpKyspCisJCQltbWNfc2VuZF9hcHBfb3BfY29uZChz YywgbW1jYnJfZ2V0X29jcihkZXYpLCBOVUxMKTsKKwl9CiAJZWxzZQogCQltbWNfc2VuZF9vcF9j b25kKHNjLCBtbWNicl9nZXRfb2NyKGRldiksIE5VTEwpOwogCW1tY19kaXNjb3Zlcl9jYXJkcyhz Yyk7Ci0KKwkKIAltbWNicl9zZXRfYnVzX21vZGUoZGV2LCBwdXNocHVsbCk7CiAJbW1jYnJfdXBk YXRlX2lvcyhkZXYpOwogCWJ1c19nZW5lcmljX2F0dGFjaChkZXYpOwpAQCAtNzkwLDQgKzgwMyw1 IEBACiAKIAogRFJJVkVSX01PRFVMRShtbWMsIGF0OTFfbWNpLCBtbWNfZHJpdmVyLCBtbWNfZGV2 Y2xhc3MsIDAsIDApOworRFJJVkVSX01PRFVMRShtbWMsIHB4YV9tY2ksIG1tY19kcml2ZXIsIG1t Y19kZXZjbGFzcywgMCwgMCk7CiBEUklWRVJfTU9EVUxFKG1tYywgc2RoLCBtbWNfZHJpdmVyLCBt bWNfZGV2Y2xhc3MsIDAsIDApOwo= ------=_Part_7236_33491345.1220262808340--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?be2f52430809010253wbd5054fn783a3b420296406>