bsp: enable KERNEL_REMAP for cvitek platform
拉取/合并请求描述:(PR description)
[
为什么提交这份PR (why to submit this PR)
enable KERNEL_REMAP for cvitek platform
你的解决方案是什么 (what is your solution)
请提供验证的bsp和config (provide the config and bsp)
- BSP:
- .config:
- action:
]
当前拉取/合并请求的状态 Intent for your PR
必须选择一项 Choose one (Mandatory):
- [ ] 本拉取/合并请求是一个草稿版本 This PR is for a code-review and is intended to get feedback
- [ ] 本拉取/合并请求是一个成熟版本 This PR is mature, and ready to be integrated into the repo
代码质量 Code Quality:
我在这个拉取/合并请求中已经考虑了 As part of this pull request, I've considered the following:
- [ ] 已经仔细查看过代码改动的对比 Already check the difference between PR and old code
- [ ] 代码风格正确,包括缩进空格,命名及其他风格 Style guide is adhered to, including spacing, naming and other styles
- [ ] 没有垃圾代码,代码尽量精简,不包含
#if 0代码,不包含已经被注释了的代码 All redundant code is removed and cleaned up - [ ] 所有变更均有原因及合理的,并且不会影响到其他软件组件代码或BSP All modifications are justified and not affect other components or BSP
- [ ] 对难懂代码均提供对应的注释 I've commented appropriately where code is tricky
- [ ] 代码是高质量的 Code in this PR is of high quality
- [ ] 已经使用formatting 等源码格式化工具确保格式符合RT-Thread代码规范 This PR complies with RT-Thread code specification
从 PR 的title来看,修改应该尽量局限在 bsp/cvitek 内部,但是我发现这个 pr 中修改了很多 common 的逻辑:
- components
- libcpu
这些是对原来 KERNEK_REMAP 修改 #9067 的补充吗?是不是应该以一个单独的 PR 的形式先行提交,然后再提交 bsp/cvitek 的部分?
是不是应该以一个单独的 PR 的形式先行提交,然后再提交 bsp/cvitek 的部分?
commit 可以拆成多个。但是部分 commits 是需要联合 PR 的。
因为 libcpu 的修改和 bsp 有些东西是完全关联的。而不是解耦的。否则会导致使用异常。
@polarvid @BernardXiong :
btw, 目前针对 rtt 的 riscv 的修改是否有工作组在推进?我刚刚参与 RTT 开发,对 rtt 的 riscv 方面的整体状态并不是很清楚,是否有渠道,SIG 等类似的组织在讨论这方面的工作,很想参与一下。这样对我理解 RTT 以及 review 这些 pr 或许会有帮助.
Thanks
是不是应该以一个单独的 PR 的形式先行提交,然后再提交 bsp/cvitek 的部分?
commit 可以拆成多个。但是部分 commits 是需要联合 PR 的。
因为 libcpu 的修改和 bsp 有些东西是完全关联的。而不是解耦的。否则会导致使用异常。
同意,必要的话,在 一个 pr 中 拆分多个 commit 会更好。
另外建议 commit message 中多写点,帮助大家理解,谢谢。
目前针对 rtt 的 riscv 的修改是否有工作组在推进?
官方渠道的报名方式在这个文章末尾有 https://mp.weixin.qq.com/s/CVbEQ4pHKVN2wVa_Uuk-bQ 😉
@polarvid @BernardXiong :
btw, 目前针对 rtt 的 riscv 的修改是否有工作组在推进?我刚刚参与 RTT 开发,对 rtt 的 riscv 方面的整体状态并不是很清楚,是否有渠道,SIG 等类似的组织在讨论这方面的工作,很想参与一下。这样对我理解 RTT 以及 review 这些 pr 或许会有帮助.
Thanks
唔,看来shell是提供了二维码,我刚看到,还想说,是否拉个群呢~~
works with persim M7 on commit https://github.com/RT-Thread/rt-thread/commit/ec2f44ea6ce84f5e466c793d58052170bccc9484
works with persim M7 on commit https://github.com/RT-Thread/rt-thread/commit/ec2f44ea6ce84f5e466c793d58052170bccc9484
Rebased to upstream
Rebased to upstream