linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/1] ARM: dts: sunxi: Add a olinuxino-lime2-emmc
@ 2016-04-28  7:19 Olliver Schinagl
  2016-05-02 10:55 ` Maxime Ripard
                   ` (2 more replies)
  0 siblings, 3 replies; 14+ messages in thread
From: Olliver Schinagl @ 2016-04-28  7:19 UTC (permalink / raw)
  To: Maxime Ripard, Chen-Yu Tsai, Tsvetan Usunov, Rob Herring,
	Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala, Russell King,
	Hans de Goede
  Cc: dev, devicetree, linux-arm-kernel, linux-kernel, Olliver Schinagl

There are 3 kinds of OLinuXino Lime2 boards.
One without any on board storage, one with NAND storage and one with
eMMC storage. This patch adds the eMMC variant of boards.

eMMC storage is different from a regular SD card in that it is soldered
on the board and cannot be changed. Additionally, it shares pins with
the NAND module and with the second SPI port.

Signed-off-by: Olliver Schinagl <oliver@schinagl.nl>
---
 .../boot/dts/sun7i-a20-olinuxino-lime2-emmc.dts    | 64 ++++++++++++++++++++++
 1 file changed, 64 insertions(+)
 create mode 100644 arch/arm/boot/dts/sun7i-a20-olinuxino-lime2-emmc.dts

diff --git a/arch/arm/boot/dts/sun7i-a20-olinuxino-lime2-emmc.dts b/arch/arm/boot/dts/sun7i-a20-olinuxino-lime2-emmc.dts
new file mode 100644
index 0000000..689da36
--- /dev/null
+++ b/arch/arm/boot/dts/sun7i-a20-olinuxino-lime2-emmc.dts
@@ -0,0 +1,64 @@
+ /*
+ * Copyright 2015 - Ultimaker B.V.
+ * Author Olliver Schinagl <oliver@schinagl.nl>
+ *
+ * This file is dual-licensed: you can use it either under the terms
+ * of the GPL or the X11 license, at your option. Note that this dual
+ * licensing only applies to this file, and not this project as a
+ * whole.
+ *
+ *  a) This file is free software; you can redistribute it and/or
+ *     modify it under the terms of the GNU General Public License as
+ *     published by the Free Software Foundation; either version 2 of the
+ *     License, or (at your option) any later version.
+ *
+ *     This file is distributed in the hope that it will be useful,
+ *     but WITHOUT ANY WARRANTY; without even the implied warranty of
+ *     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ *     GNU General Public License for more details.
+ *
+ * Or, alternatively,
+ *
+ *  b) Permission is hereby granted, free of charge, to any person
+ *     obtaining a copy of this software and associated documentation
+ *     files (the "Software"), to deal in the Software without
+ *     restriction, including without limitation the rights to use,
+ *     copy, modify, merge, publish, distribute, sublicense, and/or
+ *     sell copies of the Software, and to permit persons to whom the
+ *     Software is furnished to do so, subject to the following
+ *     conditions:
+ *
+ *     The above copyright notice and this permission notice shall be
+ *     included in all copies or substantial portions of the Software.
+ *
+ *     THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+ *     EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
+ *     OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+ *     NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
+ *     HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
+ *     WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ *     FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
+ *     OTHER DEALINGS IN THE SOFTWARE.
+ */
+
+#include "sun7i-a20-olinuxino-lime2.dts"
+
+/ {
+	model = "Olimex A20-OLinuXino-LIME2-eMMC";
+};
+
+&mmc2 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&mmc2_pins_a>;
+	vmmc-supply = <&reg_vcc3v3>;
+	bus-width = <4>;
+	non-removable;
+	no-1-8-v;
+	status = "okay";
+
+	emmc: emmc@0 {
+		reg = <0>;
+		compatible = "mmc-card";
+		broken-hpi;
+	};
+};
-- 
2.8.0.rc3

^ permalink raw reply related	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/1] ARM: dts: sunxi: Add a olinuxino-lime2-emmc
  2016-04-28  7:19 [PATCH 1/1] ARM: dts: sunxi: Add a olinuxino-lime2-emmc Olliver Schinagl
@ 2016-05-02 10:55 ` Maxime Ripard
  2016-05-03  3:33 ` Chen-Yu Tsai
  2016-05-03  7:12 ` christo.radev
  2 siblings, 0 replies; 14+ messages in thread
From: Maxime Ripard @ 2016-05-02 10:55 UTC (permalink / raw)
  To: Olliver Schinagl
  Cc: Chen-Yu Tsai, Tsvetan Usunov, Rob Herring, Pawel Moll,
	Mark Rutland, Ian Campbell, Kumar Gala, Russell King,
	Hans de Goede, dev, devicetree, linux-arm-kernel, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 720 bytes --]

Hi,

On Thu, Apr 28, 2016 at 09:19:58AM +0200, Olliver Schinagl wrote:
> There are 3 kinds of OLinuXino Lime2 boards.
> One without any on board storage, one with NAND storage and one with
> eMMC storage. This patch adds the eMMC variant of boards.
> 
> eMMC storage is different from a regular SD card in that it is soldered
> on the board and cannot be changed. Additionally, it shares pins with
> the NAND module and with the second SPI port.
> 
> Signed-off-by: Olliver Schinagl <oliver@schinagl.nl>

Is it a publicly available board, or is it a private hack you made
yourself?

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/1] ARM: dts: sunxi: Add a olinuxino-lime2-emmc
  2016-04-28  7:19 [PATCH 1/1] ARM: dts: sunxi: Add a olinuxino-lime2-emmc Olliver Schinagl
  2016-05-02 10:55 ` Maxime Ripard
@ 2016-05-03  3:33 ` Chen-Yu Tsai
  2016-05-03  7:21   ` Olliver Schinagl
  2016-05-03  7:12 ` christo.radev
  2 siblings, 1 reply; 14+ messages in thread
From: Chen-Yu Tsai @ 2016-05-03  3:33 UTC (permalink / raw)
  To: Olliver Schinagl
  Cc: Maxime Ripard, Chen-Yu Tsai, Tsvetan Usunov, Rob Herring,
	Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala, Russell King,
	Hans de Goede, dev, devicetree, linux-arm-kernel, linux-kernel

Hi,

On Thu, Apr 28, 2016 at 3:19 PM, Olliver Schinagl <oliver@schinagl.nl> wrote:
> There are 3 kinds of OLinuXino Lime2 boards.
> One without any on board storage, one with NAND storage and one with
> eMMC storage. This patch adds the eMMC variant of boards.
>
> eMMC storage is different from a regular SD card in that it is soldered
> on the board and cannot be changed. Additionally, it shares pins with
> the NAND module and with the second SPI port.
>
> Signed-off-by: Olliver Schinagl <oliver@schinagl.nl>
> ---
>  .../boot/dts/sun7i-a20-olinuxino-lime2-emmc.dts    | 64 ++++++++++++++++++++++
>  1 file changed, 64 insertions(+)
>  create mode 100644 arch/arm/boot/dts/sun7i-a20-olinuxino-lime2-emmc.dts
>
> diff --git a/arch/arm/boot/dts/sun7i-a20-olinuxino-lime2-emmc.dts b/arch/arm/boot/dts/sun7i-a20-olinuxino-lime2-emmc.dts
> new file mode 100644
> index 0000000..689da36
> --- /dev/null
> +++ b/arch/arm/boot/dts/sun7i-a20-olinuxino-lime2-emmc.dts
> @@ -0,0 +1,64 @@
> + /*
> + * Copyright 2015 - Ultimaker B.V.
> + * Author Olliver Schinagl <oliver@schinagl.nl>
> + *
> + * This file is dual-licensed: you can use it either under the terms
> + * of the GPL or the X11 license, at your option. Note that this dual
> + * licensing only applies to this file, and not this project as a
> + * whole.
> + *
> + *  a) This file is free software; you can redistribute it and/or
> + *     modify it under the terms of the GNU General Public License as
> + *     published by the Free Software Foundation; either version 2 of the
> + *     License, or (at your option) any later version.
> + *
> + *     This file is distributed in the hope that it will be useful,
> + *     but WITHOUT ANY WARRANTY; without even the implied warranty of
> + *     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> + *     GNU General Public License for more details.
> + *
> + * Or, alternatively,
> + *
> + *  b) Permission is hereby granted, free of charge, to any person
> + *     obtaining a copy of this software and associated documentation
> + *     files (the "Software"), to deal in the Software without
> + *     restriction, including without limitation the rights to use,
> + *     copy, modify, merge, publish, distribute, sublicense, and/or
> + *     sell copies of the Software, and to permit persons to whom the
> + *     Software is furnished to do so, subject to the following
> + *     conditions:
> + *
> + *     The above copyright notice and this permission notice shall be
> + *     included in all copies or substantial portions of the Software.
> + *
> + *     THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
> + *     EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
> + *     OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
> + *     NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
> + *     HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
> + *     WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> + *     FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
> + *     OTHER DEALINGS IN THE SOFTWARE.
> + */
> +
> +#include "sun7i-a20-olinuxino-lime2.dts"
> +
> +/ {
> +       model = "Olimex A20-OLinuXino-LIME2-eMMC";
> +};
> +
> +&mmc2 {
> +       pinctrl-names = "default";
> +       pinctrl-0 = <&mmc2_pins_a>;
> +       vmmc-supply = <&reg_vcc3v3>;
> +       bus-width = <4>;

Only 4 bits? We normally see eMMC with 8 bits. 4 bits are some kind of
embedded SD card.

> +       non-removable;
> +       no-1-8-v;

This flag is not supported on sunxi.

Instead, use the vqmmc-supply with the correct regulator and constraints.

ChenYu

> +       status = "okay";
> +
> +       emmc: emmc@0 {
> +               reg = <0>;
> +               compatible = "mmc-card";
> +               broken-hpi;
> +       };
> +};
> --
> 2.8.0.rc3
>

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/1] ARM: dts: sunxi: Add a olinuxino-lime2-emmc
  2016-04-28  7:19 [PATCH 1/1] ARM: dts: sunxi: Add a olinuxino-lime2-emmc Olliver Schinagl
  2016-05-02 10:55 ` Maxime Ripard
  2016-05-03  3:33 ` Chen-Yu Tsai
@ 2016-05-03  7:12 ` christo.radev
  2016-05-03 13:14   ` Maxime Ripard
  2 siblings, 1 reply; 14+ messages in thread
From: christo.radev @ 2016-05-03  7:12 UTC (permalink / raw)
  To: linux-sunxi
  Cc: maxime.ripard, wens, tsvetan, robh+dt, pawel.moll, mark.rutland,
	ijc+devicetree, galak, linux, hdegoede, dev, devicetree,
	linux-arm-kernel, linux-kernel, oliver

[-- Attachment #1: Type: text/plain, Size: 515 bytes --]

Hi to All,

I have already solved and tested this issue on Armbian build.
Find patches for both legacy (3.4.111) and mainline (4.5.2) kernels on:
http://forum.armbian.com/index.php/topic/853-armbian-customization/page-2#entry7494
There it is also described how to do eMMC bootable and much more.

About the board - Olimex already sold all 3 kinds after migration to their HW rev. E. One have to specify Lime2-eMMC as A20-Olinuxino-Lime2-eMMC instead of their old 2 options A20-Olinuxino-Lime2(-4GB).

Regards
Chris

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/1] ARM: dts: sunxi: Add a olinuxino-lime2-emmc
  2016-05-03  3:33 ` Chen-Yu Tsai
@ 2016-05-03  7:21   ` Olliver Schinagl
  2016-05-03  7:25     ` Chen-Yu Tsai
  0 siblings, 1 reply; 14+ messages in thread
From: Olliver Schinagl @ 2016-05-03  7:21 UTC (permalink / raw)
  To: Chen-Yu Tsai
  Cc: Maxime Ripard, Tsvetan Usunov, Rob Herring, Pawel Moll,
	Mark Rutland, Ian Campbell, Kumar Gala, Russell King,
	Hans de Goede, dev, devicetree, linux-arm-kernel, linux-kernel

Hey Chen,

On 03-05-16 05:33, Chen-Yu Tsai wrote:
> Hi,
>
> On Thu, Apr 28, 2016 at 3:19 PM, Olliver Schinagl <oliver@schinagl.nl> wrote:
>> There are 3 kinds of OLinuXino Lime2 boards.
>> One without any on board storage, one with NAND storage and one with
>> eMMC storage. This patch adds the eMMC variant of boards.
>>
>> eMMC storage is different from a regular SD card in that it is soldered
>> on the board and cannot be changed. Additionally, it shares pins with
>> the NAND module and with the second SPI port.
>>
>> Signed-off-by: Olliver Schinagl <oliver@schinagl.nl>
>> ---
>>   .../boot/dts/sun7i-a20-olinuxino-lime2-emmc.dts    | 64 ++++++++++++++++++++++
>>   1 file changed, 64 insertions(+)
>>   create mode 100644 arch/arm/boot/dts/sun7i-a20-olinuxino-lime2-emmc.dts
>>
>> diff --git a/arch/arm/boot/dts/sun7i-a20-olinuxino-lime2-emmc.dts b/arch/arm/boot/dts/sun7i-a20-olinuxino-lime2-emmc.dts
>> new file mode 100644
>> index 0000000..689da36
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/sun7i-a20-olinuxino-lime2-emmc.dts
>> @@ -0,0 +1,64 @@
>> + /*
>> + * Copyright 2015 - Ultimaker B.V.
>> + * Author Olliver Schinagl <oliver@schinagl.nl>
>> + *
>> + * This file is dual-licensed: you can use it either under the terms
>> + * of the GPL or the X11 license, at your option. Note that this dual
>> + * licensing only applies to this file, and not this project as a
>> + * whole.
>> + *
>> + *  a) This file is free software; you can redistribute it and/or
>> + *     modify it under the terms of the GNU General Public License as
>> + *     published by the Free Software Foundation; either version 2 of the
>> + *     License, or (at your option) any later version.
>> + *
>> + *     This file is distributed in the hope that it will be useful,
>> + *     but WITHOUT ANY WARRANTY; without even the implied warranty of
>> + *     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
>> + *     GNU General Public License for more details.
>> + *
>> + * Or, alternatively,
>> + *
>> + *  b) Permission is hereby granted, free of charge, to any person
>> + *     obtaining a copy of this software and associated documentation
>> + *     files (the "Software"), to deal in the Software without
>> + *     restriction, including without limitation the rights to use,
>> + *     copy, modify, merge, publish, distribute, sublicense, and/or
>> + *     sell copies of the Software, and to permit persons to whom the
>> + *     Software is furnished to do so, subject to the following
>> + *     conditions:
>> + *
>> + *     The above copyright notice and this permission notice shall be
>> + *     included in all copies or substantial portions of the Software.
>> + *
>> + *     THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
>> + *     EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
>> + *     OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
>> + *     NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
>> + *     HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
>> + *     WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
>> + *     FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
>> + *     OTHER DEALINGS IN THE SOFTWARE.
>> + */
>> +
>> +#include "sun7i-a20-olinuxino-lime2.dts"
>> +
>> +/ {
>> +       model = "Olimex A20-OLinuXino-LIME2-eMMC";
>> +};
>> +
>> +&mmc2 {
>> +       pinctrl-names = "default";
>> +       pinctrl-0 = <&mmc2_pins_a>;
>> +       vmmc-supply = <&reg_vcc3v3>;
>> +       bus-width = <4>;
> Only 4 bits? We normally see eMMC with 8 bits. 4 bits are some kind of
> embedded SD card.
On A20 as well? Our investigations so far have concluded that the A10 
and A20 have those pins not mapped out to pads. The IP does support it 
however we assume.
>
>> +       non-removable;
>> +       no-1-8-v;
> This flag is not supported on sunxi.
>
> Instead, use the vqmmc-supply with the correct regulator and constraints.
That was not supposed to be there, i'll fix it! Sorry.
>
> ChenYu
>
>> +       status = "okay";
>> +
>> +       emmc: emmc@0 {
>> +               reg = <0>;
>> +               compatible = "mmc-card";
>> +               broken-hpi;
>> +       };
>> +};
>> --
>> 2.8.0.rc3
>>

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/1] ARM: dts: sunxi: Add a olinuxino-lime2-emmc
  2016-05-03  7:21   ` Olliver Schinagl
@ 2016-05-03  7:25     ` Chen-Yu Tsai
  2016-05-04 12:30       ` Radoslav Kolev
  0 siblings, 1 reply; 14+ messages in thread
From: Chen-Yu Tsai @ 2016-05-03  7:25 UTC (permalink / raw)
  To: Olliver Schinagl
  Cc: Chen-Yu Tsai, Maxime Ripard, Tsvetan Usunov, Rob Herring,
	Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala, Russell King,
	Hans de Goede, dev, devicetree, linux-arm-kernel, linux-kernel

On Tue, May 3, 2016 at 3:21 PM, Olliver Schinagl <oliver@schinagl.nl> wrote:
> Hey Chen,

ChenYu :)

>
>
> On 03-05-16 05:33, Chen-Yu Tsai wrote:
>>
>> Hi,
>>
>> On Thu, Apr 28, 2016 at 3:19 PM, Olliver Schinagl <oliver@schinagl.nl>
>> wrote:
>>>
>>> There are 3 kinds of OLinuXino Lime2 boards.
>>> One without any on board storage, one with NAND storage and one with
>>> eMMC storage. This patch adds the eMMC variant of boards.
>>>
>>> eMMC storage is different from a regular SD card in that it is soldered
>>> on the board and cannot be changed. Additionally, it shares pins with
>>> the NAND module and with the second SPI port.
>>>
>>> Signed-off-by: Olliver Schinagl <oliver@schinagl.nl>
>>> ---
>>>   .../boot/dts/sun7i-a20-olinuxino-lime2-emmc.dts    | 64
>>> ++++++++++++++++++++++
>>>   1 file changed, 64 insertions(+)
>>>   create mode 100644 arch/arm/boot/dts/sun7i-a20-olinuxino-lime2-emmc.dts
>>>
>>> diff --git a/arch/arm/boot/dts/sun7i-a20-olinuxino-lime2-emmc.dts
>>> b/arch/arm/boot/dts/sun7i-a20-olinuxino-lime2-emmc.dts
>>> new file mode 100644
>>> index 0000000..689da36
>>> --- /dev/null
>>> +++ b/arch/arm/boot/dts/sun7i-a20-olinuxino-lime2-emmc.dts
>>> @@ -0,0 +1,64 @@
>>> + /*
>>> + * Copyright 2015 - Ultimaker B.V.
>>> + * Author Olliver Schinagl <oliver@schinagl.nl>
>>> + *
>>> + * This file is dual-licensed: you can use it either under the terms
>>> + * of the GPL or the X11 license, at your option. Note that this dual
>>> + * licensing only applies to this file, and not this project as a
>>> + * whole.
>>> + *
>>> + *  a) This file is free software; you can redistribute it and/or
>>> + *     modify it under the terms of the GNU General Public License as
>>> + *     published by the Free Software Foundation; either version 2 of
>>> the
>>> + *     License, or (at your option) any later version.
>>> + *
>>> + *     This file is distributed in the hope that it will be useful,
>>> + *     but WITHOUT ANY WARRANTY; without even the implied warranty of
>>> + *     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
>>> + *     GNU General Public License for more details.
>>> + *
>>> + * Or, alternatively,
>>> + *
>>> + *  b) Permission is hereby granted, free of charge, to any person
>>> + *     obtaining a copy of this software and associated documentation
>>> + *     files (the "Software"), to deal in the Software without
>>> + *     restriction, including without limitation the rights to use,
>>> + *     copy, modify, merge, publish, distribute, sublicense, and/or
>>> + *     sell copies of the Software, and to permit persons to whom the
>>> + *     Software is furnished to do so, subject to the following
>>> + *     conditions:
>>> + *
>>> + *     The above copyright notice and this permission notice shall be
>>> + *     included in all copies or substantial portions of the Software.
>>> + *
>>> + *     THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
>>> + *     EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
>>> + *     OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
>>> + *     NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
>>> + *     HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
>>> + *     WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
>>> + *     FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
>>> + *     OTHER DEALINGS IN THE SOFTWARE.
>>> + */
>>> +
>>> +#include "sun7i-a20-olinuxino-lime2.dts"
>>> +
>>> +/ {
>>> +       model = "Olimex A20-OLinuXino-LIME2-eMMC";
>>> +};
>>> +
>>> +&mmc2 {
>>> +       pinctrl-names = "default";
>>> +       pinctrl-0 = <&mmc2_pins_a>;
>>> +       vmmc-supply = <&reg_vcc3v3>;
>>> +       bus-width = <4>;
>>
>> Only 4 bits? We normally see eMMC with 8 bits. 4 bits are some kind of
>> embedded SD card.
>
> On A20 as well? Our investigations so far have concluded that the A10 and
> A20 have those pins not mapped out to pads. The IP does support it however
> we assume.

You're right. My bad. First time A10/A20 sees eMMC support.

>>
>>
>>> +       non-removable;
>>> +       no-1-8-v;
>>
>> This flag is not supported on sunxi.
>>
>> Instead, use the vqmmc-supply with the correct regulator and constraints.
>
> That was not supposed to be there, i'll fix it! Sorry.

Thanks
ChenYu

>
>>
>> ChenYu
>>
>>> +       status = "okay";
>>> +
>>> +       emmc: emmc@0 {
>>> +               reg = <0>;
>>> +               compatible = "mmc-card";
>>> +               broken-hpi;
>>> +       };
>>> +};
>>> --
>>> 2.8.0.rc3
>>>
>

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/1] ARM: dts: sunxi: Add a olinuxino-lime2-emmc
  2016-05-03  7:12 ` christo.radev
@ 2016-05-03 13:14   ` Maxime Ripard
  2016-05-03 15:02     ` christo.radev
  0 siblings, 1 reply; 14+ messages in thread
From: Maxime Ripard @ 2016-05-03 13:14 UTC (permalink / raw)
  To: christo.radev
  Cc: linux-sunxi, wens, tsvetan, robh+dt, pawel.moll, mark.rutland,
	ijc+devicetree, galak, linux, hdegoede, dev, devicetree,
	linux-arm-kernel, linux-kernel, oliver

[-- Attachment #1: Type: text/plain, Size: 795 bytes --]

Hi,

On Tue, May 03, 2016 at 12:12:06AM -0700, christo.radev@gmail.com wrote:
> Hi to All,
> 
> I have already solved and tested this issue on Armbian build.  Find
> patches for both legacy (3.4.111) and mainline (4.5.2) kernels on:
> http://forum.armbian.com/index.php/topic/853-armbian-customization/page-2#entry7494
> There it is also described how to do eMMC bootable and much more.
> 
> About the board - Olimex already sold all 3 kinds after migration to
> their HW rev. E. One have to specify Lime2-eMMC as
> A20-Olinuxino-Lime2-eMMC instead of their old 2 options
> A20-Olinuxino-Lime2(-4GB).

Interesting, you have a link to that device?

Thanks,
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/1] ARM: dts: sunxi: Add a olinuxino-lime2-emmc
  2016-05-03 13:14   ` Maxime Ripard
@ 2016-05-03 15:02     ` christo.radev
  2016-05-03 15:52       ` Olliver Schinagl
  0 siblings, 1 reply; 14+ messages in thread
From: christo.radev @ 2016-05-03 15:02 UTC (permalink / raw)
  To: linux-sunxi
  Cc: christo.radev, wens, tsvetan, robh+dt, pawel.moll, mark.rutland,
	ijc+devicetree, galak, linux, hdegoede, dev, devicetree,
	linux-arm-kernel, linux-kernel, oliver

[-- Attachment #1: Type: text/plain, Size: 1647 bytes --]

On Tuesday, May 3, 2016 at 4:14:41 PM UTC+3, Maxime Ripard wrote:
> Hi,
> 
> On Tue, May 03, 2016 at 4:12:06 PM UTC+3, Christo Radev wrote:
> > Hi to All,
> > 
> > I have already solved and tested this issue on Armbian build.  Find
> > patches for both legacy (3.4.111) and mainline (4.5.2) kernels on:
> > http://forum.armbian.com/index.php/topic/853-armbian-customization/page-2#entry7494
> > There it is also described how to do eMMC bootable and much more.
> > 
> > About the board - Olimex already sold all 3 kinds after migration to
> > their HW rev. E. One have to specify Lime2-eMMC as
> > A20-Olinuxino-Lime2-eMMC instead of their old 2 options
> > A20-Olinuxino-Lime2(-4GB).
> 
> Interesting, you have a link to that device?
> 
> Thanks,
> Maxime
> 
> -- 
> Maxime Ripard, Free Electrons
> Embedded Linux, Kernel and Android engineering
> http://free-electrons.com

I have really 2 boards delivered by their local distributor.

Unfortunately, they do not update their site. Use the link for NAND option:
https://www.olimex.com/Products/OLinuXino/A20/A20-OLinuXIno-LIME2-4GB/open-source-hardware
There you can find Users Manual where it is described that eMMC option is available from HW rev. D. The schematic for HW rev. E is also available on their repository:
https://github.com/OLIMEX/OLINUXINO/tree/master/HARDWARE/A20-OLinuXino-LIME2

On the board both 4GB NAND and eMMC Flash chips can be placed alternatively on the same place. There is difference in some other components as well.

If one want to order it from the site probably has to order A20-Olinuxino-Lime2-4GB with note that eMMC option is required. The price is the same.

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/1] ARM: dts: sunxi: Add a olinuxino-lime2-emmc
  2016-05-03 15:02     ` christo.radev
@ 2016-05-03 15:52       ` Olliver Schinagl
  2016-05-04  5:13         ` [linux-sunxi] " Priit Laes
  0 siblings, 1 reply; 14+ messages in thread
From: Olliver Schinagl @ 2016-05-03 15:52 UTC (permalink / raw)
  To: christo.radev, linux-sunxi
  Cc: wens, tsvetan, robh+dt, pawel.moll, mark.rutland, ijc+devicetree,
	galak, linux, hdegoede, dev, devicetree, linux-arm-kernel,
	linux-kernel

Hey all,

On 03-05-16 17:02, christo.radev@gmail.com wrote:
> On Tuesday, May 3, 2016 at 4:14:41 PM UTC+3, Maxime Ripard wrote:
>> Hi,
>>
>> On Tue, May 03, 2016 at 4:12:06 PM UTC+3, Christo Radev wrote:
>>> Hi to All,
>>>
>>> I have already solved and tested this issue on Armbian build.  Find
>>> patches for both legacy (3.4.111) and mainline (4.5.2) kernels on:
>>> http://forum.armbian.com/index.php/topic/853-armbian-customization/page-2#entry7494
>>> There it is also described how to do eMMC bootable and much more.
>>>
>>> About the board - Olimex already sold all 3 kinds after migration to
>>> their HW rev. E. One have to specify Lime2-eMMC as
>>> A20-Olinuxino-Lime2-eMMC instead of their old 2 options
>>> A20-Olinuxino-Lime2(-4GB).
>> Interesting, you have a link to that device?
>>
>> Thanks,
>> Maxime
>>
>> -- 
>> Maxime Ripard, Free Electrons
>> Embedded Linux, Kernel and Android engineering
>> http://free-electrons.com
> I have really 2 boards delivered by their local distributor.
>
> Unfortunately, they do not update their site. Use the link for NAND option:
> https://www.olimex.com/Products/OLinuXino/A20/A20-OLinuXIno-LIME2-4GB/open-source-hardware
> There you can find Users Manual where it is described that eMMC option is available from HW rev. D. The schematic for HW rev. E is also available on their repository:
> https://github.com/OLIMEX/OLINUXINO/tree/master/HARDWARE/A20-OLinuXino-LIME2
>
> On the board both 4GB NAND and eMMC Flash chips can be placed alternatively on the same place. There is difference in some other components as well.
>
> If one want to order it from the site probably has to order A20-Olinuxino-Lime2-4GB with note that eMMC option is required. The price is the same.
Sorry for the late reply, but yeah the board exists, we asked Olimex to 
develop the eMMC variant for us. I currently have a dozen or so on my 
desk :)

I don't know when Olimex will update their webshop with the new designs, 
but they simply might not have enough eMMC chips available yet?

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [linux-sunxi] Re: [PATCH 1/1] ARM: dts: sunxi: Add a olinuxino-lime2-emmc
  2016-05-03 15:52       ` Olliver Schinagl
@ 2016-05-04  5:13         ` Priit Laes
  0 siblings, 0 replies; 14+ messages in thread
From: Priit Laes @ 2016-05-04  5:13 UTC (permalink / raw)
  To: oliver, christo.radev, linux-sunxi
  Cc: wens, tsvetan, robh+dt, pawel.moll, mark.rutland, ijc+devicetree,
	galak, linux, hdegoede, dev, devicetree, linux-arm-kernel,
	linux-kernel

On Tue, 2016-05-03 at 17:52 +0200, Olliver Schinagl wrote:
> Hey all,
> 
> On 03-05-16 17:02, christo.radev@gmail.com wrote:
> > On Tuesday, May 3, 2016 at 4:14:41 PM UTC+3, Maxime Ripard wrote:
> > > Hi,
> > > 
> > > On Tue, May 03, 2016 at 4:12:06 PM UTC+3, Christo Radev wrote:
> > > > Hi to All,
> > > > 
> > > > I have already solved and tested this issue on Armbian build. 
> > > >  Find
> > > > patches for both legacy (3.4.111) and mainline (4.5.2) kernels
> > > > on:
> > > > http://forum.armbian.com/index.php/topic/853-armbian-customizat
> > > > ion/page-2#entry7494
> > > > There it is also described how to do eMMC bootable and much
> > > > more.
> > > > 
> > > > About the board - Olimex already sold all 3 kinds after
> > > > migration to
> > > > their HW rev. E. One have to specify Lime2-eMMC as
> > > > A20-Olinuxino-Lime2-eMMC instead of their old 2 options
> > > > A20-Olinuxino-Lime2(-4GB).
> > > Interesting, you have a link to that device?

I guess, it is this one:

https://www.olimex.com/Products/OLinuXino/A20/A20-OLinuXino-LIME2-eMMC/
open-source-hardware

> > > 
> > > Thanks,
> > > Maxime
> > > 
> > > -- 
> > > Maxime Ripard, Free Electrons
> > > Embedded Linux, Kernel and Android engineering
> > > http://free-electrons.com
> > I have really 2 boards delivered by their local distributor.
> > 
> > Unfortunately, they do not update their site. Use the link for NAND
> > option:
> > https://www.olimex.com/Products/OLinuXino/A20/A20-OLinuXIno-LIME2-4
> > GB/open-source-hardware
> > There you can find Users Manual where it is described that eMMC
> > option is available from HW rev. D. The schematic for HW rev. E is
> > also available on their repository:
> > https://github.com/OLIMEX/OLINUXINO/tree/master/HARDWARE/A20-OLinuX
> > ino-LIME2
> > 
> > On the board both 4GB NAND and eMMC Flash chips can be placed
> > alternatively on the same place. There is difference in some other
> > components as well.
> > 
> > If one want to order it from the site probably has to order A20
> > -Olinuxino-Lime2-4GB with note that eMMC option is required. The
> > price is the same.
> Sorry for the late reply, but yeah the board exists, we asked Olimex
> to 
> develop the eMMC variant for us. I currently have a dozen or so on my
> desk :)
> 
> I don't know when Olimex will update their webshop with the new
> designs, 
> but they simply might not have enough eMMC chips available yet?
> 

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/1] ARM: dts: sunxi: Add a olinuxino-lime2-emmc
  2016-05-03  7:25     ` Chen-Yu Tsai
@ 2016-05-04 12:30       ` Radoslav Kolev
  2016-05-04 12:52         ` Olliver Schinagl
  0 siblings, 1 reply; 14+ messages in thread
From: Radoslav Kolev @ 2016-05-04 12:30 UTC (permalink / raw)
  To: Chen-Yu Tsai
  Cc: Olliver Schinagl, Maxime Ripard, Tsvetan Usunov, Rob Herring,
	Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala, Russell King,
	Hans de Goede, dev, devicetree, linux-arm-kernel, linux-kernel

2016-05-03 10:25 GMT+03:00 Chen-Yu Tsai <wens@csie.org>:
> On Tue, May 3, 2016 at 3:21 PM, Olliver Schinagl <oliver@schinagl.nl> wrote:
>>>> +       bus-width = <4>;
>>>
>>> Only 4 bits? We normally see eMMC with 8 bits. 4 bits are some kind of
>>> embedded SD card.
>>
>> On A20 as well? Our investigations so far have concluded that the A10 and
>> A20 have those pins not mapped out to pads. The IP does support it however
>> we assume.
>
> You're right. My bad. First time A10/A20 sees eMMC support.

I can't say anything about A10/A20, but I have a board with A13 and
the same eMMC chip and it works fine in 8 bit mode.

Regards,
Radoslav

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/1] ARM: dts: sunxi: Add a olinuxino-lime2-emmc
  2016-05-04 12:30       ` Radoslav Kolev
@ 2016-05-04 12:52         ` Olliver Schinagl
       [not found]           ` <948be370-4401-43cb-862e-d4376755a75d@googlegroups.com>
  0 siblings, 1 reply; 14+ messages in thread
From: Olliver Schinagl @ 2016-05-04 12:52 UTC (permalink / raw)
  To: Radoslav Kolev, Chen-Yu Tsai
  Cc: Maxime Ripard, Tsvetan Usunov, Rob Herring, Pawel Moll,
	Mark Rutland, Ian Campbell, Kumar Gala, Russell King,
	Hans de Goede, dev, devicetree, linux-arm-kernel, linux-kernel

Hey Radoslav,

On 04-05-16 14:30, Radoslav Kolev wrote:
> 2016-05-03 10:25 GMT+03:00 Chen-Yu Tsai <wens@csie.org>:
>> On Tue, May 3, 2016 at 3:21 PM, Olliver Schinagl <oliver@schinagl.nl> wrote:
>>>>> +       bus-width = <4>;
>>>> Only 4 bits? We normally see eMMC with 8 bits. 4 bits are some kind of
>>>> embedded SD card.
>>> On A20 as well? Our investigations so far have concluded that the A10 and
>>> A20 have those pins not mapped out to pads. The IP does support it however
>>> we assume.
>> You're right. My bad. First time A10/A20 sees eMMC support.
> I can't say anything about A10/A20, but I have a board with A13 and
> the same eMMC chip and it works fine in 8 bit mode.
Yep, sun5i actually brings them all out to pads, the A20 however does 
not :( We first thought that the A20 would also be an 8bitter, because 
the mmc IP appears to be the same as sun5i, but initial tests show it is 
not. As for A10, it has older IP and it might not even support 8 bit 
mode, let alone bring out the pins.

But with A20's + eMMC being available via the lime2, others may repeat 
my experiments! The lime2 is 8 bit connected.

Olliver
>
> Regards,
> Radoslav

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/1] ARM: dts: sunxi: Add a olinuxino-lime2-emmc
       [not found]                                       ` <5732EB34.6070203@schinagl.nl>
@ 2016-07-15  8:39                                         ` stefan.mavrodiev
  2016-07-15  8:57                                           ` Olliver Schinagl
  0 siblings, 1 reply; 14+ messages in thread
From: stefan.mavrodiev @ 2016-07-15  8:39 UTC (permalink / raw)
  To: linux-sunxi
  Cc: christo.radev, radoslav.kolev, wens, maxime.ripard, tsvetan,
	robh+dt, pawel.moll, mark.rutland, ijc+devicetree, galak, linux,
	hdegoede, dev, devicetree, linux-arm-kernel, linux-kernel,
	oliver

[-- Attachment #1: Type: text/plain, Size: 255 bytes --]

Hi Olliver,

Why are you using nRST signal?
What I mean is this pin is inactive on this eMMC chip. To use the signal 
byte 162 of ECSD registers should be written.

On my board, this "reset" signal causes eMMC not to work.

Best regards,
Stefan Mavrodiev

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/1] ARM: dts: sunxi: Add a olinuxino-lime2-emmc
  2016-07-15  8:39                                         ` stefan.mavrodiev
@ 2016-07-15  8:57                                           ` Olliver Schinagl
  0 siblings, 0 replies; 14+ messages in thread
From: Olliver Schinagl @ 2016-07-15  8:57 UTC (permalink / raw)
  To: stefan.mavrodiev, linux-sunxi
  Cc: christo.radev, radoslav.kolev, wens, maxime.ripard, tsvetan,
	robh+dt, pawel.moll, mark.rutland, ijc+devicetree, galak, linux,
	hdegoede, dev, devicetree, linux-arm-kernel, linux-kernel

On 15-07-16 10:39, stefan.mavrodiev@gmail.com wrote:
> Hi Olliver,
>
> Why are you using nRST signal?
> What I mean is this pin is inactive on this eMMC chip. To use the signal
> byte 162 of ECSD registers should be written.
Then that sounds like a bug in the mmc layer I would say (or a missing 
attribute in the dts), we have a nRST signal it is wired, if the chip 
ignores it, then nothing lost, if the chip needs to be initialized with 
byte 162 of the ECSD register to make the nRST work, then that it sounds 
like it should be fixed there?

By ommitting the nRST signal just because the chip isn't properly 
initialized sounds more like a work-around to me, but I could be wrong 
and see things wrong.

What do the mmc guys, who have far more experience here, say?

>
> On my board, this "reset" signal causes eMMC not to work.
That supprises me, as afaik I was using it just fine...

Olliver
>
> Best regards,
> Stefan Mavrodiev

^ permalink raw reply	[flat|nested] 14+ messages in thread

end of thread, other threads:[~2016-07-15 11:16 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-04-28  7:19 [PATCH 1/1] ARM: dts: sunxi: Add a olinuxino-lime2-emmc Olliver Schinagl
2016-05-02 10:55 ` Maxime Ripard
2016-05-03  3:33 ` Chen-Yu Tsai
2016-05-03  7:21   ` Olliver Schinagl
2016-05-03  7:25     ` Chen-Yu Tsai
2016-05-04 12:30       ` Radoslav Kolev
2016-05-04 12:52         ` Olliver Schinagl
     [not found]           ` <948be370-4401-43cb-862e-d4376755a75d@googlegroups.com>
     [not found]             ` <5729F6D6.8030100@schinagl.nl>
     [not found]               ` <4704fa35-9a2a-4e6e-8fd4-f4778405c598@googlegroups.com>
     [not found]                 ` <572A0052.9060202@schinagl.nl>
     [not found]                   ` <2e745ef7-ddc0-40fc-b867-414543690276@googlegroups.com>
     [not found]                     ` <572A10B3.2020803@schinagl.nl>
     [not found]                       ` <4375220a-f939-4ed0-a6d7-2cf887b07509@googlegroups.com>
     [not found]                         ` <9fe7ebb6-6d74-4b73-b6d6-93b79650cdb6@googlegroups.com>
     [not found]                           ` <9745a80b-5444-43f7-b772-68684c96c9e5@googlegroups.com>
     [not found]                             ` <57322C8D.5060806@schinagl.nl>
     [not found]                               ` <8afbe753-4e74-4b30-ad7e-a12027d28264@googlegroups.com>
     [not found]                                 ` <57323CAB.7090202@schinagl.nl>
     [not found]                                   ` <f959ee5c-fa63-48f6-9819-4822c055b928@googlegroups.com>
     [not found]                                     ` <6bfa242e-c0ce-4dc5-9144-b3f16d8fb75c@googlegroups.com>
     [not found]                                       ` <5732EB34.6070203@schinagl.nl>
2016-07-15  8:39                                         ` stefan.mavrodiev
2016-07-15  8:57                                           ` Olliver Schinagl
2016-05-03  7:12 ` christo.radev
2016-05-03 13:14   ` Maxime Ripard
2016-05-03 15:02     ` christo.radev
2016-05-03 15:52       ` Olliver Schinagl
2016-05-04  5:13         ` [linux-sunxi] " Priit Laes

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).