Skip to content

Conversation

@comejv
Copy link
Contributor

@comejv comejv commented Jan 21, 2026

Going from CAPIOC_FREQUENCE to CAPIO_FREQUENCY according to apache/nuttx#16925

Summary

CAPIOC_FREQUENCE is a typo, should be FREQUENCY. nuttx/pull/16925 fixes this, so the app side should be fixed too.

Going from CAPIOC_FREQUENCE to CAPIO_FREQUENCY according to nuttx/pull/16925

Signed-off-by: Côme VINCENT <44554692+comejv@users.noreply.github.com>
@simbit18
Copy link
Contributor

@comejv this error occurs

====================================================================================
Configuration/Tool: esp32c6-devkitc/capture
2026-01-21 14:25:53
------------------------------------------------------------------------------------
  Cleaning...
  Configuring...
  Building NuttX...
cap_main.c: In function 'cap_main':
Error: cap_main.c:354:23: error: 'CAPIOC_FREQUENCY' undeclared (first use in this function); did you mean 'CAPIOC_FREQUENCE'?
  354 |       ret = ioctl(fd, CAPIOC_FREQUENCY,
      |                       ^~~~~~~~~~~~~~~~
      |                       CAPIOC_FREQUENCE
cap_main.c:354:23: note: each undeclared identifier is reported only once for each function it appears in
make[2]: *** [/github/workspace/sources/apps/Application.mk:330: cap_main.c.github.workspace.sources.apps.examples.capture.o] Error 1
make[2]: Target 'all' not remade because of errors.
make[1]: *** [Makefile:54: /github/workspace/sources/apps/examples/capture_all] Error 2
make[1]: Target 'all' not remade because of errors.
make: *** [tools/LibTargets.mk:248: /github/workspace/sources/apps/libapps.a] Error 2
make: Target 'all' not remade because of errors.
/github/workspace/sources/nuttx/tools/testbuild.sh: line 385: /github/workspace/sources/nuttx/../nuttx/nuttx.manifest: No such file or directory
  [1/1] Normalize esp32c6-devkitc/capture
====================================================================================

@comejv
Copy link
Contributor Author

comejv commented Jan 21, 2026

@simbit18 yes it will happen until new definition is merged from apache/nuttx#16925 but tests won't pass there unless the new definition is used here in nuttx-apps, so I don't really know how to go about this except than to force merge on one side first

@simbit18 simbit18 merged commit 85b1a1c into apache:master Jan 21, 2026
25 of 40 checks passed
@simbit18
Copy link
Contributor

OK, understood. I will force the merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants