-
Notifications
You must be signed in to change notification settings - Fork 5.3k
[bsp/phytium]修改config适配方式 #10188
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/phytium]修改config适配方式 #10188
Conversation
|
编译链不可以根据rt-smart配置切换吗,似乎rtsmart配置编译不过去?@supperthomas |
rtsmart |
|
smart部分后面再想办法吧。目前只支持rt-thread |
| CONFIG_RT_USING_RTC=y | ||
| # CONFIG_RT_USING_PIN is not set | ||
| CONFIG_RT_USING_POSIX_FS=y | ||
| CONFIG_RT_USING_POSIX_TERMIOS=y |
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.
这个文件不用加,
一些用户可以自己生成的文件,比如.config, rtconfig.h不用每次都加。不需要,可以删除,保留最基本的常用配置即可。
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先放一下,后面再解决
|
两个环境会相互干扰吗?不干扰,可以先弄好。 rt-thread/.github/workflows/bsp_buildings.yml Line 164 in 3ebad06
试试在这里先把环境弄好, |
主要是 ALL_BSP_COMPILE.json里面一个 “RTT_TOOL_CHAIN”下面对应的是一个路径。32位和64位,加上thread和smart需要4个不同的编译链,最好ci能根据宏定义切一下编译链。不然全分开也是很冗余。 |
拉取/合并请求描述:(PR description)
[
为什么提交这份PR (why to submit this PR)
1.修改config适配方式,符合最新的ci框架
你的解决方案是什么 (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