* [PATCH v2 1/5] dt-bindings: pinctrl: add the NXP LX2160A RCW pad mux
2026-11-03 18:12 [PATCH v2 0/5] lx2160a: change the pad muxes through the RCW override Vincent Jardin via B4 Relay
@ 2026-11-03 18:12 ` Vincent Jardin via B4 Relay
2026-11-03 18:23 ` sashiko-bot
2026-11-03 18:12 ` [PATCH v2 2/5] dt-bindings: soc: fsl: layerscape-dcfg: add the LX2160A pad-mux child Vincent Jardin via B4 Relay
` (3 subsequent siblings)
4 siblings, 1 reply; 8+ messages in thread
From: Vincent Jardin via B4 Relay @ 2026-11-03 18:12 UTC (permalink / raw)
To: Dong Aisheng, Fabio Estevam, Frank Li, Jacky Bai,
Pengutronix Kernel Team, NXP S32 Linux Team, Linus Walleij,
Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Christophe Leroy (CS GROUP), Shawn Guo, Sascha Hauer
Cc: linux-gpio, devicetree, linux-kernel, linuxppc-dev,
linux-arm-kernel, imx, Vladimir Oltean, Vincent Jardin
From: Vincent Jardin <vjardin@free.fr>
The LX2160A muxes its IIC, XSPI, SDHC, UART, IRQ and EVT pads from
fields of the RCW: read-only in RCWSR12..14 of DCFG, changed at runtime
through the override registers of its DCSR view.
Describe the pad mux as a node without registers, a child of DCFG: one
group per RCW field, one function per encoding.
Each function lists the groups that take it, so a pair the driver
would refuse, such as can on iic1, is refused by the schema.
Signed-off-by: Vincent Jardin <vjardin@free.fr>
---
.../bindings/pinctrl/fsl,lx2160a-rcw-pinmux.yaml | 262 +++++++++++++++++++++
1 file changed, 262 insertions(+)
diff --git a/Documentation/devicetree/bindings/pinctrl/fsl,lx2160a-rcw-pinmux.yaml b/Documentation/devicetree/bindings/pinctrl/fsl,lx2160a-rcw-pinmux.yaml
new file mode 100644
index 000000000000..b1ccd153a43b
--- /dev/null
+++ b/Documentation/devicetree/bindings/pinctrl/fsl,lx2160a-rcw-pinmux.yaml
@@ -0,0 +1,262 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/pinctrl/fsl,lx2160a-rcw-pinmux.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: NXP LX2160A RCW pad muxes
+
+maintainers:
+ - Vincent Jardin <vjardin@free.fr>
+
+description:
+ The LX2160A muxes its IIC, XSPI, SDHC, UART, IRQ and EVT pads based on its
+ RCW. The related fields are from the read-only RCWSR12 to RCWSR14 registers
+ of DCFG, and changed at runtime by writing the RCW override registers in
+ the DCSR view of DCFG. The node is a child of the DCFG node, which owns both
+ views, and needs no registers of its own. A pad group is one field, named
+ from its pads, and takes the functions its field encodes, gpio on every
+ group and the others per function below, as the RCW chapter of the
+ reference manual lists them (the *_PMUX fields). A board selects a state
+ of its own with the usual pinctrl-0 on this node, a hog.
+
+properties:
+ compatible:
+ const: fsl,lx2160a-rcw-pinmux
+
+patternProperties:
+ '-pins$':
+ type: object
+ $ref: pinmux-node.yaml#
+ additionalProperties: false
+
+ properties:
+ function:
+ enum: [ asleep, can, evt, flexspi, flextimer, gpio, i2c, irq,
+ reset-req, sdhc, sdhc-clk, sdhc-data, sdhc-data4, sdhc-spi,
+ spi, uart, uart3, uart4 ]
+
+ groups:
+ items:
+ enum: [ iic1, iic2, iic3, iic4, iic5, iic6, xspi1-data74,
+ xspi1-data30, xspi1-base, sdhc1-base, sdhc1-dir,
+ sdhc2-data74, sdhc2-base, uart1-soutsin, uart1-rtscts,
+ uart2-soutsin, uart2-rtscts, irq11-8, irq7-4, irq3-0,
+ evt4-3, evt2-0, clk-out, asleep, reset-req ]
+
+ required:
+ - function
+ - groups
+
+ allOf:
+ - if:
+ properties:
+ function:
+ const: asleep
+ then:
+ properties:
+ groups:
+ items:
+ enum: [ asleep ]
+
+ - if:
+ properties:
+ function:
+ const: can
+ then:
+ properties:
+ groups:
+ items:
+ enum: [ iic3, iic4 ]
+
+ - if:
+ properties:
+ function:
+ const: evt
+ then:
+ properties:
+ groups:
+ items:
+ enum: [ iic3, iic4, evt4-3, evt2-0, asleep ]
+
+ - if:
+ properties:
+ function:
+ const: flexspi
+ then:
+ properties:
+ groups:
+ items:
+ enum: [ xspi1-data74, xspi1-data30, xspi1-base,
+ sdhc2-data74, sdhc2-base ]
+
+ - if:
+ properties:
+ function:
+ const: flextimer
+ then:
+ properties:
+ groups:
+ items:
+ enum: [ iic2, irq7-4, irq3-0, evt4-3, evt2-0, clk-out ]
+
+ - if:
+ properties:
+ function:
+ const: gpio
+ then:
+ properties:
+ groups:
+ items:
+ enum: [ iic2, iic3, iic4, iic5, iic6, xspi1-data74,
+ xspi1-data30, xspi1-base, sdhc1-base, sdhc1-dir,
+ sdhc2-data74, sdhc2-base, uart1-soutsin,
+ uart1-rtscts, uart2-soutsin, uart2-rtscts,
+ irq11-8, irq7-4, irq3-0, evt4-3, evt2-0, clk-out,
+ asleep, reset-req, iic1 ]
+
+ - if:
+ properties:
+ function:
+ const: i2c
+ then:
+ properties:
+ groups:
+ items:
+ enum: [ iic2, iic3, iic4, iic5, iic6, sdhc2-data74, iic1 ]
+
+ - if:
+ properties:
+ function:
+ const: irq
+ then:
+ properties:
+ groups:
+ items:
+ enum: [ irq11-8, irq7-4, irq3-0 ]
+
+ - if:
+ properties:
+ function:
+ const: reset-req
+ then:
+ properties:
+ groups:
+ items:
+ enum: [ reset-req ]
+
+ - if:
+ properties:
+ function:
+ const: sdhc
+ then:
+ properties:
+ groups:
+ items:
+ enum: [ iic2, sdhc1-base, sdhc1-dir, sdhc2-data74,
+ sdhc2-base ]
+
+ - if:
+ properties:
+ function:
+ const: sdhc-clk
+ then:
+ properties:
+ groups:
+ items:
+ enum: [ iic5, iic6 ]
+
+ - if:
+ properties:
+ function:
+ const: sdhc-data
+ then:
+ properties:
+ groups:
+ items:
+ enum: [ sdhc1-dir ]
+
+ - if:
+ properties:
+ function:
+ const: sdhc-data4
+ then:
+ properties:
+ groups:
+ items:
+ enum: [ sdhc1-base ]
+
+ - if:
+ properties:
+ function:
+ const: sdhc-spi
+ then:
+ properties:
+ groups:
+ items:
+ enum: [ sdhc1-base ]
+
+ - if:
+ properties:
+ function:
+ const: spi
+ then:
+ properties:
+ groups:
+ items:
+ enum: [ iic5, sdhc1-base, sdhc1-dir, sdhc2-base ]
+
+ - if:
+ properties:
+ function:
+ const: uart
+ then:
+ properties:
+ groups:
+ items:
+ enum: [ uart1-soutsin, uart1-rtscts, uart2-soutsin,
+ uart2-rtscts ]
+
+ - if:
+ properties:
+ function:
+ const: uart3
+ then:
+ properties:
+ groups:
+ items:
+ enum: [ uart1-rtscts ]
+
+ - if:
+ properties:
+ function:
+ const: uart4
+ then:
+ properties:
+ groups:
+ items:
+ enum: [ uart2-rtscts ]
+
+required:
+ - compatible
+
+allOf:
+ - $ref: pinctrl.yaml#
+
+unevaluatedProperties: false
+
+examples:
+ - |
+ pinctrl {
+ compatible = "fsl,lx2160a-rcw-pinmux";
+
+ iic3-i2c-pins {
+ function = "i2c";
+ groups = "iic3";
+ };
+
+ iic3-gpio-pins {
+ function = "gpio";
+ groups = "iic3";
+ };
+ };
--
2.43.0
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH v2 4/5] pinctrl: freescale: add the LX2160A RCW pad-mux driver
2026-11-03 18:12 [PATCH v2 0/5] lx2160a: change the pad muxes through the RCW override Vincent Jardin via B4 Relay
` (2 preceding siblings ...)
2026-11-03 18:12 ` [PATCH v2 3/5] soc: fsl: guts: add the RCW override of any RCWSR Vincent Jardin via B4 Relay
@ 2026-11-03 18:12 ` Vincent Jardin via B4 Relay
2026-11-03 18:12 ` [PATCH v2 5/5] arm64: dts: lx2160a: describe the pad muxes as the DCFG RCW override Vincent Jardin via B4 Relay
4 siblings, 0 replies; 8+ messages in thread
From: Vincent Jardin via B4 Relay @ 2026-11-03 18:12 UTC (permalink / raw)
To: Dong Aisheng, Fabio Estevam, Frank Li, Jacky Bai,
Pengutronix Kernel Team, NXP S32 Linux Team, Linus Walleij,
Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Christophe Leroy (CS GROUP), Shawn Guo, Sascha Hauer
Cc: linux-gpio, devicetree, linux-kernel, linuxppc-dev,
linux-arm-kernel, imx, Vladimir Oltean, Vincent Jardin
From: Vincent Jardin <vjardin@free.fr>
The LX2160A muxes its IIC, XSPI, SDHC, UART, IRQ and EVT pads from
fields of the RCW, read-only in RCWSR12..14 of DCFG and changed at
runtime through the same fields of the override registers in its DCSR
view, which read 0 until written.
Since commit 8a1365c7bbc1 ("arm64: dts: lx2160a: add pinmux and i2c
gpio to support bus recovery") Linux drives them with pinctrl-single on
the override registers, so its first pin state clears every other field
of the word: a board whose RCW programs a non-zero pad mux loses those
pads at the first I2C probe.
Add a driver for the pad mux as a child of the DCFG node: the fields
of RCWSR12..14 and their encoding.
Then pin states select them with the generic function and groups
properties, and every change goes through the RCW override
of fsl-guts, which reads the status register, writes the override and
waits for the status register to follow.
Signed-off-by: Vincent Jardin <vjardin@free.fr>
---
MAINTAINERS | 7 +
drivers/pinctrl/freescale/Kconfig | 15 ++
drivers/pinctrl/freescale/Makefile | 1 +
drivers/pinctrl/freescale/pinctrl-lx2160a.c | 355 ++++++++++++++++++++++++++++
4 files changed, 378 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 93abbb315065..6c488605f5da 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -20207,6 +20207,13 @@ F: Documentation/devicetree/bindings/clock/*imx*
F: drivers/clk/imx/
F: include/dt-bindings/clock/*imx*
+NXP LX2160A RCW PAD-MUX DRIVER
+M: Vincent Jardin <vjardin@free.fr>
+L: linux-gpio@vger.kernel.org
+S: Maintained
+F: Documentation/devicetree/bindings/pinctrl/fsl,lx2160a-rcw-pinmux.yaml
+F: drivers/pinctrl/freescale/pinctrl-lx2160a.c
+
NXP NETC ETHERNET SWITCH DRIVER
M: Wei Fang <wei.fang@nxp.com>
R: Clark Wang <xiaoning.wang@nxp.com>
diff --git a/drivers/pinctrl/freescale/Kconfig b/drivers/pinctrl/freescale/Kconfig
index fd53cf5bb843..a3222714c970 100644
--- a/drivers/pinctrl/freescale/Kconfig
+++ b/drivers/pinctrl/freescale/Kconfig
@@ -283,3 +283,18 @@ config PINCTRL_IMXRT1170
select PINCTRL_IMX
help
Say Y here to enable the imxrt1170 pinctrl driver
+
+config PINCTRL_LX2160A
+ bool "NXP LX2160A RCW pad-mux driver"
+ depends on OF
+ depends on ARCH_LAYERSCAPE || COMPILE_TEST
+ default ARCH_LAYERSCAPE
+ select FSL_GUTS
+ select GENERIC_PINCTRL_GROUPS
+ select GENERIC_PINMUX_FUNCTIONS
+ select GENERIC_PINCONF
+ help
+ Say Y here to mux the IIC, XSPI, SDHC, UART, IRQ and EVT pads of the
+ LX2160A at runtime. The pads are muxed by fields of the RCW:
+ the driver changes it in the DCSR view of the DCFG that fsl-guts
+ drives, and is what the I2C bus recovery of the LX2160A needs.
diff --git a/drivers/pinctrl/freescale/Makefile b/drivers/pinctrl/freescale/Makefile
index d27085c2b4c4..5ec0d1710ad5 100644
--- a/drivers/pinctrl/freescale/Makefile
+++ b/drivers/pinctrl/freescale/Makefile
@@ -35,3 +35,4 @@ obj-$(CONFIG_PINCTRL_IMX25) += pinctrl-imx25.o
obj-$(CONFIG_PINCTRL_IMX28) += pinctrl-imx28.o
obj-$(CONFIG_PINCTRL_IMXRT1050) += pinctrl-imxrt1050.o
obj-$(CONFIG_PINCTRL_IMXRT1170) += pinctrl-imxrt1170.o
+obj-$(CONFIG_PINCTRL_LX2160A) += pinctrl-lx2160a.o
diff --git a/drivers/pinctrl/freescale/pinctrl-lx2160a.c b/drivers/pinctrl/freescale/pinctrl-lx2160a.c
new file mode 100644
index 000000000000..ba5bbcfab9fd
--- /dev/null
+++ b/drivers/pinctrl/freescale/pinctrl-lx2160a.c
@@ -0,0 +1,355 @@
+// SPDX-License-Identifier: GPL-2.0
+/*
+ * LX2160A pad muxes: fields of the RCW, changed at
+ * runtime.
+ *
+ * Copyright 2026 Free Mobile - Vincent Jardin
+ */
+
+#include <linux/bits.h>
+#include <linux/fsl/guts.h>
+#include <linux/module.h>
+#include <linux/of.h>
+#include <linux/pinctrl/pinconf-generic.h>
+#include <linux/pinctrl/pinctrl.h>
+#include <linux/pinctrl/pinmux.h>
+#include <linux/platform_device.h>
+#include <linux/seq_file.h>
+
+#include "../core.h"
+#include "../pinmux.h"
+
+struct lx2160a_pmux_func {
+ const char *name;
+ u8 val;
+};
+
+struct lx2160a_pmux_group {
+ const char *name;
+ u8 rcwsr; /* 12..14 */
+ u8 shift;
+ u8 width;
+ const struct lx2160a_pmux_func *funcs;
+ u8 nfuncs;
+};
+
+#define FUNC(_name, _val) { .name = _name, .val = _val }
+#define GROUP(_name, _rcwsr, _shift, _width, _funcs) \
+ { \
+ .name = _name, .rcwsr = _rcwsr, \
+ .shift = _shift, .width = _width, \
+ .funcs = _funcs, .nfuncs = ARRAY_SIZE(_funcs), \
+ }
+
+static const struct lx2160a_pmux_func iic2_funcs[] = {
+ FUNC("i2c", 0), FUNC("gpio", 1), FUNC("flextimer", 2), FUNC("sdhc", 6),
+};
+
+static const struct lx2160a_pmux_func iic3_funcs[] = {
+ FUNC("i2c", 0), FUNC("gpio", 1), FUNC("can", 2), FUNC("evt", 6),
+};
+
+static const struct lx2160a_pmux_func iic5_funcs[] = {
+ FUNC("i2c", 0), FUNC("gpio", 1), FUNC("sdhc-clk", 2), FUNC("spi", 3),
+};
+
+static const struct lx2160a_pmux_func iic6_funcs[] = {
+ FUNC("i2c", 0), FUNC("gpio", 1), FUNC("sdhc-clk", 2),
+};
+
+static const struct lx2160a_pmux_func xspi1_funcs[] = {
+ FUNC("flexspi", 0), FUNC("gpio", 1),
+};
+
+static const struct lx2160a_pmux_func sdhc1_base_funcs[] = {
+ FUNC("sdhc", 0), FUNC("gpio", 1), FUNC("spi", 2), FUNC("sdhc-spi", 3),
+ FUNC("sdhc-data4", 4),
+};
+
+static const struct lx2160a_pmux_func sdhc1_dir_funcs[] = {
+ FUNC("sdhc", 0), FUNC("gpio", 1), FUNC("spi", 3), FUNC("sdhc-data", 4),
+};
+
+static const struct lx2160a_pmux_func sdhc2_data74_funcs[] = {
+ FUNC("sdhc", 0), FUNC("gpio", 1), FUNC("i2c", 2), FUNC("flexspi", 6),
+};
+
+static const struct lx2160a_pmux_func sdhc2_base_funcs[] = {
+ FUNC("sdhc", 0), FUNC("gpio", 1), FUNC("spi", 2), FUNC("flexspi", 6),
+};
+
+static const struct lx2160a_pmux_func uart_funcs[] = {
+ FUNC("uart", 0), FUNC("gpio", 1),
+};
+
+static const struct lx2160a_pmux_func uart1_rtscts_funcs[] = {
+ FUNC("uart", 0), FUNC("gpio", 1), FUNC("uart3", 2),
+};
+
+static const struct lx2160a_pmux_func uart2_rtscts_funcs[] = {
+ FUNC("uart", 0), FUNC("gpio", 1), FUNC("uart4", 2),
+};
+
+static const struct lx2160a_pmux_func irq_funcs[] = {
+ FUNC("irq", 0), FUNC("gpio", 1),
+};
+
+static const struct lx2160a_pmux_func irq_flextimer_funcs[] = {
+ FUNC("irq", 0), FUNC("gpio", 1), FUNC("flextimer", 2),
+};
+
+static const struct lx2160a_pmux_func evt_funcs[] = {
+ FUNC("evt", 0), FUNC("gpio", 1), FUNC("flextimer", 2),
+};
+
+static const struct lx2160a_pmux_func clk_out_funcs[] = {
+ FUNC("gpio", 1), FUNC("flextimer", 2),
+};
+
+static const struct lx2160a_pmux_func asleep_funcs[] = {
+ FUNC("asleep", 0), FUNC("gpio", 1), FUNC("evt", 2),
+};
+
+static const struct lx2160a_pmux_func reset_req_funcs[] = {
+ FUNC("reset-req", 0), FUNC("gpio", 1),
+};
+
+static const struct lx2160a_pmux_func iic1_funcs[] = {
+ FUNC("i2c", 0), FUNC("gpio", 1),
+};
+
+static const struct lx2160a_pmux_group lx2160a_pmux_groups[] = {
+ /* RCWSR12 */
+ GROUP("iic2", 12, 0, 3, iic2_funcs),
+ GROUP("iic3", 12, 3, 3, iic3_funcs),
+ GROUP("iic4", 12, 6, 3, iic3_funcs),
+ GROUP("iic5", 12, 9, 3, iic5_funcs),
+ GROUP("iic6", 12, 12, 3, iic6_funcs),
+ GROUP("xspi1-data74", 12, 15, 3, xspi1_funcs),
+ GROUP("xspi1-data30", 12, 18, 3, xspi1_funcs),
+ GROUP("xspi1-base", 12, 21, 3, xspi1_funcs),
+ GROUP("sdhc1-base", 12, 24, 3, sdhc1_base_funcs),
+ GROUP("sdhc1-dir", 12, 27, 3, sdhc1_dir_funcs),
+ /* RCWSR13 */
+ GROUP("sdhc2-data74", 13, 0, 3, sdhc2_data74_funcs),
+ GROUP("sdhc2-base", 13, 3, 3, sdhc2_base_funcs),
+ GROUP("uart1-soutsin", 13, 6, 3, uart_funcs),
+ GROUP("uart1-rtscts", 13, 9, 3, uart1_rtscts_funcs),
+ GROUP("uart2-soutsin", 13, 12, 3, uart_funcs),
+ GROUP("uart2-rtscts", 13, 15, 3, uart2_rtscts_funcs),
+ GROUP("irq11-8", 13, 18, 3, irq_funcs),
+ GROUP("irq7-4", 13, 21, 3, irq_flextimer_funcs),
+ GROUP("irq3-0", 13, 24, 3, irq_flextimer_funcs),
+ GROUP("evt4-3", 13, 27, 3, evt_funcs),
+ /* RCWSR14 */
+ GROUP("evt2-0", 14, 0, 3, evt_funcs),
+ GROUP("clk-out", 14, 3, 3, clk_out_funcs),
+ GROUP("asleep", 14, 6, 3, asleep_funcs),
+ GROUP("reset-req", 14, 9, 1, reset_req_funcs),
+ GROUP("iic1", 14, 10, 1, iic1_funcs),
+};
+
+struct lx2160a_pmux {
+ struct device *dev;
+ struct pinctrl_dev *pctl;
+ struct pinctrl_desc desc;
+ struct pinctrl_pin_desc pins[ARRAY_SIZE(lx2160a_pmux_groups)];
+};
+
+static u32 lx2160a_pmux_group_mask(const struct lx2160a_pmux_group *group)
+{
+ return GENMASK(group->shift + group->width - 1, group->shift);
+}
+
+static void lx2160a_pmux_pin_dbg_show(struct pinctrl_dev *pctldev,
+ struct seq_file *s, unsigned int pin)
+{
+ const struct lx2160a_pmux_group *group = &lx2160a_pmux_groups[pin];
+ unsigned int i;
+ u32 val;
+
+ if (fsl_guts_rcw_read(group->rcwsr, &val))
+ return;
+
+ val = (val & lx2160a_pmux_group_mask(group)) >> group->shift;
+ for (i = 0; i < group->nfuncs; i++) {
+ if (group->funcs[i].val == val) {
+ seq_puts(s, group->funcs[i].name);
+ return;
+ }
+ }
+ seq_printf(s, "0x%x", val);
+}
+
+static const struct pinctrl_ops lx2160a_pmux_pctlops = {
+ .get_groups_count = pinctrl_generic_get_group_count,
+ .get_group_name = pinctrl_generic_get_group_name,
+ .get_group_pins = pinctrl_generic_get_group_pins,
+ .pin_dbg_show = lx2160a_pmux_pin_dbg_show,
+ .dt_node_to_map = pinconf_generic_dt_node_to_map_group,
+ .dt_free_map = pinconf_generic_dt_free_map,
+};
+
+static int lx2160a_pmux_set_mux(struct pinctrl_dev *pctldev,
+ unsigned int fsel, unsigned int gsel)
+{
+ struct lx2160a_pmux *pmux = pinctrl_dev_get_drvdata(pctldev);
+ const struct function_desc *func;
+ const struct lx2160a_pmux_group *group;
+ const struct group_desc *grp;
+ unsigned int i;
+
+ func = pinmux_generic_get_function(pctldev, fsel);
+ grp = pinctrl_generic_get_group(pctldev, gsel);
+ if (!func || !grp)
+ return -EINVAL;
+ group = grp->data;
+
+ for (i = 0; i < group->nfuncs; i++) {
+ if (!strcmp(group->funcs[i].name, func->func->name))
+ break;
+ }
+ if (i == group->nfuncs) {
+ dev_err(pmux->dev, "%s has no %s function\n",
+ group->name, func->func->name);
+ return -EINVAL;
+ }
+
+ return fsl_guts_rcw_override(group->rcwsr,
+ group->funcs[i].val << group->shift,
+ lx2160a_pmux_group_mask(group));
+}
+
+static const struct pinmux_ops lx2160a_pmux_pmxops = {
+ .get_functions_count = pinmux_generic_get_function_count,
+ .get_function_name = pinmux_generic_get_function_name,
+ .get_function_groups = pinmux_generic_get_function_groups,
+ .set_mux = lx2160a_pmux_set_mux,
+ .strict = true,
+};
+
+static int lx2160a_pmux_add_functions(struct lx2160a_pmux *pmux)
+{
+ const struct lx2160a_pmux_group *group;
+ const char **groups;
+ unsigned int i, j, k, n;
+ int ret;
+
+ for (i = 0; i < ARRAY_SIZE(lx2160a_pmux_groups); i++) {
+ group = &lx2160a_pmux_groups[i];
+ for (j = 0; j < group->nfuncs; j++) {
+ const char *name = group->funcs[j].name;
+ bool seen = false;
+
+ /* already added from an earlier group? */
+ for (k = 0; k < i && !seen; k++) {
+ const struct lx2160a_pmux_group *g =
+ &lx2160a_pmux_groups[k];
+ unsigned int l;
+
+ for (l = 0; l < g->nfuncs; l++)
+ if (!strcmp(g->funcs[l].name, name))
+ seen = true;
+ }
+ if (seen)
+ continue;
+
+ groups = devm_kcalloc(pmux->dev,
+ ARRAY_SIZE(lx2160a_pmux_groups),
+ sizeof(*groups), GFP_KERNEL);
+ if (!groups)
+ return -ENOMEM;
+ n = 0;
+ for (k = i; k < ARRAY_SIZE(lx2160a_pmux_groups); k++) {
+ const struct lx2160a_pmux_group *g =
+ &lx2160a_pmux_groups[k];
+ unsigned int l;
+
+ for (l = 0; l < g->nfuncs; l++)
+ if (!strcmp(g->funcs[l].name, name))
+ groups[n++] = g->name;
+ }
+ ret = pinmux_generic_add_function(pmux->pctl, name,
+ groups, n, NULL);
+ if (ret < 0)
+ return ret;
+ }
+ }
+
+ return 0;
+}
+
+static int lx2160a_pmux_probe(struct platform_device *pdev)
+{
+ struct device *dev = &pdev->dev;
+ struct lx2160a_pmux *pmux;
+ unsigned int i;
+ u32 val;
+ int ret;
+
+ /* The override needs the DCSR view of DCFG, mapped by fsl-guts */
+ if (!fsl_guts_rcw_override_supported())
+ return dev_err_probe(dev, -ENODEV,
+ "no RCW override through fsl-guts: the dcfg node needs its DCSR reg\n");
+
+ pmux = devm_kzalloc(dev, sizeof(*pmux), GFP_KERNEL);
+ if (!pmux)
+ return -ENOMEM;
+ pmux->dev = dev;
+
+ for (i = 0; i < ARRAY_SIZE(lx2160a_pmux_groups); i++) {
+ pmux->pins[i].number = i;
+ pmux->pins[i].name = lx2160a_pmux_groups[i].name;
+ }
+ pmux->desc.name = dev_name(dev);
+ pmux->desc.pins = pmux->pins;
+ pmux->desc.npins = ARRAY_SIZE(pmux->pins);
+ pmux->desc.pctlops = &lx2160a_pmux_pctlops;
+ pmux->desc.pmxops = &lx2160a_pmux_pmxops;
+ pmux->desc.owner = THIS_MODULE;
+
+ ret = devm_pinctrl_register_and_init(dev, &pmux->desc, pmux,
+ &pmux->pctl);
+ if (ret)
+ return dev_err_probe(dev, ret, "pinctrl registration failed\n");
+
+ for (i = 0; i < ARRAY_SIZE(lx2160a_pmux_groups); i++) {
+ ret = pinctrl_generic_add_group(pmux->pctl,
+ lx2160a_pmux_groups[i].name,
+ &pmux->pins[i].number, 1,
+ (void *)&lx2160a_pmux_groups[i]);
+ if (ret < 0)
+ return ret;
+ }
+
+ ret = lx2160a_pmux_add_functions(pmux);
+ if (ret)
+ return ret;
+
+ for (i = 12; i <= 14; i++) {
+ fsl_guts_rcw_read(i, &val);
+ dev_dbg(dev, "RCWSR%u = 0x%08x\n", i, val);
+ }
+
+ return pinctrl_enable(pmux->pctl);
+}
+
+static const struct of_device_id lx2160a_pmux_of_match[] = {
+ { .compatible = "fsl,lx2160a-rcw-pinmux" },
+ { }
+};
+MODULE_DEVICE_TABLE(of, lx2160a_pmux_of_match);
+
+static struct platform_driver lx2160a_pmux_driver = {
+ .driver = {
+ .name = "lx2160a-rcw-pinmux",
+ .of_match_table = lx2160a_pmux_of_match,
+ },
+ .probe = lx2160a_pmux_probe,
+};
+module_platform_driver(lx2160a_pmux_driver);
+
+MODULE_AUTHOR("Vincent Jardin <vjardin@free.fr>");
+MODULE_DESCRIPTION("NXP LX2160A RCW pad-mux driver");
+MODULE_LICENSE("GPL");
+MODULE_IMPORT_NS("FSL_GUTS");
--
2.43.0
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH v2 5/5] arm64: dts: lx2160a: describe the pad muxes as the DCFG RCW override
2026-11-03 18:12 [PATCH v2 0/5] lx2160a: change the pad muxes through the RCW override Vincent Jardin via B4 Relay
` (3 preceding siblings ...)
2026-11-03 18:12 ` [PATCH v2 4/5] pinctrl: freescale: add the LX2160A RCW pad-mux driver Vincent Jardin via B4 Relay
@ 2026-11-03 18:12 ` Vincent Jardin via B4 Relay
4 siblings, 0 replies; 8+ messages in thread
From: Vincent Jardin via B4 Relay @ 2026-11-03 18:12 UTC (permalink / raw)
To: Dong Aisheng, Fabio Estevam, Frank Li, Jacky Bai,
Pengutronix Kernel Team, NXP S32 Linux Team, Linus Walleij,
Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Christophe Leroy (CS GROUP), Shawn Guo, Sascha Hauer
Cc: linux-gpio, devicetree, linux-kernel, linuxppc-dev,
linux-arm-kernel, imx, Vladimir Oltean, Vincent Jardin
From: Vincent Jardin <vjardin@free.fr>
The pad muxes are fields of the RCW: read-only in RCWSR12..14, changed
at runtime through the override registers in the DCSR view of DCFG,
which read 0 until written.
pinctrl-single on those registers, since commit 8a1365c7bbc1
("arm64: dts: lx2160a: add pinmux and i2c gpio to support bus recovery"),
clears every other field of the word with its first pin state:
a board whose RCW programs a non-zero pad mux loses those pads at the
first I2C probe.
Describe the pad mux as a child of the dcfg node, which gains its DCSR
view: one group per RCW field, one function per encoding, driven by
fsl,lx2160a-rcw-pinmux through fsl-guts. Labels and node names stay,
so no consumer changes. The pinctrl-single node goes.
Signed-off-by: Vincent Jardin <vjardin@free.fr>
---
arch/arm64/boot/dts/freescale/fsl-lx216x.dtsi | 373 ++++++++++++++------------
1 file changed, 198 insertions(+), 175 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/fsl-lx216x.dtsi b/arch/arm64/boot/dts/freescale/fsl-lx216x.dtsi
index 2085362ab0ea..23e662d235a4 100644
--- a/arch/arm64/boot/dts/freescale/fsl-lx216x.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-lx216x.dtsi
@@ -669,9 +669,205 @@ clockgen: clock-controller@1300000 {
};
dcfg: syscon@1e00000 {
- compatible = "fsl,lx2160a-dcfg", "syscon";
- reg = <0x0 0x1e00000 0x0 0x10000>;
+ compatible = "fsl,lx2160a-dcfg", "syscon", "simple-mfd";
+ reg = <0x0 0x1e00000 0x0 0x10000>,
+ <0x7 0x00100000 0x0 0x10000>;
+ reg-names = "dcfg_ccsr", "dcfg_dcsr";
little-endian;
+
+ pinmux_i2crv: pinctrl {
+ compatible = "fsl,lx2160a-rcw-pinmux";
+
+ i2c1_pins: iic2-i2c-pins {
+ function = "i2c";
+ groups = "iic2";
+ };
+
+ gpio0_31_30_pins: iic2-gpio-pins {
+ function = "gpio";
+ groups = "iic2";
+ };
+
+ ftm0_ch10_pins: iic2-ftm-pins {
+ function = "flextimer";
+ groups = "iic2";
+ };
+
+ esdhc0_cd_wp_pins: iic2-sdhc-pins {
+ function = "sdhc";
+ groups = "iic2";
+ };
+
+ i2c2_pins: iic3-i2c-pins {
+ function = "i2c";
+ groups = "iic3";
+ };
+
+ gpio0_29_28_pins: iic3-gpio-pins {
+ function = "gpio";
+ groups = "iic3";
+ };
+
+ can0_pins: iic3-can-pins {
+ function = "can";
+ groups = "iic3";
+ };
+
+ event65_pins: iic3-event-pins {
+ function = "evt";
+ groups = "iic3";
+ };
+
+ i2c3_pins: iic4-i2c-pins {
+ function = "i2c";
+ groups = "iic4";
+ };
+
+ gpio0_27_26_pins: iic4-gpio-pins {
+ function = "gpio";
+ groups = "iic4";
+ };
+
+ can1_pins: iic4-can-pins {
+ function = "can";
+ groups = "iic4";
+ };
+
+ event87_pins: iic4-event-pins {
+ function = "evt";
+ groups = "iic4";
+ };
+
+ i2c4_pins: iic5-i2c-pins {
+ function = "i2c";
+ groups = "iic5";
+ };
+
+ gpio0_25_24_pins: iic5-gpio-pins {
+ function = "gpio";
+ groups = "iic5";
+ };
+
+ esdhc0_clksync_pins: iic5-sdhc-clk-pins {
+ function = "sdhc-clk";
+ groups = "iic5";
+ };
+
+ dspi2_miso_mosi_pins: iic5-spi3-pins {
+ function = "spi";
+ groups = "iic5";
+ };
+
+ i2c5_pins: iic6-i2c-pins {
+ function = "i2c";
+ groups = "iic6";
+ };
+
+ gpio0_23_22_pins: iic6-gpio-pins {
+ function = "gpio";
+ groups = "iic6";
+ };
+
+ esdhc1_clksync_pins: iic6-sdhc-clk-pins {
+ function = "sdhc-clk";
+ groups = "iic6";
+ };
+
+ fspi_data74_pins: xspi1-data74-pins {
+ function = "flexspi";
+ groups = "xspi1-data74";
+ };
+
+ gpio1_31_28_pins: xspi1-data74-gpio-pins {
+ function = "gpio";
+ groups = "xspi1-data74";
+ };
+
+ fspi_data30_pins: xspi1-data30-pins {
+ function = "flexspi";
+ groups = "xspi1-data30";
+ };
+
+ gpio1_27_24_pins: xspi1-data30-gpio-pins {
+ function = "gpio";
+ groups = "xspi1-data30";
+ };
+
+ fspi_dqs_sck_cs10_pins: xspi1-base-pins {
+ function = "flexspi";
+ groups = "xspi1-base";
+ };
+
+ gpio1_23_20_pins: xspi1-base-gpio-pins {
+ function = "gpio";
+ groups = "xspi1-base";
+ };
+
+ esdhc0_cmd_data30_clk_vsel_pins: sdhc1-base-sdhc-vsel-pins {
+ function = "sdhc";
+ groups = "sdhc1-base";
+ };
+
+ gpio0_21_15_pins: sdhc1-base-gpio-pins {
+ function = "gpio";
+ groups = "sdhc1-base";
+ };
+
+ dspi0_pins: sdhc1-base-spi1-pins {
+ function = "spi";
+ groups = "sdhc1-base";
+ };
+
+ esdhc0_cmd_data30_clk_dspi2_cs0_pins: sdhc1-base-sdhc-spi3-pins {
+ function = "sdhc-spi";
+ groups = "sdhc1-base";
+ };
+
+ esdhc0_cmd_data30_clk_data4_pins: sdhc1-base-sdhc-data4-pins {
+ function = "sdhc-data4";
+ groups = "sdhc1-base";
+ };
+
+ esdhc0_dir_pins: sdhc1-dir-pins {
+ function = "sdhc";
+ groups = "sdhc1-dir";
+ };
+
+ gpio0_14_12_pins: sdhc1-dir-gpio-pins {
+ function = "gpio";
+ groups = "sdhc1-dir";
+ };
+
+ dspi2_cs31_pins: sdhc1-dir-spi3-pins {
+ function = "spi";
+ groups = "sdhc1-dir";
+ };
+
+ esdhc0_data75_pins: sdhc1-dir-sdhc-pins {
+ function = "sdhc-data";
+ groups = "sdhc1-dir";
+ };
+
+ gpio1_18_15_pins: iic8-iic7-gpio-pins {
+ function = "gpio";
+ groups = "sdhc2-data74";
+ };
+
+ i2c6_i2c7_pins: iic8-iic7-i2c-pins {
+ function = "i2c";
+ groups = "sdhc2-data74";
+ };
+
+ i2c0_pins: iic1-i2c-pins {
+ function = "i2c";
+ groups = "iic1";
+ };
+
+ gpio0_3_2_pins: iic1-gpio-pins {
+ function = "gpio";
+ groups = "iic1";
+ };
+ };
};
sfp: efuse@1e80000 {
@@ -1531,179 +1727,6 @@ pcs18: ethernet-phy@0 {
};
};
- pinmux_i2crv: pinmux@70010012c {
- compatible = "pinctrl-single";
- reg = <0x00000007 0x0010012c 0x0 0xc>;
- #address-cells = <1>;
- #size-cells = <0>;
- #pinctrl-cells = <2>;
- pinctrl-single,bit-per-mux;
- pinctrl-single,register-width = <32>;
- pinctrl-single,function-mask = <0x7>;
-
- /* RCWSR12 */
- i2c1_pins: iic2-i2c-pins {
- pinctrl-single,bits = <0x0 0x0 0x7>;
- };
-
- gpio0_31_30_pins: iic2-gpio-pins {
- pinctrl-single,bits = <0x0 0x1 0x7>;
- };
-
- ftm0_ch10_pins: iic2-ftm-pins {
- pinctrl-single,bits = <0x0 0x2 0x7>;
- };
-
- esdhc0_cd_wp_pins: iic2-sdhc-pins {
- pinctrl-single,bits = <0x0 0x6 0x7>;
- };
-
- i2c2_pins: iic3-i2c-pins {
- pinctrl-single,bits = <0x0 0x0 (0x7 << 3)>;
- };
-
- gpio0_29_28_pins: iic3-gpio-pins {
- pinctrl-single,bits = <0x0 (0x1 << 3) (0x7 << 3)>;
- };
-
- can0_pins: iic3-can-pins {
- pinctrl-single,bits = <0x0 (0x2 << 3) (0x7 << 3)>;
- };
-
- event65_pins: iic3-event-pins {
- pinctrl-single,bits = <0x0 (0x6 << 3) (0x7 << 3)>;
- };
-
- i2c3_pins: iic4-i2c-pins {
- pinctrl-single,bits = <0x0 0x0 (0x7 << 6)>;
- };
-
- gpio0_27_26_pins: iic4-gpio-pins {
- pinctrl-single,bits = <0x0 (0x1 << 6) (0x7 << 6)>;
- };
-
- can1_pins: iic4-can-pins {
- pinctrl-single,bits = <0x0 (0x2 << 6) (0x7 << 6)>;
- };
-
- event87_pins: iic4-event-pins {
- pinctrl-single,bits = <0x0 (0x6 << 6) (0x7 << 6)>;
- };
-
- i2c4_pins: iic5-i2c-pins {
- pinctrl-single,bits = <0x0 0x0 (0x7 << 9)>;
- };
-
- gpio0_25_24_pins: iic5-gpio-pins {
- pinctrl-single,bits = <0x0 (0x1 << 9) (0x7 << 9)>;
- };
-
- esdhc0_clksync_pins: iic5-sdhc-clk-pins {
- pinctrl-single,bits = <0x0 (0x2 << 9) (0x7 << 9)>;
- };
-
- dspi2_miso_mosi_pins: iic5-spi3-pins {
- pinctrl-single,bits = <0x0 (0x3 << 9) (0x7 << 9)>;
- };
-
- i2c5_pins: iic6-i2c-pins {
- pinctrl-single,bits = <0x0 0x0 (0x7 << 12)>;
- };
-
- gpio0_23_22_pins: iic6-gpio-pins {
- pinctrl-single,bits = <0x0 (0x1 << 12) (0x7 << 12)>;
- };
-
- esdhc1_clksync_pins: iic6-sdhc-clk-pins {
- pinctrl-single,bits = <0x0 (0x2 << 12) (0x7 << 12)>;
- };
-
- fspi_data74_pins: xspi1-data74-pins {
- pinctrl-single,bits = <0x0 0x0 (0x7 << 15)>;
- };
-
- gpio1_31_28_pins: xspi1-data74-gpio-pins {
- pinctrl-single,bits = <0x0 (0x1 << 15) (0x7 << 15)>;
- };
-
- fspi_data30_pins: xspi1-data30-pins {
- pinctrl-single,bits = <0x0 0x0 (0x7 << 18)>;
- };
-
- gpio1_27_24_pins: xspi1-data30-gpio-pins {
- pinctrl-single,bits = <0x0 (0x1 << 18) (0x7 << 18)>;
- };
-
- fspi_dqs_sck_cs10_pins: xspi1-base-pins {
- pinctrl-single,bits = <0x0 0x0 (0x7 << 21)>;
- };
-
- gpio1_23_20_pins: xspi1-base-gpio-pins {
- pinctrl-single,bits = <0x0 (0x1 << 21) (0x7 << 21)>;
- };
-
- esdhc0_cmd_data30_clk_vsel_pins: sdhc1-base-sdhc-vsel-pins {
- pinctrl-single,bits = <0x0 0x0 (0x7 << 24)>;
- };
-
- gpio0_21_15_pins: sdhc1-base-gpio-pins {
- pinctrl-single,bits = <0x0 (0x1 << 24) (0x7 << 24)>;
- };
-
- dspi0_pins: sdhc1-base-spi1-pins {
- pinctrl-single,bits = <0x0 (0x2 << 24) (0x7 << 24)>;
- };
-
- esdhc0_cmd_data30_clk_dspi2_cs0_pins: sdhc1-base-sdhc-spi3-pins {
- pinctrl-single,bits = <0x0 (0x3 << 24) (0x7 << 24)>;
- };
-
- esdhc0_cmd_data30_clk_data4_pins: sdhc1-base-sdhc-data4-pins {
- pinctrl-single,bits = <0x0 (0x4 << 24) (0x7 << 24)>;
- };
-
- esdhc0_dir_pins: sdhc1-dir-pins {
- pinctrl-single,bits = <0x0 0x0 (0x7 << 27)>;
- };
-
- gpio0_14_12_pins: sdhc1-dir-gpio-pins {
- pinctrl-single,bits = <0x0 (0x1 << 27) (0x7 << 27)>;
- };
-
- dspi2_cs31_pins: sdhc1-dir-spi3-pins {
- pinctrl-single,bits = <0x0 (0x3 << 27) (0x7 << 27)>;
- };
-
- esdhc0_data75_pins: sdhc1-dir-sdhc-pins {
- pinctrl-single,bits = <0x0 (0x4 << 27) (0x7 << 27)>;
- };
-
- /* RCWSR13 */
- gpio1_18_15_pins: iic8-iic7-gpio-pins {
- pinctrl-single,bits = <0x4 0x1 0x7>;
- };
-
- i2c6_i2c7_pins: iic8-iic7-i2c-pins {
- pinctrl-single,bits = <0x4 0x2 0x7>;
- };
-
- /* RCWSR14 */
- /*
- * IIC1_PMUX is a single bit at RCWSR14[10], but bit-per-mux
- * uses the mask for the field to clear before applying the
- * value.
- * So bits 11 and 12, which are reserved, are written as 0
- * on every mux switch, it is harmless.
- */
- i2c0_pins: iic1-i2c-pins {
- pinctrl-single,bits = <0x8 0x0 (0x7 << 10)>;
- };
-
- gpio0_3_2_pins: iic1-gpio-pins {
- pinctrl-single,bits = <0x8 (0x1 << 10) (0x7 << 10)>;
- };
- };
-
fsl_mc: fsl-mc@80c000000 {
compatible = "fsl,qoriq-mc";
reg = <0x00000008 0x0c000000 0 0x40>,
--
2.43.0
^ permalink raw reply related [flat|nested] 8+ messages in thread