From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andre Heider Date: Mon, 31 Aug 2020 10:10:50 +0200 Subject: [PATCH 1/2] xenon_sdhci: support for HS200 mode In-Reply-To: <20200831075017.26mj7u2z3su26qdl@pali> References: <20200831032538.467336-1-a.heider@gmail.com> <20200831075017.26mj7u2z3su26qdl@pali> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 31/08/2020 09:50, Pali Roh?r wrote: > On Monday 31 August 2020 05:25:37 Andre Heider wrote: >> From: Wojciech Macek >> >> 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 >> Reviewed-on: http://vgitil04.il.marvell.com:8080/59230 >> Tested-by: iSoC Platform CI >> Reviewed-by: Grzegorz Jaszczyk >> Reviewed-by: Kostya Porotchkin >> Reviewed-by: Igal Liberman >> [a.heider: adapt to mainline] >> Signed-off-by: Andre Heider >> --- >> 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