Hi,
On 10/7/20 5:45 PM, sven.auhagen@xxxxxxxxxxxx wrote:
> From: Sven Auhagen <sven.auhagen@xxxxxxxxxxxx>
>
> There are two SATA ports per CP110. Each of them has a dedicated
> interrupt. Describe the real hardware by adding two SATA ports to the
> CP110 SATA node.
>
> Signed-off-by: Sven Auhagen <sven.auhagen@xxxxxxxxxxxx>
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@xxxxxxxxxxx>
> Signed-off-by: Miquel Raynal <miquel.raynal@xxxxxxxxxxx>
Patch looks good to me:
Reviewed-by: Hans de Goede <hdegoede@xxxxxxxxxx>
Regards,
Hans
> ---
> arch/arm64/boot/dts/marvell/armada-cp11x.dtsi | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi b/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi
> index 9dcf16beabf5..ec27294f097b 100644
> --- a/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi
> +++ b/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi
> @@ -300,11 +300,9 @@
> };
>
> CP11X_LABEL(sata0): sata@540000 {
> - compatible = "marvell,armada-8k-ahci",
> - "generic-ahci";
> + compatible = "marvell,armada-8k-ahci";
> reg = <0x540000 0x30000>;
> dma-coherent;
> - interrupts = <107 IRQ_TYPE_LEVEL_HIGH>;
> clocks = <&CP11X_LABEL(clk) 1 15>,
> <&CP11X_LABEL(clk) 1 16>;
> #address-cells = <1>;
> @@ -312,10 +310,12 @@
> status = "disabled";
>
> sata-port@0 {
> + interrupts = <109 IRQ_TYPE_LEVEL_HIGH>;
> reg = <0>;
> };
>
> sata-port@1 {
> + interrupts = <107 IRQ_TYPE_LEVEL_HIGH>;
> reg = <1>;
> };
> };
>
