Skip to content

Support usb with cherryusb #1057

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
sakumisu opened this issue Oct 11, 2022 · 12 comments
Open

Support usb with cherryusb #1057

sakumisu opened this issue Oct 11, 2022 · 12 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@sakumisu
Copy link

Dear raspberrypi friends,

I have ported rp2040 usb ip with cherryusb, and provide all the device class demos, see rp2040 device demo, and host driver and demo is on going.

But i do not know if i have the chance to pr cherryusb lib in pico-sdk, can someone review and tell me? If no , i will add the custom cmake script in my repo. Thank you!

@kilograham
Copy link
Contributor

nice. we might consider adding a wrapper library that uses CherryUSB if a CMake var is set (or via find_package) (if not as a submodule) ... it will be a while before we get a chance to look at it though.

@kilograham
Copy link
Contributor

@liamfraser fyi

@sakumisu
Copy link
Author

Thanks a lot.This can be a slow thing to do, just use your free time.

@kilograham
Copy link
Contributor

Hey @sakumisu ... we tend to have a lot going on, so sometimes it takes us a while to get back to things! We would like to add CherryUSB support to the Pico SDK in an upcoming release if you still have interest

@sakumisu
Copy link
Author

Hey @sakumisu ... we tend to have a lot going on, so sometimes it takes us a while to get back to things! We would like to add CherryUSB support to the Pico SDK in an upcoming release if you still have interest

Really? Yeah, i have interest. We will reschedule support for pico sdk, our pico sdk support is too old, and we will consider how to add cmake easily, now way is that we have the cherryusb.cmake and set some cmake variables to use.

@kilograham
Copy link
Contributor

Yeah, feel free to share a branch as you go on the CMake front - we had some similar fun with TinyUSB, because both.

  • TinyUSB wants to build standalone with SDK, and run its examples
  • SDK wants to include TinyUSB, and pico-examples wants to pull in the tests

@sakumisu
Copy link
Author

sakumisu commented Jan 19, 2025

I build a simple project, could you have a time to review? About demo position and library position? If you think LGTM, i will pr for them, thank you!

https://github.com/CherryUSB/pico-examples
https://github.com/CherryUSB/pico-sdk

@kilograham kilograham added this to the 2.2.0 milestone Jan 19, 2025
@kilograham
Copy link
Contributor

Excellent; thank you - i will have a little play offline to get the feel of things

@matsobdev
Copy link

matsobdev commented Jan 19, 2025

So now I know, it is 0 rather that USBCTRL_REGS_BASE as a reg_base like here: https://github.com/CherryUSB/pico-examples/blob/master/usb/cherryusb/device/cdc_msc/main.c#L15. Didn't try, but maybe this was holding RISC-V back.

PS. Nope :D Time tu use "debugprobe" finally for the first time :D

@sakumisu
Copy link
Author

So now I know, it is 0 rather that USBCTRL_REGS_BASE as a reg_base like here: https://github.com/CherryUSB/pico-examples/blob/master/usb/cherryusb/device/cdc_msc/main.c#L15. Didn't try, but maybe this was holding RISC-V back.

PS. Nope :D Time tu use "debugprobe" finally for the first time :D

Because regbase is fixed in port, so init with rebase is useless.

@sakumisu
Copy link
Author

#2212

@sakumisu
Copy link
Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

5 participants