C6: LP_I2C basic driver

[JurajSadel]

Thank you!

Thank you for your contribution.
Please make sure that your submission includes the following:

Must

• The code compiles without errors or warnings.
• All examples work.
• cargo fmt was run.
• Your changes were added to the CHANGELOG.md in the proper section.
• You updated existing examples or added examples (if applicable).
• Added examples are checked in CI

Nice to have

• You add a description of your work to this PR.
• You added proper docs for your newly added features and code.

CC @playfulFence

[bjoernQ]

I also tested on HW - apparently it works.

Given the example is not too exciting maybe adding a note that users should use a logic-analyzer or scope to actually see something would be a good thing?

[bjoernQ]

LGTM - Thanks

[MabezDev]

Am I understanding this correctly that we are copy and pasting the lp driver into esp-hal, once its complete in esp-lp-hal?

Is there any reason we aren't depending on esp-lp-hal in esp-hal and re-exporting the driver there for the HP to use? I guess the pacs have different base addresses.

I will create an issue to fix this at some point, but I'm happy to merge this as is for now.

Thanks @JurajSadel and @playfulFence for working on this and bringing it over the finish line!

[jessebraham]

Am I understanding this correctly that we are copy and pasting the lp driver into esp-hal, once its complete in esp-lp-hal?

Yeah this looks kind of strange, I don't really understand why we're duplicating so much code. This probably needs some refactoring.

[JurajSadel]

Merging this so it won't start to rot until we are sure, how to proceed.

[JurajSadel]

Thanks for reviews!

[jessebraham]

Am I understanding this correctly that we are copy and pasting the lp driver into esp-hal, once its complete in esp-lp-hal?

Yeah this looks kind of strange, I don't really understand why we're duplicating so much code. This probably needs some refactoring.

Or not...