xiaoxiang781216 commented on code in PR #10553: URL: https://github.com/apache/nuttx/pull/10553#discussion_r1325209786
########## arch/arm/src/a1x/a1x_lowputc.c: ########## @@ -27,13 +27,15 @@ #include <stdint.h> #include <arch/irq.h> -#include <arch/board/board.h> #include "arm_internal.h" #include "a1x_config.h" -#include "hardware/a1x_uart.h" #include "a1x_pio.h" +#include "hardware/a1x_uart.h" + +#include <arch/board/board.h> Review Comment: why not keep it in the original place? ########## arch/arm/src/a1x/a1x_timerisr.c: ########## @@ -31,9 +31,9 @@ #include <nuttx/arch.h> #include <arch/irq.h> -#include <arch/board/board.h> #include "arm_internal.h" + Review Comment: revert ########## arch/arm/src/a1x/a1x_serial.c: ########## @@ -42,14 +42,16 @@ #include <nuttx/fs/ioctl.h> #include <nuttx/serial/serial.h> -#include <arch/board/board.h> - #include "arm_internal.h" #include "chip.h" -#include "hardware/a1x_uart.h" +#include "a1x_config.h" #include "a1x_pio.h" #include "a1x_serial.h" +#include "hardware/a1x_uart.h" + +#include <arch/board/board.h> Review Comment: ditto ########## arch/arm/src/lpc31xx/lpc31_i2c.c: ########## @@ -38,15 +38,16 @@ #include <nuttx/mutex.h> #include <nuttx/semaphore.h> #include <nuttx/i2c/i2c_master.h> - #include <nuttx/irq.h> -#include <arch/board/board.h> #include "chip.h" #include "arm_internal.h" #include "lpc31_i2c.h" #include "lpc31_evntrtr.h" #include "lpc31_syscreg.h" +#include "lpc31_cgudrvr.h" + +#include <arch/board/board.h> Review Comment: why change tje order ########## boards/arm/a1x/pcduino-a10/src/a1x_leds.c: ########## @@ -29,12 +29,13 @@ #include <debug.h> #include <nuttx/board.h> -#include <arch/board/board.h> #include "chip.h" #include "arm_internal.h" #include "pcduino_a10.h" +#include <arch/board/board.h> Review Comment: ditto ########## boards/arm/a1x/pcduino-a10/src/a1x_buttons.c: ########## @@ -32,10 +32,11 @@ #include <nuttx/irq.h> #include <nuttx/irq.h> -#include <arch/board/board.h> #include "pcduino_a10.h" +#include <arch/board/board.h> Review Comment: why change? ########## boards/arm/eoss3/quickfeather/src/eoss3_autoleds.c: ########## @@ -28,10 +28,12 @@ #include <debug.h> #include <nuttx/board.h> -#include <arch/board/board.h> #include "chip.h" #include "arm_internal.h" +#include "eoss3_gpio.h" + +#include <arch/board/board.h> Review Comment: why change the order ########## arch/arm/src/lpc31xx/lpc31_timerisr.c: ########## @@ -29,12 +29,14 @@ #include <debug.h> #include <nuttx/arch.h> -#include <arch/board/board.h> #include "clock/clock.h" #include "arm_internal.h" #include "lpc31_timer.h" #include "lpc31.h" +#include "lpc31_cgudrvr.h" + +#include <arch/board/board.h> Review Comment: ditto ########## sched/misc/assert.c: ########## @@ -48,6 +46,8 @@ #include "sched/sched.h" #include "group/group.h" +#include <arch/board/board.h> Review Comment: ditto ########## arch/arm/src/a1x/hardware/a10_piocfg.h: ########## @@ -414,10 +414,10 @@ #define PIO_SDC0_D2 (PIO_PERIPH2 | PIO_PORT_PIOF | PIO_PIN5) #define PIO_SDC0_D3 (PIO_PERIPH2 | PIO_PORT_PIOF | PIO_PIN4) -#define PIO_SDC1_CLK (PIO_PERIPH4 | PIO_PORT_PIOG | PIO_PIN1) -// #define PIO_SDC1_CLK (PIO_PERIPH5 | PIO_PORT_PIOH | PIO_PIN23) -#define PIO_SDC1_CMD (PIO_PERIPH4 | PIO_PORT_PIOG | PIO_PIN0) -// #define PIO_SDC1_CMD (PIO_PERIPH5 | PIO_PORT_PIOH | PIO_PIN22) +#define PIO_SDC1_CLK_1 (PIO_PERIPH4 | PIO_PORT_PIOG | PIO_PIN1) +#define PIO_SDC1_CLK_2 (PIO_PERIPH5 | PIO_PORT_PIOH | PIO_PIN23) +#define PIO_SDC1_CMD_1 (PIO_PERIPH4 | PIO_PORT_PIOG | PIO_PIN0) +#define PIO_SDC1_CMD_2 (PIO_PERIPH5 | PIO_PORT_PIOH | PIO_PIN22) Review Comment: let's move the change to the original patch ########## boards/arm/stm32/emw3162/src/stm32_wlan.c: ########## @@ -29,13 +29,14 @@ #include <nuttx/wireless/ieee80211/bcmf_sdio.h> #include <nuttx/wireless/ieee80211/bcmf_board.h> -#include <arch/board/board.h> - +#include "stm32.h" #include "stm32_gpio.h" #include "stm32_sdio.h" #include "emw3162.h" +#include <arch/board/board.h> Review Comment: ditto ########## arch/arm/src/a1x/hardware/a10_piocfg.h: ########## @@ -97,68 +97,68 @@ /* Camera Sensor Interface (CSI) */ -#define PIO_CSI0_CK (PIO_PERIPH3 | PIO_PORT_PIOE | PIO_PIN1) -#define PIO_CSI0_D0 (PIO_PERIPH3 | PIO_PORT_PIOE | PIO_PIN4) -#define PIO_CSI0_D1 (PIO_PERIPH3 | PIO_PORT_PIOE | PIO_PIN5) -#define PIO_CSI0_D2 (PIO_PERIPH3 | PIO_PORT_PIOE | PIO_PIN6) -#define PIO_CSI0_D3 (PIO_PERIPH3 | PIO_PORT_PIOE | PIO_PIN7) -#define PIO_CSI0_D4 (PIO_PERIPH3 | PIO_PORT_PIOE | PIO_PIN8) -#define PIO_CSI0_D5 (PIO_PERIPH3 | PIO_PORT_PIOE | PIO_PIN9) -#define PIO_CSI0_D6 (PIO_PERIPH3 | PIO_PORT_PIOE | PIO_PIN10) -#define PIO_CSI0_D7 (PIO_PERIPH3 | PIO_PORT_PIOE | PIO_PIN11) -#define PIO_CSI0_D8 (PIO_PERIPH5 | PIO_PORT_PIOG | PIO_PIN4) -#define PIO_CSI0_D9 (PIO_PERIPH5 | PIO_PORT_PIOG | PIO_PIN5) -#define PIO_CSI0_D10 (PIO_PERIPH5 | PIO_PORT_PIOG | PIO_PIN6) -#define PIO_CSI0_D11 (PIO_PERIPH5 | PIO_PORT_PIOG | PIO_PIN7) -#define PIO_CSI0_D12 (PIO_PERIPH5 | PIO_PORT_PIOG | PIO_PIN8) -#define PIO_CSI0_D13 (PIO_PERIPH5 | PIO_PORT_PIOG | PIO_PIN9) -#define PIO_CSI0_D14 (PIO_PERIPH5 | PIO_PORT_PIOG | PIO_PIN10) -#define PIO_CSI0_D15 (PIO_PERIPH5 | PIO_PORT_PIOG | PIO_PIN11) -#define PIO_CSI0_HSYNC (PIO_PERIPH3 | PIO_PORT_PIOE | PIO_PIN2) -#define PIO_CSI0_PCK (PIO_PERIPH3 | PIO_PORT_PIOE | PIO_PIN0) -#define PIO_CSI0_VSYNC (PIO_PERIPH3 | PIO_PORT_PIOE | PIO_PIN3) - -#define PIO_CSI1_CK (PIO_PERIPH3 | PIO_PORT_PIOG | PIO_PIN1) -#define PIO_CSI1_D0_1 (PIO_PERIPH3 | PIO_PORT_PIOG | PIO_PIN4) -#define PIO_CSI1_D0_2 (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN0) -#define PIO_CSI1_D1_1 (PIO_PERIPH3 | PIO_PORT_PIOG | PIO_PIN5) -#define PIO_CSI1_D1_2 (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN1) -#define PIO_CSI1_D2_1 (PIO_PERIPH3 | PIO_PORT_PIOG | PIO_PIN6) -#define PIO_CSI1_D2_2 (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN2) -#define PIO_CSI1_D3_1 (PIO_PERIPH3 | PIO_PORT_PIOG | PIO_PIN7) -#define PIO_CSI1_D3_2 (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN3) -#define PIO_CSI1_D4_1 (PIO_PERIPH3 | PIO_PORT_PIOG | PIO_PIN8) -#define PIO_CSI1_D4_2 (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN4) -#define PIO_CSI1_D5_1 (PIO_PERIPH3 | PIO_PORT_PIOG | PIO_PIN9) -#define PIO_CSI1_D5_2 (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN5) -#define PIO_CSI1_D6_1 (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN6) -#define PIO_CSI1_D6_2 (PIO_PERIPH3 | PIO_PORT_PIOG | PIO_PIN10) -#define PIO_CSI1_D7_1 (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN7) -#define PIO_CSI1_D7_2 (PIO_PERIPH3 | PIO_PORT_PIOG | PIO_PIN11) -#define PIO_CSI1_D8 (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN8) -#define PIO_CSI1_D9 (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN9) -#define PIO_CSI1_D10 (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN10) -#define PIO_CSI1_D11 (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN11) -#define PIO_CSI1_D12 (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN12) -#define PIO_CSI1_D13 (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN13) -#define PIO_CSI1_D14 (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN14) -#define PIO_CSI1_D15 (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN15) -#define PIO_CSI1_D16 (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN16) -#define PIO_CSI1_D17 (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN17) -#define PIO_CSI1_D18 (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN18) -#define PIO_CSI1_D19 (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN19) -#define PIO_CSI1_D20 (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN20) -#define PIO_CSI1_D21 (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN21) -#define PIO_CSI1_D22 (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN22) -#define PIO_CSI1_D23 (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN23) -#define PIO_CSI1_FIELD (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN25) -#define PIO_CSI1_HSYNC (PIO_PERIPH3 | PIO_PORT_PIOG | PIO_PIN2) -// #define PIO_CSI1_HSYNC (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN26) -#define PIO_CSI1_MCLK (PIO_PERIPH3 | PIO_PORT_PIOD | PIO_PIN20) -#define PIO_CSI1_PCK (PIO_PERIPH3 | PIO_PORT_PIOG | PIO_PIN0) -#define PIO_CSI1_PCLK (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN24) -#define PIO_CSI1_VSYNC (PIO_PERIPH3 | PIO_PORT_PIOG | PIO_PIN3) -// #define PIO_CSI1_VSYNC (PIO_PERIPH7 | PIO_PORT_PIOH | PIO_PIN27) Review Comment: fix in the original patch ########## arch/arm/src/lpc31xx/lpc31_spi.c: ########## @@ -35,10 +35,11 @@ #include <nuttx/mutex.h> #include <nuttx/spi/spi.h> -#include <arch/board/board.h> - #include "lpc31_spi.h" #include "lpc31_ioconfig.h" +#include "lpc31_cgudrvr.h" + +#include <arch/board/board.h> Review Comment: ditto ########## boards/arm/stm32/mikroe-stm32f4/src/stm32_clockconfig.c: ########## @@ -26,11 +26,13 @@ #include <debug.h> -#include <arch/board/board.h> +#include "stm32.h" #include "arm_internal.h" #include "mikroe-stm32f4.h" +#include <arch/board/board.h> Review Comment: ditto ########## boards/arm/lpc31xx/ea3152/src/lpc31_spi.c: ########## @@ -29,13 +29,14 @@ #include <debug.h> #include <nuttx/spi/spi.h> -#include <arch/board/board.h> #include "arm_internal.h" #include "chip.h" #include "lpc31.h" #include "ea3152.h" +#include <arch/board/board.h> Review Comment: ditto -- 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
