platform-espressif32 icon indicating copy to clipboard operation
platform-espressif32 copied to clipboard

Feature request: LDF scan for a new entry "mcu"

Open Jason2866 opened this issue 5 years ago • 3 comments

  • [x] Feature Request. Start by telling us what problem you’re trying to solve. Often a solution already exists! Don’t send pull requests to implement new features without first getting our support. Sometimes we leave features out on purpose to keep the project small.

Configuration

Operating system: All

PlatformIO Version (platformio --version): latest development

Description of problem

Add a option to the LDF to scan for mcu in librarary.properties or in library.json mcu is already a entry in Arduino ESP32 boards.txt Esp32 has different mcu cores now (ESP32-C3 soon too) which needs different libraries. At the moment there is no possibility to differentiate this or did i missed something? Maybe as new command strict+ ?

Jason2866 avatar Feb 14 '21 17:02 Jason2866

It should be implemented in dev-platform. See https://github.com/espressif/arduino-esp32/blob/master/tools/platformio-build.py#L177

ivankravets avatar Feb 27 '21 15:02 ivankravets

HI @Jason2866 ! Could you please provide an example of any library that needs this feature?

valeros avatar Mar 01 '21 12:03 valeros

Hi @valeros The ESPP3232-C3 is available. All libs which are compatible to core esp32 are used for the C3 too, as there is no possibility to handle the mcu variant with LDF. We have some optimized libs which are mcu dependent ( xtensa vs. riscv) it would be nice not to litter the code with many #ifdefs to take care of this. More elegant would be to have lib variants for every architecture

Examples: https://github.com/arendst/Tasmota/tree/development/lib/default/Ext-printf https://github.com/arendst/Tasmota/tree/development/lib/libesp32/Berry https://github.com/arendst/Tasmota/tree/development/lib/lib_ssl/bearssl-esp8266

Jason2866 avatar Apr 20 '21 06:04 Jason2866