Skip to content

Build error while executing tests/kernel/smp on ESP32 #12654

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
spoorthik opened this issue Jan 23, 2019 · 2 comments
Closed

Build error while executing tests/kernel/smp on ESP32 #12654

spoorthik opened this issue Jan 23, 2019 · 2 comments
Assignees
Labels
bug The issue is a bug, or the PR is fixing a bug platform: ESP32 Espressif ESP32 priority: medium Medium impact/importance bug

Comments

@spoorthik
Copy link
Contributor

spoorthik commented Jan 23, 2019

Describe the bug
A build error is observed stating 'set_kernel_idle_time_in_ticks' defined but not used.

To Reproduce
Steps to reproduce the behavior:

  1. Setup build environment for ESP32
  2. cd zephyr.git/tests/kernel/smp
  3. mkdir build; cd build
  4. cmake -DBOARD=esp32 ..
  5. make flash
  6. See error

Screenshots or console output

zephyr.git/kernel/idle.c:54:13: error: 'set_kernel_idle_time_in_ticks' defined but not used [-Werror=unused-function]

 static void set_kernel_idle_time_in_ticks(s32_t ticks)
             ^
cc1: all warnings being treated as errors
[ 65%] Building C object zephyr/kernel/CMakeFiles/kernel.dir/mempool.c.obj
zephyr/kernel/CMakeFiles/kernel.dir/build.make:88: recipe for target 'zephyr/kernel/CMakeFiles/kernel.dir/idle.c.obj' failed
make[3]: *** [zephyr/kernel/CMakeFiles/kernel.dir/idle.c.obj] Error 1

Environment:

  • OS: Linux
  • Toolchain: Zephyr SDK, ESP32 toolchain
  • Commit SHA or Version used: 99aadec
@spoorthik spoorthik added bug The issue is a bug, or the PR is fixing a bug platform: ESP32 Espressif ESP32 labels Jan 23, 2019
@nashif nashif added the priority: medium Medium impact/importance bug label Jan 23, 2019
@andyross
Copy link
Collaborator

Yeah, I dropped a patch somewhere. @dcpleung reported that esp32 doesn't build earlier (and I blew him off while dealing with timer stuff -- should have listened more carefully, sorry!), and in fact qemu_x86_64 remains blacklisted on that test when I know it was working.

Will fix.

@spoorthik
Copy link
Contributor Author

#13042 has fixed the build error. The issue can be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug The issue is a bug, or the PR is fixing a bug platform: ESP32 Espressif ESP32 priority: medium Medium impact/importance bug
Projects
None yet
Development

No branches or pull requests

4 participants