Re: [PATCH] arm64: dts: imx8mq: Make GPC default interrupt parent

[Andrey Smirnov]

On Thu, Jan 31, 2019 at 1:47 PM Fabio Estevam  wrote:
>
> Hi Andrey
>
> On Thu, Jan 31, 2019 at 7:11 PM Andrey Smirnov  
> wrote:
> >
> > Now that the GPCv2 driver supports i.MX8MQ, make GPC default interrupt
> > parent.
> >
> > Signed-off-by: Andrey Smirnov 
>
> Lucas has already submitted this same change and another one that
> selects the GPC driver as part of a series.
>

OK, good to know. Please disregard this patch, then.

Thanks,
Andrey Smirnov

Re: [PATCH] arm64: dts: imx8mq: Make GPC default interrupt parent

[Fabio Estevam]

Hi Andrey

On Thu, Jan 31, 2019 at 7:11 PM Andrey Smirnov  wrote:
>
> Now that the GPCv2 driver supports i.MX8MQ, make GPC default interrupt
> parent.
>
> Signed-off-by: Andrey Smirnov 

Lucas has already submitted this same change and another one that
selects the GPC driver as part of a series.

My feedback was to send the GPC selection patch first in order to
avoid boot hang when people try to boot without the GPC driver
selected.

[PATCH] arm64: dts: imx8mq: Make GPC default interrupt parent

[Andrey Smirnov]

Now that the GPCv2 driver supports i.MX8MQ, make GPC default interrupt
parent.

Signed-off-by: Andrey Smirnov 
Cc: Fabio Estevam 
Cc: Chris Healy 
Cc: Lucas Stach 
Cc: Leonard Crestez 
Cc: "A.s. Dong" 
Cc: Richard Zhu 
Cc: linux-...@nxp.com
Cc: linux-arm-ker...@lists.infradead.org
Cc: linux-kernel@vger.kernel.org
---
 arch/arm64/boot/dts/freescale/imx8mq.dtsi | 8 ++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi 
b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
index d20e5c7e21a3..e9aa64156577 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
@@ -12,8 +12,7 @@
 #include "imx8mq-pinfunc.h"
 
 / {
-   /* This should really be the GPC, but we need a driver for this first */
-   interrupt-parent = <>;
+   interrupt-parent = <>;
 
#address-cells = <2>;
#size-cells = <2>;
@@ -265,6 +264,11 @@
compatible = "fsl,imx8mq-gpc";
reg = <0x303a 0x1>;
 
+   interrupt-controller;
+   interrupts = ;
+   #interrupt-cells = <3>;
+   interrupt-parent = <>;
+
pgc {
#address-cells = <1>;
#size-cells = <0>;
-- 
2.20.1