Re: [PATCH 1/2] ARM64: dts: meson-gxbb: odroidc2: enable sdcard UHS modes

2018-05-14 Thread Anand Moon
Hi Jerome

On 14 May 2018 at 19:18, Jerome Brunet  wrote:
> On Wed, 2018-05-02 at 00:29 +0530, Anand Moon wrote:
>> Enable UHS modes for sdcard, on odroidc2.
>>
>> Signed-off-by: Anand Moon 
>> ---
>>  arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts | 5 +
>>  1 file changed, 5 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts 
>> b/arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts
>> index 54954b314a45..c90f8b46c60c 100644
>> --- a/arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts
>> +++ b/arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts
>> @@ -258,6 +258,11 @@
>>   cap-sd-highspeed;
>>   max-frequency = <1>;
>>   disable-wp;
>> + sd-uhs-sdr12;
>> + sd-uhs-sdr25;
>> + sd-uhs-sdr50;
>> + sd-uhs-sdr104;
>> + sd-uhs-ddr50;
>>
>>   cd-gpios = <&gpio CARD_6 GPIO_ACTIVE_HIGH>;
>>   cd-inverted;
>
> Hi Anand,
>
> I tried a few sdcards on the OC2 with your 2 patches.
> Like with the libretech-cc, sdr104@200Mhz works "mostly", but, with some
> sdcards, it does not - please see below. The same sdcards appear to be working
> fine on my laptop.
>
> This is something I have not been able to crack yet on the libretech-cc.
>
> I'd suggest dropping sdr104 and keeping the max frequency at 100Mhz until we 
> can
> figure out the problem here.
>
> With this changed:
>
> Tested-by: Jerome Brunet 
>
> dd if=/dev/mmcblk1 of=/dev/zero bs=4M
> [  446.925817] mmc1: tuning execution failed: -5
> [  446.956597] mmc1: tuning execution failed: -5
> [  489.957810] print_req_error: I/O error, dev mmcblk1, sector 6654424
> [  490.141975] print_req_error: I/O error, dev mmcblk1, sector 6656440
> [  490.148304] print_req_error: I/O error, dev mmcblk1, sector 6656944
> [  490.349650] print_req_error: I/O error, dev mmcblk1, sector 6658456
> [  491.804382] print_req_error: I/O error, dev mmcblk1, sector 6747688
> [  492.281246] print_req_error: I/O error, dev mmcblk1, sector 6784992
> [  492.419034] print_req_error: I/O error, dev mmcblk1, sector 6785496
> [  492.865878] print_req_error: I/O error, dev mmcblk1, sector 6791800
> [  493.023809] print_req_error: I/O error, dev mmcblk1, sector 6792192
> [  493.024435] Buffer I/O error on dev mmcblk1, logical block 849024, async 
> page
> read
> [  493.217751] print_req_error: I/O error, dev mmcblk1, sector 6792808
> [  494.891779] mmc1: tuning execution failed: -5
> [  495.374186] print_req_error: 3 callbacks suppressed
> [  495.374193] print_req_error: I/O error, dev mmcblk1, sector 6854576
> [  495.767498] print_req_error: I/O error, dev mmcblk1, sector 686
> [  496.013104] print_req_error: I/O error, dev mmcblk1, sector 6863024
> [  496.223042] print_req_error: I/O error, dev mmcblk1, sector 6864032
> [  496.227003] print_req_error: I/O error, dev mmcblk1, sector 6864536
> [  496.375175] print_req_error: I/O error, dev mmcblk1, sector 6864176
> [  496.375806] Buffer I/O error on dev mmcblk1, logical block 858022, async 
> page
> read
> [  496.521229] print_req_error: I/O error, dev mmcblk1, sector 6864184
> [  496.521852] Buffer I/O error on dev mmcblk1, logical block 858023, async 
> page
> read
> [  503.596978] print_req_error: I/O error, dev mmcblk1, sector 6983312
> [  503.597606] Buffer I/O error on dev mmcblk1, logical block 872914, async 
> page
> read
> [  505.280621] print_req_error: I/O error, dev mmcblk1, sector 7004536
> [  505.281249] Buffer I/O error on dev mmcblk1, logical block 875567, async 
> page
> read
> [  507.372560] print_req_error: I/O error, dev mmcblk1, sector 7048696
> [  507.373192] Buffer I/O error on dev mmcblk1, logical block 881087, async 
> page
> read
> [  511.355248] print_req_error: I/O error, dev mmcblk1, sector 7131352
> [  511.355883] Buffer I/O error on dev mmcblk1, logical block 891419, async 
> page
> read
> [  511.369076] print_req_error: I/O error, dev mmcblk1, sector 7131352
> [  511.369694] Buffer I/O error on dev mmcblk1, logical block 891419, async 
> page
> read
> dd: error reading '/dev/mmcblk1': Input/output error
> 868+7 records in
> 868+7 records out
> 3651252224 bytes (3.7 GB, 3.4 GiB) copied, 66.7736 s, 54.7 MB/s


Thank for your testing, I will also do some more testing on all the
sdcard and share you the result..

Mean while Sandisk Ultra microSDHC UHS-I card @A1 32GB shows.

On my Odroid C2:

[1.165784] meson-gx-mmc d0074000.mmc: allocated mmc-pwrseq
[1.403756] meson-gx-mmc d0072000.mmc: Got CD GPIO
[1.456160] Waiting for root device /dev/mmcblk1p2...
[1.608180] mmc1: new ultra high speed SDR104 SDHC card at address 
[1.610002] mmcblk1: mmc1: JULIE 29.7 GiB
[1.617811]  mmcblk1: p1 p2

Also I did not encounter and read/write failure on the sdcard but I
will re-test them.

root@odroid64:~#  dd if=/dev/mmcblk1 of=/dev/zero bs=4M
7609+1 records in
7609+1 records out
31914983424 bytes (32 GB, 30 GiB) copied, 368.637 s, 86.6 MB/s
root@odroid64:~#

Best Regards
-Anand


Re: [PATCH 1/2] ARM64: dts: meson-gxbb: odroidc2: enable sdcard UHS modes

2018-05-14 Thread Jerome Brunet
On Wed, 2018-05-02 at 00:29 +0530, Anand Moon wrote:
> Enable UHS modes for sdcard, on odroidc2.
> 
> Signed-off-by: Anand Moon 
> ---
>  arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts | 5 +
>  1 file changed, 5 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts 
> b/arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts
> index 54954b314a45..c90f8b46c60c 100644
> --- a/arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts
> +++ b/arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts
> @@ -258,6 +258,11 @@
>   cap-sd-highspeed;
>   max-frequency = <1>;
>   disable-wp;
> + sd-uhs-sdr12;
> + sd-uhs-sdr25;
> + sd-uhs-sdr50;
> + sd-uhs-sdr104;
> + sd-uhs-ddr50;
>  
>   cd-gpios = <&gpio CARD_6 GPIO_ACTIVE_HIGH>;
>   cd-inverted;

Hi Anand,

I tried a few sdcards on the OC2 with your 2 patches.
Like with the libretech-cc, sdr104@200Mhz works "mostly", but, with some
sdcards, it does not - please see below. The same sdcards appear to be working
fine on my laptop.

This is something I have not been able to crack yet on the libretech-cc.

I'd suggest dropping sdr104 and keeping the max frequency at 100Mhz until we can
figure out the problem here.

With this changed:

Tested-by: Jerome Brunet 

dd if=/dev/mmcblk1 of=/dev/zero bs=4M   
[  446.925817] mmc1: tuning execution failed: -5   
[  446.956597] mmc1: tuning execution failed: -5   
[  489.957810] print_req_error: I/O error, dev mmcblk1, sector 6654424 
[  490.141975] print_req_error: I/O error, dev mmcblk1, sector 6656440 
[  490.148304] print_req_error: I/O error, dev mmcblk1, sector 6656944 
[  490.349650] print_req_error: I/O error, dev mmcblk1, sector 6658456 
[  491.804382] print_req_error: I/O error, dev mmcblk1, sector 6747688 
[  492.281246] print_req_error: I/O error, dev mmcblk1, sector 6784992 
[  492.419034] print_req_error: I/O error, dev mmcblk1, sector 6785496 
[  492.865878] print_req_error: I/O error, dev mmcblk1, sector 6791800 
[  493.023809] print_req_error: I/O error, dev mmcblk1, sector 6792192 
[  493.024435] Buffer I/O error on dev mmcblk1, logical block 849024, async page
read 
[  493.217751] print_req_error: I/O error, dev mmcblk1, sector 6792808 
[  494.891779] mmc1: tuning execution failed: -5   
[  495.374186] print_req_error: 3 callbacks suppressed 
[  495.374193] print_req_error: I/O error, dev mmcblk1, sector 6854576 
[  495.767498] print_req_error: I/O error, dev mmcblk1, sector 686 
[  496.013104] print_req_error: I/O error, dev mmcblk1, sector 6863024 
[  496.223042] print_req_error: I/O error, dev mmcblk1, sector 6864032 
[  496.227003] print_req_error: I/O error, dev mmcblk1, sector 6864536 
[  496.375175] print_req_error: I/O error, dev mmcblk1, sector 6864176 
[  496.375806] Buffer I/O error on dev mmcblk1, logical block 858022, async page
read 
[  496.521229] print_req_error: I/O error, dev mmcblk1, sector 6864184 
[  496.521852] Buffer I/O error on dev mmcblk1, logical block 858023, async page
read 
[  503.596978] print_req_error: I/O error, dev mmcblk1, sector 6983312 
[  503.597606] Buffer I/O error on dev mmcblk1, logical block 872914, async page
read 
[  505.280621] print_req_error: I/O error, dev mmcblk1, sector 7004536 
[  505.281249] Buffer I/O error on dev mmcblk1, logical block 875567, async page
read 
[  507.372560] print_req_error: I/O error, dev mmcblk1, sector 7048696 
[  507.373192] Buffer I/O error on dev mmcblk1, logical block 881087, async page
read 
[  511.355248] print_req_error: I/O error, dev mmcblk1, sector 7131352 
[  511.355883] Buffer I/O error on dev mmcblk1, logical block 891419, async page
read 
[  511.369076] print_req_error: I/O error, dev mmcblk1, sector 7131352 
[  511.369694] Buffer I/O error on dev mmcblk1, logical block 891419, async page
read 
dd: error reading '/dev/mmcblk1': Input/output error   
868+7 records in   
868+7 records out  
3651252224 bytes (3.7 GB, 3.4 GiB) copied, 66.7736 s, 54.7 MB/s   


[PATCH 1/2] ARM64: dts: meson-gxbb: odroidc2: enable sdcard UHS modes

2018-05-01 Thread Anand Moon
Enable UHS modes for sdcard, on odroidc2.

Signed-off-by: Anand Moon 
---
 arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts | 5 +
 1 file changed, 5 insertions(+)

diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts 
b/arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts
index 54954b314a45..c90f8b46c60c 100644
--- a/arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts
+++ b/arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts
@@ -258,6 +258,11 @@
cap-sd-highspeed;
max-frequency = <1>;
disable-wp;
+   sd-uhs-sdr12;
+   sd-uhs-sdr25;
+   sd-uhs-sdr50;
+   sd-uhs-sdr104;
+   sd-uhs-ddr50;
 
cd-gpios = <&gpio CARD_6 GPIO_ACTIVE_HIGH>;
cd-inverted;
-- 
2.17.0