doc: Add zephyr/include to the Doxygen input - #10971
Merged
Merged
Conversation
lgirdwood
requested review from
dbaluta,
kv2019i,
lbetlej,
mmaka1 and
plbossart
as code owners
July 2, 2026 18:22
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the SOF Doxygen configuration to scan Zephyr-split headers again, restoring API groups that disappeared from generated documentation after the RTOS split.
Changes:
- Add
zephyr/includeto the DoxygenINPUTpaths so headers under that tree are scanned. - Add/expand Doxygen comments in
module/generic.hfor module adapter APIs and related macros/types.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 11 comments.
| File | Description |
|---|---|
| src/include/sof/audio/module_adapter/module/generic.h | Adds extensive Doxygen annotations for module adapter/module APIs, macros, and enums. |
| doc/sof.doxygen.in | Extends Doxygen INPUT to include @top_srcdir@/zephyr/include for documentation generation. |
lgirdwood
force-pushed
the
doc-add-zephyr-include
branch
from
July 2, 2026 18:27
f28a4bd to
28d6041
Compare
lgirdwood
requested review from
abonislawski,
iuliana-prodan and
lyakh
as code owners
August 28, 2026 17:47
The Memory Allocation (alloc_api), PM Runtime (pm_runtime) and DMA (sof_dma_drivers, sof_dma_copy_func) API groups stopped appearing in the generated documentation when their headers were decoupled into zephyr/include during the RTOS split. The Doxygen INPUT still listed only src/include and xtos/include, so those Zephyr-era headers were no longer scanned and sof-docs reported "Cannot find group" for all four. Add zephyr/include to the INPUT. These headers declare exactly those four groups and no others, with no group-name overlap with xtos/include, so this recovers the missing API docs without introducing duplicates. Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Signed-off-by: Liam Girdwood <liam.r.girdwood@linux.intel.com>
Fix parameter name mismatch in module_driver_heap_remove() and correct struct field documentation for vmh_block_bundle_descriptor and vmh_heap_config so Doxygen parses zephyr/include cleanly without warnings. Signed-off-by: Liam Girdwood <liam.r.girdwood@linux.intel.com>
lgirdwood
force-pushed
the
doc-add-zephyr-include
branch
from
August 30, 2026 15:49
8ce57b5 to
16ea79f
Compare
Member
Author
|
All issues, fixed SOF docs HEAD now passing with ubuntu 26.04 and with orphaned packages removed/replaced. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Add
zephyr/includeto the DoxygenINPUTindoc/sof.doxygen.in.Why
The Memory Allocation (
alloc_api), PM Runtime (pm_runtime) and DMA(
sof_dma_drivers,sof_dma_copy_func) API doc groups stopped appearing in thegenerated documentation when their headers were decoupled into
zephyr/include/during the RTOS split. The Doxygen
INPUTstill listed onlysrc/includeandxtos/include, so those Zephyr-era headers were no longer scanned, and sof-docsreported
Cannot find groupfor all four.These headers declare exactly those four groups and no others, with no
group-name overlap with
xtos/include, so adding the directory recovers themissing API docs without introducing duplicates.
Testing
Regenerated the Doxygen XML and rebuilt sof-docs against it: all four groups now
render, and the sof-docs warning count drops from 6 to 2 (the remaining two are
a pre-existing breathe limitation, unrelated to this change).
Companion PR
Supports thesofproject/sof-docs#520, whose strict (
-W) documentation buildturns these
Cannot find groupwarnings into errors. That PR's build stays reduntil this change is merged; the two should land together.