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


Reply via email to