ARMADA espressobin SATA drive detection failure

[Shinichiro Kawasaki]

On Aug 26, 2022 / 00:15, Marek Behún wrote:
> On Thu, 25 Aug 2022 15:00:59 +0200
> Pali Rohár <pali at kernel.org> wrote:
> 
> > On Sunday 14 August 2022 01:10:50 Pali Rohár wrote:
> > > On Saturday 13 August 2022 23:02:34 Shinichiro Kawasaki wrote:

[...]

> > > Perfect! So the issue is with mvebu_a3700_comphy_reset() function.
> > > 
> > > This function is not in TF-A code and neither in my original kernel
> > > driver implementation (still available here):
> > > https://git.kernel.org/pub/scm/linux/kernel/git/pali/linux.git/commit/?h=phy-mvebu-a3700-comphy&id=4588902a3528195bcfdda9f9e1e14262a1955df1
> > > 
> > > Marek, this function mvebu_a3700_comphy_reset() was implemented by you.
> > > Could you please look at it, why you added this function and try to fix
> > > it? Is this function needed at all?  
> > 
> > PING? Any progress here?
> 
> Not yet, sorry. Maybe we could do something like I am attaching, until
> I have time to play with it?
> 
> See attached file.

Merek, thanks for the response. I applied the attached patch to v5.18.16 kernel
and observed the "ata1: SATA link down (SStatus 100 SControl 300)" message is
avoided. It looks working.

-- 
Shin'ichiro Kawasaki