MabezDev / idf2svd

Apache License 2.0
14 stars 7 forks source link

Indexed peripherals #2

Closed MabezDev closed 2 years ago

MabezDev commented 4 years ago

Some peripherals in the idf are indexed, i.e there are two identical I2C peripherals on the esp32, instead of creating register definitions for both, they are defined once with offset.

This generator doesn't properly handle this at the moment, and only creates one peripheral.

MabezDev commented 4 years ago

The current approach simply inserts a generic blank peripheral at the start, see https://github.com/MabezDev/idf2svd/blob/c433ffab105c607bd4f868765c9f2967eba09a0a/src/lib.rs#L161 . This will require some post processing of the svd to copy and give them the correct base address

sepotvin commented 2 years ago

Hi, The I2S peripheral is also indexed and should probably be added to the list. The latest esp32 PAC only shows I2S0 as I2S.

MabezDev commented 2 years ago

Hi @sepotvin, I forgot about this repo. There are now official SVD's in the https://github.com/espressif/svd repo. Please file the issues there.