July 2025 - Alsa-devel - mailman.alsa-project.org

[tiwai-sound:test/hda-reorg 17/42] sound/soc/intel/atom/sst/sst_acpi.c:272: undefined reference to `snd_intel_acpi_dsp_driver_probe'
by kernel test robot 08 Jul '25

08 Jul '25

tree: https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git test/hda-reorg head: be6979113a769a90bab2a55a30ae83b76f5971e6 commit: d1bc64a913342041aaacdde0c0920eb0d5b96281 [17/42] ALSA: hda: Move HD-audio core stuff into sound/hda/core config: i386-randconfig-063-20250522 (https://download.01.org/0day-ci/archive/20250708/202507081351.iyDgGOUP-lkp@…) compiler: gcc-12 (Debian 12.2.0-14+deb12u1) 12.2.0 reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250708/202507081351.iyDgGOUP-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/202507081351.iyDgGOUP-lkp@intel.com/ All errors (new ones prefixed by >>): ld: sound/soc/intel/atom/sst/sst_acpi.o: in function `sst_acpi_probe': >> sound/soc/intel/atom/sst/sst_acpi.c:272: undefined reference to `snd_intel_acpi_dsp_driver_probe' vim +272 sound/soc/intel/atom/sst/sst_acpi.c 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 255 5129ad6e4e6999 sound/soc/intel/sst/sst_acpi.c Wei Yongjun 2014-12-09 256 static int sst_acpi_probe(struct platform_device *pdev) 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 257 { 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 258 struct device *dev = &pdev->dev; 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 259 int ret = 0; 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 260 struct intel_sst_drv *ctx; 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 261 const struct acpi_device_id *id; 7feb2f786a46d3 sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2017-10-12 262 struct snd_soc_acpi_mach *mach; 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 263 struct platform_device *mdev; 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 264 struct platform_device *plat_dev; 12cc291b0b5850 sound/soc/intel/atom/sst/sst_acpi.c Vinod Koul 2015-11-05 265 struct sst_platform_info *pdata; 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 266 unsigned int dev_id; 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 267 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 268 id = acpi_match_device(dev->driver->acpi_match_table, dev); 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 269 if (!id) 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 270 return -ENODEV; df5f5edaef4b65 sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2020-11-12 271 df5f5edaef4b65 sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2020-11-12 @272 ret = snd_intel_acpi_dsp_driver_probe(dev, id->id); df5f5edaef4b65 sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2020-11-12 273 if (ret != SND_INTEL_DSP_DRIVER_ANY && ret != SND_INTEL_DSP_DRIVER_SST) { df5f5edaef4b65 sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2020-11-12 274 dev_dbg(dev, "SST ACPI driver not selected, aborting probe\n"); df5f5edaef4b65 sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2020-11-12 275 return -ENODEV; df5f5edaef4b65 sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2020-11-12 276 } df5f5edaef4b65 sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2020-11-12 277 1635c694124fe6 sound/soc/intel/atom/sst/sst_acpi.c Colin Ian King 2016-09-16 278 dev_dbg(dev, "for %s\n", id->id); 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 279 7feb2f786a46d3 sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2017-10-12 280 mach = (struct snd_soc_acpi_mach *)id->driver_data; 7feb2f786a46d3 sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2017-10-12 281 mach = snd_soc_acpi_find_machine(mach); 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 282 if (mach == NULL) { 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 283 dev_err(dev, "No matching machine driver found\n"); 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 284 return -ENODEV; 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 285 } 73a33f6f6d44db sound/soc/intel/atom/sst/sst_acpi.c Vinod Koul 2016-07-08 286 536cfd2f375d36 sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2019-05-30 287 if (soc_intel_is_byt()) 6004640638a2cb sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2017-10-12 288 mach->pdata = &byt_rvp_platform_data; 6004640638a2cb sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2017-10-12 289 else 6004640638a2cb sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2017-10-12 290 mach->pdata = &chv_platform_data; 12cc291b0b5850 sound/soc/intel/atom/sst/sst_acpi.c Vinod Koul 2015-11-05 291 pdata = mach->pdata; 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 292 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 293 if (!strcmp(id->id, "LPE0F28")) { 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 294 struct resource *rsrc; 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 295 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 296 /* Use regular BYT SST PCI VID:PID */ 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 297 dev_id = 0x80860F28; 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 298 byt_rvp_platform_data.res_info = &lpe8086_res_info; 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 299 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 300 /* 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 301 * The "LPE0F28" ACPI device has separate IO-mem resources for: 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 302 * DDR, SHIM, MBOX, IRAM, DRAM, CFG 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 303 * None of which covers the entire LPE base address range. 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 304 * lpe8086_res_info.acpi_lpe_res_index points to the SHIM. 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 305 * Patch this to cover the entire base address range as expected 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 306 * by sst_platform_get_resources(). 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 307 */ 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 308 rsrc = platform_get_resource(pdev, IORESOURCE_MEM, 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 309 pdata->res_info->acpi_lpe_res_index); 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 310 if (!rsrc) { c1895ba181e560 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-26 311 dev_err(dev, "Invalid SHIM base\n"); 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 312 return -EIO; 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 313 } 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 314 rsrc->start -= pdata->res_info->shim_offset; 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 315 rsrc->end = rsrc->start + 0x200000 - 1; 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 316 } else { 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 317 ret = kstrtouint(id->id, 16, &dev_id); 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 318 if (ret < 0) { 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 319 dev_err(dev, "Unique device id conversion error: %d\n", ret); 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 320 return ret; 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 321 } 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 322 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 323 if (soc_intel_is_byt_cr(pdev)) 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 324 byt_rvp_platform_data.res_info = &bytcr_res_info; 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 325 } 6668610b4d8ce9 sound/soc/intel/atom/sst/sst_acpi.c Hans de Goede 2024-10-25 326 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 327 dev_dbg(dev, "ACPI device id: %x\n", dev_id); 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 328 a68bc0d43e1b96 sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2016-08-12 329 ret = sst_alloc_drv_context(&ctx, dev, dev_id); a68bc0d43e1b96 sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2016-08-12 330 if (ret < 0) a68bc0d43e1b96 sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2016-08-12 331 return ret; a68bc0d43e1b96 sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2016-08-12 332 3ee1cd4f81e15f sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2018-11-01 333 /* update machine parameters */ 3ee1cd4f81e15f sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2018-11-01 334 mach->mach_params.acpi_ipc_irq_index = 3ee1cd4f81e15f sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2018-11-01 335 pdata->res_info->acpi_ipc_irq_index; 3ee1cd4f81e15f sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2018-11-01 336 caf94ed8629afb sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2016-01-04 337 plat_dev = platform_device_register_data(dev, pdata->platform, -1, caf94ed8629afb sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2016-01-04 338 NULL, 0); f5d40b400fe2de sound/soc/intel/sst/sst_acpi.c Wei Yongjun 2014-12-09 339 if (IS_ERR(plat_dev)) { caf94ed8629afb sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2016-01-04 340 dev_err(dev, "Failed to create machine device: %s\n", caf94ed8629afb sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2016-01-04 341 pdata->platform); f5d40b400fe2de sound/soc/intel/sst/sst_acpi.c Wei Yongjun 2014-12-09 342 return PTR_ERR(plat_dev); 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 343 } 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 344 caf94ed8629afb sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2016-01-04 345 /* caf94ed8629afb sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2016-01-04 346 * Create platform device for sst machine driver, caf94ed8629afb sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2016-01-04 347 * pass machine info as pdata caf94ed8629afb sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2016-01-04 348 */ caf94ed8629afb sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2016-01-04 349 mdev = platform_device_register_data(dev, mach->drv_name, -1, caf94ed8629afb sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2016-01-04 350 (const void *)mach, sizeof(*mach)); f5d40b400fe2de sound/soc/intel/sst/sst_acpi.c Wei Yongjun 2014-12-09 351 if (IS_ERR(mdev)) { caf94ed8629afb sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2016-01-04 352 dev_err(dev, "Failed to create machine device: %s\n", caf94ed8629afb sound/soc/intel/atom/sst/sst_acpi.c Pierre-Louis Bossart 2016-01-04 353 mach->drv_name); f5d40b400fe2de sound/soc/intel/sst/sst_acpi.c Wei Yongjun 2014-12-09 354 return PTR_ERR(mdev); 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 355 } 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 356 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 357 /* Fill sst platform data */ 12cc291b0b5850 sound/soc/intel/atom/sst/sst_acpi.c Vinod Koul 2015-11-05 358 ctx->pdata = pdata; 12cc291b0b5850 sound/soc/intel/atom/sst/sst_acpi.c Vinod Koul 2015-11-05 359 strcpy(ctx->firmware_name, mach->fw_filename); 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 360 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 361 ret = sst_platform_get_resources(ctx); 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 362 if (ret) 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 363 return ret; 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 364 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 365 ret = sst_context_init(ctx); 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 366 if (ret < 0) 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 367 return ret; 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 368 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 369 sst_configure_runtime_pm(ctx); 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 370 platform_set_drvdata(pdev, ctx); 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 371 return ret; 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 372 } 336cfbb05edf7b sound/soc/intel/sst/sst_acpi.c Vinod Koul 2014-11-11 373 :::::: The code at line 272 was first introduced by commit :::::: df5f5edaef4b653fa731dcf3753e71766f95c2cd ASoC: Intel: Atom: add dynamic selection of DSP driver :::::: TO: Pierre-Louis Bossart <pierre-louis.bossart(a)linux.intel.com> :::::: CC: Mark Brown <broonie(a)kernel.org> -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki

1 0

[PATCH v2] ALSA: hda/tas2781: Fix calibration data parser issue
by Baojun Xu 07 Jul '25

07 Jul '25

We will copy calibration data from position behind to front. We has created a variable (tmp_val) point on top of calibration data buffer, and tmp_val[1] is max of node number in original calibration data structure, it will be overwritten after first data copy, so can't be used as max node number check in for loop. So we create a new variable to save max of node number (tmp_val[1]), used to check if max node number was reached in for loop. And a point need to be increased to point at calibration data in node. Data saved position also need to be increased one byte. Fixes: 4fe238513407 ("ALSA: hda/tas2781: Move and unified the calibrated-data getting function for SPI and I2C into the tas2781_hda lib") Signed-off-by: Baojun Xu <baojun.xu(a)ti.com> --- Change in v2: - Add more description about this fix. --- sound/pci/hda/tas2781_hda.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/sound/pci/hda/tas2781_hda.c b/sound/pci/hda/tas2781_hda.c index 5f1d4b3e9688..34217ce9f28e 100644 --- a/sound/pci/hda/tas2781_hda.c +++ b/sound/pci/hda/tas2781_hda.c @@ -44,7 +44,7 @@ static void tas2781_apply_calib(struct tasdevice_priv *p) TASDEVICE_REG(0, 0x13, 0x70), TASDEVICE_REG(0, 0x18, 0x7c), }; - unsigned int crc, oft; + unsigned int crc, oft, node_num; unsigned char *buf; int i, j, k, l; @@ -80,8 +80,9 @@ static void tas2781_apply_calib(struct tasdevice_priv *p) dev_err(p->dev, "%s: CRC error\n", __func__); return; } + node_num = tmp_val[1]; - for (j = 0, k = 0; j < tmp_val[1]; j++) { + for (j = 0, k = 0; j < node_num; j++) { oft = j * 6 + 3; if (tmp_val[oft] == TASDEV_UEFI_CALI_REG_ADDR_FLG) { for (i = 0; i < TASDEV_CALIB_N; i++) { @@ -99,8 +100,9 @@ static void tas2781_apply_calib(struct tasdevice_priv *p) } data[l] = k; + oft++; for (i = 0; i < TASDEV_CALIB_N * 4; i++) - data[l + i] = data[4 * oft + i]; + data[l + i + 1] = data[4 * oft + i]; k++; } } -- 2.43.0

1 0

[PATCH v1] ALSA: hda/tas2781: Add bus name in device name check
by Baojun Xu 06 Jul '25

06 Jul '25

Device name start from bus name, as we use strstarts() to do compare, need add it for TXNW2781 device. Fixes: b2904df0a347 ("ALSA: hda/tas2781: Add compatible for hardware id TIAS2781 and TXNW2781") Signed-off-by: Baojun Xu <baojun.xu(a)ti.com> --- sound/pci/hda/tas2781_hda_i2c.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/pci/hda/tas2781_hda_i2c.c b/sound/pci/hda/tas2781_hda_i2c.c index b9cdbca951e4..530c2266ab3b 100644 --- a/sound/pci/hda/tas2781_hda_i2c.c +++ b/sound/pci/hda/tas2781_hda_i2c.c @@ -588,7 +588,7 @@ static int tas2781_hda_i2c_probe(struct i2c_client *clt) hda_priv->save_calibration = tas2781_save_calibration; tas_hda->priv->global_addr = TAS2781_GLOBAL_ADDR; } else if (strstarts(dev_name(&clt->dev), - "TXNW2781:00-tas2781-hda.0")) { + "i2c-TXNW2781:00-tas2781-hda.0")) { device_name = "TXNW2781"; hda_priv->save_calibration = tas2781_save_calibration; tas_hda->priv->global_addr = TAS2781_GLOBAL_ADDR; -- 2.43.0

1 1

[PATCH v1] ALSA: hda/tas2781: Fix calibration data parser issue
by Baojun Xu 06 Jul '25

06 Jul '25

Calibration data was overwritten during parser, it cause issue. Fixes: 4fe238513407 ("ALSA: hda/tas2781: Move and unified the calibrated-data getting function for SPI and I2C into the tas2781_hda lib") Signed-off-by: Baojun Xu <baojun.xu(a)ti.com> --- sound/pci/hda/tas2781_hda.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/sound/pci/hda/tas2781_hda.c b/sound/pci/hda/tas2781_hda.c index 5f1d4b3e9688..34217ce9f28e 100644 --- a/sound/pci/hda/tas2781_hda.c +++ b/sound/pci/hda/tas2781_hda.c @@ -44,7 +44,7 @@ static void tas2781_apply_calib(struct tasdevice_priv *p) TASDEVICE_REG(0, 0x13, 0x70), TASDEVICE_REG(0, 0x18, 0x7c), }; - unsigned int crc, oft; + unsigned int crc, oft, node_num; unsigned char *buf; int i, j, k, l; @@ -80,8 +80,9 @@ static void tas2781_apply_calib(struct tasdevice_priv *p) dev_err(p->dev, "%s: CRC error\n", __func__); return; } + node_num = tmp_val[1]; - for (j = 0, k = 0; j < tmp_val[1]; j++) { + for (j = 0, k = 0; j < node_num; j++) { oft = j * 6 + 3; if (tmp_val[oft] == TASDEV_UEFI_CALI_REG_ADDR_FLG) { for (i = 0; i < TASDEV_CALIB_N; i++) { @@ -99,8 +100,9 @@ static void tas2781_apply_calib(struct tasdevice_priv *p) } data[l] = k; + oft++; for (i = 0; i < TASDEV_CALIB_N * 4; i++) - data[l + i] = data[4 * oft + i]; + data[l + i + 1] = data[4 * oft + i]; k++; } } -- 2.43.0

1 0

test: update midifile library to ANSI C
by GitHub pull_request - opened 06 Jul '25

06 Jul '25

alsa-project/alsa-lib pull request #463 was opened from atsampson: The `midifile` library used by the `playmidi1` program dates from 1989, and used pre-ANSI function definitions and prototypes. GCC 15 now defaults to C23 where `()` means the same as `(void)` in prototypes, which causes type mismatch errors. Update the code to use ANSI function definitions and prototypes, so it'll compile happily as anything from ANSI C to C23. This revealed that `playmidi1`'s `do_tempo` had the wrong argument type, so correct that as well. Fixes #452. Request URL : https://github.com/alsa-project/alsa-lib/pull/463 Patch URL : https://github.com/alsa-project/alsa-lib/pull/463.patch Repository URL: https://github.com/alsa-project/alsa-lib

1 0

[tiwai-sound:test/hda-reorg 41/53] sound/hda/codecs/cirrus/cs421x.c:561:35: warning: 'cs421x_codec_ops' defined but not used
by kernel test robot 05 Jul '25

05 Jul '25

tree: https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git test/hda-reorg head: 6119728edf86cefcbf7b20e000414c85916e900d commit: 484072d3f99c402030fcee94e28ab3ea76d04247 [41/53] ALSA: hda/cirrus: Split to cs420x and cs421x drivers config: sh-allmodconfig (https://download.01.org/0day-ci/archive/20250705/202507051737.DMAkxayN-lkp@…) compiler: sh4-linux-gcc (GCC) 15.1.0 reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250705/202507051737.DMAkxayN-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/202507051737.DMAkxayN-lkp@intel.com/ All warnings (new ones prefixed by >>): >> sound/hda/codecs/cirrus/cs421x.c:561:35: warning: 'cs421x_codec_ops' defined but not used [-Wunused-const-variable=] 561 | static const struct hda_codec_ops cs421x_codec_ops = { | ^~~~~~~~~~~~~~~~ vim +/cs421x_codec_ops +561 sound/hda/codecs/cirrus/cs421x.c 560 > 561 static const struct hda_codec_ops cs421x_codec_ops = { 562 .probe = cs421x_probe, 563 .remove = snd_hda_gen_remove, 564 .build_controls = snd_hda_gen_build_controls, 565 .build_pcms = snd_hda_gen_build_pcms, 566 .init = cs421x_init, 567 .unsol_event = snd_hda_jack_unsol_event, 568 .suspend = cs421x_suspend, 569 .stream_pm = snd_hda_gen_stream_pm, 570 }; 571 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki

1 0

[tiwai-sound:test/hda-reorg 29/53] sound/hda/common/hwdep.c:26:6: warning: label followed by a declaration is a C23 extension
by kernel test robot 05 Jul '25

05 Jul '25

tree: https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git test/hda-reorg head: 6119728edf86cefcbf7b20e000414c85916e900d commit: 5b965be899c8e3286f5111b7e90207afe5124b67 [29/53] ALSA: hda: Move common codec driver into sound/hda/common directory config: riscv-randconfig-001-20250705 (https://download.01.org/0day-ci/archive/20250705/202507051454.VintnWOF-lkp@…) compiler: clang version 21.0.0git (https://github.com/llvm/llvm-project 61529d9e36fa86782a2458e6bdeedf7f376ef4b5) reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250705/202507051454.VintnWOF-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/202507051454.VintnWOF-lkp@intel.com/ All warnings (new ones prefixed by >>): In file included from sound/hda/common/hwdep.c:10: In file included from include/linux/compat.h:17: include/linux/fs.h:3979:15: warning: label followed by a declaration is a C23 extension [-Wc23-extensions] 3979 | if (unlikely(get_user(c, path))) | ^ arch/riscv/include/asm/uaccess.h:274:3: note: expanded from macro 'get_user' 274 | __get_user((x), __p) : \ | ^ arch/riscv/include/asm/uaccess.h:244:2: note: expanded from macro '__get_user' 244 | __get_user_error(__gu_val, __gu_ptr, __gu_err); \ | ^ arch/riscv/include/asm/uaccess.h:207:2: note: expanded from macro '__get_user_error' 207 | __get_user_nocheck(x, ptr, __gu_failed); \ | ^ arch/riscv/include/asm/uaccess.h:196:3: note: expanded from macro '__get_user_nocheck' 196 | __get_user_8((x), __gu_ptr, label); \ | ^ arch/riscv/include/asm/uaccess.h:130:2: note: expanded from macro '__get_user_8' 130 | u32 __user *__ptr = (u32 __user *)(ptr); \ | ^ >> sound/hda/common/hwdep.c:26:6: warning: label followed by a declaration is a C23 extension [-Wc23-extensions] 26 | if (get_user(verb, &arg->verb)) | ^ arch/riscv/include/asm/uaccess.h:274:3: note: expanded from macro 'get_user' 274 | __get_user((x), __p) : \ | ^ arch/riscv/include/asm/uaccess.h:244:2: note: expanded from macro '__get_user' 244 | __get_user_error(__gu_val, __gu_ptr, __gu_err); \ | ^ arch/riscv/include/asm/uaccess.h:207:2: note: expanded from macro '__get_user_error' 207 | __get_user_nocheck(x, ptr, __gu_failed); \ | ^ arch/riscv/include/asm/uaccess.h:196:3: note: expanded from macro '__get_user_nocheck' 196 | __get_user_8((x), __gu_ptr, label); \ | ^ arch/riscv/include/asm/uaccess.h:130:2: note: expanded from macro '__get_user_8' 130 | u32 __user *__ptr = (u32 __user *)(ptr); \ | ^ sound/hda/common/hwdep.c:40:6: warning: label followed by a declaration is a C23 extension [-Wc23-extensions] 40 | if (get_user(verb, &arg->verb)) | ^ arch/riscv/include/asm/uaccess.h:274:3: note: expanded from macro 'get_user' 274 | __get_user((x), __p) : \ | ^ arch/riscv/include/asm/uaccess.h:244:2: note: expanded from macro '__get_user' 244 | __get_user_error(__gu_val, __gu_ptr, __gu_err); \ | ^ arch/riscv/include/asm/uaccess.h:207:2: note: expanded from macro '__get_user_error' 207 | __get_user_nocheck(x, ptr, __gu_failed); \ | ^ arch/riscv/include/asm/uaccess.h:196:3: note: expanded from macro '__get_user_nocheck' 196 | __get_user_8((x), __gu_ptr, label); \ | ^ arch/riscv/include/asm/uaccess.h:130:2: note: expanded from macro '__get_user_8' 130 | u32 __user *__ptr = (u32 __user *)(ptr); \ | ^ 3 warnings generated. vim +26 sound/hda/common/hwdep.c 2807314d467e7d sound/pci/hda/hda_hwdep.c Takashi Iwai 2007-07-27 17 2807314d467e7d sound/pci/hda/hda_hwdep.c Takashi Iwai 2007-07-27 18 /* 2807314d467e7d sound/pci/hda/hda_hwdep.c Takashi Iwai 2007-07-27 19 * write/read an out-of-bound verb 2807314d467e7d sound/pci/hda/hda_hwdep.c Takashi Iwai 2007-07-27 20 */ 2807314d467e7d sound/pci/hda/hda_hwdep.c Takashi Iwai 2007-07-27 21 static int verb_write_ioctl(struct hda_codec *codec, 2807314d467e7d sound/pci/hda/hda_hwdep.c Takashi Iwai 2007-07-27 22 struct hda_verb_ioctl __user *arg) 2807314d467e7d sound/pci/hda/hda_hwdep.c Takashi Iwai 2007-07-27 23 { 2807314d467e7d sound/pci/hda/hda_hwdep.c Takashi Iwai 2007-07-27 24 u32 verb, res; 2807314d467e7d sound/pci/hda/hda_hwdep.c Takashi Iwai 2007-07-27 25 2807314d467e7d sound/pci/hda/hda_hwdep.c Takashi Iwai 2007-07-27 @26 if (get_user(verb, &arg->verb)) 2807314d467e7d sound/pci/hda/hda_hwdep.c Takashi Iwai 2007-07-27 27 return -EFAULT; 2807314d467e7d sound/pci/hda/hda_hwdep.c Takashi Iwai 2007-07-27 28 res = snd_hda_codec_read(codec, verb >> 24, 0, 2807314d467e7d sound/pci/hda/hda_hwdep.c Takashi Iwai 2007-07-27 29 (verb >> 8) & 0xffff, verb & 0xff); 2807314d467e7d sound/pci/hda/hda_hwdep.c Takashi Iwai 2007-07-27 30 if (put_user(res, &arg->res)) 2807314d467e7d sound/pci/hda/hda_hwdep.c Takashi Iwai 2007-07-27 31 return -EFAULT; 2807314d467e7d sound/pci/hda/hda_hwdep.c Takashi Iwai 2007-07-27 32 return 0; 2807314d467e7d sound/pci/hda/hda_hwdep.c Takashi Iwai 2007-07-27 33 } 2807314d467e7d sound/pci/hda/hda_hwdep.c Takashi Iwai 2007-07-27 34 :::::: The code at line 26 was first introduced by commit :::::: 2807314d467e7dd929c42050031aabbd28e78f0b [ALSA] hda-intel - Add hwdep interface :::::: TO: Takashi Iwai <tiwai(a)suse.de> :::::: CC: Jaroslav Kysela <perex(a)perex.cz> -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki

1 0

[GIT PULL] ASoC fixes for v6.16-rc4
by Mark Brown 04 Jul '25

04 Jul '25

The following changes since commit d0b3b7b22dfa1f4b515fd3a295b3fd958f9e81af: Linux 6.16-rc4 (2025-06-29 13:09:04 -0700) are available in the Git repository at: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git tags/asoc-fix-v6.16-rc4 for you to fetch changes up to 3b3312f28ee2d9c386602f8521e419cfc69f4823: ASoC: cs35l56: probe() should fail if the device ID is not recognized (2025-07-03 12:22:27 +0100) ---------------------------------------------------------------- ASoC: Fixes for v6.16 An update for the MAINTAINERS file, plus a number of small driver specific fixes and device quirks. ---------------------------------------------------------------- Arun Raghavan (1): ASoC: fsl_sai: Force a software reset when starting in consumer mode Bard Liao (1): ASoC: Intel: SND_SOC_INTEL_SOF_BOARD_HELPERS select SND_SOC_ACPI_INTEL_MATCH Charles Keepax (1): ASoC: Intel: soc-acpi: arl: Correct order of cs42l43 matches Richard Fitzgerald (1): ASoC: cs35l56: probe() should fail if the device ID is not recognized Shengjiu Wang (1): ASoC: fsl_asrc: use internal measured ratio for non-ideal ratio mode Simon Trimmer (1): ASoC: Intel: sof_sdw: Add quirks for Lenovo P1 and P16 Srinivas Kandagatla (1): MAINTAINERS: update Qualcomm audio codec drivers list MAINTAINERS | 16 +++++----------- sound/soc/codecs/cs35l56-shared.c | 2 +- sound/soc/fsl/fsl_asrc.c | 3 ++- sound/soc/fsl/fsl_sai.c | 14 ++++++++------ sound/soc/intel/boards/Kconfig | 1 + sound/soc/intel/boards/sof_sdw.c | 3 +++ sound/soc/intel/common/soc-acpi-intel-arl-match.c | 12 ++++++------ 7 files changed, 26 insertions(+), 25 deletions(-)

2 1

[BUG] Mackie DLZ Creator XS – USB Audio: write error -77 after accepted hw_params
by Brian Knutsson 04 Jul '25

04 Jul '25

Hi ALSA devs, I'm reporting an issue with the Mackie DLZ Creator XS USB audio device on Linux (tested on Ubuntu 24.04 and 25.05 with kernel 6.8+). The device is detected correctly via ALSA (card 2: XS), but as soon as playback is initiated – whether via `aplay` or `speaker-test` – it disconnects with `write error: -77, File descriptor in bad state`. Relevant kernel logs show: usb 3-7: Product: DLZ Creator XS usb 3-7: Manufacturer: LOUD Audio, LLC usb 3-7: USB disconnect, device number X This occurs with and without PipeWire/udev running. I’ve created a full ALSA log with `alsa-info` and opened a Kernel Bugzilla entry: <https://bugzilla.kernel.org/show_bug.cgi?id=220143> Is there any known quirk or patch needed for this hardware? Or a way to assist debugging further (usbmon trace etc.)? Thanks in advance, Brian Knutsson

2 1

[PATCH v1] ALSA: hda/tas2781: Add bus name in device name check
by Baojun Xu 03 Jul '25

03 Jul '25

Device name start from bus name, as we use strstarts() to do compare, need add it for TXNW2781 device. Signed-off-by: Baojun Xu <baojun.xu(a)ti.com> --- sound/pci/hda/tas2781_hda_i2c.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/pci/hda/tas2781_hda_i2c.c b/sound/pci/hda/tas2781_hda_i2c.c index b9cdbca95..530c2266a 100644 --- a/sound/pci/hda/tas2781_hda_i2c.c +++ b/sound/pci/hda/tas2781_hda_i2c.c @@ -588,7 +588,7 @@ static int tas2781_hda_i2c_probe(struct i2c_client *clt) hda_priv->save_calibration = tas2781_save_calibration; tas_hda->priv->global_addr = TAS2781_GLOBAL_ADDR; } else if (strstarts(dev_name(&clt->dev), - "TXNW2781:00-tas2781-hda.0")) { + "i2c-TXNW2781:00-tas2781-hda.0")) { device_name = "TXNW2781"; hda_priv->save_calibration = tas2781_save_calibration; tas_hda->priv->global_addr = TAS2781_GLOBAL_ADDR; -- 2.43.0

1 0

[PATCH 0/3] Add legacy driver support for ACP7.2 based
by Venkata Prasad Potturu 02 Jul '25

02 Jul '25

This patch series is to add I2S and PDM support for ACP7.2 based platforms. Venkata Prasad Potturu (3): ASoC: amd: acp: Add legacy driver support acp7.2 based platforms ASoC: amd: acp: Enable I2S support for acp7.2 based platforms ASoC: amd: acp: Enable acp7.2 platform based DMIC support in machine driver sound/soc/amd/acp/acp-i2s.c | 3 +++ sound/soc/amd/acp/acp-legacy-common.c | 2 ++ sound/soc/amd/acp/acp-mach-common.c | 1 + sound/soc/amd/acp/acp-pci.c | 1 + sound/soc/amd/acp/acp-platform.c | 2 ++ sound/soc/amd/acp/acp70.c | 1 + sound/soc/amd/acp/acp_common.h | 1 + 7 files changed, 11 insertions(+) -- 2.43.0

2 4

New Card: ESI Juli@ eX
by GitHub issues - opened 01 Jul '25

01 Jul '25

alsa-project/alsa-lib issue #462 was opened from Carlisle96: I am not sure where to ask or how to report this: There was a new card released ESI-Audio Juli@ eX https://www.esi-audio.com/products/juliaex/ I am wondering if this is supported or planned to be supported. It is not yet listed here: https://www.alsa-project.org/wiki/Matrix:Vendor-ESI Issue URL : https://github.com/alsa-project/alsa-lib/issues/462 Repository URL: https://github.com/alsa-project/alsa-lib

1 0

Request for help: Outputting audio as an I2S stream
by Steve Broshar 01 Jul '25

01 Jul '25

Summary: My goal is to output audio to an ADAU1467 via I2S on Linux 6.6. I tried and then abandoned using a custom, device-specific driver. Now trying to use built-in simple-sound-card. But, driver does not initialize (probe). Don't know how to debug/fix the issue. I've been working with chapgpt extensively since I have found no other source of information on how to setup linux to output an I2S audio stream. In fact, I posted a question to a linux maillist about an issue a couple weeks ago and the reply from Lars Clausen showed output from chatgpt. In the old days folks would joke about googling something for someone. Today, I guess we can do the same WRT chatgpt. But, that question was about updating a driver. Chatgpt tells me that I shouldn't need a device-specific driver since the device is configured and controlled elsewhere. I just need to pump an audio stream into it. So, I started over with my solution; no longer working on a custom driver. Apparently, I can use the simple-sound-card and a dummy codec. But, it does not work. Maybe chatgpt is wrong and I need a device-specific driver. Or maybe my setup is wrong. I have worked with chatgpt to resolve my issues, but have hit a wall. Hoping to get help from humans now. 😊 The boot-time log (attached) shows nothing about my audio setup until the last line: platform sound-i2s: deferred probe pending I think that ‘deferred probe’ means that an initial probe failed. How can I tell what failed? What prevented it from succeeding? Would be nice if the driver told me what failed. But, there’s no error message, just an info msg that probe is pending. Here's the device tree setup that I added (based on input from chatgpt): / { // Compton audio output can be accomplished via a built-in driver -- simple-audio-card sound_i2s: sound-i2s { // specify the built-in simple sound card compatible = "simple-audio-card"; // seems to be a descriptive name that shows in userland; i.e. via 'aplay -l' simple-audio-card,name = "imx8mn-adauslave"; // might specify I2S (as opposed to descriptive text) simple-audio-card,format = "i2s"; // defines CPU as master for bit-clock simple-audio-card,bitclock-master = <&sound_i2s_cpu>; // defines CPU as master for frame-clock simple-audio-card,frame-master = <&sound_i2s_cpu>; sound_i2s_cpu: simple-audio-card,cpu { sound-dai = <&sai3>; dai-tdm-slot-num = <2>; /* stereo */ dai-tdm-slot-width = <32>; /* bits per slot */ }; simple-audio-card,codec { sound-dai = <&dummy_codec>; }; }; // Compton audio output does not require a device-specific codec; the built-in, dummy driver is sufficient dummy_codec: dummy-codec { compatible = "linux,snd-soc-dummy"; #sound-dai-cells = <0>; }; Here's the device tree config that comes with the NXP IMX8M build for nano: pinctrl_sai3: sai3grp { fsl,pins = < MX8MN_IOMUXC_SAI3_TXFS_SAI3_TX_SYNC 0xd6 MX8MN_IOMUXC_SAI3_TXC_SAI3_TX_BCLK 0xd6 MX8MN_IOMUXC_SAI3_MCLK_SAI3_MCLK 0xd6 MX8MN_IOMUXC_SAI3_TXD_SAI3_TX_DATA0 0xd6 >; }; sai3: sai@30030000 { compatible = "fsl,imx8mn-sai", "fsl,imx8mq-sai"; reg = <0x30030000 0x10000>; #sound-dai-cells = <0>; interrupts = <GIC_SPI 50 IRQ_TYPE_LEVEL_HIGH>; clocks = <&clk IMX8MN_CLK_SAI3_IPG>, <&clk IMX8MN_CLK_DUMMY>, <&clk IMX8MN_CLK_SAI3_ROOT>, <&clk IMX8MN_CLK_DUMMY>, <&clk IMX8MN_CLK_DUMMY>; clock-names = "bus", "mclk0", "mclk1", "mclk2", "mclk3"; dmas = <&sdma2 4 2 0>, <&sdma2 5 2 0>; dma-names = "rx", "tx"; status = "disabled"; }; &sai3 { #sound-dai-cells = <0>; pinctrl-names = "default"; pinctrl-0 = <&pinctrl_sai3>; assigned-clocks = <&clk IMX8MN_CLK_SAI3>; assigned-clock-parents = <&clk IMX8MN_AUDIO_PLL1_OUT>; assigned-clock-rates = <24576000>; fsl,sai-mclk-direction-output; status = "okay"; }; In summary, how do I further debug this? In general, and if you know, how does one output an audio stream as I2S? Steve Broshar Palmer Wireless Medtech

2 1