Espressobin booting (Was: Re: [PATCH 1/5] arm: mvebu: Espressobin: Move env load addresses)

[Pali Rohár]

On Wednesday 26 August 2020 15:22:37 Andre Heider wrote:
> Hi,
> 
> On 26/08/2020 13:40, Pali Rohár wrote:
> > On Wednesday 26 August 2020 12:51:18 Pali Rohár wrote:
> > > On Wednesday 26 August 2020 11:57:01 Andre Heider wrote:
> > > > Hi,
> > > > 
> > > > On 26/08/2020 10:31, Pali Rohár wrote:
> > > > > Hello!
> > > > > 
> > > > > On Wednesday 26 August 2020 09:30:11 Andre Heider wrote:
> > > > > > Pali,
> > > > > > 
> > > > > > thank you for all these espressobin fixes, much appreciated!
> > > > > > 
> > > > > > This isn't directly related to the patch I'm replying to, but since you seem
> > > > > > to be syncing down- with upstream:
> > > > > > 
> > > > > > Upstream u-boot doesn't detect my sata ssd ("SATA link 0 timeout"), nor is
> > > > > 
> > > > > Could you try following Marvell SATA/AHCI patch?
> > > > > https://github.com/MarvellEmbeddedProcessors/u-boot-marvell/commit/2c1571432e4fd298be0236b53511f415bbdd21ac
> > > > 
> > > > will do that later and report my findings, thanks!
> 
> That one didn't help unfortunately. u-boot still prints "SATA link 0
> timeout". Booting kernel 5.7.0-0.bpo.2-arm64 with that and sata works, but
> booting 4.19.0-10-arm64 and it doesn't: "ata1: SATA link down (SStatus 131
> SControl 300)" is printed all over again.

I found another patch which could maybe help:
https://github.com/MarvellEmbeddedProcessors/u-boot-marvell/commit/6c6b2088f83b992d03e0d56f92c4dda7d44a7dc0

Or maybe with this patch:

diff --git a/arch/arm/dts/armada-3720-espressobin.dts b/arch/arm/dts/armada-3720-espressobin.dts
index 4534f5ff29..c5eea978f6 100644
--- a/arch/arm/dts/armada-3720-espressobin.dts
+++ b/arch/arm/dts/armada-3720-espressobin.dts
@@ -95,6 +95,7 @@
 	phy2 {
 		phy-type = <PHY_TYPE_SATA0>;
 		phy-speed = <PHY_SPEED_5G>;
+		phy-invert = <COMPHY_POLARITY_NO_INVERT>;
 	};
 };