phfbertoleti commented on a change in pull request #2888: URL: https://github.com/apache/incubator-nuttx/pull/2888#discussion_r580244943
########## File path: boards/xtensa/esp32/esp32-wrover-kit/src/esp32_bringup.c ########## @@ -259,6 +264,24 @@ int esp32_bringup(void) } #endif +#ifdef CONFIG_SENSORS_BMP180 + ret = board_bmp180_initialize(0, 0); + + if (ret < 0) + { + syslog(LOG_ERR, "Failed to initialize BMP180 + Driver for I2C0: %d\n", ret); + ret2 = board_bmp180_initialize(0, 1); Review comment: @Ouss4 , I'm searching for BMP180 in both I2C buses because of possibility to enable 2 I2Cs in ESP32. When both i2C buses are enabled and BMP180 device driver is also enabled, with recourses we have today in menuconfig, there isn't a way to know in which I2C is the BMP180 wired in. This is why I search in both buses. Is it clear? ---------------------------------------------------------------- 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