From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Mon, 05 Dec 2022 13:54:33 +0100 Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by lore.white.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1p2AzQ-00BcLW-Er for lore@lore.pengutronix.de; Mon, 05 Dec 2022 13:54:33 +0100 Received: from localhost ([127.0.0.1] helo=metis.ext.pengutronix.de) by metis.ext.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1p2AzQ-0006Xq-3I; Mon, 05 Dec 2022 13:54:32 +0100 Received: from drehscheibe.grey.stw.pengutronix.de ([2a0a:edc0:0:c01:1d::a2]) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1p2Ays-00058D-Fg; Mon, 05 Dec 2022 13:53:58 +0100 Received: from [2a0a:edc0:0:1101:1d::54] (helo=dude05.red.stw.pengutronix.de) by drehscheibe.grey.stw.pengutronix.de with esmtp (Exim 4.94.2) (envelope-from ) id 1p2Ayr-002TXl-4b; Mon, 05 Dec 2022 13:53:57 +0100 Received: from mol by dude05.red.stw.pengutronix.de with local (Exim 4.94.2) (envelope-from ) id 1p2Ayq-00A5hd-CZ; Mon, 05 Dec 2022 13:53:56 +0100 From: Michael Olbrich To: ptxdist@pengutronix.de Date: Mon, 5 Dec 2022 13:53:56 +0100 Message-Id: <20221205125356.2405140-1-m.olbrich@pengutronix.de> X-Mailer: git-send-email 2.30.2 In-Reply-To: <20221202113812.4017202-1-p.zabel@pengutronix.de> References: <20221202113812.4017202-1-p.zabel@pengutronix.de> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: Re: [ptxdist] [APPLIED] alsa-utils: fix build without rawmidi X-BeenThere: ptxdist@pengutronix.de X-Mailman-Version: 2.1.29 Precedence: list List-Id: PTXdist Development Mailing List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: ptxdist@pengutronix.de Cc: Philipp Zabel Sender: "ptxdist" X-SA-Exim-Connect-IP: 127.0.0.1 X-SA-Exim-Mail-From: ptxdist-bounces@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false Thanks, applied as 99ae5094ee1d541a84c353bc9a7eb7b15f4ef0c3. Michael [sent from post-receive hook] On Mon, 05 Dec 2022 13:53:56 +0100, Philipp Zabel wrote: > Pick up [1] to allow building with ALSA_LIB_RAWMIDI=n. Fixes the > following build error: > > info.c:83:44: error: unknown type name 'snd_rawmidi_stream_t'; did you mean 'snd_pcm_stream_t'? > > [1] https://lore.kernel.org/r/20220717172240.1386150-1-thomas.petazzoni@bootlin.com > > Signed-off-by: Philipp Zabel > Message-Id: <20221202113812.4017202-1-p.zabel@pengutronix.de> > Signed-off-by: Michael Olbrich > > diff --git a/patches/alsa-utils-1.2.8/0001-alsactl-info.c-fix-conditionals-on-__ALSA_PCM_H-and-.patch b/patches/alsa-utils-1.2.8/0001-alsactl-info.c-fix-conditionals-on-__ALSA_PCM_H-and-.patch > new file mode 100644 > index 000000000000..2298f0037eae > --- /dev/null > +++ b/patches/alsa-utils-1.2.8/0001-alsactl-info.c-fix-conditionals-on-__ALSA_PCM_H-and-.patch > @@ -0,0 +1,103 @@ > +From: Thomas Petazzoni > +Date: Sun, 17 Jul 2022 19:22:40 +0200 > +Subject: [PATCH] alsactl/info.c: fix conditionals on __ALSA_PCM_H and > + __ALSA_RAWMIDI_H > + > +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 > +--- > + alsactl/info.c | 14 ++++++++++---- > + 1 file changed, 10 insertions(+), 4 deletions(-) > + > +diff --git a/alsactl/info.c b/alsactl/info.c > +index 3e4b9486d720..af55206b8484 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) > + { > +@@ -228,17 +230,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); > diff --git a/patches/alsa-utils-1.2.8/series b/patches/alsa-utils-1.2.8/series > new file mode 100644 > index 000000000000..b39a77d00d87 > --- /dev/null > +++ b/patches/alsa-utils-1.2.8/series > @@ -0,0 +1,4 @@ > +# generated by git-ptx-patches > +#tag:base --start-number 1 > +0001-alsactl-info.c-fix-conditionals-on-__ALSA_PCM_H-and-.patch > +# 33c3f6b9e58007dd7e241f57feb385ac - git-ptx-patches magic