On 31/08/2020 09:50, Pali Rohár wrote:
On Monday 31 August 2020 05:25:37 Andre Heider wrote:
From: Wojciech Macek <w...@semihalf.com>

Add support for Marvell Xenon SDHCI HS200 mode.

Changes focue mostly on correct PHY initialization.
All procedure is similar to the one done by Linux
driver, but simplified.

Change-Id: I5e2396eeb23784f495abc18ea5a2eb7a92390730
Signed-off-by: Wojciech Macek <w...@semihalf.com>
Reviewed-on: http://vgitil04.il.marvell.com:8080/59230
Tested-by: iSoC Platform CI <ykj...@marvell.com>
Reviewed-by: Grzegorz Jaszczyk <j...@semihalf.com>
Reviewed-by: Kostya Porotchkin <kos...@marvell.com>
Reviewed-by: Igal Liberman <ig...@marvell.com>
[a.heider: adapt to mainline]
Signed-off-by: Andre Heider <a.hei...@gmail.com>
---
Missing downstream patch, noticed while diffing branches:
https://github.com/MarvellEmbeddedProcessors/u-boot-marvell/commit/387232507a0d9dda3990284221eaf87d7541dd02

Hello Andre! Why is this patch needed? Or what it is fixing?
I tested upstream U-Boot with all previous patches on Turris MOX and
Espressobin and SD cards worked fine.

My primary aim was to reduces the diff against the downstream fork, to make upstream work just as well.

But the HS200/400 mode is not used by those two boards, but by:
arch/arm/dts/armada-3720-uDPU.dts
arch/arm/dts/armada-3720-db.dts

And my other patch for espressobin-emmc is using it too. They're independent, so it works without it, just not as fast.

Having said that, I don't have either of those boards, so I haven't tested it myself. But the patches have been downstream for over two years, so I'd say they're well tested since everybody runs a downstream firmware (which will hopefully change now ;)

Regards,
Andre

Reply via email to