diff mbox series

arm64: dts: fix endianness of rcpm

Message ID 20200928090455.34364-1-ran.wang_1@nxp.com (mailing list archive)
State New, archived
Headers show
Series arm64: dts: fix endianness of rcpm | expand

Commit Message

Ran Wang Sept. 28, 2020, 9:04 a.m. UTC 
From: Biwen Li <biwen.li@nxp.com>

Add little-endian property of rcpm for ls1028a,ls1088a,ls208xa

Signed-off-by: Biwen Li <biwen.li@nxp.com>
---
 arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi | 1 +
 arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi | 1 +
 arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi | 1 +
 3 files changed, 3 insertions(+)

Comments

Leo Li Sept. 28, 2020, 4:54 p.m. UTC | #1 
> -----Original Message-----
> From: Ran Wang <ran.wang_1@nxp.com>
> Sent: Monday, September 28, 2020 4:05 AM
> To: Shawn Guo <shawnguo@kernel.org>; Leo Li <leoyang.li@nxp.com>
> Cc: moderated list:ARM/FREESCALE LAYERSCAPE ARM ARCHITECTURE <linux-
> arm-kernel@lists.infradead.org>; open list:OPEN FIRMWARE AND
> FLATTENED DEVICE TREE BINDINGS <devicetree@vger.kernel.org>; open list
> <linux-kernel@vger.kernel.org>; Biwen Li <biwen.li@nxp.com>
> Subject: [PATCH] arm64: dts: fix endianness of rcpm
> 
> From: Biwen Li <biwen.li@nxp.com>
> 
> Add little-endian property of rcpm for ls1028a,ls1088a,ls208xa
> 
> Signed-off-by: Biwen Li <biwen.li@nxp.com>

Acked-by: Li Yang <leoyang.li@nxp.com>

> ---
>  arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi | 1 +
> arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi | 1 +
> arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi | 1 +
>  3 files changed, 3 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
> b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
> index 0efeb8f..651bfe1 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
> @@ -1012,6 +1012,7 @@
>  			compatible = "fsl,ls1028a-rcpm", "fsl,qoriq-rcpm-
> 2.1+";
>  			reg = <0x0 0x1e34040 0x0 0x1c>;
>  			#fsl,rcpm-wakeup-cells = <7>;
> +			little-endian;
>  		};
> 
>  		ftm_alarm0: timer@2800000 {
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi
> b/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi
> index 169f474..2ef812d 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi
> @@ -787,6 +787,7 @@
>  			compatible = "fsl,ls1088a-rcpm", "fsl,qoriq-rcpm-
> 2.1+";
>  			reg = <0x0 0x1e34040 0x0 0x18>;
>  			#fsl,rcpm-wakeup-cells = <6>;
> +			little-endian;
>  		};
> 
>  		ftm_alarm0: timer@2800000 {
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
> b/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
> index 41102da..141b3d2 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
> @@ -769,6 +769,7 @@
>  			compatible = "fsl,ls208xa-rcpm", "fsl,qoriq-rcpm-
> 2.1+";
>  			reg = <0x0 0x1e34040 0x0 0x18>;
>  			#fsl,rcpm-wakeup-cells = <6>;
> +			little-endian;
>  		};
> 
>  		ftm_alarm0: timer@2800000 {
> --
> 2.7.4
Fabio Estevam Sept. 28, 2020, 7:20 p.m. UTC | #2 
Hi Ran,

On Mon, Sep 28, 2020 at 6:14 AM Ran Wang <ran.wang_1@nxp.com> wrote:
>
> From: Biwen Li <biwen.li@nxp.com>
>
> Add little-endian property of rcpm for ls1028a,ls1088a,ls208xa
>
> Signed-off-by: Biwen Li <biwen.li@nxp.com>

You missed your Signed-off-by tag.

What about adding a Fixes tag?
Ran Wang Sept. 29, 2020, 12:38 a.m. UTC | #3 
Hi Fabio,

On Tuesday, September 29, 2020 3:20 AM, Fabio Estevam wrote:
> 
> Hi Ran,
> 
> On Mon, Sep 28, 2020 at 6:14 AM Ran Wang <ran.wang_1@nxp.com> wrote:
> >
> > From: Biwen Li <biwen.li@nxp.com>
> >
> > Add little-endian property of rcpm for ls1028a,ls1088a,ls208xa
> >
> > Signed-off-by: Biwen Li <biwen.li@nxp.com>
> 
> You missed your Signed-off-by tag.

Oh, thanks for pointing out, will update in next version.
 
> What about adding a Fixes tag?

Sure, will add it.

Regards,
Ran
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
index 0efeb8f..651bfe1 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
@@ -1012,6 +1012,7 @@ 
 			compatible = "fsl,ls1028a-rcpm", "fsl,qoriq-rcpm-2.1+";
 			reg = <0x0 0x1e34040 0x0 0x1c>;
 			#fsl,rcpm-wakeup-cells = <7>;
+			little-endian;
 		};
 
 		ftm_alarm0: timer@2800000 {
diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi
index 169f474..2ef812d 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi
@@ -787,6 +787,7 @@ 
 			compatible = "fsl,ls1088a-rcpm", "fsl,qoriq-rcpm-2.1+";
 			reg = <0x0 0x1e34040 0x0 0x18>;
 			#fsl,rcpm-wakeup-cells = <6>;
+			little-endian;
 		};
 
 		ftm_alarm0: timer@2800000 {
diff --git a/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
index 41102da..141b3d2 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
@@ -769,6 +769,7 @@ 
 			compatible = "fsl,ls208xa-rcpm", "fsl,qoriq-rcpm-2.1+";
 			reg = <0x0 0x1e34040 0x0 0x18>;
 			#fsl,rcpm-wakeup-cells = <6>;
+			little-endian;
 		};
 
 		ftm_alarm0: timer@2800000 {