* Re: [PATCH V2] ARM: dts: imx6ul iomuxc syscon is compatible to imx6q
@ 2016-08-29 14:34 ` Shawn Guo
0 siblings, 0 replies; 12+ messages in thread
From: Shawn Guo @ 2016-08-29 14:34 UTC (permalink / raw)
To: Anson Huang
Cc: linux-arm-kernel, linux-kernel, devicetree, mark.rutland, robh+dt,
kernel, fabio.estevam, linux
On Mon, Aug 29, 2016 at 10:25:43PM +0800, Anson Huang wrote:
> The imx6ul iomuxc syscon is compatible to imx6q,
> so let's add compatible string 'fsl,imx6q-iomuxc-gpr'
> for imx6ul iomuxc syscon node.
>
> Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
Will this patch still be needed if you implement imx6ul suspend support
with PSCI?
Shawn
> ---
> changes since V1:
> improve commit message.
> arch/arm/boot/dts/imx6ul.dtsi | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/imx6ul.dtsi b/arch/arm/boot/dts/imx6ul.dtsi
> index 33b95d7..acc9486 100644
> --- a/arch/arm/boot/dts/imx6ul.dtsi
> +++ b/arch/arm/boot/dts/imx6ul.dtsi
> @@ -644,7 +644,8 @@
> };
>
> gpr: iomuxc-gpr@020e4000 {
> - compatible = "fsl,imx6ul-iomuxc-gpr", "syscon";
> + compatible = "fsl,imx6ul-iomuxc-gpr",
> + "fsl,imx6q-iomuxc-gpr", "syscon";
> reg = <0x020e4000 0x4000>;
> };
>
> --
> 1.9.1
>
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [PATCH V2] ARM: dts: imx6ul iomuxc syscon is compatible to imx6q
@ 2016-08-29 14:34 ` Shawn Guo
0 siblings, 0 replies; 12+ messages in thread
From: Shawn Guo @ 2016-08-29 14:34 UTC (permalink / raw)
To: Anson Huang
Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
devicetree-u79uwXL29TY76Z2rM5mHXA, mark.rutland-5wv7dgnIgG8,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A, kernel-bIcnvbaLZ9MEGnE8C9+IrQ,
fabio.estevam-3arQi8VN3Tc, linux-I+IVW8TIWO2tmTQ+vhA3Yw
On Mon, Aug 29, 2016 at 10:25:43PM +0800, Anson Huang wrote:
> The imx6ul iomuxc syscon is compatible to imx6q,
> so let's add compatible string 'fsl,imx6q-iomuxc-gpr'
> for imx6ul iomuxc syscon node.
>
> Signed-off-by: Anson Huang <Anson.Huang-3arQi8VN3Tc@public.gmane.org>
Will this patch still be needed if you implement imx6ul suspend support
with PSCI?
Shawn
> ---
> changes since V1:
> improve commit message.
> arch/arm/boot/dts/imx6ul.dtsi | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/imx6ul.dtsi b/arch/arm/boot/dts/imx6ul.dtsi
> index 33b95d7..acc9486 100644
> --- a/arch/arm/boot/dts/imx6ul.dtsi
> +++ b/arch/arm/boot/dts/imx6ul.dtsi
> @@ -644,7 +644,8 @@
> };
>
> gpr: iomuxc-gpr@020e4000 {
> - compatible = "fsl,imx6ul-iomuxc-gpr", "syscon";
> + compatible = "fsl,imx6ul-iomuxc-gpr",
> + "fsl,imx6q-iomuxc-gpr", "syscon";
> reg = <0x020e4000 0x4000>;
> };
>
> --
> 1.9.1
>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 12+ messages in thread* [PATCH V2] ARM: dts: imx6ul iomuxc syscon is compatible to imx6q
2016-08-29 14:34 ` Shawn Guo
(?)
@ 2016-08-29 14:44 ` Yongcai Huang
-1 siblings, 0 replies; 12+ messages in thread
From: Yongcai Huang @ 2016-08-29 14:44 UTC (permalink / raw)
To: linux-arm-kernel
Best Regards!
Anson Huang
> -----Original Message-----
> From: Shawn Guo [mailto:shawnguo at kernel.org]
> Sent: 2016-08-29 10:35 PM
> To: Yongcai Huang <anson.huang@nxp.com>
> Cc: linux-arm-kernel at lists.infradead.org; linux-kernel at vger.kernel.org;
> devicetree at vger.kernel.org; mark.rutland at arm.com; robh+dt at kernel.org;
> kernel at pengutronix.de; Fabio Estevam <fabio.estevam@nxp.com>;
> linux at armlinux.org.uk
> Subject: Re: [PATCH V2] ARM: dts: imx6ul iomuxc syscon is compatible to imx6q
>
> On Mon, Aug 29, 2016 at 10:25:43PM +0800, Anson Huang wrote:
> > The imx6ul iomuxc syscon is compatible to imx6q, so let's add
> > compatible string 'fsl,imx6q-iomuxc-gpr'
> > for imx6ul iomuxc syscon node.
> >
> > Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
>
> Will this patch still be needed if you implement imx6ul suspend support with
> PSCI?
>
> Shawn
This patch is to set the GINT bit for all low power mode transition.
Do we decide to go with PSCI for all i.MX6 SoCs? Currently i.MX6UL's suspend is
already supported with non-PCSI method, only i.MX7D uses PSCI, right? If we decide
to implement all power management feature with PSCI, then yes, all these changes
can be put into PSCI. But since current suspend/cpuidle function are already supported
using old method, this bit must be set to avoid system goes into low power mode
unexpected.
Regards!
Anson.
>
> > ---
> > changes since V1:
> > improve commit message.
> > arch/arm/boot/dts/imx6ul.dtsi | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/arch/arm/boot/dts/imx6ul.dtsi
> > b/arch/arm/boot/dts/imx6ul.dtsi index 33b95d7..acc9486 100644
> > --- a/arch/arm/boot/dts/imx6ul.dtsi
> > +++ b/arch/arm/boot/dts/imx6ul.dtsi
> > @@ -644,7 +644,8 @@
> > };
> >
> > gpr: iomuxc-gpr at 020e4000 {
> > - compatible = "fsl,imx6ul-iomuxc-gpr", "syscon";
> > + compatible = "fsl,imx6ul-iomuxc-gpr",
> > + "fsl,imx6q-iomuxc-gpr", "syscon";
> > reg = <0x020e4000 0x4000>;
> > };
> >
> > --
> > 1.9.1
> >
^ permalink raw reply [flat|nested] 12+ messages in thread* RE: [PATCH V2] ARM: dts: imx6ul iomuxc syscon is compatible to imx6q
@ 2016-08-29 14:44 ` Yongcai Huang
0 siblings, 0 replies; 12+ messages in thread
From: Yongcai Huang @ 2016-08-29 14:44 UTC (permalink / raw)
To: Shawn Guo
Cc: linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
mark.rutland@arm.com, robh+dt@kernel.org, kernel@pengutronix.de,
Fabio Estevam, linux@armlinux.org.uk
Best Regards!
Anson Huang
> -----Original Message-----
> From: Shawn Guo [mailto:shawnguo@kernel.org]
> Sent: 2016-08-29 10:35 PM
> To: Yongcai Huang <anson.huang@nxp.com>
> Cc: linux-arm-kernel@lists.infradead.org; linux-kernel@vger.kernel.org;
> devicetree@vger.kernel.org; mark.rutland@arm.com; robh+dt@kernel.org;
> kernel@pengutronix.de; Fabio Estevam <fabio.estevam@nxp.com>;
> linux@armlinux.org.uk
> Subject: Re: [PATCH V2] ARM: dts: imx6ul iomuxc syscon is compatible to imx6q
>
> On Mon, Aug 29, 2016 at 10:25:43PM +0800, Anson Huang wrote:
> > The imx6ul iomuxc syscon is compatible to imx6q, so let's add
> > compatible string 'fsl,imx6q-iomuxc-gpr'
> > for imx6ul iomuxc syscon node.
> >
> > Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
>
> Will this patch still be needed if you implement imx6ul suspend support with
> PSCI?
>
> Shawn
This patch is to set the GINT bit for all low power mode transition.
Do we decide to go with PSCI for all i.MX6 SoCs? Currently i.MX6UL's suspend is
already supported with non-PCSI method, only i.MX7D uses PSCI, right? If we decide
to implement all power management feature with PSCI, then yes, all these changes
can be put into PSCI. But since current suspend/cpuidle function are already supported
using old method, this bit must be set to avoid system goes into low power mode
unexpected.
Regards!
Anson.
>
> > ---
> > changes since V1:
> > improve commit message.
> > arch/arm/boot/dts/imx6ul.dtsi | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/arch/arm/boot/dts/imx6ul.dtsi
> > b/arch/arm/boot/dts/imx6ul.dtsi index 33b95d7..acc9486 100644
> > --- a/arch/arm/boot/dts/imx6ul.dtsi
> > +++ b/arch/arm/boot/dts/imx6ul.dtsi
> > @@ -644,7 +644,8 @@
> > };
> >
> > gpr: iomuxc-gpr@020e4000 {
> > - compatible = "fsl,imx6ul-iomuxc-gpr", "syscon";
> > + compatible = "fsl,imx6ul-iomuxc-gpr",
> > + "fsl,imx6q-iomuxc-gpr", "syscon";
> > reg = <0x020e4000 0x4000>;
> > };
> >
> > --
> > 1.9.1
> >
^ permalink raw reply [flat|nested] 12+ messages in thread* RE: [PATCH V2] ARM: dts: imx6ul iomuxc syscon is compatible to imx6q
@ 2016-08-29 14:44 ` Yongcai Huang
0 siblings, 0 replies; 12+ messages in thread
From: Yongcai Huang @ 2016-08-29 14:44 UTC (permalink / raw)
To: Shawn Guo
Cc: mark.rutland@arm.com, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, linux@armlinux.org.uk,
robh+dt@kernel.org, kernel@pengutronix.de, Fabio Estevam,
linux-arm-kernel@lists.infradead.org
Best Regards!
Anson Huang
> -----Original Message-----
> From: Shawn Guo [mailto:shawnguo@kernel.org]
> Sent: 2016-08-29 10:35 PM
> To: Yongcai Huang <anson.huang@nxp.com>
> Cc: linux-arm-kernel@lists.infradead.org; linux-kernel@vger.kernel.org;
> devicetree@vger.kernel.org; mark.rutland@arm.com; robh+dt@kernel.org;
> kernel@pengutronix.de; Fabio Estevam <fabio.estevam@nxp.com>;
> linux@armlinux.org.uk
> Subject: Re: [PATCH V2] ARM: dts: imx6ul iomuxc syscon is compatible to imx6q
>
> On Mon, Aug 29, 2016 at 10:25:43PM +0800, Anson Huang wrote:
> > The imx6ul iomuxc syscon is compatible to imx6q, so let's add
> > compatible string 'fsl,imx6q-iomuxc-gpr'
> > for imx6ul iomuxc syscon node.
> >
> > Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
>
> Will this patch still be needed if you implement imx6ul suspend support with
> PSCI?
>
> Shawn
This patch is to set the GINT bit for all low power mode transition.
Do we decide to go with PSCI for all i.MX6 SoCs? Currently i.MX6UL's suspend is
already supported with non-PCSI method, only i.MX7D uses PSCI, right? If we decide
to implement all power management feature with PSCI, then yes, all these changes
can be put into PSCI. But since current suspend/cpuidle function are already supported
using old method, this bit must be set to avoid system goes into low power mode
unexpected.
Regards!
Anson.
>
> > ---
> > changes since V1:
> > improve commit message.
> > arch/arm/boot/dts/imx6ul.dtsi | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/arch/arm/boot/dts/imx6ul.dtsi
> > b/arch/arm/boot/dts/imx6ul.dtsi index 33b95d7..acc9486 100644
> > --- a/arch/arm/boot/dts/imx6ul.dtsi
> > +++ b/arch/arm/boot/dts/imx6ul.dtsi
> > @@ -644,7 +644,8 @@
> > };
> >
> > gpr: iomuxc-gpr@020e4000 {
> > - compatible = "fsl,imx6ul-iomuxc-gpr", "syscon";
> > + compatible = "fsl,imx6ul-iomuxc-gpr",
> > + "fsl,imx6q-iomuxc-gpr", "syscon";
> > reg = <0x020e4000 0x4000>;
> > };
> >
> > --
> > 1.9.1
> >
^ permalink raw reply [flat|nested] 12+ messages in thread* [PATCH V2] ARM: dts: imx6ul iomuxc syscon is compatible to imx6q
@ 2016-08-29 15:20 ` Shawn Guo
0 siblings, 0 replies; 12+ messages in thread
From: Shawn Guo @ 2016-08-29 15:20 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Aug 29, 2016 at 02:44:47PM +0000, Yongcai Huang wrote:
> This patch is to set the GINT bit for all low power mode transition.
>
> Do we decide to go with PSCI for all i.MX6 SoCs? Currently i.MX6UL's suspend is
> already supported with non-PCSI method, only i.MX7D uses PSCI, right? If we decide
> to implement all power management feature with PSCI, then yes, all these changes
> can be put into PSCI. But since current suspend/cpuidle function are already supported
> using old method, this bit must be set to avoid system goes into low power mode
> unexpected.
My apology. I messed up it with i.MX7D. Patch applied, thanks.
Shawn
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH V2] ARM: dts: imx6ul iomuxc syscon is compatible to imx6q
@ 2016-08-29 15:20 ` Shawn Guo
0 siblings, 0 replies; 12+ messages in thread
From: Shawn Guo @ 2016-08-29 15:20 UTC (permalink / raw)
To: Yongcai Huang
Cc: mark.rutland@arm.com, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, linux@armlinux.org.uk,
robh+dt@kernel.org, kernel@pengutronix.de, Fabio Estevam,
linux-arm-kernel@lists.infradead.org
On Mon, Aug 29, 2016 at 02:44:47PM +0000, Yongcai Huang wrote:
> This patch is to set the GINT bit for all low power mode transition.
>
> Do we decide to go with PSCI for all i.MX6 SoCs? Currently i.MX6UL's suspend is
> already supported with non-PCSI method, only i.MX7D uses PSCI, right? If we decide
> to implement all power management feature with PSCI, then yes, all these changes
> can be put into PSCI. But since current suspend/cpuidle function are already supported
> using old method, this bit must be set to avoid system goes into low power mode
> unexpected.
My apology. I messed up it with i.MX7D. Patch applied, thanks.
Shawn
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH V2] ARM: dts: imx6ul iomuxc syscon is compatible to imx6q
@ 2016-08-29 15:20 ` Shawn Guo
0 siblings, 0 replies; 12+ messages in thread
From: Shawn Guo @ 2016-08-29 15:20 UTC (permalink / raw)
To: Yongcai Huang
Cc: mark.rutland-5wv7dgnIgG8@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-I+IVW8TIWO2tmTQ+vhA3Yw@public.gmane.org,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
kernel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org, Fabio Estevam,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
On Mon, Aug 29, 2016 at 02:44:47PM +0000, Yongcai Huang wrote:
> This patch is to set the GINT bit for all low power mode transition.
>
> Do we decide to go with PSCI for all i.MX6 SoCs? Currently i.MX6UL's suspend is
> already supported with non-PCSI method, only i.MX7D uses PSCI, right? If we decide
> to implement all power management feature with PSCI, then yes, all these changes
> can be put into PSCI. But since current suspend/cpuidle function are already supported
> using old method, this bit must be set to avoid system goes into low power mode
> unexpected.
My apology. I messed up it with i.MX7D. Patch applied, thanks.
Shawn
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 12+ messages in thread