ci(variant): enhance I3C support generation

HAL I3C module is enabled only if I3C is available.

Signed-off-by: Frederic Pillon <frederic.pillon@st.com>

Author: fpistm

CI/update/stm32variant.py

@@ -1478,6 +1478,25 @@ def i2c_pins_variant():
     return dict(sda=sda_pin, scl=scl_pin)
 
 
+def i3c_pins_variant():
+    sda_pin = scl_pin = "PNUM_NOT_DEFINED"
+    # Iterate to find match instance if any
+    for sda in i3csda_list:
+        sda_inst = sda[2].split("_", 1)[0]
+        for scl in i3cscl_list:
+            scl_inst = scl[2].split("_", 1)[0]
+            if sda_inst == scl_inst:
+                sda_pin = sda[0].replace("_", "", 1)
+                scl_pin = scl[0].replace("_", "", 1)
+                break
+        else:
+            continue
+        break
+    else:
+        print("No I3C found!")
+    return dict(sda=sda_pin, scl=scl_pin)
+
+
 def serial_pins_variant():
     # Manage (LP)U(S)ART pins
     if uarttx_list:
@@ -1591,6 +1610,12 @@ def print_variant(generic_list, alt_syswkup_list):
     # I2C definition
     i2c_pins = i2c_pins_variant()
 
+    # I3C definition if any
+    if i3csda_list and i3cscl_list:
+        i3c_pins = i3c_pins_variant()
+    else:
+        i3c_pins = None
+
     # Serial definition
     serial = serial_pins_variant()
 
@@ -1660,6 +1685,8 @@ def print_variant(generic_list, alt_syswkup_list):
         hal_modules_list.append("DAC")
     if eth_list:
         hal_modules_list.append("ETH")
+    if i3csda_list and i3cscl_list:
+        hal_modules_list.append("I3C")
     if xspidata0_list:
         if "OCTOSPI" in xspidata0_list[0][2]:
             hal_modules_list.append("OSPI")
@@ -1681,6 +1708,7 @@ def print_variant(generic_list, alt_syswkup_list):
             num_analog_inputs=len(analog_pins_list),
             spi_pins=spi_pins,
             i2c_pins=i2c_pins,
+            i3c_pins=i3c_pins,
             timer=timer,
             serial=serial,
             hal_modules_list=hal_modules_list,

CI/update/templates/variant_generic.h

@@ -75,6 +75,16 @@
 #ifndef PIN_WIRE_SCL
   #define PIN_WIRE_SCL          {{i2c_pins.scl}}
 #endif
+{% if i3c_pins %}
+
+// I3C definitions
+#ifndef PIN_I3C_SDA
+  #define PIN_I3C_SDA           {{i3c_pins.sda}}
+#endif
+#ifndef PIN_I3C_SCL
+  #define PIN_I3C_SCL           {{i3c_pins.scl}}
+#endif
+{% endif %}
 
 // Timer Definitions
 // Use TIM6/TIM7 when possible as servo and tone don't need GPIO output pin

cores/arduino/stm32/stm32yyxx_hal_conf.h

@@ -46,12 +46,6 @@
   #undef HAL_I2S_MODULE_ENABLED
 #endif
 
-#if !defined(HAL_I3C_MODULE_DISABLED)
-  #define HAL_I3C_MODULE_ENABLED
-#else
-  #undef HAL_I3C_MODULE_ENABLED
-#endif
-
 #if !defined(HAL_RTC_MODULE_DISABLED)
   #define HAL_RTC_MODULE_ENABLED
 #else
@@ -117,6 +111,12 @@
   #undef HAL_EXTI_MODULE_ENABLED
 #endif
 
+#if !defined(HAL_I3C_MODULE_DISABLED)
+  /*#define HAL_I3C_MODULE_ENABLED*/
+#else
+  #undef HAL_I3C_MODULE_ENABLED
+#endif
+
 #if !defined(HAL_ETH_MODULE_DISABLED)
   /*#define HAL_ETH_MODULE_ENABLED*/
 #else