-
Notifications
You must be signed in to change notification settings - Fork 5.3k
Description
RT-Thread Version
master
Hardware Type/Architectures
N/A
Develop Toolchain
Other
Describe the bug
#10058 合入后针对 audio 新增的注释部分我发现一些问题,请检查。
-
documentation/device/adudio/audio.md这个文件的路径有问题,[RFC][doxygen]Doxygen comment standard processing #10058 的代码 pr 时没有基于最新的 master,因为自从 75722c4 "doxygen: reorganize directory structure" 合入后,device 的文档路径已经移到documentation/5.device下去了。另外还要注意的是 ”adudio“ 这个目录名字拼写错误。 -
components/drivers/include/drivers/dev_audio.h中的 Data Structures 的成员都缺乏注释 -
enum 部分代码上应该给 enum 定义一个名字为好,否则在 html 上只显示一个 enum (doxygen 认为其是一个 anonymous enum),不过这个涉及代码的改动,请检查加上是否会有副作用。
-
rt_audio_ops 缺失了。作为一个需要 user 定义的 callback 函数的集合,需要对回调函数的行为做简单的描述,这对 bsp 使用还是很重要的。
-
一个改进建议,对于 marco 我看你为了分组新建了许多 group,这个没有问题。只是在对组的 brief 描述上我建议改进一下。譬如对 audio_dsp,注释是 ”Support Dsp(input/output) Units controls“ 感觉说的不是很清楚,这些 marco 是不是每个对应一种类型(type)? 这个在注释上感觉不出来。类似的还有 audio_mixer 和 audio_codec_cmd。
-
html 网页的 Modules 部分多了一项出来,见下图:
-
rt_audio_samplerate_to_speed看上去是一个无用函数,感觉可以删除,但需要另外一个 pr 并让 maintaner (熊大?)确认一下。
Other additional context
No response
