Esp32s3 i2c interface

[dimajolkin]

Was added for esp32s3 i2c interface full based in esp32c3

Tested on: esp32s3, esp32s3-wroom1, esp32c3..
For test using a co2 sensor & Tdeck keyboard

[deadprogram]

Can you please add a smoketest for this in the similar fashion as the previous PRs @dimajolkin thank you!

[dimajolkin]

I can't find any I2C tests for the ESP32XX (except for the I2C target, because the Listen(..) method isn't implemented in the ESP32XX).
I added a simple echo test.

Perhaps I can add the esp32s3 to the i2c target since it has two I2C...

[deadprogram]

I can't find any I2C tests for the ESP32XX (except for the I2C target, because the Listen(..) method isn't implemented in the ESP32XX). I added a simple echo test.

Perhaps I can add the esp32s3 to the i2c target since it has two I2C...

Sorry I should have been more specific.

The examples/blinkm uses an I2C device and is what we have been using for the I2C smoketest.

Looking, I see we are only doing this on 2 boards at present. Oops!

In any case, you can perhaps use it instead of adding another example.

[dimajolkin]

@deadprogram Maybe we should rename it to i2c-blink this blinkm?

[dimajolkin]

As I understand it, we can add the esp32s3 to the i2c-target tests, but this requires implementing slave mode. If you don't mind doing this in a separate MR, I think there's some work on that. It just needs to be tested.

[deadprogram]

@deadprogram Maybe we should rename it to i2c-blink this blinkm?

It is named this because of the https://thingm.com/products/blinkm

[deadprogram]

As I understand it, we can add the esp32s3 to the i2c-target tests, but this requires implementing slave mode. If you don't mind doing this in a separate MR, I think there's some work on that. It just needs to be tested.

Doing in a different PR seems quite fine to me.

[deadprogram]

Thanks for working on this @dimajolkin and to @soypat for helping review. Now squash/merging.