nuttx: esp32s3: fix compilation when enable mbedTLS in apps

[pkarashchenko]

No description provided.

[tmedicci]

Hi @pkarashchenko ! Thanks for submitting it. Could you please provide a background (about what is failing on NuttX) and how to reproduce the issue?

[pkarashchenko]

I'm trying to enable mbedTLS in NuttX app and build MQTT-C mbedTLS example.

[tmedicci]

I'm trying to enable mbedTLS in NuttX app and build MQTT-C mbedTLS example.

Thanks. I'll check...

Note that, for now, you can override the HAL with your own without changing any source code on NuttX. Just set env ESP_HAL_3RDPARTY_URL and ESP_HAL_3RDPARTY_VERSION before building. For instance:

ESP_HAL_3RDPARTY_URL="git@github.com:tmedicci/esp-hal-3rdparty.git" ESP_HAL_3RDPARTY_VERSION="improvement/rmt_driver" make

[pkarashchenko]

Yes. I know. Initially I passed the build with only patching a header file, but later naming conflict happened when I enabled mbedTLS for MQTT

[pkarashchenko]

Do you have results from internal CI?

[tmedicci]

Do you have results from internal CI?

I'm testing it right now. Yesterday afternoon I had another professional commitment.

We have an almost automated way of setting the prefixes (I'll make it public). There are other cases for inline functions that need to be set too. I'll let you know asap...

[tmedicci]

@pkarashchenko , can you please double-check https://github.com/tmedicci/esp-hal-3rdparty/tree/bugfix/nuttx_mbedtls?

[tmedicci]

@pkarashchenko , can you please double-check https://github.com/tmedicci/esp-hal-3rdparty/tree/bugfix/nuttx_mbedtls?

Any updates about it @pkarashchenko ?

[pkarashchenko]

Will try out till Friday. Sorry, I'm having a busy week

[pkarashchenko]

@tmedicci hi. Change works for me, but I posted one comment tmedicci@6fc8272#r139129194

[tmedicci]

@tmedicci hi. Change works for me, but I posted one comment tmedicci@6fc8272#r139129194

Could you please double-check same branch?

[pkarashchenko]

Works well for me!

[tmedicci]

Solved with 55b60d9. Thanks @pkarashchenko !