Re: [PATCH 0/6] Fix SATA support for NSP SoCs
From: Florian Fainelli
Date: Wed Oct 21 2026 - 13:06:15 EST
On 10/21/26 09:55, Florian Fainelli wrote:
This patch series fixes SATA on the Northstar Plus (BCM58xxx/8831x)
which was broken due to a number of things:
- the SATA PHY assumed separate PCB windows when a shared one is used
for this generation of SoCs
- the clock/reset (IDM) was not being configured by the driver which
meant that depending upon what the boot loader did, or did not, we
could have had the AHCI controller still in reset by then
- on some reference boards, the AHCI controller is not DMA coherent,
assuming so leads to all sorts of timeouts, data corruption, etc.
Since this touches several subsystems: DT, PHY, AHCI and DTS, I would be
keen on having Damien merge all of those via his tree if acceptable by
the DT and PHY maintainers, I am OK either way for Broadcom SoC.
Argh, looks like I submitted patches 5 and 6 twice after I had re-ordered them, the correct order is in the cover letter:
ARM: dts: broadcom: Use noncoherent SATA DMA on BCM958625HR
ARM: dts: broadcom: Enable SATA on BCM958625HR
Will wait for feedback and resubmit if necessary. Thanks!
--
Florian