2018-10-23 05:31:19

by Joakim Zhang

Subject: [PATCH V1 0/4] ARM: dts: imx: Add flexcan stop mode wakeup

This patchset intends to add flexcan stop mode wakeup support on IMX6/7
platporms.

Dong Aisheng (4):
ARM: dts: imx6sx: Add flexcan stop mode wakeup support
ARM: dts: imx6qdl: Add flexcan stop mode wakeup support
ARM: dts: imx6ul: Add flexcan stop mode wakeup support
ARM: dts: imx7s: Add flexcan stop mode wakeup support

arch/arm/boot/dts/imx6qdl.dtsi | 2 ++
arch/arm/boot/dts/imx6sx.dtsi | 2 ++
arch/arm/boot/dts/imx6ul.dtsi | 2 ++
arch/arm/boot/dts/imx7s.dtsi | 2 ++
4 files changed, 8 insertions(+)

--
2.17.1

2018-10-23 05:31:24

by Joakim Zhang

Subject: [PATCH V1 1/4] ARM: dts: imx6sx: Add flexcan stop mode wakeup support

From: Dong Aisheng <[email protected]>

Add stop-mode property which is required by stop mode wakeup
feature.

Signed-off-by: Dong Aisheng <[email protected]>
Signed-off-by: Joakim Zhang <[email protected]>
---
arch/arm/boot/dts/imx6sx.dtsi | 2 ++
1 file changed, 2 insertions(+)

diff --git a/arch/arm/boot/dts/imx6sx.dtsi b/arch/arm/boot/dts/imx6sx.dtsi
index 844caa39364f..3f459d6da7fc 100644
--- a/arch/arm/boot/dts/imx6sx.dtsi
+++ b/arch/arm/boot/dts/imx6sx.dtsi
@@ -448,6 +448,7 @@
clocks = <&clks IMX6SX_CLK_CAN1_IPG>,
<&clks IMX6SX_CLK_CAN1_SERIAL>;
clock-names = "ipg", "per";
+ stop-mode = <&gpr 0x10 1 0x10 17>;
status = "disabled";
};

@@ -458,6 +459,7 @@
clocks = <&clks IMX6SX_CLK_CAN2_IPG>,
<&clks IMX6SX_CLK_CAN2_SERIAL>;
clock-names = "ipg", "per";
+ stop-mode = <&gpr 0x10 2 0x10 18>;
status = "disabled";
};

--
2.17.1

2018-10-23 05:31:49

by Joakim Zhang

Subject: [PATCH V1 2/4] ARM: dts: imx6qdl: Add flexcan stop mode wakeup support

From: Dong Aisheng <[email protected]>

Add stop-mode property which is required by stop mode wakeup
feature.

Signed-off-by: Dong Aisheng <[email protected]>
Signed-off-by: Joakim Zhang <[email protected]>
---
arch/arm/boot/dts/imx6qdl.dtsi | 2 ++
1 file changed, 2 insertions(+)

diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi
index 61d2d26afbf4..eb84ebcbe358 100644
--- a/arch/arm/boot/dts/imx6qdl.dtsi
+++ b/arch/arm/boot/dts/imx6qdl.dtsi
@@ -548,6 +548,7 @@
clocks = <&clks IMX6QDL_CLK_CAN1_IPG>,
<&clks IMX6QDL_CLK_CAN1_SERIAL>;
clock-names = "ipg", "per";
+ stop-mode = <&gpr 0x34 28 0x10 17>;
status = "disabled";
};

@@ -558,6 +559,7 @@
clocks = <&clks IMX6QDL_CLK_CAN2_IPG>,
<&clks IMX6QDL_CLK_CAN2_SERIAL>;
clock-names = "ipg", "per";
+ stop-mode = <&gpr 0x34 29 0x10 18>;
status = "disabled";
};

--
2.17.1

2018-10-23 05:32:32

by Joakim Zhang

Subject: [PATCH V1 3/4] ARM: dts: imx6ul: Add flexcan stop mode wakeup support

From: Dong Aisheng <[email protected]>

Add stop-mode property which is required by stop mode wakeup
feature.

Signed-off-by: Dong Aisheng <[email protected]>
Signed-off-by: Joakim Zhang <[email protected]>
---
arch/arm/boot/dts/imx6ul.dtsi | 2 ++
1 file changed, 2 insertions(+)

diff --git a/arch/arm/boot/dts/imx6ul.dtsi b/arch/arm/boot/dts/imx6ul.dtsi
index 6dc0b569acdf..76486a42959c 100644
--- a/arch/arm/boot/dts/imx6ul.dtsi
+++ b/arch/arm/boot/dts/imx6ul.dtsi
@@ -407,6 +407,7 @@
clocks = <&clks IMX6UL_CLK_CAN1_IPG>,
<&clks IMX6UL_CLK_CAN1_SERIAL>;
clock-names = "ipg", "per";
+ stop-mode = <&gpr 0x10 1 0x10 17>;
status = "disabled";
};

@@ -417,6 +418,7 @@
clocks = <&clks IMX6UL_CLK_CAN2_IPG>,
<&clks IMX6UL_CLK_CAN2_SERIAL>;
clock-names = "ipg", "per";
+ stop-mode = <&gpr 0x10 2 0x10 18>;
status = "disabled";
};

--
2.17.1

2018-10-23 05:33:59

by Joakim Zhang

Subject: [PATCH V1 4/4] ARM: dts: imx7s: Add flexcan stop mode wakeup support

From: Dong Aisheng <[email protected]>

Add stop-mode property which is required by stop mode wakeup
feature.

Signed-off-by: Dong Aisheng <[email protected]>
Signed-off-by: Joakim Zhang <[email protected]>
---
arch/arm/boot/dts/imx7s.dtsi | 2 ++
1 file changed, 2 insertions(+)

diff --git a/arch/arm/boot/dts/imx7s.dtsi b/arch/arm/boot/dts/imx7s.dtsi
index a052198f6e96..382f1e1a92e3 100644
--- a/arch/arm/boot/dts/imx7s.dtsi
+++ b/arch/arm/boot/dts/imx7s.dtsi
@@ -877,6 +877,7 @@
clocks = <&clks IMX7D_CLK_DUMMY>,
<&clks IMX7D_CAN1_ROOT_CLK>;
clock-names = "ipg", "per";
+ stop-mode = <&gpr 0x10 1 0x10 17>;
status = "disabled";
};

@@ -887,6 +888,7 @@
clocks = <&clks IMX7D_CLK_DUMMY>,
<&clks IMX7D_CAN2_ROOT_CLK>;
clock-names = "ipg", "per";
+ stop-mode = <&gpr 0x10 2 0x10 18>;
status = "disabled";
};

--
2.17.1

2018-11-04 07:48:08

by Shawn Guo

Subject: Re: [PATCH V1 1/4] ARM: dts: imx6sx: Add flexcan stop mode wakeup support

On Tue, Oct 23, 2018 at 05:30:44AM +0000, Joakim Zhang wrote:
> From: Dong Aisheng <[email protected]>
>
> Add stop-mode property which is required by stop mode wakeup
> feature.
>
> Signed-off-by: Dong Aisheng <[email protected]>
> Signed-off-by: Joakim Zhang <[email protected]>
> ---
> arch/arm/boot/dts/imx6sx.dtsi | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/arch/arm/boot/dts/imx6sx.dtsi b/arch/arm/boot/dts/imx6sx.dtsi
> index 844caa39364f..3f459d6da7fc 100644
> --- a/arch/arm/boot/dts/imx6sx.dtsi
> +++ b/arch/arm/boot/dts/imx6sx.dtsi
> @@ -448,6 +448,7 @@
> clocks = <&clks IMX6SX_CLK_CAN1_IPG>,
> <&clks IMX6SX_CLK_CAN1_SERIAL>;
> clock-names = "ipg", "per";
> + stop-mode = <&gpr 0x10 1 0x10 17>;

Is it supported by upstream? I cannot find this property in ether flexcan
bindings doc or kernel driver.

Shawn

> status = "disabled";
> };
>
> @@ -458,6 +459,7 @@
> clocks = <&clks IMX6SX_CLK_CAN2_IPG>,
> <&clks IMX6SX_CLK_CAN2_SERIAL>;
> clock-names = "ipg", "per";
> + stop-mode = <&gpr 0x10 2 0x10 18>;
> status = "disabled";
> };
>
> --
> 2.17.1
>

2018-11-12 01:59:59

by Joakim Zhang

Subject: RE: [PATCH V1 1/4] ARM: dts: imx6sx: Add flexcan stop mode wakeup support

> -----Original Message-----
> From: Shawn Guo <[email protected]>
> Sent: 2018??11??4?? 15:40
> To: Joakim Zhang <[email protected]>
> Cc: [email protected]; [email protected]; Fabio Estevam
> <[email protected]>; dl-linux-imx <[email protected]>;
> [email protected]; [email protected];
> [email protected]; [email protected]; A.s. Dong
> <[email protected]>
> Subject: Re: [PATCH V1 1/4] ARM: dts: imx6sx: Add flexcan stop mode wakeup
> support
>
> On Tue, Oct 23, 2018 at 05:30:44AM +0000, Joakim Zhang wrote:
> > From: Dong Aisheng <[email protected]>
> >
> > Add stop-mode property which is required by stop mode wakeup feature.
> >
> > Signed-off-by: Dong Aisheng <[email protected]>
> > Signed-off-by: Joakim Zhang <[email protected]>
> > ---
> > arch/arm/boot/dts/imx6sx.dtsi | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/arch/arm/boot/dts/imx6sx.dtsi
> > b/arch/arm/boot/dts/imx6sx.dtsi index 844caa39364f..3f459d6da7fc
> > 100644
> > --- a/arch/arm/boot/dts/imx6sx.dtsi
> > +++ b/arch/arm/boot/dts/imx6sx.dtsi
> > @@ -448,6 +448,7 @@
> > clocks = <&clks IMX6SX_CLK_CAN1_IPG>,
> > <&clks IMX6SX_CLK_CAN1_SERIAL>;
> > clock-names = "ipg", "per";
> > + stop-mode = <&gpr 0x10 1 0x10 17>;
>
> Is it supported by upstream? I cannot find this property in ether flexcan
> bindings doc or kernel driver.
>
> Shawn

Hi Shawn,
This property has been reviewed by maintainer in flexcan bindings and kernel driver has sent to review.

Best Regards,
Joakim Zhang

> > status = "disabled";
> > };
> >
> > @@ -458,6 +459,7 @@
> > clocks = <&clks IMX6SX_CLK_CAN2_IPG>,
> > <&clks IMX6SX_CLK_CAN2_SERIAL>;
> > clock-names = "ipg", "per";
> > + stop-mode = <&gpr 0x10 2 0x10 18>;
> > status = "disabled";
> > };
> >
> > --
> > 2.17.1
> >