All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gregory CLEMENT <gregory.clement@bootlin.com>
To: sven.auhagen@voleatech.de, axboe@kernel.dk, hdegoede@redhat.com,
	robh+dt@kernel.org, tglx@linutronix.de, maz@kernel.org
Cc: linux-ide@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	devicetree@vger.kernel.org, jason@lakedaemon.net, andrew@lunn.ch,
	rjw@rjwysocki.net, viresh.kumar@linaro.org,
	antoine.tenart@bootlin.com, maxime.chevallier@bootlin.com,
	thomas.petazzoni@bootlin.com, miquel.raynal@bootlin.com
Subject: Re: [PATCH v3 9/9] arm64: dts: marvell: armada-cp110: Switch to per-port SATA interrupts
Date: Sun, 29 Nov 2020 12:25:31 +0100	[thread overview]
Message-ID: <87pn3wjtg4.fsf@BL-laptop> (raw)
In-Reply-To: <20201109173948.96663-10-sven.auhagen@voleatech.de>

Hi Sven,

> From: Sven Auhagen <sven.auhagen@voleatech.de>
>
> 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.
>
> Reviewed-by: Hans de Goede <hdegoede@redhat.com>
> Signed-off-by: Sven Auhagen <sven.auhagen@voleatech.de>
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
> Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>

Applied on mvebu/dt64

Thanks,

Gregory

> ---
>  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>;
>  			};
>  		};
> -- 
> 2.20.1
>

-- 
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com

WARNING: multiple messages have this Message-ID (diff)
From: Gregory CLEMENT <gregory.clement@bootlin.com>
To: sven.auhagen@voleatech.de, axboe@kernel.dk, hdegoede@redhat.com,
	robh+dt@kernel.org, tglx@linutronix.de, maz@kernel.org
Cc: andrew@lunn.ch, jason@lakedaemon.net, devicetree@vger.kernel.org,
	antoine.tenart@bootlin.com, viresh.kumar@linaro.org,
	rjw@rjwysocki.net, maxime.chevallier@bootlin.com,
	linux-ide@vger.kernel.org, thomas.petazzoni@bootlin.com,
	miquel.raynal@bootlin.com, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v3 9/9] arm64: dts: marvell: armada-cp110: Switch to per-port SATA interrupts
Date: Sun, 29 Nov 2020 12:25:31 +0100	[thread overview]
Message-ID: <87pn3wjtg4.fsf@BL-laptop> (raw)
In-Reply-To: <20201109173948.96663-10-sven.auhagen@voleatech.de>

Hi Sven,

> From: Sven Auhagen <sven.auhagen@voleatech.de>
>
> 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.
>
> Reviewed-by: Hans de Goede <hdegoede@redhat.com>
> Signed-off-by: Sven Auhagen <sven.auhagen@voleatech.de>
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
> Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>

Applied on mvebu/dt64

Thanks,

Gregory

> ---
>  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>;
>  			};
>  		};
> -- 
> 2.20.1
>

-- 
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2020-11-29 11:26 UTC|newest]

Thread overview: 46+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-11-09 17:39 [PATCH v3 0/9] Armada8k enable per-port SATA interrupts and drop a hack in the IRQ subsystem sven.auhagen
2020-11-09 17:39 ` sven.auhagen
2020-11-09 17:39 ` [PATCH v3 1/9] ata: ahci: mvebu: Rename a platform data flag sven.auhagen
2020-11-09 17:39   ` sven.auhagen
2020-11-09 17:39 ` [PATCH v3 2/9] ata: ahci: mvebu: Support A8k compatible sven.auhagen
2020-11-09 17:39   ` sven.auhagen
2020-11-09 17:39 ` [PATCH v3 3/9] ata: libahci_platform: Do not try to get an IRQ when AHCI_HFLAG_MULTI_MSI is set sven.auhagen
2020-11-09 17:39   ` sven.auhagen
2020-11-09 17:39 ` [PATCH v3 4/9] ata: ahci: add ack callback to multi irq handler sven.auhagen
2020-11-09 17:39   ` sven.auhagen
2020-11-09 17:39 ` [PATCH v3 5/9] ata: ahci: mvebu: Add support for A8k legacy DT bindings sven.auhagen
2020-11-09 17:39   ` sven.auhagen
2020-11-09 17:39 ` [PATCH v3 6/9] irqchip/irq-mvebu-icu: Remove the double SATA ports interrupt hack sven.auhagen
2020-11-09 17:39   ` sven.auhagen
2020-11-09 17:39 ` [PATCH v3 7/9] dt-bindings: ata: Update ahci bindings with possible per-port interrupts sven.auhagen
2020-11-09 17:39   ` sven.auhagen
2020-11-09 17:39 ` [PATCH v3 8/9] dt-bindings: ata: Update ahci_mvebu bindings sven.auhagen
2020-11-09 17:39   ` sven.auhagen
2020-11-09 17:39 ` [PATCH v3 9/9] arm64: dts: marvell: armada-cp110: Switch to per-port SATA interrupts sven.auhagen
2020-11-09 17:39   ` sven.auhagen
2020-11-29 11:25   ` Gregory CLEMENT [this message]
2020-11-29 11:25     ` Gregory CLEMENT
2021-02-27 10:57 ` [PATCH v3 0/9] Armada8k enable per-port SATA interrupts and drop a hack in the IRQ subsystem Sven Auhagen
2021-02-27 10:57   ` Sven Auhagen
2021-03-19  2:34   ` Marcin Wojtas
2021-03-19  2:34     ` Marcin Wojtas
2021-03-19  7:35     ` Gregory CLEMENT
2021-03-19  7:35       ` Gregory CLEMENT
2021-03-19  8:08       ` Marcin Wojtas
2021-03-19  8:08         ` Marcin Wojtas
2021-03-19  9:33         ` Marc Zyngier
2021-03-19  9:33           ` Marc Zyngier
2021-03-19  9:48           ` Marcin Wojtas
2021-03-19  9:48             ` Marcin Wojtas
2021-03-19 20:40             ` Mark Kettenis
2021-03-19 20:40               ` Mark Kettenis
2021-03-19 20:53               ` Andrew Lunn
2021-03-19 20:53                 ` Andrew Lunn
2021-03-19 22:13                 ` Marcin Wojtas
2021-03-19 22:13                   ` Marcin Wojtas
2021-03-19 22:28                   ` Andrew Lunn
2021-03-19 22:28                     ` Andrew Lunn
2021-03-19  7:26   ` Gregory CLEMENT
2021-03-19  7:26     ` Gregory CLEMENT
2021-03-19  8:16     ` Sven Auhagen
2021-03-19  8:16       ` Sven Auhagen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87pn3wjtg4.fsf@BL-laptop \
    --to=gregory.clement@bootlin.com \
    --cc=andrew@lunn.ch \
    --cc=antoine.tenart@bootlin.com \
    --cc=axboe@kernel.dk \
    --cc=devicetree@vger.kernel.org \
    --cc=hdegoede@redhat.com \
    --cc=jason@lakedaemon.net \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-ide@vger.kernel.org \
    --cc=maxime.chevallier@bootlin.com \
    --cc=maz@kernel.org \
    --cc=miquel.raynal@bootlin.com \
    --cc=rjw@rjwysocki.net \
    --cc=robh+dt@kernel.org \
    --cc=sven.auhagen@voleatech.de \
    --cc=tglx@linutronix.de \
    --cc=thomas.petazzoni@bootlin.com \
    --cc=viresh.kumar@linaro.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.