I2C conflict on ESP32 with Arduino (as an ESP-IDF component)

[fsender]

I Build the LovyanGFX demo project (1.simple_use)
Then run with Serial ERROR below:

E (136) i2c: CONFLICT! driver_ng is not allowed to be used with this old driver

Platform: Windows(R) 11
IDE: Visual Studio Code - Insiders 1.98.0
ESP-IDF Ver: 5.4
Use together with and Arduino (as an ESP-IDF component) Ver. 3.2
Board: ESP32 pico D4 with 4MB external flash

[fsender]

It's better for developers to give a method to use I2C by Arduino Wire.h, by software I2C or even do not use any I2C functions.
A macro like #define LGFX_USE_I2C USE_ARDUINO_WIRE_I2C or #define LGFX_USE_I2C USE_SOWTWARE_I2C may be useful.

[blathers123]

The ESP32 uses esp-idf under the hood, with the arduino parts built on-top using a core from espressif. Since all the actual calls are using esp-idf, this issue isn't related to Wire.h, but rather an outdated usage of the esp-idf i2c driver which has been deprecated in favor of a newer i2c_master and i2c_slave. LovyanGFX needs to switch to using the new i2c driver

[blathers123]

see #675

[fsender]

Just need to avoid including file <driver/i2c.h> can solve this issue.

If comment all #include <driver/i2c.h> lines, These compile errors are easy to solve.

One of the ERROR:

components/LovyanGFX/src/lgfx/v1/platforms/esp32/common.cpp:943:31: error: 'I2C_NUM_MAX' was not declared in this scope; did you mean 'UART_NUM_MAX'?
  943 |     i2c_context_t i2c_context[I2C_NUM_MAX];
      |                               ^~~~~~~~~~~
      |                               UART_NUM_MAX
components/LovyanGFX/src/lgfx/v1/platforms/esp32/common.cpp: In function 'void lgfx::v1::i2c::i2c_stop(int)':
components/LovyanGFX/src/lgfx/v1/platforms/esp32/common.cpp:993:27: error: 'i2c_context' was not declared in this scope; did you mean 'i2c_context_t'?
  993 |       gpio_num_t sda_io = i2c_context[i2c_port].pin_sda;
      |                           ^~~~~~~~~~~
      |                           i2c_context_t
components/LovyanGFX/src/lgfx/v1/platforms/esp32/common.cpp:1021:20: error: 'i2c_port_t' was not declared in this scope; did you mean 'i2c_port'?
 1021 |       i2c_set_pin((i2c_port_t)i2c_port, sda_io, scl_io, gpio_pullup_t::GPIO_PULLUP_ENABLE, gpio_pullup_t::GPIO_PULLUP_ENABLE, I2C_MODE_MASTER);
      |                    ^~~~~~~~~~
      |                    i2c_port
components/LovyanGFX/src/lgfx/v1/platforms/esp32/common.cpp:1021:127: error: 'I2C_MODE_MASTER' was not declared in this scope
 1021 |       i2c_set_pin((i2c_port_t)i2c_port, sda_io, scl_io, gpio_pullup_t::GPIO_PULLUP_ENABLE, gpio_pullup_t::GPIO_PULLUP_ENABLE, I2C_MODE_MASTER);
      |                                                                                                                               ^~~~~~~~~~~~~~~
........
........
........
components/LovyanGFX/src/lgfx/v1/platforms/esp32/common.cpp:1263:23: error: 'I2C_NUM_MAX' was not declared in this scope; did you mean 'UART_NUM_MAX'?
 1263 |       if (i2c_port >= I2C_NUM_MAX) { return cpp::fail(error_t::invalid_arg); }
      |                       ^~~~~~~~~~~
      |                       UART_NUM_MAX
components/LovyanGFX/src/lgfx/v1/platforms/esp32/common.cpp:1278:46: error: 'I2C_MASTER_READ' was not declared in this scope; did you mean 'I2C_DATE_REG'?
 1278 |         *fifo_addr = i2c_addr << 1 | (read ? I2C_MASTER_READ : I2C_MASTER_WRITE);
      |                                              ^~~~~~~~~~~~~~~
      |                                              I2C_DATE_REG
components/LovyanGFX/src/lgfx/v1/platforms/esp32/common.cpp:1278:64: error: 'I2C_MASTER_WRITE' was not declared in this scope
 1278 |         *fifo_addr = i2c_addr << 1 | (read ? I2C_MASTER_READ : I2C_MASTER_WRITE);
      |                                                                ^~~~~~~~~~~~~~~~

• Add #include <hal/i2c_types.h> to get the constants.

The other ERROR

components/LovyanGFX/src/lgfx/v1/platforms/esp32/common.cpp: In function 'void lgfx::v1::i2c::i2c_stop(int)':
components/LovyanGFX/src/lgfx/v1/platforms/esp32/common.cpp:1021:7: error: 'i2c_set_pin' was not declared in this scope; did you mean 'i2c_set_cmd'?
 1021 |       i2c_set_pin((i2c_port_t)i2c_port, sda_io, scl_io, gpio_pullup_t::GPIO_PULLUP_ENABLE, gpio_pullup_t::GPIO_PULLUP_ENABLE, I2C_MODE_MASTER);
      |       ^~~~~~~~~~~
      |       i2c_set_cmd

• Comment all calling of i2c_set_pin(). This operation will let I2C do not work.
• The best way is rewrite these functions using new I2C API.

[tobozo]

hi,

please try with the develop branch as it implements the new I2C master driver

[fsender]

I tried. It works.