From: Dan Carpenter <dan.carpenter@linaro.org>
To: oe-kbuild@lists.linux.dev, Shengjiu Wang <shengjiu.wang@nxp.com>,
shengjiu.wang@gmail.com, Xiubo.Lee@gmail.com, festevam@gmail.com,
nicoleotsuka@gmail.com, lgirdwood@gmail.com, broonie@kernel.org,
perex@perex.cz, tiwai@suse.com, linux-sound@vger.kernel.org,
linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org,
robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org,
devicetree@vger.kernel.org, shawnguo@kernel.org,
s.hauer@pengutronix.de, kernel@pengutronix.de,
imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org
Cc: lkp@intel.com, oe-kbuild-all@lists.linux.dev
Subject: Re: [PATCH 2/2] ASoC: fsl_audmix: Add support for i.MX952 platform
Date: Mon, 19 Jan 2026 09:37:53 +0300 [thread overview]
Message-ID: <202601170203.upPyGvI2-lkp@intel.com> (raw)
In-Reply-To: <20260116101648.377952-3-shengjiu.wang@nxp.com>
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@intel.com>
| Reported-by: Dan Carpenter <dan.carpenter@linaro.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
prev parent reply other threads:[~2026-01-19 6:37 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-01-16 10:16 [PATCH 0/2] ASoC: fsl_audmix: Support the i.MX952 platform Shengjiu Wang
2026-01-16 10:16 ` [PATCH 1/2] ASoC: dt-bindings: fsl,audmix: Add support for " Shengjiu Wang
2026-01-16 15:00 ` Frank Li
2026-01-17 11:44 ` Krzysztof Kozlowski
2026-01-19 6:27 ` Shengjiu Wang
2026-01-19 7:07 ` Krzysztof Kozlowski
2026-01-19 7:31 ` Shengjiu Wang
2026-01-16 10:16 ` [PATCH 2/2] ASoC: fsl_audmix: " Shengjiu Wang
2026-01-19 6:37 ` Dan Carpenter [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202601170203.upPyGvI2-lkp@intel.com \
--to=dan.carpenter@linaro.org \
--cc=Xiubo.Lee@gmail.com \
--cc=broonie@kernel.org \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=festevam@gmail.com \
--cc=imx@lists.linux.dev \
--cc=kernel@pengutronix.de \
--cc=krzk+dt@kernel.org \
--cc=lgirdwood@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-sound@vger.kernel.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=lkp@intel.com \
--cc=nicoleotsuka@gmail.com \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=oe-kbuild@lists.linux.dev \
--cc=perex@perex.cz \
--cc=robh@kernel.org \
--cc=s.hauer@pengutronix.de \
--cc=shawnguo@kernel.org \
--cc=shengjiu.wang@gmail.com \
--cc=shengjiu.wang@nxp.com \
--cc=tiwai@suse.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox