* [PATCH 5/6] arm64: dts: ls1046a: add pinctrl node
@ 2024-08-27 2:13 David Leonard
2024-08-27 13:32 ` Krzysztof Kozlowski
2024-08-27 15:11 ` Frank Li
0 siblings, 2 replies; 3+ messages in thread
From: David Leonard @ 2024-08-27 2:13 UTC (permalink / raw)
To: linux-arm-kernel
Cc: Shawn Guo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
devicetree, linux-kernel
Add a node for the LS1046A's pinmux controller and related pinctrl
properties for the nodes using the gpio and i2c functions it controls.
Signed-off-by: David Leonard <David.Leonard@digi.com>
---
.../arm64/boot/dts/freescale/fsl-ls1046a.dtsi | 36 +++++++++++++++++++
1 file changed, 36 insertions(+)
diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi
index 200e52622f99..d2286fd6f972 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi
@@ -350,6 +350,29 @@ extirq: interrupt-controller@1ac {
};
};
+ pinmux: pinmux@157040c {
+ compatible = "fsl,ls1046a-pinctrl";
+ reg = <0 0x157040c 0 4>;
+ big-endian;
+
+ pinctrl_i2c2: pinctrl-i2c2 {
+ groups = "L4", "M4";
+ function = "i2c";
+ };
+ pinctrl_i2c2_gpio: pinctrl-i2c2-gpio {
+ groups = "L4", "M4";
+ function = "gpio";
+ };
+ pinctrl_i2c3: pinctrl-i2c3 {
+ groups = "M3", "N3";
+ function = "i2c";
+ };
+ pinctrl_i2c3_gpio: pinctrl-i2c3-gpio {
+ groups = "M3", "N3";
+ function = "gpio";
+ };
+ };
+
crypto: crypto@1700000 {
compatible = "fsl,sec-v5.4", "fsl,sec-v5.0",
"fsl,sec-v4.0";
@@ -537,6 +560,10 @@ i2c2: i2c@21a0000 {
clocks = <&clockgen QORIQ_CLK_PLATFORM_PLL
QORIQ_CLK_PLL_DIV(2)>;
scl-gpios = <&gpio3 10 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>;
+ sda-gpios = <&gpio3 11 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>;
+ pinctrl-names = "default", "gpio";
+ pinctrl-0 = <&pinctrl_i2c2>;
+ pinctrl-1 = <&pinctrl_i2c2_gpio>;
status = "disabled";
};
@@ -549,6 +576,10 @@ i2c3: i2c@21b0000 {
clocks = <&clockgen QORIQ_CLK_PLATFORM_PLL
QORIQ_CLK_PLL_DIV(2)>;
scl-gpios = <&gpio3 12 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>;
+ sda-gpios = <&gpio3 13 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>;
+ pinctrl-names = "default", "gpio";
+ pinctrl-0 = <&pinctrl_i2c3>;
+ pinctrl-1 = <&pinctrl_i2c3_gpio>;
status = "disabled";
};
@@ -626,6 +657,11 @@ gpio3: gpio@2330000 {
#gpio-cells = <2>;
interrupt-controller;
#interrupt-cells = <2>;
+ gpio-ranges = <&pinmux 10 0 0>,
+ <&pinmux 11 0 0>,
+ <&pinmux 12 0 0>,
+ <&pinmux 13 0 0>;
+ gpio-ranges-group-names = "L4", "M4", "M3", "N3";
};
lpuart0: serial@2950000 {
--
2.43.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 5/6] arm64: dts: ls1046a: add pinctrl node
2024-08-27 2:13 [PATCH 5/6] arm64: dts: ls1046a: add pinctrl node David Leonard
@ 2024-08-27 13:32 ` Krzysztof Kozlowski
2024-08-27 15:11 ` Frank Li
1 sibling, 0 replies; 3+ messages in thread
From: Krzysztof Kozlowski @ 2024-08-27 13:32 UTC (permalink / raw)
To: David Leonard, linux-arm-kernel
Cc: Shawn Guo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
devicetree, linux-kernel
On 27/08/2024 04:13, David Leonard wrote:
>
You messages have weird blank line in front.
> Add a node for the LS1046A's pinmux controller and related pinctrl
> properties for the nodes using the gpio and i2c functions it controls.
>
> Signed-off-by: David Leonard <David.Leonard@digi.com>
> ---
> .../arm64/boot/dts/freescale/fsl-ls1046a.dtsi | 36 +++++++++++++++++++
> 1 file changed, 36 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi
> index 200e52622f99..d2286fd6f972 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi
> @@ -350,6 +350,29 @@ extirq: interrupt-controller@1ac {
> };
> };
>
> + pinmux: pinmux@157040c {
pinctrl@
Keep your code consistent.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 5/6] arm64: dts: ls1046a: add pinctrl node
2024-08-27 2:13 [PATCH 5/6] arm64: dts: ls1046a: add pinctrl node David Leonard
2024-08-27 13:32 ` Krzysztof Kozlowski
@ 2024-08-27 15:11 ` Frank Li
1 sibling, 0 replies; 3+ messages in thread
From: Frank Li @ 2024-08-27 15:11 UTC (permalink / raw)
To: David Leonard
Cc: linux-arm-kernel, Shawn Guo, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, devicetree, linux-kernel
On Tue, Aug 27, 2024 at 12:13:30PM +1000, David Leonard wrote:
>
> Add a node for the LS1046A's pinmux controller and related pinctrl
> properties for the nodes using the gpio and i2c functions it controls.
>
> Signed-off-by: David Leonard <David.Leonard@digi.com>
> ---
> .../arm64/boot/dts/freescale/fsl-ls1046a.dtsi | 36 +++++++++++++++++++
> 1 file changed, 36 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi
> index 200e52622f99..d2286fd6f972 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi
> @@ -350,6 +350,29 @@ extirq: interrupt-controller@1ac {
> };
> };
>
> + pinmux: pinmux@157040c {
> + compatible = "fsl,ls1046a-pinctrl";
> + reg = <0 0x157040c 0 4>;
> + big-endian;
I can't find fsl,ls1046a-pinctrl in binding doc.
git grep -r fsl,ls1046a-pinctrl Documentation/
I suggest you use pinctrl-single.
Frank
> +
> + pinctrl_i2c2: pinctrl-i2c2 {
> + groups = "L4", "M4";
> + function = "i2c";
> + };
> + pinctrl_i2c2_gpio: pinctrl-i2c2-gpio {
> + groups = "L4", "M4";
> + function = "gpio";
> + };
> + pinctrl_i2c3: pinctrl-i2c3 {
> + groups = "M3", "N3";
> + function = "i2c";
> + };
> + pinctrl_i2c3_gpio: pinctrl-i2c3-gpio {
> + groups = "M3", "N3";
> + function = "gpio";
> + };
> + };
> +
> crypto: crypto@1700000 {
> compatible = "fsl,sec-v5.4", "fsl,sec-v5.0",
> "fsl,sec-v4.0";
> @@ -537,6 +560,10 @@ i2c2: i2c@21a0000 {
> clocks = <&clockgen QORIQ_CLK_PLATFORM_PLL
> QORIQ_CLK_PLL_DIV(2)>;
> scl-gpios = <&gpio3 10 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>;
> + sda-gpios = <&gpio3 11 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>;
> + pinctrl-names = "default", "gpio";
> + pinctrl-0 = <&pinctrl_i2c2>;
> + pinctrl-1 = <&pinctrl_i2c2_gpio>;
> status = "disabled";
> };
>
> @@ -549,6 +576,10 @@ i2c3: i2c@21b0000 {
> clocks = <&clockgen QORIQ_CLK_PLATFORM_PLL
> QORIQ_CLK_PLL_DIV(2)>;
> scl-gpios = <&gpio3 12 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>;
> + sda-gpios = <&gpio3 13 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>;
> + pinctrl-names = "default", "gpio";
> + pinctrl-0 = <&pinctrl_i2c3>;
> + pinctrl-1 = <&pinctrl_i2c3_gpio>;
> status = "disabled";
> };
>
> @@ -626,6 +657,11 @@ gpio3: gpio@2330000 {
> #gpio-cells = <2>;
> interrupt-controller;
> #interrupt-cells = <2>;
> + gpio-ranges = <&pinmux 10 0 0>,
> + <&pinmux 11 0 0>,
> + <&pinmux 12 0 0>,
> + <&pinmux 13 0 0>;
> + gpio-ranges-group-names = "L4", "M4", "M3", "N3";
> };
>
> lpuart0: serial@2950000 {
> --
> 2.43.0
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-08-27 15:12 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-08-27 2:13 [PATCH 5/6] arm64: dts: ls1046a: add pinctrl node David Leonard
2024-08-27 13:32 ` Krzysztof Kozlowski
2024-08-27 15:11 ` Frank Li
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).