diff mbox

SD CRC error on mx27pdk

Message ID CAOMZO5AZ4gJZMo-DZRPffaF+NdzmzMf32YwPu2gEp9AH8gDk4Q@mail.gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Fabio Estevam April 23, 2014, 3:10 p.m. UTC 
Hi,

I am trying to add sd card support on mx27pdk with the following patch
against Shawn's for-next:

--

,but I get this error:

mxc-mmc 10013000.sdhci: mxcmci_finish_data: -EILSEQ
mmc0: error -84 whilst initialising SD card
mmc0: host does not support reading read-only switch. assuming write-enable.
mmc0: new SDHC card at address b5a1
mmcblk0: mmc0:b5a1 SD04G 3.69 GiB
 mmcblk0: unknown partition table

,which comes from STATUS_CRC_READ_ERR being set.

Any suggestions?

Thanks,

Fabio Estevam
--
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

Comments

Fabio Estevam April 23, 2014, 3:21 p.m. UTC | #1 
On Wed, Apr 23, 2014 at 12:10 PM, Fabio Estevam <festevam@gmail.com> wrote:

> ,but I get this error:
>
> mxc-mmc 10013000.sdhci: mxcmci_finish_data: -EILSEQ
> mmc0: error -84 whilst initialising SD card
> mmc0: host does not support reading read-only switch. assuming write-enable.
> mmc0: new SDHC card at address b5a1
> mmcblk0: mmc0:b5a1 SD04G 3.69 GiB
>  mmcblk0: unknown partition table

I forgot to mention that this error does not happen if I boot the
non-dt kernel, so it seems to be related to dt only.
--
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
Alexander Shiyan April 23, 2014, 6:42 p.m. UTC | #2 
V2VkLCAyMyBBcHIgMjAxNCAxMjoyMTozOSAtMDMwMCDQvtGCIEZhYmlvIEVzdGV2YW0gPGZlc3Rl
dmFtQGdtYWlsLmNvbT46Cj4gT24gV2VkLCBBcHIgMjMsIDIwMTQgYXQgMTI6MTAgUE0sIEZhYmlv
IEVzdGV2YW0gPGZlc3RldmFtQGdtYWlsLmNvbT4gd3JvdGU6Cj4gCj4gPiAsYnV0IEkgZ2V0IHRo
aXMgZXJyb3I6Cj4gPgo+ID4gbXhjLW1tYyAxMDAxMzAwMC5zZGhjaTogbXhjbWNpX2ZpbmlzaF9k
YXRhOiAtRUlMU0VRCj4gPiBtbWMwOiBlcnJvciAtODQgd2hpbHN0IGluaXRpYWxpc2luZyBTRCBj
YXJkCj4gPiBtbWMwOiBob3N0IGRvZXMgbm90IHN1cHBvcnQgcmVhZGluZyByZWFkLW9ubHkgc3dp
dGNoLiBhc3N1bWluZyB3cml0ZS1lbmFibGUuCj4gPiBtbWMwOiBuZXcgU0RIQyBjYXJkIGF0IGFk
ZHJlc3MgYjVhMQo+ID4gbW1jYmxrMDogbW1jMDpiNWExIFNEMDRHIDMuNjkgR2lCCj4gPiAgbW1j
YmxrMDogdW5rbm93biBwYXJ0aXRpb24gdGFibGUKPiAKPiBJIGZvcmdvdCB0byBtZW50aW9uIHRo
YXQgdGhpcyBlcnJvciBkb2VzIG5vdCBoYXBwZW4gaWYgSSBib290IHRoZQo+IG5vbi1kdCBrZXJu
ZWwsIHNvIGl0IHNlZW1zIHRvIGJlIHJlbGF0ZWQgdG8gZHQgb25seS4KClByb2JhYmx5IHlvdSBz
aG91bGQgYWxzbyB1c2Ugdm1tYypfcmVnIHZvbHRhZ2VzIGZyb20gUE1JQz8KCi0tLQoK
--
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
Fabio Estevam April 23, 2014, 6:47 p.m. UTC | #3 
Hi Alexander,

On Wed, Apr 23, 2014 at 3:42 PM, Alexander Shiyan <shc_work@mail.ru> wrote:
> Wed, 23 Apr 2014 12:21:39 -0300 ?? Fabio Estevam <festevam@gmail.com>:
>> On Wed, Apr 23, 2014 at 12:10 PM, Fabio Estevam <festevam@gmail.com> wrote:
>>
>> > ,but I get this error:
>> >
>> > mxc-mmc 10013000.sdhci: mxcmci_finish_data: -EILSEQ
>> > mmc0: error -84 whilst initialising SD card
>> > mmc0: host does not support reading read-only switch. assuming write-enable.
>> > mmc0: new SDHC card at address b5a1
>> > mmcblk0: mmc0:b5a1 SD04G 3.69 GiB
>> >  mmcblk0: unknown partition table
>>
>> I forgot to mention that this error does not happen if I boot the
>> non-dt kernel, so it seems to be related to dt only.
>
> Probably you should also use vmmc*_reg voltages from PMIC?

On mx27pdk board the vmmc1_reg powers up the CMOS camera instead of mmc supply.

Thanks
--
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
diff mbox

Patch

diff --git a/arch/arm/boot/dts/imx27-pdk.dts b/arch/arm/boot/dts/imx27-pdk.dts
index 0dbb16a..749aa72 100644
--- a/arch/arm/boot/dts/imx27-pdk.dts
+++ b/arch/arm/boot/dts/imx27-pdk.dts
@@ -20,6 +20,22 @@ 
         reg = <0xa0000000 0x08000000>;
     };

+    regulators {
+        compatible = "simple-bus";
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        reg_mmc: regulator@0 {
+            compatible = "regulator-fixed";
+            reg = <0>;
+            regulator-name = "3v3";
+            regulator-min-microvolt = <3300000>;
+            regulator-max-microvolt = <3300000>;
+            enable-active-high;
+            gpio = <&gpio2 25 0>;
+        };
+    };
+
     usbphy {
         compatible = "simple-bus";
         #address-cells = <1>;
@@ -92,6 +108,14 @@ 
     nand-on-flash-bbt;
 };

+&sdhci1 {
+    pinctrl-names = "default";
+    pinctrl-0 = <&pinctrl_sdhc1>;
+    cd-gpios = <&gpio2 26 GPIO_ACTIVE_HIGH>;
+    vmmc-supply = <&reg_mmc>;
+    status = "okay";
+};
+
 &uart1 {
     fsl,uart-has-rtscts;
     pinctrl-names = "default";
@@ -148,6 +172,19 @@ 
             >;
         };

+        pinctrl_sdhc1: sdhc1grp {
+            fsl,pins = <
+                MX27_PAD_SD1_D0__SD1_D0        0x0
+                MX27_PAD_SD1_D1__SD1_D1        0x0
+                MX27_PAD_SD1_D2__SD1_D2        0x0
+                MX27_PAD_SD1_D3__SD1_D3        0x0
+                MX27_PAD_SD1_CMD__SD1_CMD    0x0
+                MX27_PAD_SD1_CLK__SD1_CLK    0x0
+                MX27_PAD_USBH1_RCV__GPIO2_25    0x0
+                MX27_PAD_USBH1_FS__GPIO2_26    0x0
+            >;
+        };
+
         pinctrl_uart1: uart1grp {
             fsl,pins = <
                 MX27_PAD_UART1_TXD__UART1_TXD 0x0