[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <202601170203.upPyGvI2-lkp@intel.com>
Date: Mon, 19 Jan 2026 09:37:53 +0300
From: Dan Carpenter <dan.carpenter@...aro.org>
To: oe-kbuild@...ts.linux.dev, Shengjiu Wang <shengjiu.wang@....com>,
shengjiu.wang@...il.com, Xiubo.Lee@...il.com, festevam@...il.com,
nicoleotsuka@...il.com, lgirdwood@...il.com, broonie@...nel.org,
perex@...ex.cz, tiwai@...e.com, linux-sound@...r.kernel.org,
linuxppc-dev@...ts.ozlabs.org, linux-kernel@...r.kernel.org,
robh@...nel.org, krzk+dt@...nel.org, conor+dt@...nel.org,
devicetree@...r.kernel.org, shawnguo@...nel.org,
s.hauer@...gutronix.de, kernel@...gutronix.de, imx@...ts.linux.dev,
linux-arm-kernel@...ts.infradead.org
Cc: lkp@...el.com, oe-kbuild-all@...ts.linux.dev
Subject: Re: [PATCH 2/2] ASoC: fsl_audmix: Add support for i.MX952 platform
Hi Shengjiu,
kernel test robot noticed the following build warnings:
https://git-scm.com/docs/git-format-patch#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Shengjiu-Wang/ASoC-dt-bindings-fsl-audmix-Add-support-for-i-MX952-platform/20260116-182050
base: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next
patch link: https://lore.kernel.org/r/20260116101648.377952-3-shengjiu.wang%40nxp.com
patch subject: [PATCH 2/2] ASoC: fsl_audmix: Add support for i.MX952 platform
config: s390-randconfig-r071-20260116 (https://download.01.org/0day-ci/archive/20260117/202601170203.upPyGvI2-lkp@intel.com/config)
compiler: s390-linux-gcc (GCC) 8.5.0
smatch version: v0.5.0-8985-g2614ff1a
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@...el.com>
| Reported-by: Dan Carpenter <dan.carpenter@...aro.org>
| Closes: https://lore.kernel.org/r/202601170203.upPyGvI2-lkp@intel.com/
smatch warnings:
sound/soc/fsl/fsl_audmix.c:522 fsl_audmix_probe() warn: missing error code 'ret'
vim +/ret +522 sound/soc/fsl/fsl_audmix.c
be1df61cf06efb Viorel Suman 2019-01-22 465 static int fsl_audmix_probe(struct platform_device *pdev)
be1df61cf06efb Viorel Suman 2019-01-22 466 {
0c44e9e9e61cde Shengjiu Wang 2026-01-16 467 const struct fsl_audmix_soc_data *soc_data;
62be484f7ad844 Viorel Suman 2019-04-10 468 struct device *dev = &pdev->dev;
be1df61cf06efb Viorel Suman 2019-01-22 469 struct fsl_audmix *priv;
be1df61cf06efb Viorel Suman 2019-01-22 470 void __iomem *regs;
be1df61cf06efb Viorel Suman 2019-01-22 471 int ret;
f2a36a78423ee8 Viorel Suman 2019-04-10 472
62be484f7ad844 Viorel Suman 2019-04-10 473 priv = devm_kzalloc(dev, sizeof(*priv), GFP_KERNEL);
be1df61cf06efb Viorel Suman 2019-01-22 474 if (!priv)
be1df61cf06efb Viorel Suman 2019-01-22 475 return -ENOMEM;
be1df61cf06efb Viorel Suman 2019-01-22 476
be1df61cf06efb Viorel Suman 2019-01-22 477 /* Get the addresses */
959bb6b54d7086 YueHaibing 2019-07-27 478 regs = devm_platform_ioremap_resource(pdev, 0);
be1df61cf06efb Viorel Suman 2019-01-22 479 if (IS_ERR(regs))
be1df61cf06efb Viorel Suman 2019-01-22 480 return PTR_ERR(regs);
be1df61cf06efb Viorel Suman 2019-01-22 481
3feaba79d8f701 Shengjiu Wang 2021-03-24 482 priv->regmap = devm_regmap_init_mmio(dev, regs, &fsl_audmix_regmap_config);
be1df61cf06efb Viorel Suman 2019-01-22 483 if (IS_ERR(priv->regmap)) {
62be484f7ad844 Viorel Suman 2019-04-10 484 dev_err(dev, "failed to init regmap\n");
be1df61cf06efb Viorel Suman 2019-01-22 485 return PTR_ERR(priv->regmap);
be1df61cf06efb Viorel Suman 2019-01-22 486 }
be1df61cf06efb Viorel Suman 2019-01-22 487
62be484f7ad844 Viorel Suman 2019-04-10 488 priv->ipg_clk = devm_clk_get(dev, "ipg");
be1df61cf06efb Viorel Suman 2019-01-22 489 if (IS_ERR(priv->ipg_clk)) {
62be484f7ad844 Viorel Suman 2019-04-10 490 dev_err(dev, "failed to get ipg clock\n");
be1df61cf06efb Viorel Suman 2019-01-22 491 return PTR_ERR(priv->ipg_clk);
be1df61cf06efb Viorel Suman 2019-01-22 492 }
be1df61cf06efb Viorel Suman 2019-01-22 493
fe965096c9495d Shengjiu Wang 2019-11-11 494 spin_lock_init(&priv->lock);
be1df61cf06efb Viorel Suman 2019-01-22 495 platform_set_drvdata(pdev, priv);
62be484f7ad844 Viorel Suman 2019-04-10 496 pm_runtime_enable(dev);
be1df61cf06efb Viorel Suman 2019-01-22 497
62be484f7ad844 Viorel Suman 2019-04-10 498 ret = devm_snd_soc_register_component(dev, &fsl_audmix_component,
be1df61cf06efb Viorel Suman 2019-01-22 499 fsl_audmix_dai,
be1df61cf06efb Viorel Suman 2019-01-22 500 ARRAY_SIZE(fsl_audmix_dai));
be1df61cf06efb Viorel Suman 2019-01-22 501 if (ret) {
62be484f7ad844 Viorel Suman 2019-04-10 502 dev_err(dev, "failed to register ASoC DAI\n");
77fffa742285f2 Chuhong Yuan 2019-12-03 503 goto err_disable_pm;
be1df61cf06efb Viorel Suman 2019-01-22 504 }
be1df61cf06efb Viorel Suman 2019-01-22 505
294a60e5e98300 Shengjiu Wang 2025-02-26 506 /*
294a60e5e98300 Shengjiu Wang 2025-02-26 507 * If dais property exist, then register the imx-audmix card driver.
294a60e5e98300 Shengjiu Wang 2025-02-26 508 * otherwise, it should be linked by audio graph card.
294a60e5e98300 Shengjiu Wang 2025-02-26 509 */
294a60e5e98300 Shengjiu Wang 2025-02-26 510 if (of_find_property(pdev->dev.of_node, "dais", NULL)) {
5057d108d69a55 Fabio Estevam 2020-12-02 511 priv->pdev = platform_device_register_data(dev, "imx-audmix", 0, NULL, 0);
be1df61cf06efb Viorel Suman 2019-01-22 512 if (IS_ERR(priv->pdev)) {
be1df61cf06efb Viorel Suman 2019-01-22 513 ret = PTR_ERR(priv->pdev);
5057d108d69a55 Fabio Estevam 2020-12-02 514 dev_err(dev, "failed to register platform: %d\n", ret);
77fffa742285f2 Chuhong Yuan 2019-12-03 515 goto err_disable_pm;
be1df61cf06efb Viorel Suman 2019-01-22 516 }
294a60e5e98300 Shengjiu Wang 2025-02-26 517 }
be1df61cf06efb Viorel Suman 2019-01-22 518
0c44e9e9e61cde Shengjiu Wang 2026-01-16 519 soc_data = of_device_get_match_data(dev);
0c44e9e9e61cde Shengjiu Wang 2026-01-16 520 if (!soc_data) {
0c44e9e9e61cde Shengjiu Wang 2026-01-16 521 dev_err(dev, "failed to match device\n");
0c44e9e9e61cde Shengjiu Wang 2026-01-16 @522 goto err_disable_pm;
missing error code.
0c44e9e9e61cde Shengjiu Wang 2026-01-16 523 }
0c44e9e9e61cde Shengjiu Wang 2026-01-16 524
0c44e9e9e61cde Shengjiu Wang 2026-01-16 525 if (of_property_read_bool(pdev->dev.of_node, "fsl,amix-bypass") &&
0c44e9e9e61cde Shengjiu Wang 2026-01-16 526 soc_data->bypass_index > 0) {
0c44e9e9e61cde Shengjiu Wang 2026-01-16 527 ret = scmi_imx_misc_ctrl_set(soc_data->bypass_index, 0);
0c44e9e9e61cde Shengjiu Wang 2026-01-16 528 if (ret)
0c44e9e9e61cde Shengjiu Wang 2026-01-16 529 goto err_disable_pm;
0c44e9e9e61cde Shengjiu Wang 2026-01-16 530 }
0c44e9e9e61cde Shengjiu Wang 2026-01-16 531
77fffa742285f2 Chuhong Yuan 2019-12-03 532 return 0;
77fffa742285f2 Chuhong Yuan 2019-12-03 533
77fffa742285f2 Chuhong Yuan 2019-12-03 534 err_disable_pm:
77fffa742285f2 Chuhong Yuan 2019-12-03 535 pm_runtime_disable(dev);
be1df61cf06efb Viorel Suman 2019-01-22 536 return ret;
be1df61cf06efb Viorel Suman 2019-01-22 537 }
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
Powered by blists - more mailing lists