summaryrefslogtreecommitdiff
path: root/core
diff options
context:
space:
mode:
authorDawid Niedzwiecki <dn@semihalf.com>2021-12-09 12:20:54 +0100
committerCommit Bot <commit-bot@chromium.org>2021-12-17 08:05:06 +0000
commitcf66b5288713fedc917283ad6f86d75c3c59f4f9 (patch)
treeba1f35bd58e1a31f5c116ec7192873d812869623 /core
parentc346481f4bcc5aa52662e4dffa2c17d7b5b18509 (diff)
downloadchrome-ec-cf66b5288713fedc917283ad6f86d75c3c59f4f9.tar.gz
atomic: change atomic_t to long
Change the atomic_t type from int to long. For all chips used for EC, this change shouldn't impact the generated code, except for the string needed fot the print function. For the host build keep int, because EC codebase assumes that the atomic_t type is 4-byte wide. The change caused by a change in Zephyr upstream (atomic_t from int to long). BUG=b:207082842 Cq-Depend: chromium:3330355 TEST=make buildall && zmake testall BRANCH=main Signed-off-by: Dawid Niedzwiecki <dn@semihalf.com> Change-Id: Id4b6f8b60e73bdef741a530e3ed614b67b48720f Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3330201 Reviewed-by: Keith Short <keithshort@chromium.org> Reviewed-by: Tom Hughes <tomhughes@chromium.org> Commit-Queue: Dawid Niedzwiecki <dawidn@google.com>
Diffstat (limited to 'core')
-rw-r--r--core/cortex-m/task.c8
-rw-r--r--core/cortex-m0/task.c8
-rw-r--r--core/minute-ia/task.c25
-rw-r--r--core/nds32/task.c8
-rw-r--r--core/riscv-rv32i/task.c8
5 files changed, 29 insertions, 28 deletions
diff --git a/core/cortex-m/task.c b/core/cortex-m/task.c
index 61c180330f..6bff0b8d74 100644
--- a/core/cortex-m/task.c
+++ b/core/cortex-m/task.c
@@ -962,8 +962,8 @@ void task_print_list(void)
sp++)
stackused -= sizeof(uint32_t);
- ccprintf("%4d %c %-16s %08x %11.6lld %3d/%3d\n", i, is_ready,
- task_names[i], tasks[i].events, tasks[i].runtime,
+ ccprintf("%4d %c %-16s %08lx %11.6lld %3d/%3d\n", i, is_ready,
+ task_names[i], (long)tasks[i].events, tasks[i].runtime,
stackused, tasks_init[i].stack_size);
cflush();
}
@@ -1006,10 +1006,10 @@ DECLARE_SAFE_CONSOLE_COMMAND(taskinfo, command_task_info,
static int command_task_ready(int argc, char **argv)
{
if (argc < 2) {
- ccprintf("tasks_ready: 0x%08x\n", tasks_ready);
+ ccprintf("tasks_ready: 0x%08lx\n", (long)tasks_ready);
} else {
tasks_ready = strtoi(argv[1], NULL, 16);
- ccprintf("Setting tasks_ready to 0x%08x\n", tasks_ready);
+ ccprintf("Setting tasks_ready to 0x%08lx\n", (long)tasks_ready);
__schedule(0, 0);
}
diff --git a/core/cortex-m0/task.c b/core/cortex-m0/task.c
index 6996ad133c..64c5423a2b 100644
--- a/core/cortex-m0/task.c
+++ b/core/cortex-m0/task.c
@@ -579,8 +579,8 @@ void task_print_list(void)
sp++)
stackused -= sizeof(uint32_t);
- ccprintf("%4d %c %-16s %08x %11.6lld %3d/%3d\n", i, is_ready,
- task_names[i], tasks[i].events, tasks[i].runtime,
+ ccprintf("%4d %c %-16s %08lx %11.6lld %3d/%3d\n", i, is_ready,
+ task_names[i], (long)tasks[i].events, tasks[i].runtime,
stackused, tasks_init[i].stack_size);
cflush();
}
@@ -623,10 +623,10 @@ DECLARE_CONSOLE_COMMAND(taskinfo, command_task_info,
static int command_task_ready(int argc, char **argv)
{
if (argc < 2) {
- ccprintf("tasks_ready: 0x%08x\n", tasks_ready);
+ ccprintf("tasks_ready: 0x%08lx\n", (long)tasks_ready);
} else {
tasks_ready = strtoi(argv[1], NULL, 16);
- ccprintf("Setting tasks_ready to 0x%08x\n", tasks_ready);
+ ccprintf("Setting tasks_ready to 0x%08lx\n", (long)tasks_ready);
__schedule(0, 0);
}
diff --git a/core/minute-ia/task.c b/core/minute-ia/task.c
index 6ad7e8046e..91f996a1c6 100644
--- a/core/minute-ia/task.c
+++ b/core/minute-ia/task.c
@@ -534,15 +534,15 @@ void task_print_list(void)
if (IS_ENABLED(CONFIG_FPU)) {
char use_fpu = tasks[i].use_fpu ? 'Y' : 'N';
- ccprintf("%4d %c %-16s %08x %11.6lld %3d/%3d %c\n",
- i, is_ready, task_get_name(i), tasks[i].events,
- tasks[i].runtime, stackused,
- tasks_init[i].stack_size, use_fpu);
+ ccprintf("%4d %c %-16s %08lx %11.6lld %3d/%3d %c\n",
+ i, is_ready, task_get_name(i),
+ (long)tasks[i].events, tasks[i].runtime,
+ stackused, tasks_init[i].stack_size, use_fpu);
} else {
- ccprintf("%4d %c %-16s %08x %11.6lld %3d/%3d\n",
- i, is_ready, task_get_name(i), tasks[i].events,
- tasks[i].runtime, stackused,
- tasks_init[i].stack_size);
+ ccprintf("%4d %c %-16s %08lx %11.6lld %3d/%3d\n",
+ i, is_ready, task_get_name(i),
+ (long)tasks[i].events, tasks[i].runtime,
+ stackused, tasks_init[i].stack_size);
}
cflush();
@@ -565,8 +565,9 @@ static int command_task_info(int argc, char **argv)
total += irq_dist[i];
}
}
- ccprintf("Service calls: %11d\n", svc_calls);
- ccprintf("Total exceptions: %11d\n", total + svc_calls);
+ ccprintf("Service calls: %11ld\n", (long)svc_calls);
+ ccprintf("Total exceptions: %11ld\n",
+ total + (long)svc_calls);
ccprintf("Task switches: %11d\n", task_switches);
ccprintf("Task switching started: %11.6lld s\n",
task_start_time);
@@ -586,10 +587,10 @@ __maybe_unused
static int command_task_ready(int argc, char **argv)
{
if (argc < 2) {
- ccprintf("tasks_ready: 0x%08x\n", tasks_ready);
+ ccprintf("tasks_ready: 0x%08lx\n", (long)tasks_ready);
} else {
tasks_ready = strtoi(argv[1], NULL, 16);
- ccprintf("Setting tasks_ready to 0x%08x\n", tasks_ready);
+ ccprintf("Setting tasks_ready to 0x%08lx\n", (long)tasks_ready);
__schedule(0, 0);
}
diff --git a/core/nds32/task.c b/core/nds32/task.c
index 0503495f17..37d09c6d2c 100644
--- a/core/nds32/task.c
+++ b/core/nds32/task.c
@@ -680,8 +680,8 @@ void task_print_list(void)
sp++)
stackused -= sizeof(uint32_t);
- ccprintf("%4d %c %-16s %08x %11.6lld %3d/%3d\n", i, is_ready,
- task_names[i], tasks[i].events, tasks[i].runtime,
+ ccprintf("%4d %c %-16s %08lx %11.6lld %3d/%3d\n", i, is_ready,
+ task_names[i], (long)tasks[i].events, tasks[i].runtime,
stackused, tasks_init[i].stack_size);
cflush();
}
@@ -724,10 +724,10 @@ DECLARE_CONSOLE_COMMAND(taskinfo, command_task_info,
static int command_task_ready(int argc, char **argv)
{
if (argc < 2) {
- ccprintf("tasks_ready: 0x%08x\n", tasks_ready);
+ ccprintf("tasks_ready: 0x%08lx\n", (long)tasks_ready);
} else {
tasks_ready = strtoi(argv[1], NULL, 16);
- ccprintf("Setting tasks_ready to 0x%08x\n", tasks_ready);
+ ccprintf("Setting tasks_ready to 0x%08lx\n", (long)tasks_ready);
__schedule(0, 0, 0);
}
diff --git a/core/riscv-rv32i/task.c b/core/riscv-rv32i/task.c
index b9b126c1fd..a7c8c7c56d 100644
--- a/core/riscv-rv32i/task.c
+++ b/core/riscv-rv32i/task.c
@@ -613,8 +613,8 @@ void task_print_list(void)
sp++)
stackused -= sizeof(uint32_t);
- ccprintf("%4d %c %-16s %08x %11.6lld %3d/%3d\n", i, is_ready,
- task_names[i], tasks[i].events, tasks[i].runtime,
+ ccprintf("%4d %c %-16s %08lx %11.6lld %3d/%3d\n", i, is_ready,
+ task_names[i], (long)tasks[i].events, tasks[i].runtime,
stackused, tasks_init[i].stack_size);
cflush();
}
@@ -657,10 +657,10 @@ DECLARE_CONSOLE_COMMAND(taskinfo, command_task_info,
static int command_task_ready(int argc, char **argv)
{
if (argc < 2) {
- ccprintf("tasks_ready: 0x%08x\n", tasks_ready);
+ ccprintf("tasks_ready: 0x%08lx\n", (long)tasks_ready);
} else {
tasks_ready = strtoi(argv[1], NULL, 16);
- ccprintf("Setting tasks_ready to 0x%08x\n", tasks_ready);
+ ccprintf("Setting tasks_ready to 0x%08lx\n", (long)tasks_ready);
__schedule(0, 0, 0);
}