ChibiOS/RT free embedded RTOS / Bugs / #377 Open 407I-C Board and USB HS ULPI Support

Dave Camarillo - 2013-02-05

board files

waveshare_open_407I-C_board_files.patch

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Camarillo - 2013-02-05

Also, let me know if you need me to change anything in the patches. I couldn't find a reference document relating to how to submit patches, formatting or guidelines (other then the general C style guide).

Thanks,
-Dave

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Camarillo - 2013-02-05

Hello again, two more things...

1) I have a few more features incorporated into my branch and I will resubmit patch files.
2) If it is helpful, I would be happy to donate one of the 407I-C dev kits. Let me know if this would be helpful and we can discuss offline.

Thanks,
-Dave

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Camarillo - 2013-02-05

USB driver changes for ULPI

stm32f4_usb_high_speed_ulpi_support.patch

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Camarillo - 2013-02-05

Demo code using the full speed USB interface

waveshare_open_407I-C_usb_full_speed_demo.patch

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Camarillo - 2013-02-05

Demo code using the USB high speed ULPI USB3300 Phy

waveshare_open_407I-C_usb_high_speed_ulpi_demo.patch

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Camarillo - 2013-02-05

new patch files are attached to this ticket

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Giovanni Di Sirio - 2013-02-17

Hi,

Thanks for all the work on the driver, patches merged. I just made some style related changes and added the new options to usb_lld.h too.
Could you please verify everything is OK? I can't test it.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Giovanni Di Sirio - 2013-02-17

assigned_to: nobody --> gdisirio

priority: 5 --> 2

milestone: --> 2.5.1unstable

status: open --> open-fixed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Camarillo - 2013-02-18

Hi, it's looking pretty good, but a couple minor merge issues. I think you may have grabbed the first set of patches rather then the second.

I've merged on my end and re-generated the relevant patches and attached them.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Camarillo - 2013-02-18

update regarding serial 2 pin conflicts with ULPI pins

waveshare_open_407I-C_usb_high_speed_ulpi_demo_20130218.patch

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Camarillo - 2013-02-18

updates to fix compileation error on USBD2 symbol not found if OTG2 is not enabled

stm32f4_usb_high_speed_ulpi_support_20130218.patch

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Giovanni Di Sirio - 2013-02-19

status: open-fixed --> open-postponed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Giovanni Di Sirio - 2013-02-19

Hi,

The latest patches do not apply on trunk and I am having reports of
problems in the forum, reverting all changes until it is cleared. Right now I don't understand on which revision I have to apply each patch.

Giovanni

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Camarillo - 2013-02-19

I'd expect a couple minor compilation errors if OTG2 had been disabled given that the first set of patches had been applied. Which forum posts are you referring to?

I've been alternating back and forth between using the full-speed on board phy and high speed off board ULPI phy and things are working correctly on my end. Perhaps there are additional differences between the code bases or something went awry in the merge?

I'm generating patches from git, which appears to be updated from an SVN repo.

I plan on doing continuing development with this board, creating more USB related demos, and doing additional work on Badgers mass storage driver. Perhaps I should be doing this in a branch on github? and you could merge in from my branch?

Thoughts?

Thanks,
-Dave

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Camarillo - 2013-02-20

Hi Giovanni, I created a fork in github, https://github.com/dsigma/ChibiOS and committed and pushed the various USB changes. I'll make and push changes to this repo going forward.

Regarding the problems, if they were specific to USB, there were a number of changes to usb_lld.c that may be suspect. If the forum posts? were sufficiently detailed perhaps I could try to reproduce them?

Thanks,
-Dave

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Giovanni Di Sirio - 2013-02-20

Hi Dave,

The forum post was not much detailed, not enough to understand where the real problem was, I decided to revert because a lot of people rely on the repository code and I don't want to keep there something with an unknown problem somewhere.

About patches, when you are confident that the driver is ready then submit patches for the subversion trunk, if it does not apply there then I am unable to merge.

Giovanni

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Camarillo - 2013-02-20

Hi Giovanni, I believe this was the post you were referring to: http://forum.chibios.org/phpbb/viewtopic.php?f=3&t=971

This error is that the 2nd set of patches addressed. It was a missing symbol due STM32_USB_USE_OTG2 being set to FALSE and how the #if's in usb_lld.c were written.

Both the full speed and high speed demos are working my repo, https://github.com/dsigma/ChibiOS.git

I have a number of things I want to add for demos and USB related features. Once I have all my changes in place I'll make a pull-request, in the mean time, if others want to play with the code it's available...

Best Regards,
-Dave

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Giovanni Di Sirio - 2013-05-04

Will be re-evaluated post 2.6.0. Plese submit a single patch on the current code if possible.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Camarillo - 2013-07-25

Hi Giovanni, I've submitted a github pull request with my various changes incorporated into a single branch. I've had a number of people submit inquires to me regarding HS USB and MSD/Composite functionality and I'd like to get my changes merged upstream. Can you please take a look and let me know if you'd like me to change anything in the pull request?

Best Regards,
-Dave

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Giovanni Di Sirio - 2013-07-26

Hi Dave,

I don't control the git repository so I don't have access to the pull requests.

The repository on github is a community-handled mirror of the official subversion repository. Could you please submit a patch applicable on the current trunk code?

Giovanni

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Giovanni Di Sirio - 2013-07-26

Group: 2.5.1unstable --> 2.6.1
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Camarillo - 2013-07-27

Hi Giovanni, attached is a patch which contains changes to the OS and the demos to allow for high speed USB, USB Mass storage support and demos of composite USB devices. Please let me know if you have any questions or would like any changes made...

Thanks,
-Dave

usb_ulpi.patch

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Giovanni Di Sirio - 2013-08-01

Hi Dave,

My, probably partial, list of wishes after the inspection:

usb_lld.h does not define nor check all the options you added to mcuconf.h. Are all those new options really necessary? please consider limiting their number.

usb_lld.c enables the GPIO clocks internally, this is already done in the PAL driver and violates the rule that device drivers only manage their own HW.

4 demos are too many.

I don't want to add anymore code to the HAL because maintenance costs, put the mass storage code inside demos even if it makes sense to be in the HAL.

Please use templates for new files.

Check the code against the style guide: http://www.chibios.org/dokuwiki/doku.php?id=chibios:guides:style_guide

Last edit: Giovanni Di Sirio 2013-08-01
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Giovanni Di Sirio - 2013-08-18

Group: 2.6.1 --> 2.6.2
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Open 407I-C Board and USB HS ULPI Support

Affected Version

Milestone

Searches

Help

#377 Open 407I-C Board and USB HS ULPI Support

Discussion