drivers: pinctrl: microchip: update pinctrl driver for Port G1 IP

Update pinctrl driver for Microchip Port G1 Peripheral IPs

Signed-off-by: Mohamed Azhar <[email protected]>

Author: AzharMCHP

drivers/pinctrl/pinctrl_mchp_port_g1.c

@@ -27,23 +27,23 @@
 /* clang-format off */
 #define MCHP_PORT_ADDR_OR_NONE(nodelabel)                              \
 	IF_ENABLED(DT_NODE_EXISTS(DT_NODELABEL(nodelabel)),            \
-		(DT_REG_ADDR(DT_NODELABEL(nodelabel))))
-/* clang-format on */
+		(DT_REG_ADDR(DT_NODELABEL(nodelabel)),))
 
 /**
- * @brief Array of port addresses for the MCHP SAMD5x_E5x series.
+ * @brief Array of port addresses for the MCHP G1 series.
  *
  * This array contains the register addresses of the ports (PORTA, PORTB, PORTC, and PORTD)
- * for the MCHP SAMD5x_E5x series microcontrollers. The addresses are obtained using the
+ * for the MCHP G1 series port peripheral. The addresses are obtained using the
  * MCHP_PORT_ADDR_OR_NONE macro, which ensures that only existing ports are included.
- * This can be updated for other devices using conditional comiplation directives
+ * This can be updated for other devices using conditional compilation directives
  */
 static const uint32_t mchp_port_addrs[] = {
-	MCHP_PORT_ADDR_OR_NONE(porta),
-	MCHP_PORT_ADDR_OR_NONE(portb),
-	MCHP_PORT_ADDR_OR_NONE(portc),
-	MCHP_PORT_ADDR_OR_NONE(portd),
+	MCHP_PORT_ADDR_OR_NONE(porta)
+	MCHP_PORT_ADDR_OR_NONE(portb)
+	MCHP_PORT_ADDR_OR_NONE(portc)
+	MCHP_PORT_ADDR_OR_NONE(portd)
 };
+/* clang-format on */
 
 /**
  * @brief Set pinmux registers using odd/even logic

soc/microchip/pic32c/pic32cm_jh/Kconfig.soc

@@ -16,6 +16,12 @@ config GPIO_MCHP_PORT_U2210_3_1_0
 	help
 	  Enable PORT GPIO peripheral IP version id="U2210" version="3.1.0".
 
+config PINCTRL_MCHP_PORT_U2210_3_1_0
+	bool
+	default y
+	help
+	  Enable PORT pinctrl peripheral IP version 3.1.0 (U2210).
+
 endif # SOC_FAMILY_MICROCHIP_PIC32CM_JH
 
 rsource "*/Kconfig.soc"

soc/microchip/pic32c/pic32cm_jh/common/CMakeLists.txt

@@ -2,5 +2,6 @@
 # SPDX-License-Identifier: Apache-2.0
 
 zephyr_sources(soc.c)
+zephyr_include_directories(.)
 
 set(SOC_LINKER_SCRIPT ${ZEPHYR_BASE}/include/zephyr/arch/arm/cortex_m/scripts/linker.ld CACHE INTERNAL "")

soc/microchip/pic32c/pic32cm_jh/common/pinctrl_soc.h

@@ -0,0 +1,105 @@
+/*
+ * Copyright (c) 2025 Microchip Technology Inc.
+ *
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+#include <zephyr/devicetree.h>
+#include <zephyr/types.h>
+#include <dt-bindings/pic32c/pic32cm_jh/common/mchp_pinctrl_pinmux_pic32c.h>
+
+#ifdef __cplusplus
+extern "C" {
+#endif
+
+/** @cond INTERNAL_HIDDEN */
+
+/**
+ * @brief Structure representing the pin control settings for a SOC pin.
+ *
+ * This structure is used to define the pinmux and pin configuration settings
+ * for a specific pin on the SOC. It includes information about the port, pin,
+ * function, bias, drive, and other configuration options.
+ */
+typedef struct pinctrl_soc_pin {
+	/** Pinmux settings (port, pin and function). */
+	uint16_t pinmux;
+	/** Pin configuration (bias, drive etc). */
+	uint16_t pinflag;
+} pinctrl_soc_pin_t;
+
+/**
+ * @brief Utility macro to initialize pinmux field.
+ *
+ * @param node_id Node identifier.
+ * @param prop Property name.
+ * @param idx Property entry index.
+ */
+#define Z_PINCTRL_MCHP_PINMUX_INIT(node_id, prop, idx) DT_PROP_BY_IDX(node_id, prop, idx)
+
+/**
+ * @brief Utility macro to initialize each pin flag.
+ *
+ * @param node_id Node identifier.
+ * @param prop Property name.
+ * @param idx Property entry index.
+ */
+#define Z_PINCTRL_MCHP_PINFLAG_INIT(node_id, prop, idx)                                            \
+	((DT_PROP(node_id, bias_pull_up) << MCHP_PINCTRL_PULLUP_POS) |                             \
+	 (DT_PROP(node_id, bias_pull_down) << MCHP_PINCTRL_PULLDOWN_POS) |                         \
+	 (DT_PROP(node_id, input_enable) << MCHP_PINCTRL_INPUTENABLE_POS) |                        \
+	 (DT_PROP(node_id, output_enable) << MCHP_PINCTRL_OUTPUTENABLE_POS) |                      \
+	 (DT_ENUM_IDX(node_id, drive_strength) << MCHP_PINCTRL_DRIVESTRENGTH_POS)),
+
+/**
+ * @brief Utility macro to initialize each pin.
+ *
+ * @param node_id Node identifier.
+ * @param prop Property name.
+ * @param idx Property entry index.
+ */
+#define Z_PINCTRL_STATE_PIN_INIT(node_id, prop, idx)                                               \
+	{.pinmux = Z_PINCTRL_MCHP_PINMUX_INIT(node_id, prop, idx),                                 \
+	 .pinflag = Z_PINCTRL_MCHP_PINFLAG_INIT(node_id, prop, idx)},
+
+/**
+ * @brief Utility macro to initialize state pins contained in a given property.
+ *
+ * @param node_id Node identifier.
+ * @param prop Property name describing state pins.
+ */
+#define Z_PINCTRL_STATE_PINS_INIT(node_id, prop)                                                   \
+	{DT_FOREACH_CHILD_VARGS(DT_PHANDLE(node_id, prop), DT_FOREACH_PROP_ELEM, pinmux,           \
+				Z_PINCTRL_STATE_PIN_INIT)}
+
+/** @endcond */
+
+/**
+ * @brief Pin flags/attributes
+ * @anchor MCHP_PINFLAGS
+ *
+ * @{
+ */
+
+#define MCHP_PINCTRL_FLAGS_DEFAULT     (0U)
+#define MCHP_PINCTRL_FLAGS_POS         (0U)
+#define MCHP_PINCTRL_FLAGS_MASK        (0x3F << MCHP_PINCTRL_FLAGS_POS)
+#define MCHP_PINCTRL_FLAG_MASK         (1U)
+#define MCHP_PINCTRL_PULLUP_POS        (MCHP_PINCTRL_FLAGS_POS)
+#define MCHP_PINCTRL_PULLUP            (1U << MCHP_PINCTRL_PULLUP_POS)
+#define MCHP_PINCTRL_PULLDOWN_POS      (MCHP_PINCTRL_PULLUP_POS + 1U)
+#define MCHP_PINCTRL_PULLDOWN          (1U << MCHP_PINCTRL_PULLDOWN_POS)
+#define MCHP_PINCTRL_OPENDRAIN_POS     (MCHP_PINCTRL_PULLDOWN_POS + 1U)
+#define MCHP_PINCTRL_OPENDRAIN         (1U << MCHP_PINCTRL_OPENDRAIN_POS)
+#define MCHP_PINCTRL_INPUTENABLE_POS   (MCHP_PINCTRL_OPENDRAIN_POS + 1U)
+#define MCHP_PINCTRL_INPUTENABLE       (1U << MCHP_PINCTRL_INPUTENABLE_POS)
+#define MCHP_PINCTRL_OUTPUTENABLE_POS  (MCHP_PINCTRL_INPUTENABLE_POS + 1U)
+#define MCHP_PINCTRL_OUTPUTENABLE      (1U << MCHP_PINCTRL_OUTPUTENABLE_POS)
+#define MCHP_PINCTRL_DRIVESTRENGTH_POS (MCHP_PINCTRL_OUTPUTENABLE_POS + 1U)
+#define MCHP_PINCTRL_DRIVESTRENGTH     (1U << MCHP_PINCTRL_DRIVESTRENGTH_POS)
+
+/** @} */
+
+#ifdef __cplusplus
+}
+#endif