Re: [PATCH] ata: ahci: clear HOST_IRQ_STAT before the ports on Marvell 88SE61xx

On 8/25/26 16:58, Hajo Noerenberg wrote:
> 
> Damien,
> 
> thank you for the quick review.
> 
> On 8/25/26 02:51, Damien Le Moal wrote:
>> The commit title is not great as "before the ports" is not very clear.
>> What does "clear the ports" mean? Though I do not have a better alternative
>> title (yet).
> 
> Agreed, and "the ports" was meant to be the ports' PxIS registers, which the
> title does not say.  Would this be better?
> 
>   ata: ahci: work around lost interrupts on Marvell 88SE61xx

That is better!

> I tried exactly that: a build that registers the existing
> ahci_thunderx_irq_handler() unchanged for board_ahci_mv, taken out of the
> #ifdef CONFIG_ARM64, with a dev_info() so I could be certain which handler was
> really active.  It does not work.  The controller fails exactly as it does
> without any patch.

OK. Understood. Please resend the patch with the fixed-up commit title and we'll
apply it.

-- 
Damien Le Moal
Western Digital Research