Skip to content

Add HC32F4A2/HC32F467 BSP support and update configurations#11589

Closed
Leeto1970 wants to merge 4 commits into
RT-Thread:masterfrom
levizh:hc32_pr_20260706
Closed

Add HC32F4A2/HC32F467 BSP support and update configurations#11589
Leeto1970 wants to merge 4 commits into
RT-Thread:masterfrom
levizh:hc32_pr_20260706

Conversation

@Leeto1970

Copy link
Copy Markdown

拉取/合并请求描述:(PR description)

[
Add HC32F4A2/HC32F467 BSP support and update configurations
]

当前拉取/合并请求的状态 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
  • 如果是新增bsp, 已经添加ci检查到.github/ALL_BSP_COMPILE.json 详细请参考链接BSP自查

…IZE' to 512, modify 'HWTIMER' to 'CLOCK_TIMER' in ci and readme.
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

👋 感谢您对 RT-Thread 的贡献!Thank you for your contribution to RT-Thread!

为确保代码符合 RT-Thread 的编码规范,请在你的仓库中执行以下步骤运行代码格式化工作流(如果格式化CI运行失败)。
To ensure your code complies with RT-Thread's coding style, please run the code formatting workflow by following the steps below (If the formatting of CI fails to run).


🛠 操作步骤 | Steps

  1. 前往 Actions 页面 | Go to the Actions page
    点击进入工作流 → | Click to open workflow →

  2. 点击 Run workflow | Click Run workflow

  • 设置需排除的文件/目录(目录请以"/"结尾)
    Set files/directories to exclude (directories should end with "/")
  • 将目标分支设置为 \ Set the target branch to:hc32_pr_20260706
  • 设置PR number为 \ Set the PR number to:11589
  1. 等待工作流完成 | Wait for the workflow to complete
    格式化后的代码将自动推送至你的分支。
    The formatted code will be automatically pushed to your branch.

完成后,提交将自动更新至 hc32_pr_20260706 分支,关联的 Pull Request 也会同步更新。
Once completed, commits will be pushed to the hc32_pr_20260706 branch automatically, and the related Pull Request will be updated.

如有问题欢迎联系我们,再次感谢您的贡献!💐
If you have any questions, feel free to reach out. Thanks again for your contribution!

@github-actions github-actions Bot added BSP BSP: HC32 BSP related with HC32 action github action yml imporve labels Jul 8, 2026
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

📌 Code Review Assignment

🏷️ Tag: bsp_hc32

Reviewers: @levizh

Changed Files (Click to expand)
  • bsp/hc32/docs/HC32系列BSP制作教程.md
  • bsp/hc32/docs/HC32系列驱动介绍.md
  • bsp/hc32/ev_hc32f334_lqfp64/.ci/attachconfig/ci.attachconfig.yml
  • bsp/hc32/ev_hc32f334_lqfp64/README.md
  • bsp/hc32/ev_hc32f334_lqfp64/rtconfig.h
  • bsp/hc32/ev_hc32f448_lqfp80/.ci/attachconfig/ci.attachconfig.yml
  • bsp/hc32/ev_hc32f448_lqfp80/README.md
  • bsp/hc32/ev_hc32f448_lqfp80/rtconfig.h
  • bsp/hc32/ev_hc32f460_lqfp100_v2/.ci/attachconfig/ci.attachconfig.yml
  • bsp/hc32/ev_hc32f460_lqfp100_v2/README.md
  • bsp/hc32/ev_hc32f460_lqfp100_v2/rtconfig.h
  • bsp/hc32/ev_hc32f467_lqfp144/.ci/attachconfig/ci.attachconfig.yml
  • bsp/hc32/ev_hc32f467_lqfp144/.config
  • bsp/hc32/ev_hc32f467_lqfp144/.cproject
  • bsp/hc32/ev_hc32f467_lqfp144/.gitignore
  • bsp/hc32/ev_hc32f467_lqfp144/.project
  • bsp/hc32/ev_hc32f467_lqfp144/EventRecorderStub.scvd
  • bsp/hc32/ev_hc32f467_lqfp144/Kconfig
  • bsp/hc32/ev_hc32f467_lqfp144/README.md
  • bsp/hc32/ev_hc32f467_lqfp144/SConscript
  • bsp/hc32/ev_hc32f467_lqfp144/SConstruct
  • bsp/hc32/ev_hc32f467_lqfp144/applications/SConscript
  • bsp/hc32/ev_hc32f467_lqfp144/applications/main.c
  • bsp/hc32/ev_hc32f467_lqfp144/applications/xtal32_fcm.c
  • bsp/hc32/ev_hc32f467_lqfp144/board/Kconfig
  • bsp/hc32/ev_hc32f467_lqfp144/board/SConscript
  • bsp/hc32/ev_hc32f467_lqfp144/board/board.c
  • bsp/hc32/ev_hc32f467_lqfp144/board/board.h
  • bsp/hc32/ev_hc32f467_lqfp144/board/board_config.c

🏷️ Tag: workflow

Reviewers: @Rbb666 @kurisaW @supperthomas

Changed Files (Click to expand)
  • .github/ALL_BSP_COMPILE.json

📊 Current Review Status (Last Updated: 2026-07-13 11:16 CST)


📝 Review Instructions

  1. 维护者可以通过单击此处来刷新审查状态: 🔄 刷新状态
    Maintainers can refresh the review status by clicking here: 🔄 Refresh Status

  2. 确认审核通过后评论 LGTM/lgtm
    Comment LGTM/lgtm after confirming approval

  3. PR合并前需至少一位维护者确认
    PR must be confirmed by at least one maintainer before merging

ℹ️ 刷新CI状态操作需要具备仓库写入权限。
ℹ️ Refresh CI status operation requires repository Write permission.

@CLAassistant

CLAassistant commented Jul 8, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@Leeto1970

Copy link
Copy Markdown
Author

Hi @Rbb666, Could you please help review and merge these PR when convenient?
HC32F4A2 CMSIS
HC32F467 CMSIS
HC32 BSP need these resource files.

Thanks!

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个脚本貌似是测试脚本?无关的话可以删除

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

您好,这个脚本是我们用于检查BSP编译错误,希望保留

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个是studio生成的中间文件,可以删除

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

好的,谢谢提醒

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

MemBrowse Memory Report

No memory changes detected for:

@CYFS3

CYFS3 commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

@Leeto1970 CI有报错。麻烦解决一下

@Leeto1970 Leeto1970 requested a review from Rbb666 July 9, 2026 07:22
@Leeto1970

Copy link
Copy Markdown
Author

@Leeto1970 CI有报错。麻烦解决一下

@CYFS3 @Rbb666 你们好,我们已重新解决了静态检查错误,请同意重新执行CI

@Rbb666 Rbb666 closed this Jul 10, 2026
@Rbb666 Rbb666 reopened this Jul 10, 2026
@Leeto1970

Copy link
Copy Markdown
Author

@Rbb666 要重新跑CI是否要关闭PR再重开,更新了packages,BSP构建应该不会报错了。
按照RTT官方文档的参数对提交的代码都做了格式化,代码静态分析也按照之前的提示做了,本次提交后还是报有格式化和代码静态分析问题

@CYFS3

CYFS3 commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

重新跑CI是否要关闭PR再重开,更新了packages,BSP构建应该不会报

推送代码会重新跑ci的,格式化是否使用的是clang-format.

@Leeto1970

Copy link
Copy Markdown
Author

重新跑CI是否要关闭PR再重开,更新了packages,BSP构建应该不会报

推送代码会重新跑ci的,格式化是否使用的是clang-format.
@CYFS3 我提了packages的PR,已合并了,这边没有更改的。格式化我是用的Astyle,我试试clang-format重新格式化代码。

@CYFS3

CYFS3 commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

重新跑CI是否要关闭PR再重开,更新了packages,BSP构建应该不会报

推送代码会重新跑ci的,格式化是否使用的是clang-format.
@CYFS3 我提了packages的PR,已合并了,这边没有更改的。格式化我是用的Astyle,我试试clang-format重新格式化代码。

嗯嗯
image
看看这个报错是不是软件包导致的,如果是可以尝试关闭然后重新打开pr跑一下

@Rbb666 Rbb666 closed this Jul 10, 2026
@Rbb666 Rbb666 reopened this Jul 10, 2026
@Leeto1970 Leeto1970 closed this Jul 13, 2026
@Leeto1970 Leeto1970 reopened this Jul 13, 2026
@Leeto1970

Copy link
Copy Markdown
Author

@CYFS3 请问RTT使用clang-format格式化代码的命令是什么样的,我这边使用clang-format -i bsp\hc32\tests\test_usbd.c命令,没有对文件做格式化操作。另外CI报这种格式化错误确实是需要修改的吗
Image

@CYFS3

CYFS3 commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

@CYFS3 请问RTT使用clang-format格式化代码的命令是什么样的,我这边使用clang-format -i bsp\hc32\tests\test_usbd.c命令,没有对文件做格式化操作。另外CI报这种格式化错误确实是需要修改的吗 Image

image

我这边本地操作是没有问题的,或者你可以运行action的格式化,如下图
image

另外rtconfig.h的检查确实该去掉

@Leeto1970

Copy link
Copy Markdown
Author

@CYFS3 请问RTT使用clang-format格式化代码的命令是什么样的,我这边使用clang-format -i bsp\hc32\tests\test_usbd.c命令,没有对文件做格式化操作。另外CI报这种格式化错误确实是需要修改的吗 Image

image 我这边本地操作是没有问题的,或者你可以运行action的格式化,如下图 image

另外rtconfig.h的检查确实该去掉

@CYFS3 你好,我这边也试过通过workflow来格式化代码,任务结束了,但是代码没有变更
image
image
是否是因为这个文件屏蔽了格式化
image

@CYFS3

CYFS3 commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

@CYFS3 请问RTT使用clang-format格式化代码的命令是什么样的,我这边使用clang-format -i bsp\hc32\tests\test_usbd.c命令,没有对文件做格式化操作。另外CI报这种格式化错误确实是需要修改的吗 Image

image 我这边本地操作是没有问题的,或者你可以运行action的格式化,如下图 image 另外rtconfig.h的检查确实该去掉

@CYFS3 你好,我这边也试过通过workflow来格式化代码,任务结束了,但是代码没有变更 image image 是否是因为这个文件屏蔽了格式化 image

是的,需要您变基到最新主线

@Leeto1970

Copy link
Copy Markdown
Author

@CYFS3 请问RTT使用clang-format格式化代码的命令是什么样的,我这边使用clang-format -i bsp\hc32\tests\test_usbd.c命令,没有对文件做格式化操作。另外CI报这种格式化错误确实是需要修改的吗 Image

image 我这边本地操作是没有问题的,或者你可以运行action的格式化,如下图 image 另外rtconfig.h的检查确实该去掉

@CYFS3 你好,我这边也试过通过workflow来格式化代码,任务结束了,但是代码没有变更 image image 是否是因为这个文件屏蔽了格式化 image

是的,需要您变基到最新主线

好的,了解了,我将变基合并commit后重新提交PR。请问关于静态检查在本地可以通过什么方式执行?本次PR的几个commit都被静态检查卡住

@CYFS3

CYFS3 commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

本次PR的几个commit都被静态检查卡住

参考ci相关命令
image

@Leeto1970

Copy link
Copy Markdown
Author

本次PR的几个commit都被静态检查卡住

参考ci相关命令 image

好的,谢谢,这个PR我先关闭了

@Leeto1970 Leeto1970 closed this Jul 13, 2026
@CYFS3

CYFS3 commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

本次PR的几个commit都被静态检查卡住

参考ci相关命令 image

好的,谢谢,这个PR我先关闭了

不需要关闭的,本地rebase一下推送上来即可,不需要新建分支

@Leeto1970

Copy link
Copy Markdown
Author

本次PR的几个commit都被静态检查卡住

参考ci相关命令 image

好的,谢谢,这个PR我先关闭了

不需要关闭的,本地rebase一下推送上来即可,不需要新建分支

好的,那我本地rebase后重新提交吧,谢谢提醒

@Leeto1970

Copy link
Copy Markdown
Author

重新提交了

@CYFS3

CYFS3 commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

重新提交了

重新打开此pr即可

@Leeto1970

Copy link
Copy Markdown
Author

重新打开此pr即可

image 我对原分支做了rebase,将之前修改过程的4条commit合并成1条了(官方建议PR不超过5条commit),PR提示分支被强制推送或重建了,好像不能重开PR了,如果我恢复4条commit是不是还能重新PR

@CYFS3

CYFS3 commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

重新打开此pr即可

image 我对原分支做了rebase,将之前修改过程的4条commit合并成1条了(官方建议PR不超过5条commit),PR提示分支被强制推送或重建了,好像不能重开PR了,如果我恢复4条commit是不是还能重新PR

那你新建pr,你刚才不关闭pr,直接压缩强推应该是没有问题的

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action github action yml imporve BSP: HC32 BSP related with HC32 BSP

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants