Skip to content
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

App "example_power_gating_core" cannot be built #261

Closed
JuanSapriza opened this issue May 15, 2023 · 2 comments · Fixed by #267
Closed

App "example_power_gating_core" cannot be built #261

JuanSapriza opened this issue May 15, 2023 · 2 comments · Fixed by #267

Comments

@JuanSapriza
Copy link
Contributor

During the test for PR #239 it was seen that a parameter number issue was introduced into the example_power_gating_core app, preventing it from being built. The problem was existing before, but was hiding in a #ifdef that evaluated as false, so the code was not compiled.
This blocks the deployment of the CI of #239
Below the output of the compiler. This was built with GCC on the X-HEEP builder docker container.

 /__w/x-heep/x-heep/sw/applications/example_power_gating_core/main.c: In function 'main':
/__w/x-heep/x-heep/sw/applications/example_power_gating_core/main.c:126:5: error: too few arguments to function 'gpio_irq_set_trigger'
  126 |     gpio_irq_set_trigger(&gpio, 1 << 31, kGpioIrqTriggerLevelHigh);
      |     ^~~~~~~~~~~~~~~~~~~~
In file included from /__w/x-heep/x-heep/sw/applications/example_power_gating_core/main.c:16:
/__w/x-heep/x-heep/sw/device/lib/drivers/gpio/gpio.h:224:15: note: declared here
  224 | gpio_result_t gpio_irq_set_trigger(const gpio_t *gpio,
      |               ^~~~~~~~~~~~~~~~~~~~
make[4]: *** [CMakeFiles/main.elf.dir/build.make:66: CMakeFiles/main.elf.dir/applications/example_power_gating_core/main.c.obj] Error 1
@JuanSapriza JuanSapriza changed the title App "" cannot be built App "example_power_gating_core" cannot be built May 15, 2023
@JoseCalero
Copy link
Collaborator

This needs to be fixed by @Ho3eintaji in #246

@Ho3eintaji
Copy link
Contributor

it is still previous version, however, pass pin input w/o shift and state is not used so just pass sth, then should be fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants