* [PATCH 1/5] dt-bindings: soc: imx: add IOMUXC GPR support
2022-12-23 2:56 [PATCH 0/5] binding: imx8m: add IOMUXC GPR Peng Fan (OSS)
@ 2022-12-23 2:56 ` Peng Fan (OSS)
2022-12-26 18:49 ` Rob Herring
2022-12-23 2:57 ` [PATCH 2/5] arm64: dts: imx8mq: correct iomuxc-gpr compatible Peng Fan (OSS)
` (3 subsequent siblings)
4 siblings, 1 reply; 11+ messages in thread
From: Peng Fan (OSS) @ 2022-12-23 2:56 UTC (permalink / raw)
To: robh+dt, krzysztof.kozlowski+dt, shawnguo, s.hauer
Cc: kernel, festevam, linux-imx, devicetree, linux-kernel,
linux-arm-kernel, Peng Fan
From: Peng Fan <peng.fan@nxp.com>
Add binding doc for i.MX IOMUX Controller General Purpose Registers
Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
.../bindings/soc/imx/fsl,imx-iomuxc-gpr.yaml | 57 +++++++++++++++++++
1 file changed, 57 insertions(+)
create mode 100644 Documentation/devicetree/bindings/soc/imx/fsl,imx-iomuxc-gpr.yaml
diff --git a/Documentation/devicetree/bindings/soc/imx/fsl,imx-iomuxc-gpr.yaml b/Documentation/devicetree/bindings/soc/imx/fsl,imx-iomuxc-gpr.yaml
new file mode 100644
index 000000000000..1da1b758b4ae
--- /dev/null
+++ b/Documentation/devicetree/bindings/soc/imx/fsl,imx-iomuxc-gpr.yaml
@@ -0,0 +1,57 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/soc/imx/fsl,imx-iomuxc-gpr.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Freescale IOMUX Controller General Purpose Registers
+
+maintainers:
+ - Peng Fan <peng.fan@nxp.com>
+
+description:
+ i.MX Processors have an IOMUXC General Purpose Register group for
+ various System Settings
+
+properties:
+ compatible:
+ oneOf:
+ - items:
+ - const: fsl,imx8mq-iomuxc-gpr
+ - const: syscon
+ - const: simple-mfd
+ - items:
+ - enum:
+ - fsl,imx8mm-iomuxc-gpr
+ - fsl,imx8mn-iomuxc-gpr
+ - fsl,imx8mp-iomuxc-gpr
+ - const: syscon
+
+ reg:
+ maxItems: 1
+
+ mux-controller:
+ type: object
+ $ref: /schemas/mux/reg-mux.yaml
+
+additionalProperties: false
+
+required:
+ - compatible
+ - reg
+
+examples:
+ # Pinmux controller node
+ - |
+ iomuxc_gpr: syscon@30340000 {
+ compatible = "fsl,imx8mq-iomuxc-gpr", "syscon", "simple-mfd";
+ reg = <0x30340000 0x10000>;
+
+ mux: mux-controller {
+ compatible = "mmio-mux";
+ #mux-control-cells = <1>;
+ mux-reg-masks = <0x34 0x00000004>; /* MIPI_MUX_SEL */
+ };
+ };
+
+...
--
2.37.1
^ permalink raw reply related [flat|nested] 11+ messages in thread* Re: [PATCH 1/5] dt-bindings: soc: imx: add IOMUXC GPR support
2022-12-23 2:56 ` [PATCH 1/5] dt-bindings: soc: imx: add IOMUXC GPR support Peng Fan (OSS)
@ 2022-12-26 18:49 ` Rob Herring
0 siblings, 0 replies; 11+ messages in thread
From: Rob Herring @ 2022-12-26 18:49 UTC (permalink / raw)
To: Peng Fan (OSS)
Cc: s.hauer, linux-imx, Peng Fan, krzysztof.kozlowski+dt, festevam,
shawnguo, robh+dt, linux-kernel, kernel, devicetree,
linux-arm-kernel
On Fri, 23 Dec 2022 10:56:59 +0800, Peng Fan (OSS) wrote:
> From: Peng Fan <peng.fan@nxp.com>
>
> Add binding doc for i.MX IOMUX Controller General Purpose Registers
>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> ---
> .../bindings/soc/imx/fsl,imx-iomuxc-gpr.yaml | 57 +++++++++++++++++++
> 1 file changed, 57 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/soc/imx/fsl,imx-iomuxc-gpr.yaml
>
Reviewed-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH 2/5] arm64: dts: imx8mq: correct iomuxc-gpr compatible
2022-12-23 2:56 [PATCH 0/5] binding: imx8m: add IOMUXC GPR Peng Fan (OSS)
2022-12-23 2:56 ` [PATCH 1/5] dt-bindings: soc: imx: add IOMUXC GPR support Peng Fan (OSS)
@ 2022-12-23 2:57 ` Peng Fan (OSS)
2022-12-23 8:48 ` Marco Felsch
2022-12-23 2:57 ` [PATCH 3/5] arm64: dts: imx8mm: " Peng Fan (OSS)
` (2 subsequent siblings)
4 siblings, 1 reply; 11+ messages in thread
From: Peng Fan (OSS) @ 2022-12-23 2:57 UTC (permalink / raw)
To: robh+dt, krzysztof.kozlowski+dt, shawnguo, s.hauer
Cc: kernel, festevam, linux-imx, devicetree, linux-kernel,
linux-arm-kernel, Peng Fan
From: Peng Fan <peng.fan@nxp.com>
The IOMUX Controller General purpose register group are unique almost
per SoC, i.MX8MQ is not compatible with i.MX6Q. So correct it.
Also update name with syscon.
Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
arch/arm64/boot/dts/freescale/imx8mq.dtsi | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
index 7ce99c084e54..6971c47d11fa 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
@@ -575,8 +575,7 @@ iomuxc: pinctrl@30330000 {
};
iomuxc_gpr: syscon@30340000 {
- compatible = "fsl,imx8mq-iomuxc-gpr", "fsl,imx6q-iomuxc-gpr",
- "syscon", "simple-mfd";
+ compatible = "fsl,imx8mq-iomuxc-gpr", "syscon", "simple-mfd";
reg = <0x30340000 0x10000>;
mux: mux-controller {
--
2.37.1
^ permalink raw reply related [flat|nested] 11+ messages in thread* Re: [PATCH 2/5] arm64: dts: imx8mq: correct iomuxc-gpr compatible
2022-12-23 2:57 ` [PATCH 2/5] arm64: dts: imx8mq: correct iomuxc-gpr compatible Peng Fan (OSS)
@ 2022-12-23 8:48 ` Marco Felsch
0 siblings, 0 replies; 11+ messages in thread
From: Marco Felsch @ 2022-12-23 8:48 UTC (permalink / raw)
To: Peng Fan (OSS)
Cc: robh+dt, krzysztof.kozlowski+dt, shawnguo, s.hauer, devicetree,
Peng Fan, linux-kernel, linux-imx, kernel, festevam,
linux-arm-kernel
On 22-12-23, Peng Fan (OSS) wrote:
> From: Peng Fan <peng.fan@nxp.com>
>
> The IOMUX Controller General purpose register group are unique almost
> per SoC, i.MX8MQ is not compatible with i.MX6Q. So correct it.
> Also update name with syscon.
>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
Reviewed-by: Marco Felsch <m.felsch@pengutronix.de>
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH 3/5] arm64: dts: imx8mm: correct iomuxc-gpr compatible
2022-12-23 2:56 [PATCH 0/5] binding: imx8m: add IOMUXC GPR Peng Fan (OSS)
2022-12-23 2:56 ` [PATCH 1/5] dt-bindings: soc: imx: add IOMUXC GPR support Peng Fan (OSS)
2022-12-23 2:57 ` [PATCH 2/5] arm64: dts: imx8mq: correct iomuxc-gpr compatible Peng Fan (OSS)
@ 2022-12-23 2:57 ` Peng Fan (OSS)
2022-12-23 8:47 ` Marco Felsch
2022-12-23 2:57 ` [PATCH 4/5] arm64: dts: imx8mn: update iomuxc-gpr node name Peng Fan (OSS)
2022-12-23 2:57 ` [PATCH 5/5] arm64: dts: imx8mp: use syscon for iomuxc-gpr Peng Fan (OSS)
4 siblings, 1 reply; 11+ messages in thread
From: Peng Fan (OSS) @ 2022-12-23 2:57 UTC (permalink / raw)
To: robh+dt, krzysztof.kozlowski+dt, shawnguo, s.hauer
Cc: kernel, festevam, linux-imx, devicetree, linux-kernel,
linux-arm-kernel, Peng Fan
From: Peng Fan <peng.fan@nxp.com>
The IOMUX Controller General purpose register group are unique almost
per SoC, i.MX8MM is not compatible with i.MX6Q. So correct it.
Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
arch/arm64/boot/dts/freescale/imx8mm.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi
index 4ee89fdcf59b..6bcc170d7b84 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi
@@ -551,7 +551,7 @@ iomuxc: pinctrl@30330000 {
};
gpr: iomuxc-gpr@30340000 {
- compatible = "fsl,imx8mm-iomuxc-gpr", "fsl,imx6q-iomuxc-gpr", "syscon";
+ compatible = "fsl,imx8mm-iomuxc-gpr", "syscon";
reg = <0x30340000 0x10000>;
};
--
2.37.1
^ permalink raw reply related [flat|nested] 11+ messages in thread* Re: [PATCH 3/5] arm64: dts: imx8mm: correct iomuxc-gpr compatible
2022-12-23 2:57 ` [PATCH 3/5] arm64: dts: imx8mm: " Peng Fan (OSS)
@ 2022-12-23 8:47 ` Marco Felsch
0 siblings, 0 replies; 11+ messages in thread
From: Marco Felsch @ 2022-12-23 8:47 UTC (permalink / raw)
To: Peng Fan (OSS)
Cc: robh+dt, krzysztof.kozlowski+dt, shawnguo, s.hauer, devicetree,
Peng Fan, linux-kernel, linux-imx, kernel, festevam,
linux-arm-kernel
Hi Peng,
On 22-12-23, Peng Fan (OSS) wrote:
> From: Peng Fan <peng.fan@nxp.com>
>
> The IOMUX Controller General purpose register group are unique almost
> per SoC, i.MX8MM is not compatible with i.MX6Q. So correct it.
>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> ---
> arch/arm64/boot/dts/freescale/imx8mm.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi
> index 4ee89fdcf59b..6bcc170d7b84 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi
> @@ -551,7 +551,7 @@ iomuxc: pinctrl@30330000 {
> };
>
> gpr: iomuxc-gpr@30340000 {
For 8mn/p you adapting the name to syscon, this should be done here as
well.
Regards,
Marco
> - compatible = "fsl,imx8mm-iomuxc-gpr", "fsl,imx6q-iomuxc-gpr", "syscon";
> + compatible = "fsl,imx8mm-iomuxc-gpr", "syscon";
> reg = <0x30340000 0x10000>;
> };
>
> --
> 2.37.1
>
>
>
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH 4/5] arm64: dts: imx8mn: update iomuxc-gpr node name
2022-12-23 2:56 [PATCH 0/5] binding: imx8m: add IOMUXC GPR Peng Fan (OSS)
` (2 preceding siblings ...)
2022-12-23 2:57 ` [PATCH 3/5] arm64: dts: imx8mm: " Peng Fan (OSS)
@ 2022-12-23 2:57 ` Peng Fan (OSS)
2022-12-23 8:48 ` Marco Felsch
2022-12-23 2:57 ` [PATCH 5/5] arm64: dts: imx8mp: use syscon for iomuxc-gpr Peng Fan (OSS)
4 siblings, 1 reply; 11+ messages in thread
From: Peng Fan (OSS) @ 2022-12-23 2:57 UTC (permalink / raw)
To: robh+dt, krzysztof.kozlowski+dt, shawnguo, s.hauer
Cc: kernel, festevam, linux-imx, devicetree, linux-kernel,
linux-arm-kernel, Peng Fan
From: Peng Fan <peng.fan@nxp.com>
It is better use syscon for IOMUXC GPR, since it contains various
bits for system control
Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
arch/arm64/boot/dts/freescale/imx8mn.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mn.dtsi b/arch/arm64/boot/dts/freescale/imx8mn.dtsi
index b7d91df71cc2..08ca91bb1887 100644
--- a/arch/arm64/boot/dts/freescale/imx8mn.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mn.dtsi
@@ -552,7 +552,7 @@ iomuxc: pinctrl@30330000 {
reg = <0x30330000 0x10000>;
};
- gpr: iomuxc-gpr@30340000 {
+ gpr: syscon@30340000 {
compatible = "fsl,imx8mn-iomuxc-gpr", "syscon";
reg = <0x30340000 0x10000>;
};
--
2.37.1
^ permalink raw reply related [flat|nested] 11+ messages in thread* Re: [PATCH 4/5] arm64: dts: imx8mn: update iomuxc-gpr node name
2022-12-23 2:57 ` [PATCH 4/5] arm64: dts: imx8mn: update iomuxc-gpr node name Peng Fan (OSS)
@ 2022-12-23 8:48 ` Marco Felsch
0 siblings, 0 replies; 11+ messages in thread
From: Marco Felsch @ 2022-12-23 8:48 UTC (permalink / raw)
To: Peng Fan (OSS)
Cc: robh+dt, krzysztof.kozlowski+dt, shawnguo, s.hauer, devicetree,
Peng Fan, linux-kernel, linux-imx, kernel, festevam,
linux-arm-kernel
On 22-12-23, Peng Fan (OSS) wrote:
> From: Peng Fan <peng.fan@nxp.com>
>
> It is better use syscon for IOMUXC GPR, since it contains various
> bits for system control
>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
Reviewed-by: Marco Felsch <m.felsch@pengutronix.de>
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH 5/5] arm64: dts: imx8mp: use syscon for iomuxc-gpr
2022-12-23 2:56 [PATCH 0/5] binding: imx8m: add IOMUXC GPR Peng Fan (OSS)
` (3 preceding siblings ...)
2022-12-23 2:57 ` [PATCH 4/5] arm64: dts: imx8mn: update iomuxc-gpr node name Peng Fan (OSS)
@ 2022-12-23 2:57 ` Peng Fan (OSS)
2022-12-23 8:48 ` Marco Felsch
4 siblings, 1 reply; 11+ messages in thread
From: Peng Fan (OSS) @ 2022-12-23 2:57 UTC (permalink / raw)
To: robh+dt, krzysztof.kozlowski+dt, shawnguo, s.hauer
Cc: kernel, festevam, linux-imx, devicetree, linux-kernel,
linux-arm-kernel, Peng Fan
From: Peng Fan <peng.fan@nxp.com>
It is preferred to use syscon per bindind doc
Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi b/arch/arm64/boot/dts/freescale/imx8mp.dtsi
index 7a6e6221f421..9e5260721931 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mp.dtsi
@@ -412,7 +412,7 @@ iomuxc: pinctrl@30330000 {
reg = <0x30330000 0x10000>;
};
- gpr: iomuxc-gpr@30340000 {
+ gpr: syscon@30340000 {
compatible = "fsl,imx8mp-iomuxc-gpr", "syscon";
reg = <0x30340000 0x10000>;
};
--
2.37.1
^ permalink raw reply related [flat|nested] 11+ messages in thread* Re: [PATCH 5/5] arm64: dts: imx8mp: use syscon for iomuxc-gpr
2022-12-23 2:57 ` [PATCH 5/5] arm64: dts: imx8mp: use syscon for iomuxc-gpr Peng Fan (OSS)
@ 2022-12-23 8:48 ` Marco Felsch
0 siblings, 0 replies; 11+ messages in thread
From: Marco Felsch @ 2022-12-23 8:48 UTC (permalink / raw)
To: Peng Fan (OSS)
Cc: robh+dt, krzysztof.kozlowski+dt, shawnguo, s.hauer, devicetree,
Peng Fan, linux-kernel, linux-imx, kernel, festevam,
linux-arm-kernel
On 22-12-23, Peng Fan (OSS) wrote:
> From: Peng Fan <peng.fan@nxp.com>
>
> It is preferred to use syscon per bindind doc
>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
Reviewed-by: Marco Felsch <m.felsch@pengutronix.de>
^ permalink raw reply [flat|nested] 11+ messages in thread