fix(CMSIS): Call PinInit from SystemInit in RISC-V startup (#1430)

Signed-off-by: Michael Perkins <[email protected]>

Author: perkinsmg

Libraries/CMSIS/Device/Maxim/MAX32520/Source/system_max32520.c

@@ -2,7 +2,7 @@
  *
  * Copyright (C) 2022-2023 Maxim Integrated Products, Inc. (now owned by 
  * Analog Devices, Inc.),
- * Copyright (C) 2023-2024 Analog Devices, Inc.
+ * Copyright (C) 2023-2025 Analog Devices, Inc.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -81,7 +81,7 @@ __weak void SystemCoreClockUpdate(void)
  * implemented by the application for early initializations. If a value other
  * than '0' is returned, the C runtime initialization will be skipped.
  *
- * You may over-ride this function in your program by defining a custom 
+ * You may override this function in your program by defining a custom 
  *  PreInit(), but care should be taken to reproduce the initialization steps
  *  or a non-functional system may result.
  */
@@ -92,7 +92,7 @@ __weak int PreInit(void)
 }
 
 /* This function is called before the Board_Init function.  This weak 
- * implementation does nothing, but you may over-ride this function in your 
+ * implementation does nothing, but you may override this function in your 
  * program if you want to configure the state of all pins prior to the 
  * application running.  This is useful when using external tools (like a
  * Pin Mux configuration tool) that generate code to initialize the pins.
@@ -117,7 +117,7 @@ __weak int Board_Init(void)
 
 /* This function is called just before control is transferred to main().
  *
- * You may over-ride this function in your program by defining a custom 
+ * You may override this function in your program by defining a custom 
  *  SystemInit(), but care should be taken to reproduce the initialization
  *  steps or a non-functional system may result.
  */

Libraries/CMSIS/Device/Maxim/MAX32570/Source/system_max32570.c

@@ -2,7 +2,7 @@
  *
  * Copyright (C) 2022-2023 Maxim Integrated Products, Inc. (now owned by 
  * Analog Devices, Inc.),
- * Copyright (C) 2023-2024 Analog Devices, Inc.
+ * Copyright (C) 2023-2025 Analog Devices, Inc.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -87,7 +87,7 @@ __weak void SystemCoreClockUpdate(void)
  * implemented by the application for early initializations. If a value other
  * than '0' is returned, the C runtime initialization will be skipped.
  *
- * You may over-ride this function in your program by defining a custom 
+ * You may override this function in your program by defining a custom 
  *  PreInit(), but care should be taken to reproduce the initialization steps
  *  or a non-functional system may result.
  */
@@ -98,7 +98,7 @@ __weak int PreInit(void)
 }
 
 /* This function is called before the Board_Init function.  This weak 
- * implementation does nothing, but you may over-ride this function in your 
+ * implementation does nothing, but you may override this function in your 
  * program if you want to configure the state of all pins prior to the 
  * application running.  This is useful when using external tools (like a
  * Pin Mux configuration tool) that generate code to initialize the pins.
@@ -123,7 +123,7 @@ __weak int Board_Init(void)
 
 /* This function is called just before control is transferred to main().
  *
- * You may over-ride this function in your program by defining a custom 
+ * You may override this function in your program by defining a custom 
  *  SystemInit(), but care should be taken to reproduce the initialization
  *  steps or a non-functional system may result.
  */

Libraries/CMSIS/Device/Maxim/MAX32572/Source/system_max32572.c

@@ -2,7 +2,7 @@
  *
  * Copyright (C) 2022-2023 Maxim Integrated Products, Inc. (now owned by 
  * Analog Devices, Inc.),
- * Copyright (C) 2023-2024 Analog Devices, Inc.
+ * Copyright (C) 2023-2025 Analog Devices, Inc.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -87,7 +87,7 @@ __weak void SystemCoreClockUpdate(void)
  * implemented by the application for early initializations. If a value other
  * than '0' is returned, the C runtime initialization will be skipped.
  *
- * You may over-ride this function in your program by defining a custom 
+ * You may override this function in your program by defining a custom 
  *  PreInit(), but care should be taken to reproduce the initialization steps
  *  or a non-functional system may result.
  */
@@ -98,7 +98,7 @@ __weak int PreInit(void)
 }
 
 /* This function is called before the Board_Init function.  This weak 
- * implementation does nothing, but you may over-ride this function in your 
+ * implementation does nothing, but you may override this function in your 
  * program if you want to configure the state of all pins prior to the 
  * application running.  This is useful when using external tools (like a
  * Pin Mux configuration tool) that generate code to initialize the pins.
@@ -123,7 +123,7 @@ __weak int Board_Init(void)
 
 /* This function is called just before control is transferred to main().
  *
- * You may over-ride this function in your program by defining a custom 
+ * You may override this function in your program by defining a custom 
  *  SystemInit(), but care should be taken to reproduce the initialization
  *  steps or a non-functional system may result.
  */

Libraries/CMSIS/Device/Maxim/MAX32572/Source/system_riscv_max32572.c

@@ -2,7 +2,7 @@
  *
  * Copyright (C) 2022-2023 Maxim Integrated Products, Inc. (now owned by 
  * Analog Devices, Inc.),
- * Copyright (C) 2023-2024 Analog Devices, Inc.
+ * Copyright (C) 2023-2025 Analog Devices, Inc.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -84,7 +84,7 @@ __weak void SystemCoreClockUpdate(void)
  * implemented by the application for early initializations. If a value other
  * than '0' is returned, the C runtime initialization will be skipped.
  *
- * You may over-ride this function in your program by defining a custom 
+ * You may override this function in your program by defining a custom 
  *  PreInit(), but care should be taken to reproduce the initilization steps
  *  or a non-functional system may result.
  */
@@ -94,6 +94,17 @@ __weak int PreInit(void)
     return 0;
 }
 
+/* This function is called before the Board_Init function.  This weak
+ * implementation does nothing, but you may override this function in your
+ * program if you want to configure the state of all pins prior to the
+ * application running.  This is useful when using external tools (like a
+ * Pin Mux configuration tool) that generate code to initialize the pins.
+ */
+__weak void PinInit(void)
+{
+    /* Do nothing */
+}
+
 __weak int PeripheralInit(void)
 {
     /* Do nothing */
@@ -115,7 +126,7 @@ void __enable_irq(void)
 
 /* This function is called just before control is transferred to main().
  *
- * You may over-ride this function in your program by defining a custom 
+ * You may override this function in your program by defining a custom 
  *  SystemInit(), but care should be taken to reproduce the initialization
  *  steps or a non-functional system may result.
  */
@@ -125,6 +136,7 @@ __weak void SystemInit(void)
 
     __enable_irq();
 
+    PinInit();
     Board_Init();
 
     /* Call peripheral init after board init to ensure the user's configuration

Libraries/CMSIS/Device/Maxim/MAX32650/Source/system_max32650.c

@@ -2,7 +2,7 @@
  *
  * Copyright (C) 2022-2023 Maxim Integrated Products, Inc. (now owned by 
  * Analog Devices, Inc.),
- * Copyright (C) 2023-2024 Analog Devices, Inc.
+ * Copyright (C) 2023-2025 Analog Devices, Inc.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -85,7 +85,7 @@ __weak void SystemCoreClockUpdate(void)
  * implemented by the application for early initializations. If a value other
  * than '0' is returned, the C runtime initialization will be skipped.
  *
- * You may over-ride this function in your program by defining a custom
+ * You may override this function in your program by defining a custom
  *  PreInit(), but care should be taken to reproduce the initialization steps
  *  or a non-functional system may result.
  */
@@ -97,7 +97,7 @@ __weak int PreInit(void)
 }
 
 /* This function is called before the Board_Init function.  This weak 
- * implementation does nothing, but you may over-ride this function in your 
+ * implementation does nothing, but you may override this function in your 
  * program if you want to configure the state of all pins prior to the 
  * application running.  This is useful when using external tools (like a
  * Pin Mux configuration tool) that generate code to initialize the pins.
@@ -122,7 +122,7 @@ __weak int Board_Init(void)
 
 /* This function is called just before control is transferred to main().
  *
- * You may over-ride this function in your program by defining a custom
+ * You may override this function in your program by defining a custom
  *  SystemInit(), but care should be taken to reproduce the initialization
  *  steps or a non-functional system may result.
  */

Libraries/CMSIS/Device/Maxim/MAX32655/Source/system_max32655.c

@@ -2,7 +2,7 @@
  *
  * Copyright (C) 2022-2023 Maxim Integrated Products, Inc. (now owned by 
  * Analog Devices, Inc.),
- * Copyright (C) 2023-2024 Analog Devices, Inc.
+ * Copyright (C) 2023-2025 Analog Devices, Inc.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -92,7 +92,7 @@ __weak void SystemCoreClockUpdate(void)
  * implemented by the application for early initializations. If a value other
  * than '0' is returned, the C runtime initialization will be skipped.
  *
- * You may over-ride this function in your program by defining a custom
+ * You may override this function in your program by defining a custom
  *  PreInit(), but care should be taken to reproduce the initialization steps
  *  or a non-functional system may result.
  */
@@ -115,7 +115,7 @@ __weak int PreInit(void)
 }
 
 /* This function is called before the Board_Init function.  This weak 
- * implementation does nothing, but you may over-ride this function in your 
+ * implementation does nothing, but you may override this function in your 
  * program if you want to configure the state of all pins prior to the 
  * application running.  This is useful when using external tools (like a
  * Pin Mux configuration tool) that generate code to initialize the pins.
@@ -142,7 +142,7 @@ __weak void PalSysInit(void) {}
 
 /* This function is called just before control is transferred to main().
  *
- * You may over-ride this function in your program by defining a custom
+ * You may override this function in your program by defining a custom
  *  SystemInit(), but care should be taken to reproduce the initialization
  *  steps or a non-functional system may result.
  */

Libraries/CMSIS/Device/Maxim/MAX32655/Source/system_riscv_max32655.c

@@ -2,7 +2,7 @@
  *
  * Copyright (C) 2022-2023 Maxim Integrated Products, Inc. (now owned by 
  * Analog Devices, Inc.),
- * Copyright (C) 2023-2024 Analog Devices, Inc.
+ * Copyright (C) 2023-2025 Analog Devices, Inc.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -734,7 +734,7 @@ void SystemCoreClockUpdate(void)
  * implemented by the application for early initializations. If a value other
  * than '0' is returned, the C runtime initialization will be skipped.
  *
- * You may over-ride this function in your program by defining a custom
+ * You may override this function in your program by defining a custom
  *  PreInit(), but care should be taken to reproduce the initilization steps
  *  or a non-functional system may result.
  */
@@ -744,6 +744,17 @@ __weak int PreInit(void)
     return 0;
 }
 
+/* This function is called before the Board_Init function.  This weak
+ * implementation does nothing, but you may override this function in your
+ * program if you want to configure the state of all pins prior to the
+ * application running.  This is useful when using external tools (like a
+ * Pin Mux configuration tool) that generate code to initialize the pins.
+ */
+__weak void PinInit(void)
+{
+    /* Do nothing */
+}
+
 __weak int PeripheralInit(void)
 {
     /* Do nothing */
@@ -761,7 +772,7 @@ __weak void PalSysInit(void) {}
 
 /* This function is called just before control is transferred to main().
  *
- * You may over-ride this function in your program by defining a custom
+ * You may override this function in your program by defining a custom
  *  SystemInit(), but care should be taken to reproduce the initialization
  *  steps or a non-functional system may result.
  */
@@ -778,6 +789,7 @@ __weak void SystemInit(void)
 
     __enable_irq();
 
+    PinInit();
     Board_Init();
 
     /* Call peripheral init after board init to ensure the user's configuration

Libraries/CMSIS/Device/Maxim/MAX32657/Source/system_max32657.c

@@ -112,7 +112,7 @@ __weak void SystemCoreClockUpdate(void)
  * implemented by the application for early initializations. If a value other
  * than '0' is returned, the C runtime initialization will be skipped.
  *
- * You may over-ride this function in your program by defining a custom
+ * You may override this function in your program by defining a custom
  *  PreInit(), but care should be taken to reproduce the initialization steps
  *  or a non-functional system may result.
  */
@@ -124,7 +124,7 @@ __weak int PreInit(void)
 
 /**
  * This function is called before the Board_Init function.  This weak 
- * implementation does nothing, but you may over-ride this function in your 
+ * implementation does nothing, but you may override this function in your 
  * program if you want to configure the state of all pins prior to the 
  * application running.  This is useful when using external tools (like a
  * Pin Mux configuration tool) that generate code to initialize the pins.
@@ -153,7 +153,7 @@ __weak void PalSysInit(void) {}
 /**
  * This function is called just before control is transferred to main().
  *
- * You may over-ride this function in your program by defining a custom
+ * You may override this function in your program by defining a custom
  *  SystemInit(), but care should be taken to reproduce the initialization
  *  steps or a non-functional system may result.
  */
@@ -219,7 +219,7 @@ __weak void SystemInit(void)
  *  is transferred to non-secure world. Only available when
  *  trustzone feature is used.
  *
- * You may over-ride this function in your program by defining a custom
+ * You may override this function in your program by defining a custom
  *  NonSecure_Init(), but care should be taken to reproduce the initialization
  *  steps to non-secure code.
  * 

Libraries/CMSIS/Device/Maxim/MAX32660/Source/system_max32660.c

@@ -7,7 +7,7 @@
  *
  * Copyright (C) 2022-2023 Maxim Integrated Products, Inc. (now owned by 
  * Analog Devices, Inc.),
- * Copyright (C) 2023-2024 Analog Devices, Inc.
+ * Copyright (C) 2023-2025 Analog Devices, Inc.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -87,7 +87,7 @@ __weak void SystemCoreClockUpdate(void)
  * implemented by the application for early initializations. If a value other
  * than '0' is returned, the C runtime initialization will be skipped.
  *
- * You may over-ride this function in your program by defining a custom
+ * You may override this function in your program by defining a custom
  *  PreInit(), but care should be taken to reproduce the initilization steps
  *  or a non-functional system may result.
  */
@@ -98,7 +98,7 @@ __weak int PreInit(void)
 }
 
 /* This function is called before the Board_Init function.  This weak 
- * implementation does nothing, but you may over-ride this function in your 
+ * implementation does nothing, but you may override this function in your 
  * program if you want to configure the state of all pins prior to the 
  * application running.  This is useful when using external tools (like a
  * Pin Mux configuration tool) that generate code to initialize the pins.
@@ -123,7 +123,7 @@ __weak int Board_Init(void)
 
 /* This function is called just before control is transferred to main().
  *
- * You may over-ride this function in your program by defining a custom
+ * You may override this function in your program by defining a custom
  *  SystemInit(), but care should be taken to reproduce the initialization
  *  steps or a non-functional system may result.
  */

Libraries/CMSIS/Device/Maxim/MAX32662/Source/system_max32662.c

@@ -6,7 +6,7 @@
  *
  * Copyright (C) 2022-2023 Maxim Integrated Products, Inc. (now owned by 
  * Analog Devices, Inc.),
- * Copyright (C) 2023-2024 Analog Devices, Inc.
+ * Copyright (C) 2023-2025 Analog Devices, Inc.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -110,7 +110,7 @@ __weak void SystemCoreClockUpdate(void)
  * implemented by the application for early initializations. If a value other
  * than '0' is returned, the C runtime initialization will be skipped.
  *
- * You may over-ride this function in your program by defining a custom 
+ * You may override this function in your program by defining a custom 
  *  PreInit(), but care should be taken to reproduce the initialization steps
  *  or a non-functional system may result.
  */
@@ -121,7 +121,7 @@ __weak int PreInit(void)
 }
 
 /* This function is called before the Board_Init function.  This weak 
- * implementation does nothing, but you may over-ride this function in your 
+ * implementation does nothing, but you may override this function in your 
  * program if you want to configure the state of all pins prior to the 
  * application running.  This is useful when using external tools (like a
  * Pin Mux configuration tool) that generate code to initialize the pins.
@@ -148,7 +148,7 @@ __weak void PalSysInit(void) {}
 
 /* This function is called just before control is transferred to main().
  *
- * You may over-ride this function in your program by defining a custom 
+ * You may override this function in your program by defining a custom 
  *  SystemInit(), but care should be taken to reproduce the initialization
  *  steps or a non-functional system may result.
  */