Re: [PATCH v2] ARM64: dts: mcbin: fix SATA ports on Macchiatobin

["Russell King (Oracle)" <linux@armlinux.org.uk>]

On Thu, Aug 21, 2025 at 06:43:28PM +0100, Russell King (Oracle) wrote:
> Booting 6.16 on the Macchiatobin, I discover that I can no longer
> access my disks, and thus the userspace boot fails. The cause appears
> to be that one of the SATA controllers doesn't have any ports:
> 
> [    1.190312] ahci f4540000.sata: supply ahci not found, using dummy regulator
> [    1.196255] ahci f4540000.sata: supply phy not found, using dummy regulator
> [    1.202026] ahci f4540000.sata: No port enabled
> 
> This is as a result of the blamed commit below which added a default
> disabled status to the .dtsi, but didn't properly update the mcbin
> dtsi file. Fix this regression.
> 
> Fixes: 30023876aef4 ("arm64: dts: marvell: only enable complete sata nodes")
> Signed-off-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk>

Frank,

I think this is also similarly broken by your patch:

arch/arm64/boot/dts/marvell/armada-8040-db.dts

as you've updated the ports on one SATA controller but not the other
in the same way as you omitted the second controller on mcbin.

I'd also question this:

arch/arm64/boot/dts/marvell/cn9132-clearfog.dts

as you updated the other cn9132, but not this one which was introduced
in 6.11, and your change was in 6.13. Please can you look at both of
these and send appropriate fixes?

Thanks.

-- 
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last!