diff mbox

ARM: dts: imx6qdl: Fix CODA960 interrupt order

Message ID 1417188226-21756-1-git-send-email-p.zabel@pengutronix.de (mailing list archive)
State New, archived
Headers show

Commit Message

Philipp Zabel Nov. 28, 2014, 3:23 p.m. UTC
Commit a04a0b6fed4f ("ARM: dts: imx6qdl: Enable CODA960 VPU") lost the
fix for the CODA960 interrupt order during a rebase before being applied.
This patch adds the missing bit and brings the interrupts and
interrupt-names properties back in sync.

Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
---
 arch/arm/boot/dts/imx6qdl.dtsi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Fabio Estevam Nov. 28, 2014, 3:52 p.m. UTC | #1
On Fri, Nov 28, 2014 at 1:23 PM, Philipp Zabel <p.zabel@pengutronix.de> wrote:
> Commit a04a0b6fed4f ("ARM: dts: imx6qdl: Enable CODA960 VPU") lost the
> fix for the CODA960 interrupt order during a rebase before being applied.
> This patch adds the missing bit and brings the interrupts and
> interrupt-names properties back in sync.
>
> Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>

Thanks, Philipp. This fixes the VPU timeout I was seeing:

Reviewed-by: Fabio Estevam <fabio.estevam@freescale.com>
Shawn Guo Dec. 2, 2014, 2:20 p.m. UTC | #2
On Fri, Nov 28, 2014 at 04:23:46PM +0100, Philipp Zabel wrote:
> Commit a04a0b6fed4f ("ARM: dts: imx6qdl: Enable CODA960 VPU") lost the
> fix for the CODA960 interrupt order during a rebase before being applied.
> This patch adds the missing bit and brings the interrupts and
> interrupt-names properties back in sync.
> 
> Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>

Applied, thanks.
Fabio Estevam Dec. 8, 2014, 3:13 p.m. UTC | #3
Hi Shawn,

On Tue, Dec 2, 2014 at 12:20 PM, Shawn Guo <shawn.guo@linaro.org> wrote:
> On Fri, Nov 28, 2014 at 04:23:46PM +0100, Philipp Zabel wrote:
>> Commit a04a0b6fed4f ("ARM: dts: imx6qdl: Enable CODA960 VPU") lost the
>> fix for the CODA960 interrupt order during a rebase before being applied.
>> This patch adds the missing bit and brings the interrupts and
>> interrupt-names properties back in sync.
>>
>> Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
>
> Applied, thanks.

Could you also apply it into for-next?
diff mbox

Patch

diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi
index 4fc03b7..2109d07 100644
--- a/arch/arm/boot/dts/imx6qdl.dtsi
+++ b/arch/arm/boot/dts/imx6qdl.dtsi
@@ -335,8 +335,8 @@ 
 			vpu: vpu@02040000 {
 				compatible = "cnm,coda960";
 				reg = <0x02040000 0x3c000>;
-				interrupts = <0 3 IRQ_TYPE_LEVEL_HIGH>,
-				             <0 12 IRQ_TYPE_LEVEL_HIGH>;
+				interrupts = <0 12 IRQ_TYPE_LEVEL_HIGH>,
+					     <0 3 IRQ_TYPE_LEVEL_HIGH>;
 				interrupt-names = "bit", "jpeg";
 				clocks = <&clks IMX6QDL_CLK_VPU_AXI>,
 					 <&clks IMX6QDL_CLK_MMDC_CH0_AXI>,