search

raspberrypi / pico-sdk

BSD 3-Clause "New" or "Revised" License
3.66k stars 911 forks source link

CFG_TUSB_OS as cmake parameter #1326

Open rapzak opened 1 year ago

rapzak commented 1 year ago

Hi,

Could you add a cmake parameter to set the CFG_TUSB_OS

This way we can use the SDK's tinyusb lib in freertos programs as well...

see: https://www.eevblog.com/forum/microcontrollers/rp2040-how-to-use-freertos-with-tinyusb/

peterharperuk commented 1 year ago

Doesn't this require a change to tinyusb and not pico-sdk?

rapzak commented 1 year ago

Doesn't this require a change to tinyusb and not pico-sdk?

I am not an expert, but i see it as a mix, because the tinyusb is both used as a part of SDK usage and final program....

I think it is a user config, the SDK set behind the user program....

Another way is to not use any Tinyusb from sdk, and link in your own version, but think it would be a pitty...

/Kasper

peterharperuk commented 1 year ago

What I mean is, you can already define CFG_TUSB_OS on the cmake command line. The issue is that the tinyusb cmake file won't use it. I guess we can raise a PR against tinyusb to fix this.

rapzak commented 1 year ago

https://github.com/hathach/tinyusb/discussions/1951

mablinov commented 1 year ago

Without intending to derail the topic - I'm a bit surprised that FreeRTOS isn't provided as part of the pico-sdk by default. It's a very common framework, and the rp2040 is particularly unique in that it provides possibility of SMP FreeRTOS. I think it would make sense to include it out of the box.

kilograham commented 2 months ago

@liamfraser just in case you are passing; though enabling it may not be much help until we go and check FreeRTOS support is integrated properly