Hi Liam,
FYI, the error/warning still remains.
tree: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-4.18 head: 221dd96c30a7c65b24ead7fdd7645abb99506ce2 commit: 81e9b0a078894841a50a8dd666fd64ca452a2a50 [48/51] ASoC: topology: Give more data to clients via callbacks config: x86_64-federa-25 (attached as .config) compiler: gcc-7 (Debian 7.3.0-16) 7.3.0 reproduce: git checkout 81e9b0a078894841a50a8dd666fd64ca452a2a50 # save the attached .config to linux build tree make ARCH=x86_64
All errors (new ones prefixed by >>):
sound/soc/intel/skylake/skl-pcm.c: In function 'skl_populate_modules':
sound/soc/intel/skylake/skl-pcm.c:1351:4: error: implicit declaration of function 'skl_tplg_add_moduleid_in_bind_params'; did you mean 'skl_tplg_update_pipe_params'? [-Werror=implicit-function-declaration]
skl_tplg_add_moduleid_in_bind_params(skl, w); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ skl_tplg_update_pipe_params sound/soc/intel/skylake/skl-pcm.c: In function 'skl_platform_soc_probe':
sound/soc/intel/skylake/skl-pcm.c:1372:23: error: passing argument 1 of 'skl_tplg_init' from incompatible pointer type [-Werror=incompatible-pointer-types]
ret = skl_tplg_init(component, ebus); ^~~~~~~~~ In file included from sound/soc/intel/skylake/skl-pcm.c:28:0: sound/soc/intel/skylake/skl-topology.h:463:5: note: expected 'struct snd_soc_platform *' but argument is of type 'struct snd_soc_component *' int skl_tplg_init(struct snd_soc_platform *platform, ^~~~~~~~~~~~~ cc1: some warnings being treated as errors -- sound/soc/intel/skylake/skl-topology.c: In function 'skl_fill_sink_instance_id':
sound/soc/intel/skylake/skl-topology.c:833:45: error: 'struct skl_kpb_params' has no member named 'u'
struct skl_mod_inst_map *inst = kpb_params->u.map; ^~ sound/soc/intel/skylake/skl-topology.c: In function 'skl_tplg_find_moduleid_from_uuid': sound/soc/intel/skylake/skl-topology.c:953:17: error: 'struct skl_kpb_params' has no member named 'u' &uuid_params->u.map_uuid[i].mod_uuid); ^~ sound/soc/intel/skylake/skl-topology.c:959:10: error: 'struct skl_kpb_params' has no member named 'u' params->u.map[i].mod_id = module_id; ^~ sound/soc/intel/skylake/skl-topology.c:960:10: error: 'struct skl_kpb_params' has no member named 'u' params->u.map[i].inst_id = ^~ sound/soc/intel/skylake/skl-topology.c:961:16: error: 'struct skl_kpb_params' has no member named 'u' uuid_params->u.map_uuid[i].inst_id; ^~ sound/soc/intel/skylake/skl-topology.c: At top level:
sound/soc/intel/skylake/skl-topology.c:3544:5: error: conflicting types for 'skl_tplg_init'
int skl_tplg_init(struct snd_soc_component *component, struct hdac_ext_bus *ebus) ^~~~~~~~~~~~~ In file included from sound/soc/intel/skylake/skl-topology.c:27:0: sound/soc/intel/skylake/skl-topology.h:463:5: note: previous declaration of 'skl_tplg_init' was here int skl_tplg_init(struct snd_soc_platform *platform, ^~~~~~~~~~~~~
vim +1351 sound/soc/intel/skylake/skl-pcm.c
b26199ea Jeeja KP 2017-03-24 1330 64cb1d0a Vinod Koul 2016-08-10 1331 static int skl_populate_modules(struct skl *skl) 64cb1d0a Vinod Koul 2016-08-10 1332 { 64cb1d0a Vinod Koul 2016-08-10 1333 struct skl_pipeline *p; 64cb1d0a Vinod Koul 2016-08-10 1334 struct skl_pipe_module *m; 64cb1d0a Vinod Koul 2016-08-10 1335 struct snd_soc_dapm_widget *w; 64cb1d0a Vinod Koul 2016-08-10 1336 struct skl_module_cfg *mconfig; b26199ea Jeeja KP 2017-03-24 1337 int ret = 0; 64cb1d0a Vinod Koul 2016-08-10 1338 64cb1d0a Vinod Koul 2016-08-10 1339 list_for_each_entry(p, &skl->ppl_list, node) { 64cb1d0a Vinod Koul 2016-08-10 1340 list_for_each_entry(m, &p->pipe->w_list, node) { 64cb1d0a Vinod Koul 2016-08-10 1341 w = m->w; 64cb1d0a Vinod Koul 2016-08-10 1342 mconfig = w->priv; 64cb1d0a Vinod Koul 2016-08-10 1343 b26199ea Jeeja KP 2017-03-24 1344 ret = skl_get_module_info(skl, mconfig); 64cb1d0a Vinod Koul 2016-08-10 1345 if (ret < 0) { 64cb1d0a Vinod Koul 2016-08-10 1346 dev_err(skl->skl_sst->dev, b26199ea Jeeja KP 2017-03-24 1347 "query module info failed\n"); b26199ea Jeeja KP 2017-03-24 1348 return ret; 64cb1d0a Vinod Koul 2016-08-10 1349 } f7a9f772 Sriram Periyasamy 2018-01-27 1350 f7a9f772 Sriram Periyasamy 2018-01-27 @1351 skl_tplg_add_moduleid_in_bind_params(skl, w); 64cb1d0a Vinod Koul 2016-08-10 1352 } 64cb1d0a Vinod Koul 2016-08-10 1353 } b26199ea Jeeja KP 2017-03-24 1354 64cb1d0a Vinod Koul 2016-08-10 1355 return ret; 64cb1d0a Vinod Koul 2016-08-10 1356 } 64cb1d0a Vinod Koul 2016-08-10 1357 56b03b4c Kuninori Morimoto 2018-01-29 1358 static int skl_platform_soc_probe(struct snd_soc_component *component) b663a8c5 Jeeja KP 2015-10-07 1359 { 56b03b4c Kuninori Morimoto 2018-01-29 1360 struct hdac_ext_bus *ebus = dev_get_drvdata(component->dev); fe3f4442 Dharageswari R 2016-06-03 1361 struct skl *skl = ebus_to_skl(ebus); 78cdbbda Vinod Koul 2016-07-26 1362 const struct skl_dsp_ops *ops; fe3f4442 Dharageswari R 2016-06-03 1363 int ret; b663a8c5 Jeeja KP 2015-10-07 1364 56b03b4c Kuninori Morimoto 2018-01-29 1365 pm_runtime_get_sync(component->dev); ec8ae570 Vinod Koul 2016-08-04 1366 if ((ebus_to_hbus(ebus))->ppcap) { 56b03b4c Kuninori Morimoto 2018-01-29 1367 skl->component = component; 5cdf6c09 Vinod Koul 2017-06-30 1368 5cdf6c09 Vinod Koul 2017-06-30 1369 /* init debugfs */ 5cdf6c09 Vinod Koul 2017-06-30 1370 skl->debugfs = skl_debugfs_init(skl); 5cdf6c09 Vinod Koul 2017-06-30 1371 56b03b4c Kuninori Morimoto 2018-01-29 @1372 ret = skl_tplg_init(component, ebus); fe3f4442 Dharageswari R 2016-06-03 1373 if (ret < 0) { 56b03b4c Kuninori Morimoto 2018-01-29 1374 dev_err(component->dev, "Failed to init topology!\n"); fe3f4442 Dharageswari R 2016-06-03 1375 return ret; fe3f4442 Dharageswari R 2016-06-03 1376 } 78cdbbda Vinod Koul 2016-07-26 1377 78cdbbda Vinod Koul 2016-07-26 1378 /* load the firmwares, since all is set */ 78cdbbda Vinod Koul 2016-07-26 1379 ops = skl_get_dsp_ops(skl->pci->device); 78cdbbda Vinod Koul 2016-07-26 1380 if (!ops) 78cdbbda Vinod Koul 2016-07-26 1381 return -EIO; 78cdbbda Vinod Koul 2016-07-26 1382 78cdbbda Vinod Koul 2016-07-26 1383 if (skl->skl_sst->is_first_boot == false) { 56b03b4c Kuninori Morimoto 2018-01-29 1384 dev_err(component->dev, "DSP reports first boot done!!!\n"); 78cdbbda Vinod Koul 2016-07-26 1385 return -EIO; 78cdbbda Vinod Koul 2016-07-26 1386 } 78cdbbda Vinod Koul 2016-07-26 1387 fc9fdd61 Sanyog Kale 2018-03-13 1388 /* fc9fdd61 Sanyog Kale 2018-03-13 1389 * Disable dynamic clock and power gating during firmware fc9fdd61 Sanyog Kale 2018-03-13 1390 * and library download fc9fdd61 Sanyog Kale 2018-03-13 1391 */ 56b03b4c Kuninori Morimoto 2018-01-29 1392 skl->skl_sst->enable_miscbdcge(component->dev, false); fc9fdd61 Sanyog Kale 2018-03-13 1393 skl->skl_sst->clock_power_gating(component->dev, false); d5cc0a1f Pardha Saradhi K 2018-01-02 1394 56b03b4c Kuninori Morimoto 2018-01-29 1395 ret = ops->init_fw(component->dev, skl->skl_sst); 56b03b4c Kuninori Morimoto 2018-01-29 1396 skl->skl_sst->enable_miscbdcge(component->dev, true); fc9fdd61 Sanyog Kale 2018-03-13 1397 skl->skl_sst->clock_power_gating(component->dev, true); 78cdbbda Vinod Koul 2016-07-26 1398 if (ret < 0) { 56b03b4c Kuninori Morimoto 2018-01-29 1399 dev_err(component->dev, "Failed to boot first fw: %d\n", ret); 78cdbbda Vinod Koul 2016-07-26 1400 return ret; 78cdbbda Vinod Koul 2016-07-26 1401 } 64cb1d0a Vinod Koul 2016-08-10 1402 skl_populate_modules(skl); a26a3f53 Pardha Saradhi K 2016-11-03 1403 skl->skl_sst->update_d0i3c = skl_update_d0i3c; cb729d80 G Kranthi 2017-03-13 1404 skl_dsp_enable_notification(skl->skl_sst, false); 9452314d Pradeep Tewani 2017-12-06 1405 9452314d Pradeep Tewani 2017-12-06 1406 if (skl->cfg.astate_cfg != NULL) { 9452314d Pradeep Tewani 2017-12-06 1407 skl_dsp_set_astate_cfg(skl->skl_sst, 9452314d Pradeep Tewani 2017-12-06 1408 skl->cfg.astate_cfg->count, 9452314d Pradeep Tewani 2017-12-06 1409 skl->cfg.astate_cfg); 9452314d Pradeep Tewani 2017-12-06 1410 } fe3f4442 Dharageswari R 2016-06-03 1411 } 56b03b4c Kuninori Morimoto 2018-01-29 1412 pm_runtime_mark_last_busy(component->dev); 56b03b4c Kuninori Morimoto 2018-01-29 1413 pm_runtime_put_autosuspend(component->dev); b663a8c5 Jeeja KP 2015-10-07 1414 b663a8c5 Jeeja KP 2015-10-07 1415 return 0; b663a8c5 Jeeja KP 2015-10-07 1416 } 56b03b4c Kuninori Morimoto 2018-01-29 1417
:::::: The code at line 1351 was first introduced by commit :::::: f7a9f77256e7fbd6150651bfab44f60c39f0b7a2 ASoC: Intel: Skylake: Find module id from UUID for bind params
:::::: TO: Sriram Periyasamy sriramx.periyasamy@intel.com :::::: CC: Mark Brown broonie@kernel.org
--- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation