-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
[bsp/stm32] add usbcdc for openmv #5702
Conversation
34bdc3c
to
26b85d1
Compare
26b85d1
to
8b921b1
Compare
如果是正常的STM32开发板或者最小系统板子,是要求只默认开启GPIO和串口。 |
@supperthomas 涛哥 这位贡献者你认识吗 如果不认识可以考虑拉进你们的小组呀 |
@Miaowulue 非常感谢PR。建议下次一些可以不用提交的文件,下次可以不用提交。尽量减少git的大小。比如.config .mxproject 这些不影响后面开发者使用的文件以及不是特别需要修改的文件,可以选择不用提交,有需要大修改的时候,可以一次性提交。欢迎参加社区小组,一起来PR。 |
应该是群里的小雨吧。 |
@supperthomas .config .mxproject 这些文件我是建议要提交的 因为.config是保存了kconfig的一些设置,如果没有.config,就会导致kconfig设置都是默认设置,会出问题。 |
嗯,我的意思是不用每次提交。整理好了,一次性修改即可。 |
这个到没啥事,提交的时候不看就是了,但是该提交还是得提交的。否则比如,一些int类型的数据我有一些个性化的设置,这个是不能通过类似于select的方式预设的。如果漏提交.config 这些数据就会丢失掉恢复成默认值就很难受了 |
@Miaowulue 请发邮件ping我一下,jiantingman@foxmail.com 我会拉你进群 |
不是我,是我们团队的另一个人 |
拉取/合并请求描述:(PR description)
[
Add usbcdc for openmv.
Has tested on openmv-h7plus
完成了引脚、时钟配置和Kconfig修改。
]
以下的内容不应该在提交PR时的message修改,修改下述message,PR会被直接关闭。请在提交PR后,浏览器查看PR并对以下检查项逐项check,没问题后逐条在页面上打钩。
The following content must not be changed in the submitted PR message. Otherwise, the PR will be closed immediately. After submitted PR, please use a web browser to visit PR, and check items one by one, and ticked them if no problem.
当前拉取/合并请求的状态 Intent for your PR
必须选择一项 Choose one (Mandatory):
代码质量 Code Quality:
我在这个拉取/合并请求中已经考虑了 As part of this pull request, I've considered the following:
#if 0
代码,不包含已经被注释了的代码 All redundant code is removed and cleaned up