rt-thread icon indicating copy to clipboard operation
rt-thread copied to clipboard

bsp: enable KERNEL_REMAP for cvitek platform

Open polarvid opened this issue 1 year ago • 1 comments

拉取/合并请求描述:(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

polarvid avatar Jun 30 '24 08:06 polarvid

从 PR 的title来看,修改应该尽量局限在 bsp/cvitek 内部,但是我发现这个 pr 中修改了很多 common 的逻辑:

  • components
  • libcpu

这些是对原来 KERNEK_REMAP 修改 #9067 的补充吗?是不是应该以一个单独的 PR 的形式先行提交,然后再提交 bsp/cvitek 的部分?

unicornx avatar Jul 03 '24 02:07 unicornx

是不是应该以一个单独的 PR 的形式先行提交,然后再提交 bsp/cvitek 的部分?

commit 可以拆成多个。但是部分 commits 是需要联合 PR 的。

因为 libcpu 的修改和 bsp 有些东西是完全关联的。而不是解耦的。否则会导致使用异常。

polarvid avatar Jul 03 '24 02:07 polarvid

@polarvid @BernardXiong :

btw, 目前针对 rtt 的 riscv 的修改是否有工作组在推进?我刚刚参与 RTT 开发,对 rtt 的 riscv 方面的整体状态并不是很清楚,是否有渠道,SIG 等类似的组织在讨论这方面的工作,很想参与一下。这样对我理解 RTT 以及 review 这些 pr 或许会有帮助.

Thanks

unicornx avatar Jul 03 '24 02:07 unicornx

是不是应该以一个单独的 PR 的形式先行提交,然后再提交 bsp/cvitek 的部分?

commit 可以拆成多个。但是部分 commits 是需要联合 PR 的。

因为 libcpu 的修改和 bsp 有些东西是完全关联的。而不是解耦的。否则会导致使用异常。

同意,必要的话,在 一个 pr 中 拆分多个 commit 会更好。

另外建议 commit message 中多写点,帮助大家理解,谢谢。

unicornx avatar Jul 03 '24 02:07 unicornx

目前针对 rtt 的 riscv 的修改是否有工作组在推进?

官方渠道的报名方式在这个文章末尾有 https://mp.weixin.qq.com/s/CVbEQ4pHKVN2wVa_Uuk-bQ 😉

1719974054672

polarvid avatar Jul 03 '24 02:07 polarvid

@polarvid @BernardXiong :

btw, 目前针对 rtt 的 riscv 的修改是否有工作组在推进?我刚刚参与 RTT 开发,对 rtt 的 riscv 方面的整体状态并不是很清楚,是否有渠道,SIG 等类似的组织在讨论这方面的工作,很想参与一下。这样对我理解 RTT 以及 review 这些 pr 或许会有帮助.

Thanks

唔,看来shell是提供了二维码,我刚看到,还想说,是否拉个群呢~~

BernardXiong avatar Jul 04 '24 07:07 BernardXiong

works with persim M7 on commit https://github.com/RT-Thread/rt-thread/commit/ec2f44ea6ce84f5e466c793d58052170bccc9484

polarvid avatar Jul 08 '24 05:07 polarvid

works with persim M7 on commit https://github.com/RT-Thread/rt-thread/commit/ec2f44ea6ce84f5e466c793d58052170bccc9484

polarvid avatar Jul 08 '24 05:07 polarvid

Rebased to upstream

polarvid avatar Jul 08 '24 09:07 polarvid

Rebased to upstream

polarvid avatar Jul 08 '24 09:07 polarvid