-
Notifications
You must be signed in to change notification settings - Fork 5.3k
[doxygen][lwp] add doxygen comment for lwp_dbg.h #9924
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
Closed
Closed
Changes from all commits
Commits
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,11 +1,12 @@ | ||
| /* | ||
| * Copyright (c) 2006-2023, RT-Thread Development Team | ||
| * Copyright (c) 2006-2025 RT-Thread Development Team | ||
| * | ||
| * SPDX-License-Identifier: Apache-2.0 | ||
| * | ||
| * Change Logs: | ||
| * Date Author Notes | ||
| * 2023-07-11 RT-Thread first version | ||
| * 2025-01-16 wumingzi add doxyzen comment | ||
| */ | ||
|
|
||
| #ifndef __LWP_DBG_H__ | ||
|
|
@@ -15,15 +16,75 @@ | |
| #include <rthw.h> | ||
| #include <lwp.h> | ||
|
|
||
| /** | ||
| * addtogroup KernelService | ||
| * @defgroup lwp lwp | ||
| * @brief lwp api | ||
| * @ingroup KernelService | ||
| * @addtogroup lwp | ||
| *@{ | ||
| */ | ||
|
|
||
| /** | ||
| * @brief Get lwp debug state | ||
| * @return int lwp debug state code | ||
| */ | ||
| int dbg_thread_in_debug(void); | ||
|
|
||
| /** | ||
| * @brief Register dbg_ops to struct dbg_ops_t | ||
| * @param dbg_ops Pointer to dbg_ops_t that contains function pointers for various debugging operations. | ||
| */ | ||
| void dbg_register(struct dbg_ops_t *dbg_ops); | ||
|
|
||
| /** | ||
| * @brief Get lwp instructure state | ||
| * @return Return the architecture number | ||
| */ | ||
| uint32_t dbg_get_ins(void); | ||
|
|
||
| /** | ||
| * @brief Enable step debug feature | ||
| */ | ||
| void dbg_activate_step(void); | ||
|
|
||
| /** | ||
| * @brief Disable step debug feature | ||
| */ | ||
| void dbg_deactivate_step(void); | ||
|
|
||
| /** | ||
| * @brief Check event for related instruction by exception state register | ||
| * @param regs Pointer to exception stack with type struct rt_hw_exp_stack * | ||
| * @param esr Address of exception state register | ||
| * @return Return the debug event number | ||
| */ | ||
| int dbg_check_event(struct rt_hw_exp_stack *regs, unsigned long esr); | ||
|
|
||
| /** | ||
| * @brief Get gdb server channel and prepared by rt_channel_t | ||
| * @return Return a Pointer to rt_channel_t | ||
| */ | ||
| rt_channel_t gdb_server_channel(void); | ||
|
|
||
| /** | ||
| * @brief Get debug step type | ||
| * @return Return the debug step type | ||
| */ | ||
| int dbg_step_type(void); | ||
|
|
||
| /** | ||
| * @brief Start debug attach request | ||
| * @param pc Pointer to programe counter | ||
| */ | ||
| void dbg_attach_req(void *pc); | ||
|
|
||
| /** | ||
| * @brief Suspend debug attach request | ||
| * @return Return result of suspend operation and 0 is failed. | ||
| */ | ||
| int dbg_check_suspend(void); | ||
|
|
||
| #endif /* __LWP_DBG_H__ */ | ||
| /*! @}*/ | ||
|
|
||
| #endif /* __LWP_DBG_H__ */ | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 这是什么改动?你是不是多删除了行尾的空行?
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 是的,我使用formatting工具格式化出来的就是这样。之前我给写驱动的时候尝试添加空行,可是ci过不去。虽然在某些ide上可能会有warning,但我没发现社区有反馈过这个问题的,所以先确保合入主线吧 |
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
不需要,采用 git 后,写好你的 commit message,加上你的 signed-off 即可。
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.
commit message我改一下,但是change log也应该保留?有的人开发可能不使用git,况且为了代码风格一致也应该保留吧
Uh oh!
There was an error while loading. Please reload this page.
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.
change log 随便你吧。
??? 这样的人如果也能给 RTT 提 PR,那么我服了