cisvanmierlo commented on code in PR #6315: URL: https://github.com/apache/incubator-nuttx/pull/6315#discussion_r880442001
########## drivers/lcd/ssd1306_base.c: ########## @@ -782,7 +783,7 @@ static int ssd1306_setpower(FAR struct lcd_dev_s *dev, int power) { /* Configure display and turn the display on */ - ret = ssd1306_configuredisplay(priv); + ret = ssd1306_configuredisplay(dev); Review Comment: Hi @xiaoxiang781216, In our case, this SSD1306 display is pulling power from the I2C lines because the supply voltage of the display is not active yet. This causes the I2C transactions to look OK, but in fact it is not being configured correctly. I've checked, but there is no way to read out this display if it went OK. We thought of a better solution and opened an other PR, see closing comment. -- 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