Re: [PATCH 1/3] firmware: qcom_scm: Always try to consume all three clocks
From: kernel test robot
Date: Fri Jun 23 2023 - 20:16:29 EST
Hi Konrad,
kernel test robot noticed the following build warnings:
[auto build test WARNING on c87d46a9e8ebd2f2c3960927b1d21687096d1109]
url: https://github.com/intel-lab-lkp/linux/commits/Konrad-Dybcio/firmware-qcom_scm-Always-try-to-consume-all-three-clocks/20230624-055215
base: c87d46a9e8ebd2f2c3960927b1d21687096d1109
patch link: https://lore.kernel.org/r/20230623-topic-scm_cleanup-v1-1-383089eae98d%40linaro.org
patch subject: [PATCH 1/3] firmware: qcom_scm: Always try to consume all three clocks
config: arm64-randconfig-r023-20230622 (https://download.01.org/0day-ci/archive/20230624/202306240841.iVZhYL4l-lkp@xxxxxxxxx/config)
compiler: aarch64-linux-gcc (GCC) 12.3.0
reproduce: (https://download.01.org/0day-ci/archive/20230624/202306240841.iVZhYL4l-lkp@xxxxxxxxx/reproduce)
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@xxxxxxxxx>
| Closes: https://lore.kernel.org/oe-kbuild-all/202306240841.iVZhYL4l-lkp@xxxxxxxxx/
All warnings (new ones prefixed by >>):
drivers/firmware/qcom_scm.c: In function 'qcom_scm_probe':
>> drivers/firmware/qcom_scm.c:1404:23: warning: variable 'clks' set but not used [-Wunused-but-set-variable]
1404 | unsigned long clks;
| ^~~~
vim +/clks +1404 drivers/firmware/qcom_scm.c
6bf32599223634 Guru Das Srinagesh 2023-01-13 1400
d0f6fa7ba2d624 Andy Gross 2016-06-03 1401 static int qcom_scm_probe(struct platform_device *pdev)
d0f6fa7ba2d624 Andy Gross 2016-06-03 1402 {
d0f6fa7ba2d624 Andy Gross 2016-06-03 1403 struct qcom_scm *scm;
ab0822d57d8ccd Sarangdhar Joshi 2016-11-15 @1404 unsigned long clks;
6bf32599223634 Guru Das Srinagesh 2023-01-13 1405 int irq, ret;
d0f6fa7ba2d624 Andy Gross 2016-06-03 1406
d0f6fa7ba2d624 Andy Gross 2016-06-03 1407 scm = devm_kzalloc(&pdev->dev, sizeof(*scm), GFP_KERNEL);
d0f6fa7ba2d624 Andy Gross 2016-06-03 1408 if (!scm)
d0f6fa7ba2d624 Andy Gross 2016-06-03 1409 return -ENOMEM;
d0f6fa7ba2d624 Andy Gross 2016-06-03 1410
8c1b7dc9ba2294 Bjorn Andersson 2017-08-14 1411 ret = qcom_scm_find_dload_address(&pdev->dev, &scm->dload_mode_addr);
8c1b7dc9ba2294 Bjorn Andersson 2017-08-14 1412 if (ret < 0)
8c1b7dc9ba2294 Bjorn Andersson 2017-08-14 1413 return ret;
8c1b7dc9ba2294 Bjorn Andersson 2017-08-14 1414
65b7ebda502861 Sibi Sankar 2022-05-23 1415 mutex_init(&scm->scm_bw_lock);
65b7ebda502861 Sibi Sankar 2022-05-23 1416
ab0822d57d8ccd Sarangdhar Joshi 2016-11-15 1417 clks = (unsigned long)of_device_get_match_data(&pdev->dev);
60cd420c91e28c Bjorn Andersson 2018-08-29 1418
65b7ebda502861 Sibi Sankar 2022-05-23 1419 scm->path = devm_of_icc_get(&pdev->dev, NULL);
65b7ebda502861 Sibi Sankar 2022-05-23 1420 if (IS_ERR(scm->path))
65b7ebda502861 Sibi Sankar 2022-05-23 1421 return dev_err_probe(&pdev->dev, PTR_ERR(scm->path),
65b7ebda502861 Sibi Sankar 2022-05-23 1422 "failed to acquire interconnect path\n");
65b7ebda502861 Sibi Sankar 2022-05-23 1423
06987a4b9a3c31 Konrad Dybcio 2023-06-23 1424 scm->core_clk = devm_clk_get_optional(&pdev->dev, "core");
d0f6fa7ba2d624 Andy Gross 2016-06-03 1425 if (IS_ERR(scm->core_clk)) {
60cd420c91e28c Bjorn Andersson 2018-08-29 1426 if (PTR_ERR(scm->core_clk) == -EPROBE_DEFER)
60cd420c91e28c Bjorn Andersson 2018-08-29 1427 return PTR_ERR(scm->core_clk);
ab0822d57d8ccd Sarangdhar Joshi 2016-11-15 1428 }
d0f6fa7ba2d624 Andy Gross 2016-06-03 1429
06987a4b9a3c31 Konrad Dybcio 2023-06-23 1430 scm->iface_clk = devm_clk_get_optional(&pdev->dev, "iface");
d0f6fa7ba2d624 Andy Gross 2016-06-03 1431 if (IS_ERR(scm->iface_clk)) {
60cd420c91e28c Bjorn Andersson 2018-08-29 1432 if (PTR_ERR(scm->iface_clk) == -EPROBE_DEFER)
60cd420c91e28c Bjorn Andersson 2018-08-29 1433 return PTR_ERR(scm->iface_clk);
d0f6fa7ba2d624 Andy Gross 2016-06-03 1434 }
60cd420c91e28c Bjorn Andersson 2018-08-29 1435
06987a4b9a3c31 Konrad Dybcio 2023-06-23 1436 scm->bus_clk = devm_clk_get_optional(&pdev->dev, "bus");
d0f6fa7ba2d624 Andy Gross 2016-06-03 1437 if (IS_ERR(scm->bus_clk)) {
60cd420c91e28c Bjorn Andersson 2018-08-29 1438 if (PTR_ERR(scm->bus_clk) == -EPROBE_DEFER)
60cd420c91e28c Bjorn Andersson 2018-08-29 1439 return PTR_ERR(scm->bus_clk);
d0f6fa7ba2d624 Andy Gross 2016-06-03 1440 }
d0f6fa7ba2d624 Andy Gross 2016-06-03 1441
dd4fe5b292226f Bjorn Andersson 2016-06-17 1442 scm->reset.ops = &qcom_scm_pas_reset_ops;
dd4fe5b292226f Bjorn Andersson 2016-06-17 1443 scm->reset.nr_resets = 1;
dd4fe5b292226f Bjorn Andersson 2016-06-17 1444 scm->reset.of_node = pdev->dev.of_node;
bd4760ca031567 Wei Yongjun 2016-08-28 1445 ret = devm_reset_controller_register(&pdev->dev, &scm->reset);
bd4760ca031567 Wei Yongjun 2016-08-28 1446 if (ret)
bd4760ca031567 Wei Yongjun 2016-08-28 1447 return ret;
dd4fe5b292226f Bjorn Andersson 2016-06-17 1448
d0f6fa7ba2d624 Andy Gross 2016-06-03 1449 /* vote for max clk rate for highest performance */
d0f6fa7ba2d624 Andy Gross 2016-06-03 1450 ret = clk_set_rate(scm->core_clk, INT_MAX);
d0f6fa7ba2d624 Andy Gross 2016-06-03 1451 if (ret)
d0f6fa7ba2d624 Andy Gross 2016-06-03 1452 return ret;
d0f6fa7ba2d624 Andy Gross 2016-06-03 1453
d0f6fa7ba2d624 Andy Gross 2016-06-03 1454 __scm = scm;
d0f6fa7ba2d624 Andy Gross 2016-06-03 1455 __scm->dev = &pdev->dev;
d0f6fa7ba2d624 Andy Gross 2016-06-03 1456
6bf32599223634 Guru Das Srinagesh 2023-01-13 1457 init_completion(&__scm->waitq_comp);
6bf32599223634 Guru Das Srinagesh 2023-01-13 1458
f3d0fbad6765da Johan Hovold 2023-03-09 1459 irq = platform_get_irq_optional(pdev, 0);
6bf32599223634 Guru Das Srinagesh 2023-01-13 1460 if (irq < 0) {
6bf32599223634 Guru Das Srinagesh 2023-01-13 1461 if (irq != -ENXIO)
6bf32599223634 Guru Das Srinagesh 2023-01-13 1462 return irq;
6bf32599223634 Guru Das Srinagesh 2023-01-13 1463 } else {
6bf32599223634 Guru Das Srinagesh 2023-01-13 1464 ret = devm_request_threaded_irq(__scm->dev, irq, NULL, qcom_scm_irq_handler,
6bf32599223634 Guru Das Srinagesh 2023-01-13 1465 IRQF_ONESHOT, "qcom-scm", __scm);
6bf32599223634 Guru Das Srinagesh 2023-01-13 1466 if (ret < 0)
6bf32599223634 Guru Das Srinagesh 2023-01-13 1467 return dev_err_probe(scm->dev, ret, "Failed to request qcom-scm irq\n");
6bf32599223634 Guru Das Srinagesh 2023-01-13 1468 }
6bf32599223634 Guru Das Srinagesh 2023-01-13 1469
f6ea568f0ddcdf Stephen Boyd 2021-02-23 1470 __get_convention();
6b1751a86ce2eb Kumar Gala 2016-06-03 1471
8c1b7dc9ba2294 Bjorn Andersson 2017-08-14 1472 /*
8c1b7dc9ba2294 Bjorn Andersson 2017-08-14 1473 * If requested enable "download mode", from this point on warmboot
c19698a9e41bd6 Jiang Jian 2022-06-21 1474 * will cause the boot stages to enter download mode, unless
8c1b7dc9ba2294 Bjorn Andersson 2017-08-14 1475 * disabled below by a clean shutdown/reboot.
8c1b7dc9ba2294 Bjorn Andersson 2017-08-14 1476 */
8c1b7dc9ba2294 Bjorn Andersson 2017-08-14 1477 if (download_mode)
8c1b7dc9ba2294 Bjorn Andersson 2017-08-14 1478 qcom_scm_set_download_mode(true);
8c1b7dc9ba2294 Bjorn Andersson 2017-08-14 1479
d0f6fa7ba2d624 Andy Gross 2016-06-03 1480 return 0;
d0f6fa7ba2d624 Andy Gross 2016-06-03 1481 }
d0f6fa7ba2d624 Andy Gross 2016-06-03 1482
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki