search

espressif / idf-extra-components

Additional components for ESP-IDF, maintained by Espressif
149 stars 89 forks source link

fix(ccomp_timer): require perfmon only on xtensa #318

Closed fhrbata closed 4 months ago

fhrbata commented 5 months ago

The perfmon component is supported[1] only on xtensa arch, so it should not be in requires for other archs.

[1] https://github.com/espressif/esp-idf/blob/master/components/perfmon/CMakeLists.txt#L9-L11

Fixes: 5d61b5047f37 ("test_utils: implement performance timer")

igrr commented 5 months ago

@fhrbata LGTM! Could you please also bump the version in ccomp_timer/idf_component.yml so that this fix gets released?

fhrbata commented 4 months ago

@fhrbata LGTM! Could you please also bump the version in ccomp_timer/idf_component.yml so that this fix gets released?

@igrr I'm sorry, but I somehow missed your comment. It should be done. Thank you