From 17fdd141a31a55dc5a51d0421ea167698f039b2f Mon Sep 17 00:00:00 2001 From: Emilio Benavente Date: Thu, 7 Aug 2025 09:41:53 -0500 Subject: [PATCH 1/2] mcux: scripts: pinctrl: Updated Offset value for PMIC I2C Updated the scripts offset value for the PMIC I2C signals when using RT7XX Devices. An accidental prefix '0x' was added to the offset's actual decimal value. e.g. 96 -> 0x96. Real value should be 96 -> 0x60. Signed-off-by: Emilio Benavente --- mcux/scripts/pinctrl/lpc/lpc_cfg_utils.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/mcux/scripts/pinctrl/lpc/lpc_cfg_utils.py b/mcux/scripts/pinctrl/lpc/lpc_cfg_utils.py index 4fff304ad7..0b7df0a40a 100644 --- a/mcux/scripts/pinctrl/lpc/lpc_cfg_utils.py +++ b/mcux/scripts/pinctrl/lpc/lpc_cfg_utils.py @@ -122,11 +122,11 @@ def __init__(self, connection, imx_rt = ''): self._mux = 0 elif self._name == 'PMIC_I2C_SCL' and imx_rt == "MIMXRT7XX": self._index = 1 - self._offset = 0x96 + self._offset = 0x60 self._mux = 0 elif self._name == 'PMIC_I2C_SDA' and imx_rt == "MIMXRT7XX": self._index = 1 - self._offset = 0x97 + self._offset = 0x61 self._mux = 0 if re.match(r'^\d', self._name): # If string starts with a digit, it will not be a valid C name From d0a9971e7e95f2c0d868bc6ea8ce3c7c144877de Mon Sep 17 00:00:00 2001 From: Emilio Benavente Date: Thu, 7 Aug 2025 09:51:07 -0500 Subject: [PATCH 2/2] dts: nxp: nxp_imx: rt: Updated offset values for PMIC I2C Updated the offset values for the PMIC I2C to point to the correct offset values. Signed-off-by: Emilio Benavente --- dts/nxp/nxp_imx/rt/MIMXRT735SGAWAR-pinctrl.h | 4 ++-- dts/nxp/nxp_imx/rt/MIMXRT735SGFOA-pinctrl.h | 4 ++-- dts/nxp/nxp_imx/rt/MIMXRT758SGAWAR-pinctrl.h | 4 ++-- dts/nxp/nxp_imx/rt/MIMXRT758SGFOA-pinctrl.h | 4 ++-- dts/nxp/nxp_imx/rt/MIMXRT798SGAWAR-pinctrl.h | 4 ++-- dts/nxp/nxp_imx/rt/MIMXRT798SGFOA-pinctrl.h | 4 ++-- 6 files changed, 12 insertions(+), 12 deletions(-) diff --git a/dts/nxp/nxp_imx/rt/MIMXRT735SGAWAR-pinctrl.h b/dts/nxp/nxp_imx/rt/MIMXRT735SGAWAR-pinctrl.h index 9271791aec..f965de04bd 100644 --- a/dts/nxp/nxp_imx/rt/MIMXRT735SGAWAR-pinctrl.h +++ b/dts/nxp/nxp_imx/rt/MIMXRT735SGAWAR-pinctrl.h @@ -14,8 +14,8 @@ (((offset) & 0xFFF) << 20) | \ (((mux) & 0xF) << 0)) -#define PMIC_I2C_SCL IOPCTL_MUX(1, 150, 0) /* PIO0_0 */ -#define PMIC_I2C_SDA IOPCTL_MUX(1, 151, 0) /* PIO0_0 */ +#define PMIC_I2C_SCL IOPCTL_MUX(1, 96, 0) /* PIO0_0 */ +#define PMIC_I2C_SDA IOPCTL_MUX(1, 97, 0) /* PIO0_0 */ #define CTIMER0_CAPTURE0_PIO0_0 IOPCTL_MUX(0, 0, 4) /* PIO0_0 */ #define CTIMER0_CAPTURE1_PIO0_0 IOPCTL_MUX(0, 0, 4) /* PIO0_0 */ #define CTIMER0_CAPTURE2_PIO0_0 IOPCTL_MUX(0, 0, 4) /* PIO0_0 */ diff --git a/dts/nxp/nxp_imx/rt/MIMXRT735SGFOA-pinctrl.h b/dts/nxp/nxp_imx/rt/MIMXRT735SGFOA-pinctrl.h index 9dae09ddf0..29ba8fe030 100644 --- a/dts/nxp/nxp_imx/rt/MIMXRT735SGFOA-pinctrl.h +++ b/dts/nxp/nxp_imx/rt/MIMXRT735SGFOA-pinctrl.h @@ -14,8 +14,8 @@ (((offset) & 0xFFF) << 20) | \ (((mux) & 0xF) << 0)) -#define PMIC_I2C_SCL IOPCTL_MUX(1, 150, 0) /* PIO0_0 */ -#define PMIC_I2C_SDA IOPCTL_MUX(1, 151, 0) /* PIO0_0 */ +#define PMIC_I2C_SCL IOPCTL_MUX(1, 96, 0) /* PIO0_0 */ +#define PMIC_I2C_SDA IOPCTL_MUX(1, 97, 0) /* PIO0_0 */ #define CTIMER0_CAPTURE0_PIO0_0 IOPCTL_MUX(0, 0, 4) /* PIO0_0 */ #define CTIMER0_CAPTURE1_PIO0_0 IOPCTL_MUX(0, 0, 4) /* PIO0_0 */ #define CTIMER0_CAPTURE2_PIO0_0 IOPCTL_MUX(0, 0, 4) /* PIO0_0 */ diff --git a/dts/nxp/nxp_imx/rt/MIMXRT758SGAWAR-pinctrl.h b/dts/nxp/nxp_imx/rt/MIMXRT758SGAWAR-pinctrl.h index 979fc31983..4249112ce8 100644 --- a/dts/nxp/nxp_imx/rt/MIMXRT758SGAWAR-pinctrl.h +++ b/dts/nxp/nxp_imx/rt/MIMXRT758SGAWAR-pinctrl.h @@ -14,8 +14,8 @@ (((offset) & 0xFFF) << 20) | \ (((mux) & 0xF) << 0)) -#define PMIC_I2C_SCL IOPCTL_MUX(1, 150, 0) /* PIO0_0 */ -#define PMIC_I2C_SDA IOPCTL_MUX(1, 151, 0) /* PIO0_0 */ +#define PMIC_I2C_SCL IOPCTL_MUX(1, 96, 0) /* PIO0_0 */ +#define PMIC_I2C_SDA IOPCTL_MUX(1, 97, 0) /* PIO0_0 */ #define CTIMER0_CAPTURE0_PIO0_0 IOPCTL_MUX(0, 0, 4) /* PIO0_0 */ #define CTIMER0_CAPTURE1_PIO0_0 IOPCTL_MUX(0, 0, 4) /* PIO0_0 */ #define CTIMER0_CAPTURE2_PIO0_0 IOPCTL_MUX(0, 0, 4) /* PIO0_0 */ diff --git a/dts/nxp/nxp_imx/rt/MIMXRT758SGFOA-pinctrl.h b/dts/nxp/nxp_imx/rt/MIMXRT758SGFOA-pinctrl.h index b87e61e452..01a1545032 100644 --- a/dts/nxp/nxp_imx/rt/MIMXRT758SGFOA-pinctrl.h +++ b/dts/nxp/nxp_imx/rt/MIMXRT758SGFOA-pinctrl.h @@ -14,8 +14,8 @@ (((offset) & 0xFFF) << 20) | \ (((mux) & 0xF) << 0)) -#define PMIC_I2C_SCL IOPCTL_MUX(1, 150, 0) /* PIO0_0 */ -#define PMIC_I2C_SDA IOPCTL_MUX(1, 151, 0) /* PIO0_0 */ +#define PMIC_I2C_SCL IOPCTL_MUX(1, 96, 0) /* PIO0_0 */ +#define PMIC_I2C_SDA IOPCTL_MUX(1, 97, 0) /* PIO0_0 */ #define CTIMER0_CAPTURE0_PIO0_0 IOPCTL_MUX(0, 0, 4) /* PIO0_0 */ #define CTIMER0_CAPTURE1_PIO0_0 IOPCTL_MUX(0, 0, 4) /* PIO0_0 */ #define CTIMER0_CAPTURE2_PIO0_0 IOPCTL_MUX(0, 0, 4) /* PIO0_0 */ diff --git a/dts/nxp/nxp_imx/rt/MIMXRT798SGAWAR-pinctrl.h b/dts/nxp/nxp_imx/rt/MIMXRT798SGAWAR-pinctrl.h index d6b5cd0d6a..6b22f434f4 100644 --- a/dts/nxp/nxp_imx/rt/MIMXRT798SGAWAR-pinctrl.h +++ b/dts/nxp/nxp_imx/rt/MIMXRT798SGAWAR-pinctrl.h @@ -14,8 +14,8 @@ (((offset) & 0xFFF) << 20) | \ (((mux) & 0xF) << 0)) -#define PMIC_I2C_SCL IOPCTL_MUX(1, 150, 0) /* PIO0_0 */ -#define PMIC_I2C_SDA IOPCTL_MUX(1, 151, 0) /* PIO0_0 */ +#define PMIC_I2C_SCL IOPCTL_MUX(1, 96, 0) /* PIO0_0 */ +#define PMIC_I2C_SDA IOPCTL_MUX(1, 97, 0) /* PIO0_0 */ #define CTIMER0_CAPTURE0_PIO0_0 IOPCTL_MUX(0, 0, 4) /* PIO0_0 */ #define CTIMER0_CAPTURE1_PIO0_0 IOPCTL_MUX(0, 0, 4) /* PIO0_0 */ #define CTIMER0_CAPTURE2_PIO0_0 IOPCTL_MUX(0, 0, 4) /* PIO0_0 */ diff --git a/dts/nxp/nxp_imx/rt/MIMXRT798SGFOA-pinctrl.h b/dts/nxp/nxp_imx/rt/MIMXRT798SGFOA-pinctrl.h index 26e430c65d..4ac14bdd09 100644 --- a/dts/nxp/nxp_imx/rt/MIMXRT798SGFOA-pinctrl.h +++ b/dts/nxp/nxp_imx/rt/MIMXRT798SGFOA-pinctrl.h @@ -14,8 +14,8 @@ (((offset) & 0xFFF) << 20) | \ (((mux) & 0xF) << 0)) -#define PMIC_I2C_SCL IOPCTL_MUX(1, 150, 0) /* PIO0_0 */ -#define PMIC_I2C_SDA IOPCTL_MUX(1, 151, 0) /* PIO0_0 */ +#define PMIC_I2C_SCL IOPCTL_MUX(1, 96, 0) /* PIO0_0 */ +#define PMIC_I2C_SDA IOPCTL_MUX(1, 97, 0) /* PIO0_0 */ #define CTIMER0_CAPTURE0_PIO0_0 IOPCTL_MUX(0, 0, 4) /* PIO0_0 */ #define CTIMER0_CAPTURE1_PIO0_0 IOPCTL_MUX(0, 0, 4) /* PIO0_0 */ #define CTIMER0_CAPTURE2_PIO0_0 IOPCTL_MUX(0, 0, 4) /* PIO0_0 */