Set of patches that brings no new functionality but makes it easier to
maintain and read the avs-driver code. There is one 'fix' among them -
the third patch, described below.
First, debug-related code - mainly the data-probing feature - is mixed
with non-debug PCM code. Separate it into debug.h and update the pcm.c
file.
Next, as the probing-board is not tied to any topology file, it shall
not using struct avs_soc_component descriptor but the basic struct
snd_soc_component one. While on the first sight this is just bump in
LOCs, runtime is simplified as no topology checks and related code will
be engaged.
With probing-board addressed and debug code relocated:
- address the component-teardown issues when the
componet->name points to a string which is shared by multiple components
and/or machine boards. devm_kstrdup() makes this go away.
- streamline the naming scheme for the functions engaged in the
components registration. This is purely a readability improvement.
Cezary Rojewski (4):
ASoC: Intel: avs: Separate debug symbols
ASoC: Intel: avs: Drop pcm.h dependency for probes
ASoC: Intel: avs: Do not share the name pointer between components
ASoC: Intel: avs: Streamline register-component function names
sound/soc/intel/avs/apl.c | 1 +
sound/soc/intel/avs/avs.h | 89 +++-----------------------
sound/soc/intel/avs/board_selection.c | 10 +--
sound/soc/intel/avs/cnl.c | 1 +
sound/soc/intel/avs/core.c | 1 +
sound/soc/intel/avs/debug.h | 91 +++++++++++++++++++++++++++
sound/soc/intel/avs/debugfs.c | 1 +
sound/soc/intel/avs/icl.c | 1 +
sound/soc/intel/avs/ipc.c | 1 +
sound/soc/intel/avs/lnl.c | 1 +
sound/soc/intel/avs/mtl.c | 1 +
sound/soc/intel/avs/pcm.c | 35 ++++++-----
sound/soc/intel/avs/probes.c | 23 +++++--
sound/soc/intel/avs/ptl.c | 1 +
sound/soc/intel/avs/skl.c | 1 +
sound/soc/intel/avs/tgl.c | 1 +
16 files changed, 153 insertions(+), 106 deletions(-)
create mode 100644 sound/soc/intel/avs/debug.h
--
2.25.1
[Index of Archives]
[Pulseaudio]
[Linux Audio Users]
[ALSA Devel]
[Fedora Desktop]
[Fedora SELinux]
[Big List of Linux Books]
[Yosemite News]
[KDE Users]