[GitHub] [mynewt-core] apache-mynewt-bot commented on pull request #2348: Add NRF9160 BSP with I2C, SPI, UART and GPIO support
apache-mynewt-bot commented on pull request #2348: URL: https://github.com/apache/mynewt-core/pull/2348#issuecomment-671541951 ## Style check summary ### Our coding style is [here!](https://github.com/apache/mynewt-core/blob/master/CODING_STANDARDS.md) hw/bus/drivers/i2c_nrf91_twim/src/i2c_nrf91_twim.c ```diff @@ -40,17 +40,17 @@ #if MYNEWT_VAL(I2C_NRF91_TWIM_STAT) STATS_SECT_START(twim_stats_section) -STATS_SECT_ENTRY(sda_lo_err)/* SDA pulled low on r/w */ -STATS_SECT_ENTRY(sda_lo_err_nrecov) /* SDA pulled low on r/w (not recovered)*/ -STATS_SECT_ENTRY(scl_hi_err)/* SCL unresponsive */ -STATS_SECT_ENTRY(scl_hi_err_nrecov) /* SCL unresponsive (not recovered)*/ +STATS_SECT_ENTRY(sda_lo_err)/* SDA pulled low on r/w */ +STATS_SECT_ENTRY(sda_lo_err_nrecov) /* SDA pulled low on r/w (not recovered)*/ +STATS_SECT_ENTRY(scl_hi_err)/* SCL unresponsive */ +STATS_SECT_ENTRY(scl_hi_err_nrecov) /* SCL unresponsive (not recovered)*/ STATS_SECT_END STATS_NAME_START(twim_stats_section) -STATS_NAME(twim_stats_section, sda_lo_err) -STATS_NAME(twim_stats_section, sda_lo_err_nrecov) -STATS_NAME(twim_stats_section, scl_hi_err) -STATS_NAME(twim_stats_section, scl_hi_err_nrecov) +STATS_NAME(twim_stats_section, sda_lo_err) +STATS_NAME(twim_stats_section, sda_lo_err_nrecov) +STATS_NAME(twim_stats_section, scl_hi_err) +STATS_NAME(twim_stats_section, scl_hi_err_nrecov) STATS_NAME_END(twim_stats_section) #endif @@ -352,7 +352,8 @@ nrf_twim->TASKS_STOP = 1; nrf_twim->ENABLE = TWIM_ENABLE_ENABLE_Disabled; nrf_twim->ENABLE = TWIM_ENABLE_ENABLE_Enabled; -}; +} +; #if MYNEWT_VAL(I2C_NRF91_TWIM_STAT) STATS_INC(dd->stats, scl_hi_err_nrecov); ``` hw/mcu/nordic/nrf91xx/src/hal_flash.c ```diff @@ -25,7 +25,7 @@ #include "nrfx_config_nrf9160.h" #include "nrf_nvmc.h" -#define NRF91K_FLASH_SECTOR_SZ 4096 +#define NRF91K_FLASH_SECTOR_SZ 4096 static int nrf91k_flash_read(const struct hal_flash *dev, uint32_t address, void *dst, uint32_t num_bytes); @@ -49,8 +49,8 @@ const struct hal_flash nrf91k_flash_dev = { .hf_itf = _flash_funcs, .hf_base_addr = 0x, -.hf_size = 1024 * 1024, /* XXX read from factory info? */ -.hf_sector_cnt = 256, /* XXX read from factory info? */ +.hf_size = 1024 * 1024, /* XXX read from factory info? */ +.hf_sector_cnt = 256, /* XXX read from factory info? */ .hf_align = 1, .hf_erased_val = 0xff, }; ``` hw/mcu/nordic/nrf91xx/src/hal_gpio.c ```diff @@ -116,7 +116,7 @@ port->OUTCLR = HAL_GPIO_MASK(pin); } port->PIN_CNF[pin_index] = GPIO_PIN_CNF_DIR_Output | -(GPIO_PIN_CNF_INPUT_Disconnect << GPIO_PIN_CNF_INPUT_Pos); + (GPIO_PIN_CNF_INPUT_Disconnect << GPIO_PIN_CNF_INPUT_Pos); port->DIRSET = HAL_GPIO_MASK(pin); return 0; ``` hw/mcu/nordic/nrf91xx/src/hal_i2c.c ```diff @@ -38,18 +38,18 @@ #define NRF91_SCL_PIN_CONF \ ((GPIO_PIN_CNF_SENSE_Disabled << GPIO_PIN_CNF_SENSE_Pos) | \ - (GPIO_PIN_CNF_DRIVE_S0D1<< GPIO_PIN_CNF_DRIVE_Pos) | \ - (GPIO_PIN_CNF_PULL_Pullup << GPIO_PIN_CNF_PULL_Pos) | \ + (GPIO_PIN_CNF_DRIVE_S0D1 << GPIO_PIN_CNF_DRIVE_Pos) | \ + (GPIO_PIN_CNF_PULL_Pullup << GPIO_PIN_CNF_PULL_Pos) | \ (GPIO_PIN_CNF_INPUT_Connect << GPIO_PIN_CNF_INPUT_Pos) | \ - (GPIO_PIN_CNF_DIR_Input << GPIO_PIN_CNF_DIR_Pos)) + (GPIO_PIN_CNF_DIR_Input << GPIO_PIN_CNF_DIR_Pos)) #define NRF91_SDA_PIN_CONF NRF91_SCL_PIN_CONF #define NRF91_SCL_PIN_CONF_CLR \ - ((GPIO_PIN_CNF_SENSE_Disabled << GPIO_PIN_CNF_SENSE_Pos) | \ - (GPIO_PIN_CNF_DRIVE_S0D1 << GPIO_PIN_CNF_DRIVE_Pos) | \ - (GPIO_PIN_CNF_PULL_Pullup<< GPIO_PIN_CNF_PULL_Pos) | \ - (GPIO_PIN_CNF_INPUT_Connect << GPIO_PIN_CNF_INPUT_Pos) | \ - (GPIO_PIN_CNF_DIR_Output << GPIO_PIN_CNF_DIR_Pos)) +((GPIO_PIN_CNF_SENSE_Disabled << GPIO_PIN_CNF_SENSE_Pos) | \ + (GPIO_PIN_CNF_DRIVE_S0D1 << GPIO_PIN_CNF_DRIVE_Pos) | \ + (GPIO_PIN_CNF_PULL_Pullup << GPIO_PIN_CNF_PULL_Pos) | \ + (GPIO_PIN_CNF_INPUT_Connect << GPIO_PIN_CNF_INPUT_Pos) | \ + (GPIO_PIN_CNF_DIR_Output << GPIO_PIN_CNF_DIR_Pos)) #define NRF91_SDA_PIN_CONF_CLRNRF91_SCL_PIN_CONF_CLR struct nrf91_hal_i2c { ``` hw/mcu/nordic/nrf91xx/src/hal_qspi.c ```diff @@ -162,7 +162,7 @@ while
[GitHub] [mynewt-core] apache-mynewt-bot commented on pull request #2348: Add NRF9160 BSP with I2C, SPI, UART and GPIO support
apache-mynewt-bot commented on pull request #2348: URL: https://github.com/apache/mynewt-core/pull/2348#issuecomment-671533692 ## Style check summary ### Our coding style is [here!](https://github.com/apache/mynewt-core/blob/master/CODING_STANDARDS.md) hw/bus/drivers/i2c_nrf91_twim/src/i2c_nrf91_twim.c ```diff @@ -40,17 +40,17 @@ #if MYNEWT_VAL(I2C_NRF91_TWIM_STAT) STATS_SECT_START(twim_stats_section) -STATS_SECT_ENTRY(sda_lo_err)/* SDA pulled low on r/w */ -STATS_SECT_ENTRY(sda_lo_err_nrecov) /* SDA pulled low on r/w (not recovered)*/ -STATS_SECT_ENTRY(scl_hi_err)/* SCL unresponsive */ -STATS_SECT_ENTRY(scl_hi_err_nrecov) /* SCL unresponsive (not recovered)*/ +STATS_SECT_ENTRY(sda_lo_err)/* SDA pulled low on r/w */ +STATS_SECT_ENTRY(sda_lo_err_nrecov) /* SDA pulled low on r/w (not recovered)*/ +STATS_SECT_ENTRY(scl_hi_err)/* SCL unresponsive */ +STATS_SECT_ENTRY(scl_hi_err_nrecov) /* SCL unresponsive (not recovered)*/ STATS_SECT_END STATS_NAME_START(twim_stats_section) -STATS_NAME(twim_stats_section, sda_lo_err) -STATS_NAME(twim_stats_section, sda_lo_err_nrecov) -STATS_NAME(twim_stats_section, scl_hi_err) -STATS_NAME(twim_stats_section, scl_hi_err_nrecov) +STATS_NAME(twim_stats_section, sda_lo_err) +STATS_NAME(twim_stats_section, sda_lo_err_nrecov) +STATS_NAME(twim_stats_section, scl_hi_err) +STATS_NAME(twim_stats_section, scl_hi_err_nrecov) STATS_NAME_END(twim_stats_section) #endif @@ -352,7 +352,8 @@ nrf_twim->TASKS_STOP = 1; nrf_twim->ENABLE = TWIM_ENABLE_ENABLE_Disabled; nrf_twim->ENABLE = TWIM_ENABLE_ENABLE_Enabled; -}; +} +; #if MYNEWT_VAL(I2C_NRF91_TWIM_STAT) STATS_INC(dd->stats, scl_hi_err_nrecov); ``` hw/mcu/nordic/nrf91xx/src/hal_flash.c ```diff @@ -25,7 +25,7 @@ #include "nrfx_config_nrf9160.h" #include "nrf_nvmc.h" -#define NRF91K_FLASH_SECTOR_SZ 4096 +#define NRF91K_FLASH_SECTOR_SZ 4096 static int nrf91k_flash_read(const struct hal_flash *dev, uint32_t address, void *dst, uint32_t num_bytes); @@ -49,8 +49,8 @@ const struct hal_flash nrf91k_flash_dev = { .hf_itf = _flash_funcs, .hf_base_addr = 0x, -.hf_size = 1024 * 1024, /* XXX read from factory info? */ -.hf_sector_cnt = 256, /* XXX read from factory info? */ +.hf_size = 1024 * 1024, /* XXX read from factory info? */ +.hf_sector_cnt = 256, /* XXX read from factory info? */ .hf_align = 1, .hf_erased_val = 0xff, }; ``` hw/mcu/nordic/nrf91xx/src/hal_gpio.c ```diff @@ -116,7 +116,7 @@ port->OUTCLR = HAL_GPIO_MASK(pin); } port->PIN_CNF[pin_index] = GPIO_PIN_CNF_DIR_Output | -(GPIO_PIN_CNF_INPUT_Disconnect << GPIO_PIN_CNF_INPUT_Pos); + (GPIO_PIN_CNF_INPUT_Disconnect << GPIO_PIN_CNF_INPUT_Pos); port->DIRSET = HAL_GPIO_MASK(pin); return 0; ``` hw/mcu/nordic/nrf91xx/src/hal_i2c.c ```diff @@ -38,18 +38,18 @@ #define NRF91_SCL_PIN_CONF \ ((GPIO_PIN_CNF_SENSE_Disabled << GPIO_PIN_CNF_SENSE_Pos) | \ - (GPIO_PIN_CNF_DRIVE_S0D1<< GPIO_PIN_CNF_DRIVE_Pos) | \ - (GPIO_PIN_CNF_PULL_Pullup << GPIO_PIN_CNF_PULL_Pos) | \ + (GPIO_PIN_CNF_DRIVE_S0D1 << GPIO_PIN_CNF_DRIVE_Pos) | \ + (GPIO_PIN_CNF_PULL_Pullup << GPIO_PIN_CNF_PULL_Pos) | \ (GPIO_PIN_CNF_INPUT_Connect << GPIO_PIN_CNF_INPUT_Pos) | \ - (GPIO_PIN_CNF_DIR_Input << GPIO_PIN_CNF_DIR_Pos)) + (GPIO_PIN_CNF_DIR_Input << GPIO_PIN_CNF_DIR_Pos)) #define NRF91_SDA_PIN_CONF NRF91_SCL_PIN_CONF #define NRF91_SCL_PIN_CONF_CLR \ - ((GPIO_PIN_CNF_SENSE_Disabled << GPIO_PIN_CNF_SENSE_Pos) | \ - (GPIO_PIN_CNF_DRIVE_S0D1 << GPIO_PIN_CNF_DRIVE_Pos) | \ - (GPIO_PIN_CNF_PULL_Pullup<< GPIO_PIN_CNF_PULL_Pos) | \ - (GPIO_PIN_CNF_INPUT_Connect << GPIO_PIN_CNF_INPUT_Pos) | \ - (GPIO_PIN_CNF_DIR_Output << GPIO_PIN_CNF_DIR_Pos)) +((GPIO_PIN_CNF_SENSE_Disabled << GPIO_PIN_CNF_SENSE_Pos) | \ + (GPIO_PIN_CNF_DRIVE_S0D1 << GPIO_PIN_CNF_DRIVE_Pos) | \ + (GPIO_PIN_CNF_PULL_Pullup << GPIO_PIN_CNF_PULL_Pos) | \ + (GPIO_PIN_CNF_INPUT_Connect << GPIO_PIN_CNF_INPUT_Pos) | \ + (GPIO_PIN_CNF_DIR_Output << GPIO_PIN_CNF_DIR_Pos)) #define NRF91_SDA_PIN_CONF_CLRNRF91_SCL_PIN_CONF_CLR struct nrf91_hal_i2c { ``` hw/mcu/nordic/nrf91xx/src/hal_qspi.c ```diff @@ -162,7 +162,7 @@ while
[GitHub] [mynewt-core] apache-mynewt-bot commented on pull request #2348: Add NRF9160 BSP with I2C, SPI, UART and GPIO support
apache-mynewt-bot commented on pull request #2348: URL: https://github.com/apache/mynewt-core/pull/2348#issuecomment-671533318 ## RAT Report (2020-08-10 19:04:49) ## New files with unknown licenses * https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/src/arch/cortex_m33/gcc_startup_nrf9160_split.s;>hw/bsp/nordic_pca10090/src/arch/cortex_m33/gcc_startup_nrf9160_split.s ## 53 new files were excluded from check (.rat-excludes) Detailed analysis ## New files in this PR | License | File | |-|--| | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/approtect_detect;>hw/bsp/nordic_pca10090/approtect_detect | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/bsp.yml;>hw/bsp/nordic_pca10090/bsp.yml | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/nordic_pca10090_debug.cmd;>hw/bsp/nordic_pca10090/nordic_pca10090_debug.cmd | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/nordic_pca10090_debug.sh;>hw/bsp/nordic_pca10090/nordic_pca10090_debug.sh | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/nordic_pca10090_download.cmd;>hw/bsp/nordic_pca10090/nordic_pca10090_download.cmd | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/nordic_pca10090_download.sh;>hw/bsp/nordic_pca10090/nordic_pca10090_download.sh | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/nrf9160dk_common.sh;>hw/bsp/nordic_pca10090/nrf9160dk_common.sh | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/pkg.yml;>hw/bsp/nordic_pca10090/pkg.yml | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/syscfg.yml;>hw/bsp/nordic_pca10090/syscfg.yml | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/targetpoweron;>hw/bsp/nordic_pca10090/targetpoweron | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/include/bsp/bsp.h;>hw/bsp/nordic_pca10090/include/bsp/bsp.h | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/src/hal_bsp.c;>hw/bsp/nordic_pca10090/src/hal_bsp.c | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/src/sbrk.c;>hw/bsp/nordic_pca10090/src/sbrk.c | | ? | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/src/arch/cortex_m33/gcc_startup_nrf9160_split.s;>hw/bsp/nordic_pca10090/src/arch/cortex_m33/gcc_startup_nrf9160_split.s | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bus/drivers/i2c_nrf91_twim/pkg.yml;>hw/bus/drivers/i2c_nrf91_twim/pkg.yml | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bus/drivers/i2c_nrf91_twim/syscfg.yml;>hw/bus/drivers/i2c_nrf91_twim/syscfg.yml | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bus/drivers/i2c_nrf91_twim/include/bus/drivers/i2c_nrf91_twim.h;>hw/bus/drivers/i2c_nrf91_twim/include/bus/drivers/i2c_nrf91_twim.h | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bus/drivers/i2c_nrf91_twim/src/i2c_nrf91_twim.c;>hw/bus/drivers/i2c_nrf91_twim/src/i2c_nrf91_twim.c | 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [mynewt-core] apache-mynewt-bot commented on pull request #2348: Add NRF9160 BSP with I2C, SPI, UART and GPIO support
apache-mynewt-bot commented on pull request #2348: URL: https://github.com/apache/mynewt-core/pull/2348#issuecomment-671528563 ## Style check summary ### Our coding style is [here!](https://github.com/apache/mynewt-core/blob/master/CODING_STANDARDS.md) hw/bus/drivers/i2c_nrf91_twim/src/i2c_nrf91_twim.c ```diff @@ -40,17 +40,17 @@ #if MYNEWT_VAL(I2C_NRF91_TWIM_STAT) STATS_SECT_START(twim_stats_section) -STATS_SECT_ENTRY(sda_lo_err)/* SDA pulled low on r/w */ -STATS_SECT_ENTRY(sda_lo_err_nrecov) /* SDA pulled low on r/w (not recovered)*/ -STATS_SECT_ENTRY(scl_hi_err)/* SCL unresponsive */ -STATS_SECT_ENTRY(scl_hi_err_nrecov) /* SCL unresponsive (not recovered)*/ +STATS_SECT_ENTRY(sda_lo_err)/* SDA pulled low on r/w */ +STATS_SECT_ENTRY(sda_lo_err_nrecov) /* SDA pulled low on r/w (not recovered)*/ +STATS_SECT_ENTRY(scl_hi_err)/* SCL unresponsive */ +STATS_SECT_ENTRY(scl_hi_err_nrecov) /* SCL unresponsive (not recovered)*/ STATS_SECT_END STATS_NAME_START(twim_stats_section) -STATS_NAME(twim_stats_section, sda_lo_err) -STATS_NAME(twim_stats_section, sda_lo_err_nrecov) -STATS_NAME(twim_stats_section, scl_hi_err) -STATS_NAME(twim_stats_section, scl_hi_err_nrecov) +STATS_NAME(twim_stats_section, sda_lo_err) +STATS_NAME(twim_stats_section, sda_lo_err_nrecov) +STATS_NAME(twim_stats_section, scl_hi_err) +STATS_NAME(twim_stats_section, scl_hi_err_nrecov) STATS_NAME_END(twim_stats_section) #endif @@ -352,7 +352,8 @@ nrf_twim->TASKS_STOP = 1; nrf_twim->ENABLE = TWIM_ENABLE_ENABLE_Disabled; nrf_twim->ENABLE = TWIM_ENABLE_ENABLE_Enabled; -}; +} +; #if MYNEWT_VAL(I2C_NRF91_TWIM_STAT) STATS_INC(dd->stats, scl_hi_err_nrecov); ``` hw/mcu/nordic/nrf91xx/src/hal_flash.c ```diff @@ -25,7 +25,7 @@ #include "nrfx_config_nrf9160.h" #include "nrf_nvmc.h" -#define NRF91K_FLASH_SECTOR_SZ 4096 +#define NRF91K_FLASH_SECTOR_SZ 4096 static int nrf91k_flash_read(const struct hal_flash *dev, uint32_t address, void *dst, uint32_t num_bytes); @@ -49,8 +49,8 @@ const struct hal_flash nrf91k_flash_dev = { .hf_itf = _flash_funcs, .hf_base_addr = 0x, -.hf_size = 1024 * 1024, /* XXX read from factory info? */ -.hf_sector_cnt = 256, /* XXX read from factory info? */ +.hf_size = 1024 * 1024, /* XXX read from factory info? */ +.hf_sector_cnt = 256, /* XXX read from factory info? */ .hf_align = 1, .hf_erased_val = 0xff, }; ``` hw/mcu/nordic/nrf91xx/src/hal_gpio.c ```diff @@ -116,7 +116,7 @@ port->OUTCLR = HAL_GPIO_MASK(pin); } port->PIN_CNF[pin_index] = GPIO_PIN_CNF_DIR_Output | -(GPIO_PIN_CNF_INPUT_Disconnect << GPIO_PIN_CNF_INPUT_Pos); + (GPIO_PIN_CNF_INPUT_Disconnect << GPIO_PIN_CNF_INPUT_Pos); port->DIRSET = HAL_GPIO_MASK(pin); return 0; ``` hw/mcu/nordic/nrf91xx/src/hal_i2c.c ```diff @@ -38,18 +38,18 @@ #define NRF91_SCL_PIN_CONF \ ((GPIO_PIN_CNF_SENSE_Disabled << GPIO_PIN_CNF_SENSE_Pos) | \ - (GPIO_PIN_CNF_DRIVE_S0D1<< GPIO_PIN_CNF_DRIVE_Pos) | \ - (GPIO_PIN_CNF_PULL_Pullup << GPIO_PIN_CNF_PULL_Pos) | \ + (GPIO_PIN_CNF_DRIVE_S0D1 << GPIO_PIN_CNF_DRIVE_Pos) | \ + (GPIO_PIN_CNF_PULL_Pullup << GPIO_PIN_CNF_PULL_Pos) | \ (GPIO_PIN_CNF_INPUT_Connect << GPIO_PIN_CNF_INPUT_Pos) | \ - (GPIO_PIN_CNF_DIR_Input << GPIO_PIN_CNF_DIR_Pos)) + (GPIO_PIN_CNF_DIR_Input << GPIO_PIN_CNF_DIR_Pos)) #define NRF91_SDA_PIN_CONF NRF91_SCL_PIN_CONF #define NRF91_SCL_PIN_CONF_CLR \ - ((GPIO_PIN_CNF_SENSE_Disabled << GPIO_PIN_CNF_SENSE_Pos) | \ - (GPIO_PIN_CNF_DRIVE_S0D1 << GPIO_PIN_CNF_DRIVE_Pos) | \ - (GPIO_PIN_CNF_PULL_Pullup<< GPIO_PIN_CNF_PULL_Pos) | \ - (GPIO_PIN_CNF_INPUT_Connect << GPIO_PIN_CNF_INPUT_Pos) | \ - (GPIO_PIN_CNF_DIR_Output << GPIO_PIN_CNF_DIR_Pos)) +((GPIO_PIN_CNF_SENSE_Disabled << GPIO_PIN_CNF_SENSE_Pos) | \ + (GPIO_PIN_CNF_DRIVE_S0D1 << GPIO_PIN_CNF_DRIVE_Pos) | \ + (GPIO_PIN_CNF_PULL_Pullup << GPIO_PIN_CNF_PULL_Pos) | \ + (GPIO_PIN_CNF_INPUT_Connect << GPIO_PIN_CNF_INPUT_Pos) | \ + (GPIO_PIN_CNF_DIR_Output << GPIO_PIN_CNF_DIR_Pos)) #define NRF91_SDA_PIN_CONF_CLRNRF91_SCL_PIN_CONF_CLR struct nrf91_hal_i2c { ``` hw/mcu/nordic/nrf91xx/src/hal_qspi.c ```diff @@ -162,7 +162,7 @@ while
[GitHub] [mynewt-core] apache-mynewt-bot commented on pull request #2348: Add NRF9160 BSP with I2C, SPI, UART and GPIO support
apache-mynewt-bot commented on pull request #2348: URL: https://github.com/apache/mynewt-core/pull/2348#issuecomment-670961274 ## Style check summary ### Our coding style is [here!](https://github.com/apache/mynewt-core/blob/master/CODING_STANDARDS.md) hw/bus/drivers/i2c_nrf91_twim/src/i2c_nrf91_twim.c ```diff @@ -40,17 +40,17 @@ #if MYNEWT_VAL(I2C_NRF91_TWIM_STAT) STATS_SECT_START(twim_stats_section) -STATS_SECT_ENTRY(sda_lo_err)/* SDA pulled low on r/w */ -STATS_SECT_ENTRY(sda_lo_err_nrecov) /* SDA pulled low on r/w (not recovered)*/ -STATS_SECT_ENTRY(scl_hi_err)/* SCL unresponsive */ -STATS_SECT_ENTRY(scl_hi_err_nrecov) /* SCL unresponsive (not recovered)*/ +STATS_SECT_ENTRY(sda_lo_err)/* SDA pulled low on r/w */ +STATS_SECT_ENTRY(sda_lo_err_nrecov) /* SDA pulled low on r/w (not recovered)*/ +STATS_SECT_ENTRY(scl_hi_err)/* SCL unresponsive */ +STATS_SECT_ENTRY(scl_hi_err_nrecov) /* SCL unresponsive (not recovered)*/ STATS_SECT_END STATS_NAME_START(twim_stats_section) -STATS_NAME(twim_stats_section, sda_lo_err) -STATS_NAME(twim_stats_section, sda_lo_err_nrecov) -STATS_NAME(twim_stats_section, scl_hi_err) -STATS_NAME(twim_stats_section, scl_hi_err_nrecov) +STATS_NAME(twim_stats_section, sda_lo_err) +STATS_NAME(twim_stats_section, sda_lo_err_nrecov) +STATS_NAME(twim_stats_section, scl_hi_err) +STATS_NAME(twim_stats_section, scl_hi_err_nrecov) STATS_NAME_END(twim_stats_section) #endif @@ -352,7 +352,8 @@ nrf_twim->TASKS_STOP = 1; nrf_twim->ENABLE = TWIM_ENABLE_ENABLE_Disabled; nrf_twim->ENABLE = TWIM_ENABLE_ENABLE_Enabled; -}; +} +; #if MYNEWT_VAL(I2C_NRF91_TWIM_STAT) STATS_INC(dd->stats, scl_hi_err_nrecov); ``` hw/mcu/nordic/nrf91xx/src/hal_flash.c ```diff @@ -25,7 +25,7 @@ #include "nrfx_config_nrf9160.h" #include "nrf_nvmc.h" -#define NRF91K_FLASH_SECTOR_SZ 4096 +#define NRF91K_FLASH_SECTOR_SZ 4096 static int nrf91k_flash_read(const struct hal_flash *dev, uint32_t address, void *dst, uint32_t num_bytes); @@ -49,8 +49,8 @@ const struct hal_flash nrf91k_flash_dev = { .hf_itf = _flash_funcs, .hf_base_addr = 0x, -.hf_size = 1024 * 1024, /* XXX read from factory info? */ -.hf_sector_cnt = 256, /* XXX read from factory info? */ +.hf_size = 1024 * 1024, /* XXX read from factory info? */ +.hf_sector_cnt = 256, /* XXX read from factory info? */ .hf_align = 1, .hf_erased_val = 0xff, }; ``` hw/mcu/nordic/nrf91xx/src/hal_gpio.c ```diff @@ -116,7 +116,7 @@ port->OUTCLR = HAL_GPIO_MASK(pin); } port->PIN_CNF[pin_index] = GPIO_PIN_CNF_DIR_Output | -(GPIO_PIN_CNF_INPUT_Disconnect << GPIO_PIN_CNF_INPUT_Pos); + (GPIO_PIN_CNF_INPUT_Disconnect << GPIO_PIN_CNF_INPUT_Pos); port->DIRSET = HAL_GPIO_MASK(pin); return 0; ``` hw/mcu/nordic/nrf91xx/src/hal_i2c.c ```diff @@ -38,18 +38,18 @@ #define NRF91_SCL_PIN_CONF \ ((GPIO_PIN_CNF_SENSE_Disabled << GPIO_PIN_CNF_SENSE_Pos) | \ - (GPIO_PIN_CNF_DRIVE_S0D1<< GPIO_PIN_CNF_DRIVE_Pos) | \ - (GPIO_PIN_CNF_PULL_Pullup << GPIO_PIN_CNF_PULL_Pos) | \ + (GPIO_PIN_CNF_DRIVE_S0D1 << GPIO_PIN_CNF_DRIVE_Pos) | \ + (GPIO_PIN_CNF_PULL_Pullup << GPIO_PIN_CNF_PULL_Pos) | \ (GPIO_PIN_CNF_INPUT_Connect << GPIO_PIN_CNF_INPUT_Pos) | \ - (GPIO_PIN_CNF_DIR_Input << GPIO_PIN_CNF_DIR_Pos)) + (GPIO_PIN_CNF_DIR_Input << GPIO_PIN_CNF_DIR_Pos)) #define NRF91_SDA_PIN_CONF NRF91_SCL_PIN_CONF #define NRF91_SCL_PIN_CONF_CLR \ - ((GPIO_PIN_CNF_SENSE_Disabled << GPIO_PIN_CNF_SENSE_Pos) | \ - (GPIO_PIN_CNF_DRIVE_S0D1 << GPIO_PIN_CNF_DRIVE_Pos) | \ - (GPIO_PIN_CNF_PULL_Pullup<< GPIO_PIN_CNF_PULL_Pos) | \ - (GPIO_PIN_CNF_INPUT_Connect << GPIO_PIN_CNF_INPUT_Pos) | \ - (GPIO_PIN_CNF_DIR_Output << GPIO_PIN_CNF_DIR_Pos)) +((GPIO_PIN_CNF_SENSE_Disabled << GPIO_PIN_CNF_SENSE_Pos) | \ + (GPIO_PIN_CNF_DRIVE_S0D1 << GPIO_PIN_CNF_DRIVE_Pos) | \ + (GPIO_PIN_CNF_PULL_Pullup << GPIO_PIN_CNF_PULL_Pos) | \ + (GPIO_PIN_CNF_INPUT_Connect << GPIO_PIN_CNF_INPUT_Pos) | \ + (GPIO_PIN_CNF_DIR_Output << GPIO_PIN_CNF_DIR_Pos)) #define NRF91_SDA_PIN_CONF_CLRNRF91_SCL_PIN_CONF_CLR struct nrf91_hal_i2c { ``` hw/mcu/nordic/nrf91xx/src/hal_qspi.c ```diff @@ -162,7 +162,7 @@ while
[GitHub] [mynewt-core] apache-mynewt-bot commented on pull request #2348: Add NRF9160 BSP with I2C, SPI, UART and GPIO support
apache-mynewt-bot commented on pull request #2348: URL: https://github.com/apache/mynewt-core/pull/2348#issuecomment-670797910 ## Style check summary ### Our coding style is [here!](https://github.com/apache/mynewt-core/blob/master/CODING_STANDARDS.md) hw/bus/drivers/i2c_nrf91_twim/src/i2c_nrf91_twim.c ```diff @@ -40,17 +40,17 @@ #if MYNEWT_VAL(I2C_NRF91_TWIM_STAT) STATS_SECT_START(twim_stats_section) -STATS_SECT_ENTRY(sda_lo_err)/* SDA pulled low on r/w */ -STATS_SECT_ENTRY(sda_lo_err_nrecov) /* SDA pulled low on r/w (not recovered)*/ -STATS_SECT_ENTRY(scl_hi_err)/* SCL unresponsive */ -STATS_SECT_ENTRY(scl_hi_err_nrecov) /* SCL unresponsive (not recovered)*/ +STATS_SECT_ENTRY(sda_lo_err)/* SDA pulled low on r/w */ +STATS_SECT_ENTRY(sda_lo_err_nrecov) /* SDA pulled low on r/w (not recovered)*/ +STATS_SECT_ENTRY(scl_hi_err)/* SCL unresponsive */ +STATS_SECT_ENTRY(scl_hi_err_nrecov) /* SCL unresponsive (not recovered)*/ STATS_SECT_END STATS_NAME_START(twim_stats_section) -STATS_NAME(twim_stats_section, sda_lo_err) -STATS_NAME(twim_stats_section, sda_lo_err_nrecov) -STATS_NAME(twim_stats_section, scl_hi_err) -STATS_NAME(twim_stats_section, scl_hi_err_nrecov) +STATS_NAME(twim_stats_section, sda_lo_err) +STATS_NAME(twim_stats_section, sda_lo_err_nrecov) +STATS_NAME(twim_stats_section, scl_hi_err) +STATS_NAME(twim_stats_section, scl_hi_err_nrecov) STATS_NAME_END(twim_stats_section) #endif @@ -352,7 +352,8 @@ nrf_twim->TASKS_STOP = 1; nrf_twim->ENABLE = TWIM_ENABLE_ENABLE_Disabled; nrf_twim->ENABLE = TWIM_ENABLE_ENABLE_Enabled; -}; +} +; #if MYNEWT_VAL(I2C_NRF91_TWIM_STAT) STATS_INC(dd->stats, scl_hi_err_nrecov); ``` hw/mcu/nordic/nrf91xx/src/hal_flash.c ```diff @@ -25,7 +25,7 @@ #include "nrfx_config_nrf9160.h" #include "nrf_nvmc.h" -#define NRF91K_FLASH_SECTOR_SZ 4096 +#define NRF91K_FLASH_SECTOR_SZ 4096 static int nrf91k_flash_read(const struct hal_flash *dev, uint32_t address, void *dst, uint32_t num_bytes); @@ -49,8 +49,8 @@ const struct hal_flash nrf91k_flash_dev = { .hf_itf = _flash_funcs, .hf_base_addr = 0x, -.hf_size = 1024 * 1024, /* XXX read from factory info? */ -.hf_sector_cnt = 256, /* XXX read from factory info? */ +.hf_size = 1024 * 1024, /* XXX read from factory info? */ +.hf_sector_cnt = 256, /* XXX read from factory info? */ .hf_align = 1, .hf_erased_val = 0xff, }; ``` hw/mcu/nordic/nrf91xx/src/hal_gpio.c ```diff @@ -116,7 +116,7 @@ port->OUTCLR = HAL_GPIO_MASK(pin); } port->PIN_CNF[pin_index] = GPIO_PIN_CNF_DIR_Output | -(GPIO_PIN_CNF_INPUT_Disconnect << GPIO_PIN_CNF_INPUT_Pos); + (GPIO_PIN_CNF_INPUT_Disconnect << GPIO_PIN_CNF_INPUT_Pos); port->DIRSET = HAL_GPIO_MASK(pin); return 0; ``` hw/mcu/nordic/nrf91xx/src/hal_i2c.c ```diff @@ -38,18 +38,18 @@ #define NRF91_SCL_PIN_CONF \ ((GPIO_PIN_CNF_SENSE_Disabled << GPIO_PIN_CNF_SENSE_Pos) | \ - (GPIO_PIN_CNF_DRIVE_S0D1<< GPIO_PIN_CNF_DRIVE_Pos) | \ - (GPIO_PIN_CNF_PULL_Pullup << GPIO_PIN_CNF_PULL_Pos) | \ + (GPIO_PIN_CNF_DRIVE_S0D1 << GPIO_PIN_CNF_DRIVE_Pos) | \ + (GPIO_PIN_CNF_PULL_Pullup << GPIO_PIN_CNF_PULL_Pos) | \ (GPIO_PIN_CNF_INPUT_Connect << GPIO_PIN_CNF_INPUT_Pos) | \ - (GPIO_PIN_CNF_DIR_Input << GPIO_PIN_CNF_DIR_Pos)) + (GPIO_PIN_CNF_DIR_Input << GPIO_PIN_CNF_DIR_Pos)) #define NRF91_SDA_PIN_CONF NRF91_SCL_PIN_CONF #define NRF91_SCL_PIN_CONF_CLR \ - ((GPIO_PIN_CNF_SENSE_Disabled << GPIO_PIN_CNF_SENSE_Pos) | \ - (GPIO_PIN_CNF_DRIVE_S0D1 << GPIO_PIN_CNF_DRIVE_Pos) | \ - (GPIO_PIN_CNF_PULL_Pullup<< GPIO_PIN_CNF_PULL_Pos) | \ - (GPIO_PIN_CNF_INPUT_Connect << GPIO_PIN_CNF_INPUT_Pos) | \ - (GPIO_PIN_CNF_DIR_Output << GPIO_PIN_CNF_DIR_Pos)) +((GPIO_PIN_CNF_SENSE_Disabled << GPIO_PIN_CNF_SENSE_Pos) | \ + (GPIO_PIN_CNF_DRIVE_S0D1 << GPIO_PIN_CNF_DRIVE_Pos) | \ + (GPIO_PIN_CNF_PULL_Pullup << GPIO_PIN_CNF_PULL_Pos) | \ + (GPIO_PIN_CNF_INPUT_Connect << GPIO_PIN_CNF_INPUT_Pos) | \ + (GPIO_PIN_CNF_DIR_Output << GPIO_PIN_CNF_DIR_Pos)) #define NRF91_SDA_PIN_CONF_CLRNRF91_SCL_PIN_CONF_CLR struct nrf91_hal_i2c { ``` hw/mcu/nordic/nrf91xx/src/hal_qspi.c ```diff @@ -162,7 +162,7 @@ while
[GitHub] [mynewt-core] apache-mynewt-bot commented on pull request #2348: Add NRF9160 BSP with I2C, SPI, UART and GPIO support
apache-mynewt-bot commented on pull request #2348: URL: https://github.com/apache/mynewt-core/pull/2348#issuecomment-670796817 ## Style check summary ### Our coding style is [here!](https://github.com/apache/mynewt-core/blob/master/CODING_STANDARDS.md) hw/bus/drivers/i2c_nrf91_twim/src/i2c_nrf91_twim.c ```diff @@ -40,17 +40,17 @@ #if MYNEWT_VAL(I2C_NRF91_TWIM_STAT) STATS_SECT_START(twim_stats_section) -STATS_SECT_ENTRY(sda_lo_err)/* SDA pulled low on r/w */ -STATS_SECT_ENTRY(sda_lo_err_nrecov) /* SDA pulled low on r/w (not recovered)*/ -STATS_SECT_ENTRY(scl_hi_err)/* SCL unresponsive */ -STATS_SECT_ENTRY(scl_hi_err_nrecov) /* SCL unresponsive (not recovered)*/ +STATS_SECT_ENTRY(sda_lo_err)/* SDA pulled low on r/w */ +STATS_SECT_ENTRY(sda_lo_err_nrecov) /* SDA pulled low on r/w (not recovered)*/ +STATS_SECT_ENTRY(scl_hi_err)/* SCL unresponsive */ +STATS_SECT_ENTRY(scl_hi_err_nrecov) /* SCL unresponsive (not recovered)*/ STATS_SECT_END STATS_NAME_START(twim_stats_section) -STATS_NAME(twim_stats_section, sda_lo_err) -STATS_NAME(twim_stats_section, sda_lo_err_nrecov) -STATS_NAME(twim_stats_section, scl_hi_err) -STATS_NAME(twim_stats_section, scl_hi_err_nrecov) +STATS_NAME(twim_stats_section, sda_lo_err) +STATS_NAME(twim_stats_section, sda_lo_err_nrecov) +STATS_NAME(twim_stats_section, scl_hi_err) +STATS_NAME(twim_stats_section, scl_hi_err_nrecov) STATS_NAME_END(twim_stats_section) #endif @@ -352,7 +352,8 @@ nrf_twim->TASKS_STOP = 1; nrf_twim->ENABLE = TWIM_ENABLE_ENABLE_Disabled; nrf_twim->ENABLE = TWIM_ENABLE_ENABLE_Enabled; -}; +} +; #if MYNEWT_VAL(I2C_NRF91_TWIM_STAT) STATS_INC(dd->stats, scl_hi_err_nrecov); ``` hw/mcu/nordic/nrf91xx/src/hal_flash.c ```diff @@ -25,7 +25,7 @@ #include "nrfx_config_nrf9160.h" #include "nrf_nvmc.h" -#define NRF91K_FLASH_SECTOR_SZ 4096 +#define NRF91K_FLASH_SECTOR_SZ 4096 static int nrf91k_flash_read(const struct hal_flash *dev, uint32_t address, void *dst, uint32_t num_bytes); @@ -49,8 +49,8 @@ const struct hal_flash nrf91k_flash_dev = { .hf_itf = _flash_funcs, .hf_base_addr = 0x, -.hf_size = 1024 * 1024, /* XXX read from factory info? */ -.hf_sector_cnt = 256, /* XXX read from factory info? */ +.hf_size = 1024 * 1024, /* XXX read from factory info? */ +.hf_sector_cnt = 256, /* XXX read from factory info? */ .hf_align = 1, .hf_erased_val = 0xff, }; ``` hw/mcu/nordic/nrf91xx/src/hal_gpio.c ```diff @@ -116,7 +116,7 @@ port->OUTCLR = HAL_GPIO_MASK(pin); } port->PIN_CNF[pin_index] = GPIO_PIN_CNF_DIR_Output | -(GPIO_PIN_CNF_INPUT_Disconnect << GPIO_PIN_CNF_INPUT_Pos); + (GPIO_PIN_CNF_INPUT_Disconnect << GPIO_PIN_CNF_INPUT_Pos); port->DIRSET = HAL_GPIO_MASK(pin); return 0; ``` hw/mcu/nordic/nrf91xx/src/hal_i2c.c ```diff @@ -38,18 +38,18 @@ #define NRF91_SCL_PIN_CONF \ ((GPIO_PIN_CNF_SENSE_Disabled << GPIO_PIN_CNF_SENSE_Pos) | \ - (GPIO_PIN_CNF_DRIVE_S0D1<< GPIO_PIN_CNF_DRIVE_Pos) | \ - (GPIO_PIN_CNF_PULL_Pullup << GPIO_PIN_CNF_PULL_Pos) | \ + (GPIO_PIN_CNF_DRIVE_S0D1 << GPIO_PIN_CNF_DRIVE_Pos) | \ + (GPIO_PIN_CNF_PULL_Pullup << GPIO_PIN_CNF_PULL_Pos) | \ (GPIO_PIN_CNF_INPUT_Connect << GPIO_PIN_CNF_INPUT_Pos) | \ - (GPIO_PIN_CNF_DIR_Input << GPIO_PIN_CNF_DIR_Pos)) + (GPIO_PIN_CNF_DIR_Input << GPIO_PIN_CNF_DIR_Pos)) #define NRF91_SDA_PIN_CONF NRF91_SCL_PIN_CONF #define NRF91_SCL_PIN_CONF_CLR \ - ((GPIO_PIN_CNF_SENSE_Disabled << GPIO_PIN_CNF_SENSE_Pos) | \ - (GPIO_PIN_CNF_DRIVE_S0D1 << GPIO_PIN_CNF_DRIVE_Pos) | \ - (GPIO_PIN_CNF_PULL_Pullup<< GPIO_PIN_CNF_PULL_Pos) | \ - (GPIO_PIN_CNF_INPUT_Connect << GPIO_PIN_CNF_INPUT_Pos) | \ - (GPIO_PIN_CNF_DIR_Output << GPIO_PIN_CNF_DIR_Pos)) +((GPIO_PIN_CNF_SENSE_Disabled << GPIO_PIN_CNF_SENSE_Pos) | \ + (GPIO_PIN_CNF_DRIVE_S0D1 << GPIO_PIN_CNF_DRIVE_Pos) | \ + (GPIO_PIN_CNF_PULL_Pullup << GPIO_PIN_CNF_PULL_Pos) | \ + (GPIO_PIN_CNF_INPUT_Connect << GPIO_PIN_CNF_INPUT_Pos) | \ + (GPIO_PIN_CNF_DIR_Output << GPIO_PIN_CNF_DIR_Pos)) #define NRF91_SDA_PIN_CONF_CLRNRF91_SCL_PIN_CONF_CLR struct nrf91_hal_i2c { ``` hw/mcu/nordic/nrf91xx/src/hal_qspi.c ```diff @@ -162,7 +162,7 @@ while
[GitHub] [mynewt-core] apache-mynewt-bot commented on pull request #2348: Add NRF9160 BSP with I2C, SPI, UART and GPIO support
apache-mynewt-bot commented on pull request #2348: URL: https://github.com/apache/mynewt-core/pull/2348#issuecomment-670794461 ## Style check summary ### Our coding style is [here!](https://github.com/apache/mynewt-core/blob/master/CODING_STANDARDS.md) hw/bus/drivers/i2c_nrf91_twim/src/i2c_nrf91_twim.c ```diff @@ -40,17 +40,17 @@ #if MYNEWT_VAL(I2C_NRF91_TWIM_STAT) STATS_SECT_START(twim_stats_section) -STATS_SECT_ENTRY(sda_lo_err)/* SDA pulled low on r/w */ -STATS_SECT_ENTRY(sda_lo_err_nrecov) /* SDA pulled low on r/w (not recovered)*/ -STATS_SECT_ENTRY(scl_hi_err)/* SCL unresponsive */ -STATS_SECT_ENTRY(scl_hi_err_nrecov) /* SCL unresponsive (not recovered)*/ +STATS_SECT_ENTRY(sda_lo_err)/* SDA pulled low on r/w */ +STATS_SECT_ENTRY(sda_lo_err_nrecov) /* SDA pulled low on r/w (not recovered)*/ +STATS_SECT_ENTRY(scl_hi_err)/* SCL unresponsive */ +STATS_SECT_ENTRY(scl_hi_err_nrecov) /* SCL unresponsive (not recovered)*/ STATS_SECT_END STATS_NAME_START(twim_stats_section) -STATS_NAME(twim_stats_section, sda_lo_err) -STATS_NAME(twim_stats_section, sda_lo_err_nrecov) -STATS_NAME(twim_stats_section, scl_hi_err) -STATS_NAME(twim_stats_section, scl_hi_err_nrecov) +STATS_NAME(twim_stats_section, sda_lo_err) +STATS_NAME(twim_stats_section, sda_lo_err_nrecov) +STATS_NAME(twim_stats_section, scl_hi_err) +STATS_NAME(twim_stats_section, scl_hi_err_nrecov) STATS_NAME_END(twim_stats_section) #endif @@ -352,7 +352,8 @@ nrf_twim->TASKS_STOP = 1; nrf_twim->ENABLE = TWIM_ENABLE_ENABLE_Disabled; nrf_twim->ENABLE = TWIM_ENABLE_ENABLE_Enabled; -}; +} +; #if MYNEWT_VAL(I2C_NRF91_TWIM_STAT) STATS_INC(dd->stats, scl_hi_err_nrecov); ``` hw/mcu/nordic/nrf91xxx/src/hal_flash.c ```diff @@ -25,7 +25,7 @@ #include "nrfx_config_nrf9160.h" #include "nrf_nvmc.h" -#define NRF91K_FLASH_SECTOR_SZ 4096 +#define NRF91K_FLASH_SECTOR_SZ 4096 static int nrf91k_flash_read(const struct hal_flash *dev, uint32_t address, void *dst, uint32_t num_bytes); @@ -49,8 +49,8 @@ const struct hal_flash nrf91k_flash_dev = { .hf_itf = _flash_funcs, .hf_base_addr = 0x, -.hf_size = 1024 * 1024, /* XXX read from factory info? */ -.hf_sector_cnt = 256, /* XXX read from factory info? */ +.hf_size = 1024 * 1024, /* XXX read from factory info? */ +.hf_sector_cnt = 256, /* XXX read from factory info? */ .hf_align = 1, .hf_erased_val = 0xff, }; ``` hw/mcu/nordic/nrf91xxx/src/hal_gpio.c ```diff @@ -116,7 +116,7 @@ port->OUTCLR = HAL_GPIO_MASK(pin); } port->PIN_CNF[pin_index] = GPIO_PIN_CNF_DIR_Output | -(GPIO_PIN_CNF_INPUT_Disconnect << GPIO_PIN_CNF_INPUT_Pos); + (GPIO_PIN_CNF_INPUT_Disconnect << GPIO_PIN_CNF_INPUT_Pos); port->DIRSET = HAL_GPIO_MASK(pin); return 0; ``` hw/mcu/nordic/nrf91xxx/src/hal_i2c.c ```diff @@ -38,18 +38,18 @@ #define NRF91_SCL_PIN_CONF \ ((GPIO_PIN_CNF_SENSE_Disabled << GPIO_PIN_CNF_SENSE_Pos) | \ - (GPIO_PIN_CNF_DRIVE_S0D1<< GPIO_PIN_CNF_DRIVE_Pos) | \ - (GPIO_PIN_CNF_PULL_Pullup << GPIO_PIN_CNF_PULL_Pos) | \ + (GPIO_PIN_CNF_DRIVE_S0D1 << GPIO_PIN_CNF_DRIVE_Pos) | \ + (GPIO_PIN_CNF_PULL_Pullup << GPIO_PIN_CNF_PULL_Pos) | \ (GPIO_PIN_CNF_INPUT_Connect << GPIO_PIN_CNF_INPUT_Pos) | \ - (GPIO_PIN_CNF_DIR_Input << GPIO_PIN_CNF_DIR_Pos)) + (GPIO_PIN_CNF_DIR_Input << GPIO_PIN_CNF_DIR_Pos)) #define NRF91_SDA_PIN_CONF NRF91_SCL_PIN_CONF #define NRF91_SCL_PIN_CONF_CLR \ - ((GPIO_PIN_CNF_SENSE_Disabled << GPIO_PIN_CNF_SENSE_Pos) | \ - (GPIO_PIN_CNF_DRIVE_S0D1 << GPIO_PIN_CNF_DRIVE_Pos) | \ - (GPIO_PIN_CNF_PULL_Pullup<< GPIO_PIN_CNF_PULL_Pos) | \ - (GPIO_PIN_CNF_INPUT_Connect << GPIO_PIN_CNF_INPUT_Pos) | \ - (GPIO_PIN_CNF_DIR_Output << GPIO_PIN_CNF_DIR_Pos)) +((GPIO_PIN_CNF_SENSE_Disabled << GPIO_PIN_CNF_SENSE_Pos) | \ + (GPIO_PIN_CNF_DRIVE_S0D1 << GPIO_PIN_CNF_DRIVE_Pos) | \ + (GPIO_PIN_CNF_PULL_Pullup << GPIO_PIN_CNF_PULL_Pos) | \ + (GPIO_PIN_CNF_INPUT_Connect << GPIO_PIN_CNF_INPUT_Pos) | \ + (GPIO_PIN_CNF_DIR_Output << GPIO_PIN_CNF_DIR_Pos)) #define NRF91_SDA_PIN_CONF_CLRNRF91_SCL_PIN_CONF_CLR struct nrf91_hal_i2c { ``` hw/mcu/nordic/nrf91xxx/src/hal_qspi.c ```diff @@ -162,7 +162,7 @@ while
[GitHub] [mynewt-core] apache-mynewt-bot commented on pull request #2348: Add NRF9160 BSP with I2C, SPI, UART and GPIO support
apache-mynewt-bot commented on pull request #2348: URL: https://github.com/apache/mynewt-core/pull/2348#issuecomment-670794368 ## RAT Report (2020-08-08 00:24:24) ## New files with unknown licenses * https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/approtect_detect;>hw/bsp/nordic_pca10090/approtect_detect * https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/boot-nrf91xxaa.ld;>hw/bsp/nordic_pca10090/boot-nrf91xxaa.ld * https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/nordic_pca10090_no_boot.ld;>hw/bsp/nordic_pca10090/nordic_pca10090_no_boot.ld * https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/nrf91xxaa.ld;>hw/bsp/nordic_pca10090/nrf91xxaa.ld * https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/split-nordic_pca10090.ld;>hw/bsp/nordic_pca10090/split-nordic_pca10090.ld * https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/targetpoweron;>hw/bsp/nordic_pca10090/targetpoweron * https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/src/arch/cortex_m33/gcc_startup_nrf9160.s;>hw/bsp/nordic_pca10090/src/arch/cortex_m33/gcc_startup_nrf9160.s * https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/src/arch/cortex_m33/gcc_startup_nrf9160_split.s;>hw/bsp/nordic_pca10090/src/arch/cortex_m33/gcc_startup_nrf9160_split.s * https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/mcu/nordic/nrf91xxx/nrf91.ld;>hw/mcu/nordic/nrf91xxx/nrf91.ld * https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/mcu/nordic/nrf91xxx/nrf91_ram_resident.ld;>hw/mcu/nordic/nrf91xxx/nrf91_ram_resident.ld * https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/mcu/nordic/nrf91xxx/src/system_nrf9160.c;>hw/mcu/nordic/nrf91xxx/src/system_nrf9160.c ## 53 new files were excluded from check (.rat-excludes) Detailed analysis ## New files in this PR | License | File | |-|--| | ? | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/approtect_detect;>hw/bsp/nordic_pca10090/approtect_detect | | ? | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/boot-nrf91xxaa.ld;>hw/bsp/nordic_pca10090/boot-nrf91xxaa.ld | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/bsp.yml;>hw/bsp/nordic_pca10090/bsp.yml | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/nordic_pca10090_debug.cmd;>hw/bsp/nordic_pca10090/nordic_pca10090_debug.cmd | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/nordic_pca10090_debug.sh;>hw/bsp/nordic_pca10090/nordic_pca10090_debug.sh | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/nordic_pca10090_download.cmd;>hw/bsp/nordic_pca10090/nordic_pca10090_download.cmd | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/nordic_pca10090_download.sh;>hw/bsp/nordic_pca10090/nordic_pca10090_download.sh | | ? | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/nordic_pca10090_no_boot.ld;>hw/bsp/nordic_pca10090/nordic_pca10090_no_boot.ld | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/nrf9160dk_common.sh;>hw/bsp/nordic_pca10090/nrf9160dk_common.sh | | ? | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/nrf91xxaa.ld;>hw/bsp/nordic_pca10090/nrf91xxaa.ld | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/pkg.yml;>hw/bsp/nordic_pca10090/pkg.yml | | ? | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/split-nordic_pca10090.ld;>hw/bsp/nordic_pca10090/split-nordic_pca10090.ld | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/syscfg.yml;>hw/bsp/nordic_pca10090/syscfg.yml | | ? |
[GitHub] [mynewt-core] apache-mynewt-bot commented on pull request #2348: Add NRF9160 BSP with I2C, SPI, UART and GPIO support
apache-mynewt-bot commented on pull request #2348: URL: https://github.com/apache/mynewt-core/pull/2348#issuecomment-670652063 ## Style check summary ### Our coding style is [here!](https://github.com/apache/mynewt-core/blob/master/CODING_STANDARDS.md) hw/bus/drivers/i2c_nrf91_twim/src/i2c_nrf91_twim.c ```diff @@ -40,17 +40,17 @@ #if MYNEWT_VAL(I2C_NRF91_TWIM_STAT) STATS_SECT_START(twim_stats_section) -STATS_SECT_ENTRY(sda_lo_err)/* SDA pulled low on r/w */ -STATS_SECT_ENTRY(sda_lo_err_nrecov) /* SDA pulled low on r/w (not recovered)*/ -STATS_SECT_ENTRY(scl_hi_err)/* SCL unresponsive */ -STATS_SECT_ENTRY(scl_hi_err_nrecov) /* SCL unresponsive (not recovered)*/ +STATS_SECT_ENTRY(sda_lo_err)/* SDA pulled low on r/w */ +STATS_SECT_ENTRY(sda_lo_err_nrecov) /* SDA pulled low on r/w (not recovered)*/ +STATS_SECT_ENTRY(scl_hi_err)/* SCL unresponsive */ +STATS_SECT_ENTRY(scl_hi_err_nrecov) /* SCL unresponsive (not recovered)*/ STATS_SECT_END STATS_NAME_START(twim_stats_section) -STATS_NAME(twim_stats_section, sda_lo_err) -STATS_NAME(twim_stats_section, sda_lo_err_nrecov) -STATS_NAME(twim_stats_section, scl_hi_err) -STATS_NAME(twim_stats_section, scl_hi_err_nrecov) +STATS_NAME(twim_stats_section, sda_lo_err) +STATS_NAME(twim_stats_section, sda_lo_err_nrecov) +STATS_NAME(twim_stats_section, scl_hi_err) +STATS_NAME(twim_stats_section, scl_hi_err_nrecov) STATS_NAME_END(twim_stats_section) #endif @@ -352,7 +352,8 @@ nrf_twim->TASKS_STOP = 1; nrf_twim->ENABLE = TWIM_ENABLE_ENABLE_Disabled; nrf_twim->ENABLE = TWIM_ENABLE_ENABLE_Enabled; -}; +} +; #if MYNEWT_VAL(I2C_NRF91_TWIM_STAT) STATS_INC(dd->stats, scl_hi_err_nrecov); ``` hw/mcu/nordic/nrf91xxx/src/hal_flash.c ```diff @@ -25,7 +25,7 @@ #include "nrfx_config_nrf9160.h" #include "nrf_nvmc.h" -#define NRF91K_FLASH_SECTOR_SZ 4096 +#define NRF91K_FLASH_SECTOR_SZ 4096 static int nrf91k_flash_read(const struct hal_flash *dev, uint32_t address, void *dst, uint32_t num_bytes); @@ -49,8 +49,8 @@ const struct hal_flash nrf91k_flash_dev = { .hf_itf = _flash_funcs, .hf_base_addr = 0x, -.hf_size = 1024 * 1024, /* XXX read from factory info? */ -.hf_sector_cnt = 256, /* XXX read from factory info? */ +.hf_size = 1024 * 1024, /* XXX read from factory info? */ +.hf_sector_cnt = 256, /* XXX read from factory info? */ .hf_align = 1, .hf_erased_val = 0xff, }; ``` hw/mcu/nordic/nrf91xxx/src/hal_gpio.c ```diff @@ -116,7 +116,7 @@ port->OUTCLR = HAL_GPIO_MASK(pin); } port->PIN_CNF[pin_index] = GPIO_PIN_CNF_DIR_Output | -(GPIO_PIN_CNF_INPUT_Disconnect << GPIO_PIN_CNF_INPUT_Pos); + (GPIO_PIN_CNF_INPUT_Disconnect << GPIO_PIN_CNF_INPUT_Pos); port->DIRSET = HAL_GPIO_MASK(pin); return 0; ``` hw/mcu/nordic/nrf91xxx/src/hal_i2c.c ```diff @@ -38,18 +38,18 @@ #define NRF91_SCL_PIN_CONF \ ((GPIO_PIN_CNF_SENSE_Disabled << GPIO_PIN_CNF_SENSE_Pos) | \ - (GPIO_PIN_CNF_DRIVE_S0D1<< GPIO_PIN_CNF_DRIVE_Pos) | \ - (GPIO_PIN_CNF_PULL_Pullup << GPIO_PIN_CNF_PULL_Pos) | \ + (GPIO_PIN_CNF_DRIVE_S0D1 << GPIO_PIN_CNF_DRIVE_Pos) | \ + (GPIO_PIN_CNF_PULL_Pullup << GPIO_PIN_CNF_PULL_Pos) | \ (GPIO_PIN_CNF_INPUT_Connect << GPIO_PIN_CNF_INPUT_Pos) | \ - (GPIO_PIN_CNF_DIR_Input << GPIO_PIN_CNF_DIR_Pos)) + (GPIO_PIN_CNF_DIR_Input << GPIO_PIN_CNF_DIR_Pos)) #define NRF91_SDA_PIN_CONF NRF91_SCL_PIN_CONF #define NRF91_SCL_PIN_CONF_CLR \ - ((GPIO_PIN_CNF_SENSE_Disabled << GPIO_PIN_CNF_SENSE_Pos) | \ - (GPIO_PIN_CNF_DRIVE_S0D1 << GPIO_PIN_CNF_DRIVE_Pos) | \ - (GPIO_PIN_CNF_PULL_Pullup<< GPIO_PIN_CNF_PULL_Pos) | \ - (GPIO_PIN_CNF_INPUT_Connect << GPIO_PIN_CNF_INPUT_Pos) | \ - (GPIO_PIN_CNF_DIR_Output << GPIO_PIN_CNF_DIR_Pos)) +((GPIO_PIN_CNF_SENSE_Disabled << GPIO_PIN_CNF_SENSE_Pos) | \ + (GPIO_PIN_CNF_DRIVE_S0D1 << GPIO_PIN_CNF_DRIVE_Pos) | \ + (GPIO_PIN_CNF_PULL_Pullup << GPIO_PIN_CNF_PULL_Pos) | \ + (GPIO_PIN_CNF_INPUT_Connect << GPIO_PIN_CNF_INPUT_Pos) | \ + (GPIO_PIN_CNF_DIR_Output << GPIO_PIN_CNF_DIR_Pos)) #define NRF91_SDA_PIN_CONF_CLRNRF91_SCL_PIN_CONF_CLR struct nrf91_hal_i2c { ``` hw/mcu/nordic/nrf91xxx/src/hal_qspi.c ```diff @@ -162,7 +162,7 @@ while
[GitHub] [mynewt-core] apache-mynewt-bot commented on pull request #2348: Add NRF9160 BSP with I2C, SPI, UART and GPIO support
apache-mynewt-bot commented on pull request #2348: URL: https://github.com/apache/mynewt-core/pull/2348#issuecomment-670651943 ## RAT Report (2020-08-07 18:31:47) ## New files with unknown licenses * https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/approtect_detect;>hw/bsp/nordic_pca10090/approtect_detect * https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/boot-nrf91xxaa.ld;>hw/bsp/nordic_pca10090/boot-nrf91xxaa.ld * https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/nordic_pca10090_no_boot.ld;>hw/bsp/nordic_pca10090/nordic_pca10090_no_boot.ld * https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/nrf91xxaa.ld;>hw/bsp/nordic_pca10090/nrf91xxaa.ld * https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/split-nordic_pca10090.ld;>hw/bsp/nordic_pca10090/split-nordic_pca10090.ld * https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/targetpoweron;>hw/bsp/nordic_pca10090/targetpoweron * https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/src/arch/cortex_m33/gcc_startup_nrf9160.s;>hw/bsp/nordic_pca10090/src/arch/cortex_m33/gcc_startup_nrf9160.s * https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/src/arch/cortex_m33/gcc_startup_nrf9160_split.s;>hw/bsp/nordic_pca10090/src/arch/cortex_m33/gcc_startup_nrf9160_split.s * https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/mcu/nordic/nrf91xxx/nrf91.ld;>hw/mcu/nordic/nrf91xxx/nrf91.ld * https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/mcu/nordic/nrf91xxx/nrf91_ram_resident.ld;>hw/mcu/nordic/nrf91xxx/nrf91_ram_resident.ld * https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/mcu/nordic/nrf91xxx/src/system_nrf9160.c;>hw/mcu/nordic/nrf91xxx/src/system_nrf9160.c ## 53 new files were excluded from check (.rat-excludes) Detailed analysis ## New files in this PR | License | File | |-|--| | ? | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/approtect_detect;>hw/bsp/nordic_pca10090/approtect_detect | | ? | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/boot-nrf91xxaa.ld;>hw/bsp/nordic_pca10090/boot-nrf91xxaa.ld | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/bsp.yml;>hw/bsp/nordic_pca10090/bsp.yml | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/nordic_pca10090_debug.cmd;>hw/bsp/nordic_pca10090/nordic_pca10090_debug.cmd | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/nordic_pca10090_debug.sh;>hw/bsp/nordic_pca10090/nordic_pca10090_debug.sh | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/nordic_pca10090_download.cmd;>hw/bsp/nordic_pca10090/nordic_pca10090_download.cmd | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/nordic_pca10090_download.sh;>hw/bsp/nordic_pca10090/nordic_pca10090_download.sh | | ? | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/nordic_pca10090_no_boot.ld;>hw/bsp/nordic_pca10090/nordic_pca10090_no_boot.ld | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/nrf9160dk_common.sh;>hw/bsp/nordic_pca10090/nrf9160dk_common.sh | | ? | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/nrf91xxaa.ld;>hw/bsp/nordic_pca10090/nrf91xxaa.ld | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/pkg.yml;>hw/bsp/nordic_pca10090/pkg.yml | | ? | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/split-nordic_pca10090.ld;>hw/bsp/nordic_pca10090/split-nordic_pca10090.ld | | AL | https://github.com/apache/mynewt-core/blob/dc40231c4894a0ccf639ed8be09a86c91c41421f/hw/bsp/nordic_pca10090/syscfg.yml;>hw/bsp/nordic_pca10090/syscfg.yml | | ? |
[GitHub] [mynewt-core] apache-mynewt-bot commented on pull request #2348: Add NRF9160 BSP with I2C, SPI, UART and GPIO support
apache-mynewt-bot commented on pull request #2348: URL: https://github.com/apache/mynewt-core/pull/2348#issuecomment-670647757 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [mynewt-core] apache-mynewt-bot commented on pull request #2348: Add NRF9160 BSP with I2C, SPI, UART and GPIO support
apache-mynewt-bot commented on pull request #2348: URL: https://github.com/apache/mynewt-core/pull/2348#issuecomment-670289247 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org