Thank you!

[RobMeades]

Not an issue at all, I just wanted to say that I've spent the last week adding SPI support to all of the platforms we use (Zephyr, NRF5SDK, STM32F4 and ESP-IDF) and, leaving ESP-IDF to last, it was a JOY to get here. Your APIs and your documentation are Just Right: I often tell people "boring" [a compliment in this context, not an insult]: this is the way I want to receive embedded code, I want it to be boring, just do what it is meant to do and no more, without any excitement, minimal "cleverness", getting the job done well.

This is what you, ESP-IDF, does.

Thank you.

[bidrohini]

Thanks for your review.

[ESP_Sprite]

Tsk, a fair few of my colleagues must have seen this post (it got linked in our internal chat) but no responded yet. So I'll do it: Thank you. As developers, you tend to only see the negative user experiences directly (bugs, documentation flaws, hardware issues, complaining users) and it's very nice to see something positive once every now and then. On the 'boring' part: I absolutely understand what you're saying; if you ever seek different way of saying it, I'd think 'predictable' would be it: APIs in places and formats that people who know other bits of ESP-IDF would expect it.

[RobMeades]

Glad to give you something to smile about. The contrast was particularly noticeable in this case: I had not looked at your SPI interface before I wrote my own abstraction to add SPI for our supported MCUs and your SPI interface was [IMHO] just the right way to do it, matched mine pretty much exactly.

And it worked as predicted .