Skip to content

shell: kernel_service: reorg the commands #148904

shell: kernel_service: reorg the commands

shell: kernel_service: reorg the commands #148904

Triggered via pull request September 12, 2024 07:54
Status Success
Total duration 3m 2s
Artifacts 1

compliance.yml

on: pull_request
Run compliance checks on patch series (PR)
2m 52s
Run compliance checks on patch series (PR)
Fit to window
Zoom out
Zoom in

Annotations

1 error, 1 warning, and 10 notices
Run compliance checks on patch series (PR)
Process completed with exit code 1.
Run compliance checks on patch series (PR): ClangFormat.txt#L1
See https://docs.zephyrproject.org/latest/contribute/guidelines.html#clang-format for more details. You may want to run clang-format on this change: -static int cmd_kernel_reboot_warm(const struct shell *sh, - size_t argc, char **argv) +static int cmd_kernel_reboot_warm(const struct shell *sh, size_t argc, char **argv) File:subsys/shell/modules/kernel_service/reboot.c Line:14 You may want to run clang-format on this change: -static int cmd_kernel_reboot_cold(const struct shell *sh, - size_t argc, char **argv) +static int cmd_kernel_reboot_cold(const struct shell *sh, size_t argc, char **argv) File:subsys/shell/modules/kernel_service/reboot.c Line:26 You may want to run clang-format on this change: - SHELL_CMD(cold, NULL, "Cold reboot.", cmd_kernel_reboot_cold), - SHELL_CMD(warm, NULL, "Warm reboot.", cmd_kernel_reboot_warm), - SHELL_SUBCMD_SET_END /* Array terminated. */ + SHELL_CMD(cold, NULL, "Cold reboot.", cmd_kernel_reboot_cold), + SHELL_CMD(warm, NULL, "Warm reboot.", cmd_kernel_reboot_warm), + SHELL_SUBCMD_SET_END /* Array terminated. */ File:subsys/shell/modules/kernel_service/reboot.c Line:40 You may want to run clang-format on this change: - pcnt = (rt_stats_thread.execution_cycles * 100U) / - rt_stats_all.execution_cycles; + pcnt = (rt_stats_thread.execution_cycles * 100U) / rt_stats_all.execution_cycles; File:subsys/shell/modules/kernel_service/thread/list.c Line:32 You may want to run clang-format on this change: - (uint32_t)rt_stats_thread.execution_cycles, - pcnt); + (uint32_t)rt_stats_thread.execution_cycles, pcnt); File:subsys/shell/modules/kernel_service/thread/list.c Line:43 You may want to run clang-format on this change: - shell_print(sh, "%s%p %-10s", - (thread == k_current_get()) ? "*" : " ", - thread, - tname ? tname : "NA"); + shell_print(sh, "%s%p %-10s", (thread == k_current_get()) ? "*" : " ", thread, + tname ? tname : "NA"); /* Cannot use lld as it's less portable. */ shell_print(sh, "\toptions: 0x%x, priority: %d timeout: %" PRId64, - thread->base.user_options, - thread->base.prio, - (int64_t)thread->base.timeout.dticks); + thread->base.user_options, thread->base.prio, + (int64_t)thread->base.timeout.dticks); shell_print(sh, "\tstate: %s, entry: %p", - k_thread_state_str(thread, state_str, sizeof(state_str)), - thread->entry.pEntry); + k_thread_state_str(thread, state_str, sizeof(state_str)), thread->entry.pEntry); File:subsys/shell/modules/kernel_service/thread/list.c Line:87 You may want to run clang-format on this change: - shell_print(sh, - "Unable to determine unused stack size (%d)\n", - ret); + shell_print(sh, "Unable to determine unused stack size (%d)\n", ret); } else { /* Calculate the real size reserved for the stack */ pcnt = ((size - unused) * 100U) / size; - shell_print(sh, - "\tstack size %zu, unused %zu, usage %zu / %zu (%u %%)\n", - size, unused, size - unused, size, pcnt); + shell_print(sh, "\tstack size %zu, unused %zu, usage %zu / %zu (%u %%)\n", size, + unused, size - unused, size, pcnt); } - File:subsys/shell/modules/kernel_service/thread/list.c Line:108 You may want to run clang-format on this change: -SHELL_STATIC_SUBCMD_SET_CREATE(sub_kernel_thread_mask, +SHELL_STATIC_SUBCMD_SET_CREATE( + sub_kernel_thread_mask, File:subsys/shell/modules/kernel_service/thread/mask.c Line:140 You may want to run clang-format on this change: -static int cmd_kernel_thread_pin(const struct shell *sh, - size_t argc, char **argv) +static int cmd_kernel_thread_pin(const struct shell *sh, size_t argc, char **argv) File:subsys/shell/modules/kernel_service/thread/pin.c Line:13 You may want to run clang-format on this change: - shell_print(sh, - "Unable to determine unused stack size (%d)\n", - ret); + shell_print(sh, "Unable to determine unused stack size (%d)\n", ret); File:subsys/shell/modules/kernel_service/thread/stacks.c Line:36 You may want to run clan
You may want to run clang-format on this change: subsys/shell/modules/kernel_service/reboot.c#L14
subsys/shell/modules/kernel_service/reboot.c:14 -static int cmd_kernel_reboot_warm(const struct shell *sh, - size_t argc, char **argv) +static int cmd_kernel_reboot_warm(const struct shell *sh, size_t argc, char **argv)
You may want to run clang-format on this change: subsys/shell/modules/kernel_service/reboot.c#L26
subsys/shell/modules/kernel_service/reboot.c:26 -static int cmd_kernel_reboot_cold(const struct shell *sh, - size_t argc, char **argv) +static int cmd_kernel_reboot_cold(const struct shell *sh, size_t argc, char **argv)
You may want to run clang-format on this change: subsys/shell/modules/kernel_service/reboot.c#L40
subsys/shell/modules/kernel_service/reboot.c:40 - SHELL_CMD(cold, NULL, "Cold reboot.", cmd_kernel_reboot_cold), - SHELL_CMD(warm, NULL, "Warm reboot.", cmd_kernel_reboot_warm), - SHELL_SUBCMD_SET_END /* Array terminated. */ + SHELL_CMD(cold, NULL, "Cold reboot.", cmd_kernel_reboot_cold), + SHELL_CMD(warm, NULL, "Warm reboot.", cmd_kernel_reboot_warm), + SHELL_SUBCMD_SET_END /* Array terminated. */
You may want to run clang-format on this change: subsys/shell/modules/kernel_service/thread/list.c#L32
subsys/shell/modules/kernel_service/thread/list.c:32 - pcnt = (rt_stats_thread.execution_cycles * 100U) / - rt_stats_all.execution_cycles; + pcnt = (rt_stats_thread.execution_cycles * 100U) / rt_stats_all.execution_cycles;
You may want to run clang-format on this change: subsys/shell/modules/kernel_service/thread/list.c#L43
subsys/shell/modules/kernel_service/thread/list.c:43 - (uint32_t)rt_stats_thread.execution_cycles, - pcnt); + (uint32_t)rt_stats_thread.execution_cycles, pcnt);
You may want to run clang-format on this change: subsys/shell/modules/kernel_service/thread/list.c#L87
subsys/shell/modules/kernel_service/thread/list.c:87 - shell_print(sh, "%s%p %-10s", - (thread == k_current_get()) ? "*" : " ", - thread, - tname ? tname : "NA"); + shell_print(sh, "%s%p %-10s", (thread == k_current_get()) ? "*" : " ", thread, + tname ? tname : "NA"); /* Cannot use lld as it's less portable. */ shell_print(sh, "\toptions: 0x%x, priority: %d timeout: %" PRId64, - thread->base.user_options, - thread->base.prio, - (int64_t)thread->base.timeout.dticks); + thread->base.user_options, thread->base.prio, + (int64_t)thread->base.timeout.dticks); shell_print(sh, "\tstate: %s, entry: %p", - k_thread_state_str(thread, state_str, sizeof(state_str)), - thread->entry.pEntry); + k_thread_state_str(thread, state_str, sizeof(state_str)), thread->entry.pEntry);
You may want to run clang-format on this change: subsys/shell/modules/kernel_service/thread/list.c#L108
subsys/shell/modules/kernel_service/thread/list.c:108 - shell_print(sh, - "Unable to determine unused stack size (%d)\n", - ret); + shell_print(sh, "Unable to determine unused stack size (%d)\n", ret); } else { /* Calculate the real size reserved for the stack */ pcnt = ((size - unused) * 100U) / size; - shell_print(sh, - "\tstack size %zu, unused %zu, usage %zu / %zu (%u %%)\n", - size, unused, size - unused, size, pcnt); + shell_print(sh, "\tstack size %zu, unused %zu, usage %zu / %zu (%u %%)\n", size, + unused, size - unused, size, pcnt); } -
You may want to run clang-format on this change: subsys/shell/modules/kernel_service/thread/mask.c#L140
subsys/shell/modules/kernel_service/thread/mask.c:140 -SHELL_STATIC_SUBCMD_SET_CREATE(sub_kernel_thread_mask, +SHELL_STATIC_SUBCMD_SET_CREATE( + sub_kernel_thread_mask,
You may want to run clang-format on this change: subsys/shell/modules/kernel_service/thread/pin.c#L13
subsys/shell/modules/kernel_service/thread/pin.c:13 -static int cmd_kernel_thread_pin(const struct shell *sh, - size_t argc, char **argv) +static int cmd_kernel_thread_pin(const struct shell *sh, size_t argc, char **argv)
You may want to run clang-format on this change: subsys/shell/modules/kernel_service/thread/stacks.c#L36
subsys/shell/modules/kernel_service/thread/stacks.c:36 - shell_print(sh, - "Unable to determine unused stack size (%d)\n", - ret); + shell_print(sh, "Unable to determine unused stack size (%d)\n", ret);

Artifacts

Produced during runtime
Name Size
compliance.xml
1.9 KB