[PATCH alsa-utils] alsactl/info.c: fix conditionals on __ALSA_PCM_H and __ALSA_RAWMIDI_H

Thomas Petazzoni thomas.petazzoni at bootlin.com
Sun Jul 17 19:22:40 CEST 2022


Commit bbc74a61ac7c35e506c3d7f76ecf943cb55736a6 ("alsactl: implement
'info' command") implemented an alsactl info command. In this
implementation, there was an attempt to properly address optional
features from alsa-lib by using conditions on __ALSA_PCM_H,
__ALSA_RAWMIDI_H.

Unfortunately, this attempt does not work entirely: only the code
inside pcm_device_list(), rawmidi_device_list() was conditionally
compiled, but their very prototype also use type definitions provided
in pcm.h and rawmidi.h. So really, it's the entire function that needs
to be conditionally implemented.

Also, snd_rawmidi_stream_name() was not handled properly, for the same
reason.

This commit implements pcm_device_list() only if __ALSA_PCM_H is
defined, and implements snd_rawmidi_stream_name() and
rawmidi_device_list() only if __ALSA_RAWMIDI_H is defined.

general_card_info() is modified to not call the PCM or raw MIDI
functions when support is not available.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni at bootlin.com>
---
 alsactl/info.c | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/alsactl/info.c b/alsactl/info.c
index 253539d..9bd72af 100644
--- a/alsactl/info.c
+++ b/alsactl/info.c
@@ -22,9 +22,9 @@
 #include "aconfig.h"
 #include "alsactl.h"
 
+#ifdef __ALSA_PCM_H
 static int pcm_device_list(snd_ctl_t *ctl, snd_pcm_stream_t stream, bool *first)
 {
-#ifdef __ALSA_PCM_H
 	int err, dev, idx;
 	unsigned int count;
 	snd_pcm_info_t *pcminfo;
@@ -76,10 +76,12 @@ static int pcm_device_list(snd_ctl_t *ctl, snd_pcm_stream_t stream, bool *first)
 						idx, snd_pcm_info_get_subdevice_name(pcminfo));
 		}
 	}
-#endif
+
 	return 0;
 }
+#endif
 
+#ifdef __ALSA_RAWMIDI_H
 static const char *snd_rawmidi_stream_name(snd_rawmidi_stream_t stream)
 {
 	if (stream == SND_RAWMIDI_STREAM_INPUT)
@@ -91,7 +93,6 @@ static const char *snd_rawmidi_stream_name(snd_rawmidi_stream_t stream)
 
 static int rawmidi_device_list(snd_ctl_t *ctl, snd_rawmidi_stream_t stream, bool *first)
 {
-#ifdef __ALSA_RAWMIDI_H
 	int err, dev, idx;
 	unsigned int count;
 	snd_rawmidi_info_t *info;
@@ -143,9 +144,10 @@ static int rawmidi_device_list(snd_ctl_t *ctl, snd_rawmidi_stream_t stream, bool
 						idx, snd_rawmidi_info_get_subdevice_name(info));
 		}
 	}
-#endif
+
 	return 0;
 }
+#endif
 
 static int hwdep_device_list(snd_ctl_t *ctl)
 {
@@ -227,17 +229,21 @@ int general_card_info(int cardno)
 	}
 	err = card_info(ctl);
 
+#ifdef __ALSA_PCM_H
 	first = true;
 	if (err >= 0)
 		err = pcm_device_list(ctl, SND_PCM_STREAM_PLAYBACK, &first);
 	if (err >= 0)
 		err = pcm_device_list(ctl, SND_PCM_STREAM_CAPTURE, &first);
+#endif
 
+#ifdef __ALSA_RAWMIDI_H
 	first = true;
 	if (err >= 0)
 		err = rawmidi_device_list(ctl, SND_PCM_STREAM_PLAYBACK, &first);
 	if (err >= 0)
 		err = rawmidi_device_list(ctl, SND_PCM_STREAM_CAPTURE, &first);
+#endif
 
 	if (err >= 0)
 		err = hwdep_device_list(ctl);
-- 
2.36.1



More information about the Alsa-devel mailing list