Hi Jon, Am Mittwoch, den 17.02.2016, 13:19 +0000 schrieb Jon Hunter: > Hi Lucas, > > On 22/12/15 18:40, Lucas Stach wrote: > > Hi all, > > > > this series implements UHS-I signaling for the Tegra SDHCI host, > > which mainly means putting a proper tuning sequence in place. > > > > I've tested this on Jetson TK1 and got the following speed results, > > where mmcblk0 is the on-board eMMC and mmcblk1 is a micro SDXC > > card: > > > > Without series applied: > > hdparm -t /dev/mmcblk0 > > Timing buffered disk reads: 110 MB in 3.05 seconds = 36.02 > > MB/sec > > hdparm -t /dev/mmcblk1 > > Timing buffered disk reads: 56 MB in 3.01 seconds = 18.63 > > MB/sec > > > > With series applied: > > hdparm -t /dev/mmcblk0 > > Timing buffered disk reads: 236 MB in 3.00 seconds = 78.58 > > MB/sec > > hdparm -t /dev/mmcblk1 > > Timing buffered disk reads: 102 MB in 3.04 seconds = 33.51 > > MB/sec > > > > Tegra 30 does support UHS-I speeds too, but currently has problems > > when lowering the card voltage, which is needed in order to switch > > to UHS-I signaling. I have some more patches to fix this, but they > > need a bit more cleanup, with them applied the gains on Tegra30 are > > similar to the results above. > > > > For now the gains are limited to Tegra124+, with no regressions on > > Tegra30 and Tegra20. > > > > V2 fixes some minor style problems and is rebased on top of > > mmc/next. > > This means it enables the same tuning logic on Tegra210 also. I > > don't have a way to test this myself, so any testing on Tegra210 > > much > > appreciated. > > I have recently noticed that on my tegra114-dalmore-a04, SD cards are > no > longer recognised on boot. It appears that the problem started after > this series was merged. > > On tegra114 I see: > > [ 1.777006] mmc0: error -110 whilst initialising SD card > > If I disable the UHS-I changes then it works again (see below). I am > guessing you don't have a tegra114 board to test on? I have not had > time > to look into this further, but I am not sure if we should disable > UHS-I > on tegra114 for now? > > Cheers > Jon > Right I don't have a Tegra114 board to test with. I already know that Tegra30 needs some more fixes to get UHS-I to work and I have patches for that, but they are not yet ready for sending out. I would guess that Tegra114 may need the same fix as Tegra30. I can Cc you on those patches if you prefer. Please submit a formal patch to disable UHS-I on Tegra114, I think it's the right thing to do at this point and will happily ACK it. Regards, Lucas -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
