Skip to content

Conversation

@unicornx
Copy link
Contributor

clk.c 和 clk.h中重复定义 group_clk.

删除 clk.c 中的定义,只保留 clk.h 中的定义。
同时更改名字 group_clkgroup_driver_clock,符合其他 driver 的 group 的命名习惯。

Note:本 PR 只修改了注释文档,不涉及代码逻辑修改。

Problen:
Duplicated definition of group_clk in clk.c and clk.h.

Solution:
Delete the definition in clk.c and keep only the definition
in clk.h.

Also change the name of group_clk to group_driver_clock to
conform to the naming convention of other driver groups.

Signed-off-by: Chen Wang <unicorn_wang@outlook.com>
@github-actions
Copy link

📌 Code Review Assignment

🏷️ Tag: components

Reviewers: @Maihuanyi

Changed Files (Click to expand)
  • components/drivers/clk/clk.c
  • components/drivers/include/drivers/clk.h

📊 Current Review Status (Last Updated: 2025-07-22 15:51 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.

@Rbb666 Rbb666 merged commit 895b272 into RT-Thread:master Jul 22, 2025
61 checks passed
@unicornx unicornx deleted the dev-doxygen-clks branch July 22, 2025 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants