raiden00pl commented on code in PR #16293:
URL: https://github.com/apache/nuttx/pull/16293#discussion_r2069963907


##########
arch/arm/src/stm32f0l0g0/Kconfig:
##########
@@ -531,6 +531,86 @@ config ARCH_CHIP_STM32G071RB
        select STM32F0L0G0_STM32G0
        depends on ARCH_CHIP_STM32G0
 
+config ARCH_CHIP_STM32G0B1KB

Review Comment:
   please split changes to `/arch` and `/boards` into separate commits. 
Something like this:
   
   * /arch/stm32f0l0g0: add stm32g0bXXX support
   * /boards/stm32f0l0g0: Add Nucleo G0B1RE support



##########
arch/arm/src/stm32f0l0g0/stm32_crypto.c:
##########
@@ -79,7 +79,7 @@ static int stm32_newsession(uint32_t *sid, struct cryptoini 
*cri)
 
         *sid = g_stm32_sesnum++;
         break;
-      default :
+      default:

Review Comment:
   please move this change to a separate commit



##########
arch/arm/src/stm32f0l0g0/stm32_serial_v1.c:
##########
@@ -2320,25 +2320,25 @@ static void stm32serial_pmnotify(struct pm_callback_s 
*cb, int domain,
 {
   switch (pmstate)
     {
-      case(PM_NORMAL):

Review Comment:
   please move serial related changes to a separate commit



##########
arch/arm/include/stm32f0l0g0/chip.h:
##########
@@ -281,6 +281,39 @@
 #  define STM32_NCAP            0  /* No Capacitive sensing channels */
 #  define STM32_NPORTS          6  /* Six GPIO ports, GPIOA-F */
 
+#elif defined(CONFIG_ARCH_CHIP_STM32G0B1KB) || 
defined(CONFIG_ARCH_CHIP_STM32G0B1CB) || \
+      defined(CONFIG_ARCH_CHIP_STM32G0B1RB) || 
defined(CONFIG_ARCH_CHIP_STM32G0B1MB) || \
+      defined(CONFIG_ARCH_CHIP_STM32G0B1VB) || 
defined(CONFIG_ARCH_CHIP_STM32G0B1KC) || \
+      defined(CONFIG_ARCH_CHIP_STM32G0B1CC) || 
defined(CONFIG_ARCH_CHIP_STM32G0B1RC) || \
+      defined(CONFIG_ARCH_CHIP_STM32G0B1MC) || 
defined(CONFIG_ARCH_CHIP_STM32G0B1VC) || \
+      defined(CONFIG_ARCH_CHIP_STM32G0B1KE) || 
defined(CONFIG_ARCH_CHIP_STM32G0B1CE) || \
+      defined(CONFIG_ARCH_CHIP_STM32G0B1RE) || 
defined(CONFIG_ARCH_CHIP_STM32G0B1NE) || \
+      defined(CONFIG_ARCH_CHIP_STM32G0B1ME) || 
defined(CONFIG_ARCH_CHIP_STM32G0B1VE)
+
+#  define STM32_NATIM           1  /* One advanced timer TIM1 */              
// Checked

Review Comment:
   remove c++ comments



##########
boards/Kconfig:
##########
@@ -3552,6 +3562,7 @@ config ARCH_BOARD
        default "stm32g071b-disco"          if ARCH_BOARD_STM32G071B_DISCO
        default "nucleo-g070rb"             if ARCH_BOARD_NUCLEO_G070RB
        default "nucleo-g071rb"             if ARCH_BOARD_NUCLEO_G071RB
+       default "nucleo-g0b1re"                         if 
ARCH_BOARD_NUCLEO_G0B1RE

Review Comment:
   please fix this indentation



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to