August 2024 - Alsa-devel - mailman.alsa-project.org

[tiwai-sound:for-next 66/82] sound/usb/usx2y/usb_stream.c:668:17: warning: variable 'iters' set but not used
by kernel test robot 08 Aug '24

08 Aug '24

tree: https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git for-next head: e95b9f7f2ee05bbef3bf6a4cd7da6e887f17f652 commit: f8466d91f36d97e237ea1b3c6f19de980054a1b6 [66/82] ALSA: usx2y: Use standard print API config: x86_64-randconfig-161-20240808 (https://download.01.org/0day-ci/archive/20240809/202408090249.8LE9qrae-lkp@…) compiler: clang version 18.1.5 (https://github.com/llvm/llvm-project 617a15a9eac96088ae5e9134248d8236e34b91b1) reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240809/202408090249.8LE9qrae-lkp@…) If you fix the issue in a separate patch/commit (i.e. not just a new version of the same patch/commit), kindly add following tags | Reported-by: kernel test robot <lkp(a)intel.com> | Closes: https://lore.kernel.org/oe-kbuild-all/202408090249.8LE9qrae-lkp@intel.com/ All warnings (new ones prefixed by >>): In file included from sound/usb/usx2y/us122l.c:15: >> sound/usb/usx2y/usb_stream.c:668:17: warning: variable 'iters' set but not used [-Wunused-but-set-variable] 668 | int frame = 0, iters = 0; | ^ 1 warning generated. vim +/iters +668 sound/usb/usx2y/usb_stream.c 030a07e441296c Karsten Wiese 2008-07-30 664 030a07e441296c Karsten Wiese 2008-07-30 665 int usb_stream_start(struct usb_stream_kernel *sk) 030a07e441296c Karsten Wiese 2008-07-30 666 { 030a07e441296c Karsten Wiese 2008-07-30 667 struct usb_stream *s = sk->s; 030a07e441296c Karsten Wiese 2008-07-30 @668 int frame = 0, iters = 0; 030a07e441296c Karsten Wiese 2008-07-30 669 int u, err; 030a07e441296c Karsten Wiese 2008-07-30 670 int try = 0; 030a07e441296c Karsten Wiese 2008-07-30 671 030a07e441296c Karsten Wiese 2008-07-30 672 if (s->state != usb_stream_stopped) 030a07e441296c Karsten Wiese 2008-07-30 673 return -EAGAIN; 030a07e441296c Karsten Wiese 2008-07-30 674 030a07e441296c Karsten Wiese 2008-07-30 675 subs_set_complete(sk->inurb, i_capture_start); 030a07e441296c Karsten Wiese 2008-07-30 676 subs_set_complete(sk->outurb, i_playback_start); 030a07e441296c Karsten Wiese 2008-07-30 677 memset(sk->write_page, 0, s->write_size); 030a07e441296c Karsten Wiese 2008-07-30 678 dotry: 030a07e441296c Karsten Wiese 2008-07-30 679 s->insize_done = 0; 030a07e441296c Karsten Wiese 2008-07-30 680 s->idle_insize = 0; 030a07e441296c Karsten Wiese 2008-07-30 681 s->idle_outsize = 0; 030a07e441296c Karsten Wiese 2008-07-30 682 s->sync_packet = -1; 030a07e441296c Karsten Wiese 2008-07-30 683 s->inpacket_head = -1; 030a07e441296c Karsten Wiese 2008-07-30 684 sk->iso_frame_balance = 0; 030a07e441296c Karsten Wiese 2008-07-30 685 ++try; 030a07e441296c Karsten Wiese 2008-07-30 686 for (u = 0; u < 2; u++) { 030a07e441296c Karsten Wiese 2008-07-30 687 struct urb *inurb = sk->inurb[u]; 030a07e441296c Karsten Wiese 2008-07-30 688 struct urb *outurb = sk->outurb[u]; 4c0a58ef36f3de Takashi Iwai 2021-05-17 689 030a07e441296c Karsten Wiese 2008-07-30 690 playback_prep_freqn(sk, outurb); 030a07e441296c Karsten Wiese 2008-07-30 691 inurb->number_of_packets = outurb->number_of_packets; 030a07e441296c Karsten Wiese 2008-07-30 692 inurb->transfer_buffer_length = 030a07e441296c Karsten Wiese 2008-07-30 693 inurb->number_of_packets * 030a07e441296c Karsten Wiese 2008-07-30 694 inurb->iso_frame_desc[0].length; d0f3a2eb906256 Karsten Wiese 2011-12-30 695 030a07e441296c Karsten Wiese 2008-07-30 696 if (u == 0) { 030a07e441296c Karsten Wiese 2008-07-30 697 int now; 030a07e441296c Karsten Wiese 2008-07-30 698 struct usb_device *dev = inurb->dev; 4c0a58ef36f3de Takashi Iwai 2021-05-17 699 030a07e441296c Karsten Wiese 2008-07-30 700 frame = usb_get_current_frame_number(dev); 030a07e441296c Karsten Wiese 2008-07-30 701 do { 030a07e441296c Karsten Wiese 2008-07-30 702 now = usb_get_current_frame_number(dev); 030a07e441296c Karsten Wiese 2008-07-30 703 ++iters; 030a07e441296c Karsten Wiese 2008-07-30 704 } while (now > -1 && now == frame); 030a07e441296c Karsten Wiese 2008-07-30 705 } 030a07e441296c Karsten Wiese 2008-07-30 706 err = usb_submit_urb(inurb, GFP_ATOMIC); 030a07e441296c Karsten Wiese 2008-07-30 707 if (err < 0) { f8466d91f36d97 Takashi Iwai 2024-08-07 708 dev_err(&sk->dev->dev, f8466d91f36d97 Takashi Iwai 2024-08-07 709 "%s: usb_submit_urb(sk->inurb[%i]) returned %i\n", f8466d91f36d97 Takashi Iwai 2024-08-07 710 __func__, u, err); 030a07e441296c Karsten Wiese 2008-07-30 711 return err; 030a07e441296c Karsten Wiese 2008-07-30 712 } 030a07e441296c Karsten Wiese 2008-07-30 713 err = usb_submit_urb(outurb, GFP_ATOMIC); 030a07e441296c Karsten Wiese 2008-07-30 714 if (err < 0) { f8466d91f36d97 Takashi Iwai 2024-08-07 715 dev_err(&sk->dev->dev, f8466d91f36d97 Takashi Iwai 2024-08-07 716 "%s: usb_submit_urb(sk->outurb[%i]) returned %i\n", f8466d91f36d97 Takashi Iwai 2024-08-07 717 __func__, u, err); 030a07e441296c Karsten Wiese 2008-07-30 718 return err; 030a07e441296c Karsten Wiese 2008-07-30 719 } d0f3a2eb906256 Karsten Wiese 2011-12-30 720 030a07e441296c Karsten Wiese 2008-07-30 721 if (inurb->start_frame != outurb->start_frame) { f8466d91f36d97 Takashi Iwai 2024-08-07 722 dev_dbg(&sk->dev->dev, f8466d91f36d97 Takashi Iwai 2024-08-07 723 "%s: u[%i] start_frames differ in:%u out:%u\n", f8466d91f36d97 Takashi Iwai 2024-08-07 724 __func__, u, inurb->start_frame, outurb->start_frame); 030a07e441296c Karsten Wiese 2008-07-30 725 goto check_retry; 030a07e441296c Karsten Wiese 2008-07-30 726 } 030a07e441296c Karsten Wiese 2008-07-30 727 } 030a07e441296c Karsten Wiese 2008-07-30 728 try = 0; 030a07e441296c Karsten Wiese 2008-07-30 729 check_retry: 030a07e441296c Karsten Wiese 2008-07-30 730 if (try) { 030a07e441296c Karsten Wiese 2008-07-30 731 usb_stream_stop(sk); 030a07e441296c Karsten Wiese 2008-07-30 732 if (try < 5) { 030a07e441296c Karsten Wiese 2008-07-30 733 msleep(1500); f8466d91f36d97 Takashi Iwai 2024-08-07 734 dev_dbg(&sk->dev->dev, "goto dotry;\n"); 030a07e441296c Karsten Wiese 2008-07-30 735 goto dotry; 030a07e441296c Karsten Wiese 2008-07-30 736 } f8466d91f36d97 Takashi Iwai 2024-08-07 737 dev_warn(&sk->dev->dev, f8466d91f36d97 Takashi Iwai 2024-08-07 738 "%s: couldn't start all urbs on the same start_frame.\n", f8466d91f36d97 Takashi Iwai 2024-08-07 739 __func__); 030a07e441296c Karsten Wiese 2008-07-30 740 return -EFAULT; 030a07e441296c Karsten Wiese 2008-07-30 741 } 030a07e441296c Karsten Wiese 2008-07-30 742 030a07e441296c Karsten Wiese 2008-07-30 743 sk->idle_inurb = sk->inurb[USB_STREAM_NURBS - 2]; 030a07e441296c Karsten Wiese 2008-07-30 744 sk->idle_outurb = sk->outurb[USB_STREAM_NURBS - 2]; 030a07e441296c Karsten Wiese 2008-07-30 745 sk->completed_inurb = sk->inurb[USB_STREAM_NURBS - 1]; 030a07e441296c Karsten Wiese 2008-07-30 746 sk->completed_outurb = sk->outurb[USB_STREAM_NURBS - 1]; 030a07e441296c Karsten Wiese 2008-07-30 747 :::::: The code at line 668 was first introduced by commit :::::: 030a07e441296c372f946cd4065b5d831d8dc40c ALSA: Add USB US122L driver :::::: TO: Karsten Wiese <fzu(a)wemgehoertderstaat.de> :::::: CC: Jaroslav Kysela <perex(a)perex.cz> -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki

1 0

[tiwai-sound:for-next 69/82] sound/pcmcia/vx/vxpocket.c:273:26: error: 'pdev' undeclared; did you mean 'p_dev'?
by kernel test robot 08 Aug '24

08 Aug '24

tree: https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git for-next head: e95b9f7f2ee05bbef3bf6a4cd7da6e887f17f652 commit: 2acbb5e57230830016e0fb2d61886e7a8c7f59ce [69/82] ALSA: vxpocket: Use standard print API config: i386-randconfig-141-20240808 (https://download.01.org/0day-ci/archive/20240809/202408090110.t0mWbTyh-lkp@…) compiler: gcc-12 (Debian 12.2.0-14) 12.2.0 reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240809/202408090110.t0mWbTyh-lkp@…) If you fix the issue in a separate patch/commit (i.e. not just a new version of the same patch/commit), kindly add following tags | Reported-by: kernel test robot <lkp(a)intel.com> | Closes: https://lore.kernel.org/oe-kbuild-all/202408090110.t0mWbTyh-lkp@intel.com/ All errors (new ones prefixed by >>): sound/pcmcia/vx/vxpocket.c: In function 'vxpocket_config': sound/pcmcia/vx/vxpocket.c:207:13: error: 'struct vx_core' has no member named 'dev' 207 | chip->dev = &link->dev; | ^~ In file included from include/linux/device.h:15, from include/sound/core.h:10, from sound/pcmcia/vx/vxpocket.c:13: sound/pcmcia/vx/vxpocket.c: In function 'vxpocket_probe': >> sound/pcmcia/vx/vxpocket.c:273:26: error: 'pdev' undeclared (first use in this function); did you mean 'p_dev'? 273 | dev_err(&pdev->dev, "vxpocket: cannot create a card instance\n"); | ^~~~ include/linux/dev_printk.h:110:25: note: in definition of macro 'dev_printk_index_wrap' 110 | _p_func(dev, fmt, ##__VA_ARGS__); \ | ^~~ sound/pcmcia/vx/vxpocket.c:273:17: note: in expansion of macro 'dev_err' 273 | dev_err(&pdev->dev, "vxpocket: cannot create a card instance\n"); | ^~~~~~~ sound/pcmcia/vx/vxpocket.c:273:26: note: each undeclared identifier is reported only once for each function it appears in 273 | dev_err(&pdev->dev, "vxpocket: cannot create a card instance\n"); | ^~~~ include/linux/dev_printk.h:110:25: note: in definition of macro 'dev_printk_index_wrap' 110 | _p_func(dev, fmt, ##__VA_ARGS__); \ | ^~~ sound/pcmcia/vx/vxpocket.c:273:17: note: in expansion of macro 'dev_err' 273 | dev_err(&pdev->dev, "vxpocket: cannot create a card instance\n"); | ^~~~~~~ vim +273 sound/pcmcia/vx/vxpocket.c 247 248 249 /* 250 */ 251 static int vxpocket_probe(struct pcmcia_device *p_dev) 252 { 253 struct snd_card *card; 254 struct snd_vxpocket *vxp; 255 int i, err; 256 257 /* find an empty slot from the card list */ 258 for (i = 0; i < SNDRV_CARDS; i++) { 259 if (!(card_alloc & (1 << i))) 260 break; 261 } 262 if (i >= SNDRV_CARDS) { 263 dev_err(&p_dev->dev, "vxpocket: too many cards found\n"); 264 return -EINVAL; 265 } 266 if (! enable[i]) 267 return -ENODEV; /* disabled explicitly */ 268 269 /* ok, create a card instance */ 270 err = snd_card_new(&p_dev->dev, index[i], id[i], THIS_MODULE, 271 0, &card); 272 if (err < 0) { > 273 dev_err(&pdev->dev, "vxpocket: cannot create a card instance\n"); 274 return err; 275 } 276 277 err = snd_vxpocket_new(card, ibl[i], p_dev, &vxp); 278 if (err < 0) { 279 snd_card_free(card); 280 return err; 281 } 282 card->private_data = vxp; 283 284 vxp->index = i; 285 card_alloc |= 1 << i; 286 287 vxp->p_dev = p_dev; 288 289 return vxpocket_config(p_dev); 290 } 291 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki

1 0

[PATCH] ALSA: snd-usb-caiaq: use snd_pcm_rate_to_rate_bit
by Jerome Brunet 08 Aug '24

08 Aug '24

Use snd_pcm_rate_to_rate_bit() helper provided by Alsa instead re-implementing it. This reduce code duplication and helps when changing some Alsa definition is necessary. Signed-off-by: Jerome Brunet <jbrunet(a)baylibre.com> --- sound/usb/caiaq/audio.c | 13 +------------ 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/sound/usb/caiaq/audio.c b/sound/usb/caiaq/audio.c index 4981753652a7..e62a4ea1d19c 100644 --- a/sound/usb/caiaq/audio.c +++ b/sound/usb/caiaq/audio.c @@ -174,14 +174,6 @@ static int snd_usb_caiaq_pcm_hw_free(struct snd_pcm_substream *sub) return 0; } -/* this should probably go upstream */ -#if SNDRV_PCM_RATE_5512 != 1 << 0 || SNDRV_PCM_RATE_192000 != 1 << 12 -#error "Change this table" -#endif - -static const unsigned int rates[] = { 5512, 8000, 11025, 16000, 22050, 32000, 44100, - 48000, 64000, 88200, 96000, 176400, 192000 }; - static int snd_usb_caiaq_pcm_prepare(struct snd_pcm_substream *substream) { int bytes_per_sample, bpp, ret, i; @@ -233,10 +225,7 @@ static int snd_usb_caiaq_pcm_prepare(struct snd_pcm_substream *substream) /* the first client that opens a stream defines the sample rate * setting for all subsequent calls, until the last client closed. */ - for (i=0; i < ARRAY_SIZE(rates); i++) - if (runtime->rate == rates[i]) - cdev->pcm_info.rates = 1 << i; - + cdev->pcm_info.rates = snd_pcm_rate_to_rate_bit(runtime->rate); snd_pcm_limit_hw_rates(runtime); bytes_per_sample = BYTES_PER_SAMPLE; -- 2.43.0

2 1

[tiwai-sound:for-next 32/79] sound/pcmcia/vx/vxpocket.c:208:13: error: 'struct vx_core' has no member named 'dev'
by kernel test robot 08 Aug '24

08 Aug '24

tree: https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git for-next head: 9b88d0890ed9adab413ea991fac90842688e1017 commit: b426b3ba9f6fe17990893c5324727dd217d420b6 [32/79] ALSA: vx_core: Drop unused dev field config: i386-randconfig-141-20240808 (https://download.01.org/0day-ci/archive/20240809/202408090016.kW0TA6fc-lkp@…) compiler: gcc-12 (Debian 12.2.0-14) 12.2.0 reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240809/202408090016.kW0TA6fc-lkp@…) If you fix the issue in a separate patch/commit (i.e. not just a new version of the same patch/commit), kindly add following tags | Reported-by: kernel test robot <lkp(a)intel.com> | Closes: https://lore.kernel.org/oe-kbuild-all/202408090016.kW0TA6fc-lkp@intel.com/ All errors (new ones prefixed by >>): sound/pcmcia/vx/vxpocket.c: In function 'vxpocket_config': >> sound/pcmcia/vx/vxpocket.c:208:13: error: 'struct vx_core' has no member named 'dev' 208 | chip->dev = &link->dev; | ^~ vim +208 sound/pcmcia/vx/vxpocket.c 6d00a3127972e7 Takashi Iwai 2005-06-30 170 6d00a3127972e7 Takashi Iwai 2005-06-30 171 6d00a3127972e7 Takashi Iwai 2005-06-30 172 /* 6d00a3127972e7 Takashi Iwai 2005-06-30 173 * configuration callback 6d00a3127972e7 Takashi Iwai 2005-06-30 174 */ 6d00a3127972e7 Takashi Iwai 2005-06-30 175 15b99ac1729503 Dominik Brodowski 2006-03-31 176 static int vxpocket_config(struct pcmcia_device *link) 6d00a3127972e7 Takashi Iwai 2005-06-30 177 { af26367f69a474 Takashi Iwai 2005-11-17 178 struct vx_core *chip = link->priv; 7c5af6ffd69bb2 Dominik Brodowski 2009-10-24 179 int ret; 6d00a3127972e7 Takashi Iwai 2005-06-30 180 6d00a3127972e7 Takashi Iwai 2005-06-30 181 snd_printdd(KERN_DEBUG "vxpocket_config called\n"); 6d00a3127972e7 Takashi Iwai 2005-06-30 182 6d00a3127972e7 Takashi Iwai 2005-06-30 183 /* redefine hardware record according to the VERSION1 string */ af2b3b503ad1b0 Dominik Brodowski 2006-10-25 184 if (!strcmp(link->prod_id[1], "VX-POCKET")) { 6d00a3127972e7 Takashi Iwai 2005-06-30 185 snd_printdd("VX-pocket is detected\n"); 6d00a3127972e7 Takashi Iwai 2005-06-30 186 } else { 6d00a3127972e7 Takashi Iwai 2005-06-30 187 snd_printdd("VX-pocket 440 is detected\n"); 6d00a3127972e7 Takashi Iwai 2005-06-30 188 /* overwrite the hardware information */ 6d00a3127972e7 Takashi Iwai 2005-06-30 189 chip->hw = &vxp440_hw; 6d00a3127972e7 Takashi Iwai 2005-06-30 190 chip->type = vxp440_hw.type; 6d00a3127972e7 Takashi Iwai 2005-06-30 191 strcpy(chip->card->driver, vxp440_hw.name); 6d00a3127972e7 Takashi Iwai 2005-06-30 192 } 6d00a3127972e7 Takashi Iwai 2005-06-30 193 90abdc3b973229 Dominik Brodowski 2010-07-24 194 ret = pcmcia_request_io(link); 7c5af6ffd69bb2 Dominik Brodowski 2009-10-24 195 if (ret) db0a5214b8d6cc Takashi Iwai 2014-09-09 196 goto failed_preirq; 7c5af6ffd69bb2 Dominik Brodowski 2009-10-24 197 db0a5214b8d6cc Takashi Iwai 2014-09-09 198 ret = request_threaded_irq(link->irq, snd_vx_irq_handler, db0a5214b8d6cc Takashi Iwai 2014-09-09 199 snd_vx_threaded_irq_handler, db0a5214b8d6cc Takashi Iwai 2014-09-09 200 IRQF_SHARED, link->devname, link->priv); 7c5af6ffd69bb2 Dominik Brodowski 2009-10-24 201 if (ret) db0a5214b8d6cc Takashi Iwai 2014-09-09 202 goto failed_preirq; 7c5af6ffd69bb2 Dominik Brodowski 2009-10-24 203 1ac71e5a35eebe Dominik Brodowski 2010-07-29 204 ret = pcmcia_enable_device(link); 7c5af6ffd69bb2 Dominik Brodowski 2009-10-24 205 if (ret) 7c5af6ffd69bb2 Dominik Brodowski 2009-10-24 206 goto failed; 6d00a3127972e7 Takashi Iwai 2005-06-30 207 dd2e5a156525f1 Dominik Brodowski 2009-11-03 @208 chip->dev = &link->dev; 6d00a3127972e7 Takashi Iwai 2005-06-30 209 9a017a910346af Dominik Brodowski 2010-07-24 210 if (snd_vxpocket_assign_resources(chip, link->resource[0]->start, 9a017a910346af Dominik Brodowski 2010-07-24 211 link->irq) < 0) 6d00a3127972e7 Takashi Iwai 2005-06-30 212 goto failed; 6d00a3127972e7 Takashi Iwai 2005-06-30 213 79ca4f3f625e14 Takashi Iwai 2006-04-12 214 return 0; 6d00a3127972e7 Takashi Iwai 2005-06-30 215 6d00a3127972e7 Takashi Iwai 2005-06-30 216 failed: db0a5214b8d6cc Takashi Iwai 2014-09-09 217 free_irq(link->irq, link->priv); db0a5214b8d6cc Takashi Iwai 2014-09-09 218 failed_preirq: fba395eee7d3f3 Dominik Brodowski 2006-03-31 219 pcmcia_disable_device(link); 15b99ac1729503 Dominik Brodowski 2006-03-31 220 return -ENODEV; 6d00a3127972e7 Takashi Iwai 2005-06-30 221 } 6d00a3127972e7 Takashi Iwai 2005-06-30 222 :::::: The code at line 208 was first introduced by commit :::::: dd2e5a156525f11754d9b1e0583f6bb49c253d62 pcmcia: remove deprecated handle_to_dev() macro :::::: TO: Dominik Brodowski <linux(a)dominikbrodowski.net> :::::: CC: Dominik Brodowski <linux(a)dominikbrodowski.net> -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki

1 0

[PATCH 0/5] ASoC/SOF/PCI/Intel: add PantherLake support
by Pierre-Louis Bossart 08 Aug '24

08 Aug '24

Add initial support for the PantherLake platform, and initial ACPI configurations. This patchset depends on the first patch of "[PATCH 0/3] ALSA/PCI: add PantherLake audio support" Bard Liao (2): ASoC: Intel: soc-acpi-intel-ptl-match: add rt711-sdca table ASoC: Intel: soc-acpi-intel-ptl-match: Add rt722 support Fred Oh (1): ASoC: SOF: Intel: add PTL specific power control register Pierre-Louis Bossart (2): ASoC: SOF: Intel: add initial support for PTL ASoC: Intel: soc-acpi: add PTL match tables include/sound/soc-acpi-intel-match.h | 2 + sound/soc/intel/common/Makefile | 1 + .../intel/common/soc-acpi-intel-ptl-match.c | 121 ++++++++++++++++++ sound/soc/sof/intel/Kconfig | 17 +++ sound/soc/sof/intel/Makefile | 2 + sound/soc/sof/intel/hda-dsp.c | 1 + sound/soc/sof/intel/hda.h | 1 + sound/soc/sof/intel/lnl.c | 27 ++++ sound/soc/sof/intel/mtl.c | 16 ++- sound/soc/sof/intel/mtl.h | 2 + sound/soc/sof/intel/pci-ptl.c | 77 +++++++++++ sound/soc/sof/intel/shim.h | 1 + 12 files changed, 266 insertions(+), 2 deletions(-) create mode 100644 sound/soc/intel/common/soc-acpi-intel-ptl-match.c create mode 100644 sound/soc/sof/intel/pci-ptl.c -- 2.43.0

2 9

[PATCH 2/2] ASoC: cs35l56: Stop creating ALSA controls for firmware coefficients
by Simon Trimmer 07 Aug '24

07 Aug '24

A number of laptops have gone to market with old firmware versions that export controls that have since been hidden, but we can't just install a newer firmware because the firmware for each product is customized and qualified by the OEM. The issue is that alsactl save and restore has no idea what controls are good to persist which can lead to misconfiguration. There is no reason that the UCM or user should need to interact with any of the ALSA controls for the firmware coefficients so they can be removed entirely. Fixes: e49611252900 ("ASoC: cs35l56: Add driver for Cirrus Logic CS35L56") Signed-off-by: Simon Trimmer <simont(a)opensource.cirrus.com> --- sound/soc/codecs/cs35l56.c | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/sound/soc/codecs/cs35l56.c b/sound/soc/codecs/cs35l56.c index 84c34f5b1a51..757ade6373ed 100644 --- a/sound/soc/codecs/cs35l56.c +++ b/sound/soc/codecs/cs35l56.c @@ -1095,6 +1095,11 @@ int cs35l56_system_resume(struct device *dev) } EXPORT_SYMBOL_GPL(cs35l56_system_resume); +static int cs35l56_control_add_nop(struct wm_adsp *dsp, struct cs_dsp_coeff_ctl *cs_ctl) +{ + return 0; +} + static int cs35l56_dsp_init(struct cs35l56_private *cs35l56) { struct wm_adsp *dsp; @@ -1117,6 +1122,12 @@ static int cs35l56_dsp_init(struct cs35l56_private *cs35l56) dsp->fw = 12; dsp->wmfw_optional = true; + /* + * None of the firmware controls need to be exported so add a no-op + * callback that suppresses creating an ALSA control. + */ + dsp->control_add = &cs35l56_control_add_nop; + dev_dbg(cs35l56->base.dev, "DSP system name: '%s'\n", dsp->system_name); ret = wm_halo_init(dsp); -- 2.43.0

1 0

[PATCH alsa-lib] seq: Add snd_seq_{get|set}_ump_is_midi1() API functions
by Takashi Iwai 07 Aug '24

07 Aug '24

Implement the API function calls corresponding to the new sequencer port flag bit that has been added recently to the kernel. A UMP MIDI 2.0 device allow to have an optional MIDI 1.0 port while speaking in MIDI 2.0 protocol for other UMP Groups. The new seq port flag indicates that. This is rather a minor difference, and since ALSA sequencer core covers the all conversions, usually you don't have to worry about it at all. Signed-off-by: Takashi Iwai <tiwai(a)suse.de> --- include/seq.h | 2 ++ include/sound/uapi/asequencer.h | 2 ++ src/Versions.in.in | 2 ++ src/seq/seq.c | 29 +++++++++++++++++++++++++++++ 4 files changed, 35 insertions(+) diff --git a/include/seq.h b/include/seq.h index 4f03df438288..2eee95a643b4 100644 --- a/include/seq.h +++ b/include/seq.h @@ -318,6 +318,7 @@ int snd_seq_port_info_get_timestamp_real(const snd_seq_port_info_t *info); int snd_seq_port_info_get_timestamp_queue(const snd_seq_port_info_t *info); int snd_seq_port_info_get_direction(const snd_seq_port_info_t *info); int snd_seq_port_info_get_ump_group(const snd_seq_port_info_t *info); +int snd_seq_port_info_get_ump_is_midi1(const snd_seq_port_info_t *info); void snd_seq_port_info_set_client(snd_seq_port_info_t *info, int client); void snd_seq_port_info_set_port(snd_seq_port_info_t *info, int port); @@ -334,6 +335,7 @@ void snd_seq_port_info_set_timestamp_real(snd_seq_port_info_t *info, int realtim void snd_seq_port_info_set_timestamp_queue(snd_seq_port_info_t *info, int queue); void snd_seq_port_info_set_direction(snd_seq_port_info_t *info, int direction); void snd_seq_port_info_set_ump_group(snd_seq_port_info_t *info, int ump_group); +void snd_seq_port_info_set_ump_is_midi1(snd_seq_port_info_t *info, int is_midi1); int snd_seq_create_port(snd_seq_t *handle, snd_seq_port_info_t *info); int snd_seq_delete_port(snd_seq_t *handle, int port); diff --git a/include/sound/uapi/asequencer.h b/include/sound/uapi/asequencer.h index 923dfdddfc76..b3e9df5894af 100644 --- a/include/sound/uapi/asequencer.h +++ b/include/sound/uapi/asequencer.h @@ -477,6 +477,8 @@ struct snd_seq_remove_events { #define SNDRV_SEQ_PORT_FLG_TIMESTAMP (1<<1) #define SNDRV_SEQ_PORT_FLG_TIME_REAL (1<<2) +#define SNDRV_SEQ_PORT_FLG_IS_MIDI1 (1<<3) /* Keep MIDI 1.0 protocol */ + /* port direction */ #define SNDRV_SEQ_PORT_DIR_UNKNOWN 0 #define SNDRV_SEQ_PORT_DIR_INPUT 1 diff --git a/src/Versions.in.in b/src/Versions.in.in index 90849277c983..298e610c2099 100644 --- a/src/Versions.in.in +++ b/src/Versions.in.in @@ -209,6 +209,8 @@ ALSA_1.2.13 { @SYMBOL_PREFIX@snd_seq_queue_tempo_get_tempo_base; @SYMBOL_PREFIX@snd_seq_queue_tempo_set_tempo_base; @SYMBOL_PREFIX@snd_seq_has_tempo_base; + @SYMBOL_PREFIX@snd_seq_port_info_get_ump_is_midi1; + @SYMBOL_PREFIX@snd_seq_port_info_set_ump_is_midi1; #endif #ifdef HAVE_RAWMIDI_SYMS @SYMBOL_PREFIX@snd_ump_endpoint_info_clear; diff --git a/src/seq/seq.c b/src/seq/seq.c index d0ecedf0838f..347ff45544fa 100644 --- a/src/seq/seq.c +++ b/src/seq/seq.c @@ -2431,6 +2431,19 @@ int snd_seq_port_info_get_ump_group(const snd_seq_port_info_t *info) return info->ump_group; } +/** + * \brief Get the status of the optional MIDI 1.0 port in MIDI 2.0 UMP Endpoint + * \param info port_info container + * \return 1 if it's an optional MIDI 1.0 port in MIDI 2.0 UMP Endpoint + * + * \sa snd_seq_get_port_info(), snd_seq_port_info_set_ump_is_midi1() + */ +int snd_seq_port_info_get_ump_is_midi1(const snd_seq_port_info_t *info) +{ + assert(info); + return !!(info->flags & SNDRV_SEQ_PORT_FLG_IS_MIDI1); +} + /** * \brief Set the client id of a port_info container * \param info port_info container @@ -2635,6 +2648,22 @@ void snd_seq_port_info_set_ump_group(snd_seq_port_info_t *info, int ump_group) info->ump_group = ump_group; } +/** + * \brief Set the optional MIDI 1.0 port in MIDI 2.0 UMP Endpoint + * \param info port_info container + * \param is_midi1 non-zero for MIDI 1.0 port in MIDI 2.0 EP + * + * \sa snd_seq_get_port_info(), snd_seq_port_info_get_ump_is_midi1() + */ +void snd_seq_port_info_set_ump_is_midi1(snd_seq_port_info_t *info, int is_midi1) +{ + assert(info); + if (is_midi1) + info->flags |= SNDRV_SEQ_PORT_FLG_IS_MIDI1; + else + info->flags &= ~SNDRV_SEQ_PORT_FLG_IS_MIDI1; +} + /** * \brief create a sequencer port on the current client * \param seq sequencer handle -- 2.43.0

1 0

Re: [PATCH v11 07/10] mtd: spi-nor: Add stacked memories support in spi-nor
by Michael Walle 07 Aug '24

07 Aug '24

Hi, > Based on the inputs/suggestions from Tudor, i am planning to add a new > layer between the SPI-NOR and MTD layers to support stacked and parallel > configurations. This new layer will be part of the spi-nor and located in > mtd/spi-nor/ Will AMD submit to maintain this layer? What happens if the maintainer will leave AMD? TBH, personally, I don't like to maintain such a niche feature. I'd really like to see some use cases and performance reports for this, like actual boards (and no evaluation boards don't count). Why wouldn't someone just use an octal flash? And as already mentioned there is also mtdcat, which seems to duplicate some features? -michael

5 13

[PATCH v1] ALSA: ASoC/tas2781: fix wrong calibrated data order
by Shenghao Ding 07 Aug '24

07 Aug '24

From: Baojun Xu <baojun.xu(a)ti.com> Wrong calibration data order cause sound too low in some device. Fix wrong calibrated data order, add calibration data converssion by get_unaligned_be32() after reading from UEFI. Fixes: 5be27f1e3ec9 ("ALSA: hda/tas2781: Add tas2781 HDA driver") Signed-off-by: Baojun Xu <baojun.xu(a)ti.com> --- v1: - Change copyright date, and add new maintainer. - Add unaligned.h included for get_unaligned_be32(). - In tas2781_apply_calib(), change data address transfer directly to get data by get_unaligned_be32(), and send address to device. --- sound/pci/hda/tas2781_hda_i2c.c | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/sound/pci/hda/tas2781_hda_i2c.c b/sound/pci/hda/tas2781_hda_i2c.c index 49bd7097d..4dc3350d5 100644 --- a/sound/pci/hda/tas2781_hda_i2c.c +++ b/sound/pci/hda/tas2781_hda_i2c.c @@ -2,10 +2,12 @@ // // TAS2781 HDA I2C driver // -// Copyright 2023 Texas Instruments, Inc. +// Copyright 2023 - 2024 Texas Instruments, Inc. // // Author: Shenghao Ding <shenghao-ding(a)ti.com> +// Current maintainer: Baojun Xu <baojun.xu(a)ti.com> +#include <asm/unaligned.h> #include <linux/acpi.h> #include <linux/crc8.h> #include <linux/crc32.h> @@ -519,20 +521,21 @@ static void tas2781_apply_calib(struct tasdevice_priv *tas_priv) static const unsigned char rgno_array[CALIB_MAX] = { 0x74, 0x0c, 0x14, 0x70, 0x7c, }; - unsigned char *data; - int i, j, rc; + int i, j, rc, data; + int offset = 0; for (i = 0; i < tas_priv->ndev; i++) { - data = tas_priv->cali_data.data + - i * TASDEVICE_SPEAKER_CALIBRATION_SIZE; for (j = 0; j < CALIB_MAX; j++) { + data = get_unaligned_be32( + &tas_priv->cali_data.data[offset]); rc = tasdevice_dev_bulk_write(tas_priv, i, TASDEVICE_REG(0, page_array[j], rgno_array[j]), - &(data[4 * j]), 4); + (unsigned char *)&data, 4); if (rc < 0) dev_err(tas_priv->dev, "chn %d calib %d bulk_wr err = %d\n", i, j, rc); + offset += 4; } } } -- 2.40.1

2 1

[PATCH] ALSA: hda/realtek: Add Framework Laptop 13 (Intel Core Ultra) to quirks
by Dustin L. Howett 07 Aug '24

07 Aug '24

The Framework Laptop 13 (Intel Core Ultra) has an ALC285 that ships in a similar configuration to the ALC295 in previous models. It requires the same quirk for headset detection. --- Signed-off-by: Dustin L. Howett <dustin(a)howett.net> --- sound/pci/hda/patch_realtek.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index 1645d21d422f..480e82df7a4c 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -10678,6 +10678,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = { SND_PCI_QUIRK(0x8086, 0x3038, "Intel NUC 13", ALC295_FIXUP_CHROME_BOOK), SND_PCI_QUIRK(0xf111, 0x0001, "Framework Laptop", ALC295_FIXUP_FRAMEWORK_LAPTOP_MIC_NO_PRESENCE), SND_PCI_QUIRK(0xf111, 0x0006, "Framework Laptop", ALC295_FIXUP_FRAMEWORK_LAPTOP_MIC_NO_PRESENCE), + SND_PCI_QUIRK(0xf111, 0x0009, "Framework Laptop", ALC295_FIXUP_FRAMEWORK_LAPTOP_MIC_NO_PRESENCE), #if 0 /* Below is a quirk table taken from the old code. --- base-commit: de9c2c66ad8e787abec7c9d7eff4f8c3cdd28aed change-id: 20240606-alsa-hda-realtek-add-framework-laptop-13-intel-core-ultra-to-quirks-2af5081e2f2c Best regards, -- Dustin L. Howett <dustin(a)howett.net>

2 1