fix: Handle member structure pointers inside a structure (#471)

Signed-off-by: shubham kumar <shubham.kumar@intel.com>

Author: shubskmr

bindings/sysman/python/test/validate_structures.py

@@ -101,13 +101,15 @@ def parse_python_structure(
     fields_str = match.group(1)
     fields = []
 
-    # Parse each field tuple
-    field_pattern = r'\(\s*"([^"]+)"\s*,\s*([^\)]+?)\s*\)'
-
-    for field_match in re.finditer(field_pattern, fields_str):
-        field_name = field_match.group(1).strip()
-        field_type = field_match.group(2).strip().rstrip(",")
-        fields.append((field_name, field_type))
+    # Parse line by line to handle nested parentheses like POINTER(...)
+    for line in fields_str.split("\n"):
+        # Match: ("field_name", field_type), with optional comment
+        pattern = r'\s*\(\s*"([^"]+)"\s*,\s*(.+?)\s*\)\s*,?\s*(?:#.*)?$'
+        field_match = re.match(pattern, line)
+        if field_match:
+            field_name = field_match.group(1)
+            field_type = field_match.group(2).strip().rstrip(",")
+            fields.append((field_name, field_type))
 
     return fields if fields else None
 
@@ -126,6 +128,19 @@ def normalize_c_type(c_type: str) -> str:
         python_base = C_TO_PYTHON_TYPE_MAP.get(base_type, base_type)
         return f"{python_base} * {size}"
 
+    # Handle pointers to custom structures:
+    # e.g., zes_power_limit_ext_desc_t* -> POINTER(zes_power_limit_ext_desc_t)
+    if c_type.endswith("*"):
+        # Check if it's a mapped type first (like void*, const void*)
+        if c_type in C_TO_PYTHON_TYPE_MAP:
+            return C_TO_PYTHON_TYPE_MAP[c_type]
+
+        # Otherwise, it's a pointer to a custom type
+        base_type = c_type[:-1].strip()  # Remove * and trailing spaces
+        # Check if base type is in map (unlikely for pointers, but be safe)
+        python_base = C_TO_PYTHON_TYPE_MAP.get(base_type, base_type)
+        return f"POINTER({python_base})"
+
     # Map type if in dictionary, otherwise keep as-is
     return C_TO_PYTHON_TYPE_MAP.get(c_type, c_type)
 
@@ -150,7 +165,9 @@ def types_are_equivalent(type1: str, type2: str) -> bool:
 
 
 def compare_structures(
-    c_fields: List[Tuple[str, str]], py_fields: List[Tuple[str, str]], struct_name: str
+    c_fields: List[Tuple[str, str]],
+    py_fields: List[Tuple[str, str]],
+    struct_name: str,
 ) -> List[str]:
     """
     Compare C and Python structure fields.
@@ -181,18 +198,22 @@ def compare_structures(
 
         # Check field name
         if c_name != py_name:
-            errors.append(
-                f"Field {i}: name mismatch - C: '{c_name}' ({c_type}), Python: '{py_name}' ({py_type})"
+            msg = (
+                f"Field {i}: name mismatch - "
+                f"C: '{c_name}' ({c_type}), Python: '{py_name}' ({py_type})"
             )
-            # Skip type comparison when names don't match - they're different fields
+            errors.append(msg)
+            # Skip type comparison when names don't match
             continue
 
         # Check field type (only if names match)
         expected_type = normalize_c_type(c_type)
         if not types_are_equivalent(expected_type, py_type):
-            errors.append(
-                f"Field '{c_name}' (position {i}): type mismatch - C: '{c_type}' -> '{expected_type}', Python: '{py_type}'"
+            msg = (
+                f"Field '{c_name}' (position {i}): type mismatch - "
+                f"C: '{c_type}' -> '{expected_type}', Python: '{py_type}'"
             )
+            errors.append(msg)
 
     return errors