-
Notifications
You must be signed in to change notification settings - Fork 5k
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
[action] add nrf5340 ci config #9570
Conversation
640affd
to
6db9a4a
Compare
CONFIG_BSP_USING_I2C0=n | ||
CONFIG_BSP_USING_I2C1=n |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
=n不用写呀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
发现i2c开了之后默认就会打开BSP_USING_I2C0然后跟串口的SERIAL0_IRQHandler冲突,导致编译报错,这里设置=n就可以关掉了
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
可以的,这里有坑,非常好。感谢反馈
请帮忙添加其他的attach,覆盖driver下面的所有文件。 |
已添加完所有attach,另外nrfx-latest软件包仓库那边也提交了一份修复兼容的pr,等待那边pr通过后,这边的ci才能过 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
非常好的pr
拉取/合并请求描述:(PR description)
[
为什么提交这份PR (why to submit this PR)
分别添加nrf5340中driver下面的所有文件的attach
你的解决方案是什么 (what is your solution)
请提供验证的bsp和config (provide the config and bsp)
]
当前拉取/合并请求的状态 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