From: Dan Carpenter <dan.carpenter@linaro.org>
To: oe-kbuild@lists.linux.dev,
Cosmin Tanislav <cosmin-gabriel.tanislav.xa@renesas.com>
Cc: lkp@intel.com, oe-kbuild-all@lists.linux.dev,
Fabrizio Castro <fabrizio.castro.jz@renesas.com>,
Mark Brown <broonie@kernel.org>, Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Geert Uytterhoeven <geert+renesas@glider.be>,
Magnus Damm <magnus.damm@gmail.com>,
Michael Turquette <mturquette@baylibre.com>,
Stephen Boyd <sboyd@kernel.org>,
Philipp Zabel <p.zabel@pengutronix.de>,
linux-spi@vger.kernel.org, linux-renesas-soc@vger.kernel.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-clk@vger.kernel.org,
Cosmin Tanislav <cosmin-gabriel.tanislav.xa@renesas.com>
Subject: Re: [PATCH 09/14] spi: rzv2h-rspi: add support for variable transfer clock
Date: Fri, 31 Oct 2025 15:02:56 +0300 [thread overview]
Message-ID: <202510311957.SRNglGAw-lkp@intel.com> (raw)
In-Reply-To: <20251028133151.1487327-10-cosmin-gabriel.tanislav.xa@renesas.com>
Hi Cosmin,
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/Cosmin-Tanislav/clk-renesas-r9a09g077-add-SPI-module-clocks/20251028-213853
base: https://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-drivers.git renesas-clk
patch link: https://lore.kernel.org/r/20251028133151.1487327-10-cosmin-gabriel.tanislav.xa%40renesas.com
patch subject: [PATCH 09/14] spi: rzv2h-rspi: add support for variable transfer clock
config: hexagon-randconfig-r071-20251031 (https://download.01.org/0day-ci/archive/20251031/202510311957.SRNglGAw-lkp@intel.com/config)
compiler: clang version 22.0.0git (https://github.com/llvm/llvm-project d1c086e82af239b245fe8d7832f2753436634990)
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/202510311957.SRNglGAw-lkp@intel.com/
smatch warnings:
drivers/spi/spi-rzv2h-rspi.c:510 rzv2h_rspi_probe() warn: missing unwind goto?
vim +510 drivers/spi/spi-rzv2h-rspi.c
8b61c8919dff08 Fabrizio Castro 2025-07-04 428 static int rzv2h_rspi_probe(struct platform_device *pdev)
8b61c8919dff08 Fabrizio Castro 2025-07-04 429 {
8b61c8919dff08 Fabrizio Castro 2025-07-04 430 struct spi_controller *controller;
8b61c8919dff08 Fabrizio Castro 2025-07-04 431 struct device *dev = &pdev->dev;
8b61c8919dff08 Fabrizio Castro 2025-07-04 432 struct rzv2h_rspi_priv *rspi;
8b61c8919dff08 Fabrizio Castro 2025-07-04 433 struct clk_bulk_data *clks;
8b61c8919dff08 Fabrizio Castro 2025-07-04 434 int irq_rx, ret, i;
c4e1a7fda7e52d Cosmin Tanislav 2025-10-28 435 long tclk_rate;
8b61c8919dff08 Fabrizio Castro 2025-07-04 436
8b61c8919dff08 Fabrizio Castro 2025-07-04 437 controller = devm_spi_alloc_host(dev, sizeof(*rspi));
8b61c8919dff08 Fabrizio Castro 2025-07-04 438 if (!controller)
8b61c8919dff08 Fabrizio Castro 2025-07-04 439 return -ENOMEM;
8b61c8919dff08 Fabrizio Castro 2025-07-04 440
8b61c8919dff08 Fabrizio Castro 2025-07-04 441 rspi = spi_controller_get_devdata(controller);
8b61c8919dff08 Fabrizio Castro 2025-07-04 442 platform_set_drvdata(pdev, rspi);
8b61c8919dff08 Fabrizio Castro 2025-07-04 443
8b61c8919dff08 Fabrizio Castro 2025-07-04 444 rspi->controller = controller;
8b61c8919dff08 Fabrizio Castro 2025-07-04 445
a1ddc1bd04968e Cosmin Tanislav 2025-10-28 446 rspi->info = device_get_match_data(dev);
a1ddc1bd04968e Cosmin Tanislav 2025-10-28 447
8b61c8919dff08 Fabrizio Castro 2025-07-04 448 rspi->base = devm_platform_ioremap_resource(pdev, 0);
8b61c8919dff08 Fabrizio Castro 2025-07-04 449 if (IS_ERR(rspi->base))
8b61c8919dff08 Fabrizio Castro 2025-07-04 450 return PTR_ERR(rspi->base);
8b61c8919dff08 Fabrizio Castro 2025-07-04 451
8b61c8919dff08 Fabrizio Castro 2025-07-04 452 ret = devm_clk_bulk_get_all_enabled(dev, &clks);
2cfbd980e14a68 Cosmin Tanislav 2025-10-28 453 if (ret != rspi->info->num_clks)
8b61c8919dff08 Fabrizio Castro 2025-07-04 454 return dev_err_probe(dev, ret >= 0 ? -EINVAL : ret,
8b61c8919dff08 Fabrizio Castro 2025-07-04 455 "cannot get clocks\n");
2cfbd980e14a68 Cosmin Tanislav 2025-10-28 456 for (i = 0; i < rspi->info->num_clks; i++) {
2cfbd980e14a68 Cosmin Tanislav 2025-10-28 457 if (!strcmp(clks[i].id, rspi->info->tclk_name)) {
8b61c8919dff08 Fabrizio Castro 2025-07-04 458 rspi->tclk = clks[i].clk;
92eb6cc239b9d2 Cosmin Tanislav 2025-10-28 459 } else if (rspi->info->find_pclk_rate &&
92eb6cc239b9d2 Cosmin Tanislav 2025-10-28 460 !strcmp(clks[i].id, "pclk")) {
92eb6cc239b9d2 Cosmin Tanislav 2025-10-28 461 rspi->pclk = clks[i].clk;
8b61c8919dff08 Fabrizio Castro 2025-07-04 462 }
8b61c8919dff08 Fabrizio Castro 2025-07-04 463 }
8b61c8919dff08 Fabrizio Castro 2025-07-04 464
8b61c8919dff08 Fabrizio Castro 2025-07-04 465 if (!rspi->tclk)
8b61c8919dff08 Fabrizio Castro 2025-07-04 466 return dev_err_probe(dev, -EINVAL, "Failed to get tclk\n");
8b61c8919dff08 Fabrizio Castro 2025-07-04 467
8b61c8919dff08 Fabrizio Castro 2025-07-04 468 rspi->resets[0].id = "presetn";
8b61c8919dff08 Fabrizio Castro 2025-07-04 469 rspi->resets[1].id = "tresetn";
c10c8eb75d9db8 Cosmin Tanislav 2025-10-28 470 ret = devm_reset_control_bulk_get_optional_exclusive(dev, RSPI_RESET_NUM,
8b61c8919dff08 Fabrizio Castro 2025-07-04 471 rspi->resets);
8b61c8919dff08 Fabrizio Castro 2025-07-04 472 if (ret)
8b61c8919dff08 Fabrizio Castro 2025-07-04 473 return dev_err_probe(dev, ret, "cannot get resets\n");
8b61c8919dff08 Fabrizio Castro 2025-07-04 474
8b61c8919dff08 Fabrizio Castro 2025-07-04 475 irq_rx = platform_get_irq_byname(pdev, "rx");
8b61c8919dff08 Fabrizio Castro 2025-07-04 476 if (irq_rx < 0)
8b61c8919dff08 Fabrizio Castro 2025-07-04 477 return dev_err_probe(dev, irq_rx, "cannot get IRQ 'rx'\n");
8b61c8919dff08 Fabrizio Castro 2025-07-04 478
8b61c8919dff08 Fabrizio Castro 2025-07-04 479 ret = reset_control_bulk_deassert(RSPI_RESET_NUM, rspi->resets);
deasserted here.
8b61c8919dff08 Fabrizio Castro 2025-07-04 480 if (ret)
8b61c8919dff08 Fabrizio Castro 2025-07-04 481 return dev_err_probe(dev, ret, "failed to deassert resets\n");
8b61c8919dff08 Fabrizio Castro 2025-07-04 482
8b61c8919dff08 Fabrizio Castro 2025-07-04 483 init_waitqueue_head(&rspi->wait);
8b61c8919dff08 Fabrizio Castro 2025-07-04 484
8b61c8919dff08 Fabrizio Castro 2025-07-04 485 ret = devm_request_irq(dev, irq_rx, rzv2h_rx_irq_handler, 0,
8b61c8919dff08 Fabrizio Castro 2025-07-04 486 dev_name(dev), rspi);
8b61c8919dff08 Fabrizio Castro 2025-07-04 487 if (ret) {
8b61c8919dff08 Fabrizio Castro 2025-07-04 488 dev_err(dev, "cannot request `rx` IRQ\n");
8b61c8919dff08 Fabrizio Castro 2025-07-04 489 goto quit_resets;
8b61c8919dff08 Fabrizio Castro 2025-07-04 490 }
8b61c8919dff08 Fabrizio Castro 2025-07-04 491
8b61c8919dff08 Fabrizio Castro 2025-07-04 492 controller->mode_bits = SPI_CPHA | SPI_CPOL | SPI_CS_HIGH |
8b61c8919dff08 Fabrizio Castro 2025-07-04 493 SPI_LSB_FIRST;
8b61c8919dff08 Fabrizio Castro 2025-07-04 494 controller->bits_per_word_mask = SPI_BPW_RANGE_MASK(4, 32);
8b61c8919dff08 Fabrizio Castro 2025-07-04 495 controller->prepare_message = rzv2h_rspi_prepare_message;
8b61c8919dff08 Fabrizio Castro 2025-07-04 496 controller->unprepare_message = rzv2h_rspi_unprepare_message;
8b61c8919dff08 Fabrizio Castro 2025-07-04 497 controller->num_chipselect = 4;
8b61c8919dff08 Fabrizio Castro 2025-07-04 498 controller->transfer_one = rzv2h_rspi_transfer_one;
c4e1a7fda7e52d Cosmin Tanislav 2025-10-28 499
c4e1a7fda7e52d Cosmin Tanislav 2025-10-28 500 tclk_rate = clk_round_rate(rspi->tclk, 0);
c4e1a7fda7e52d Cosmin Tanislav 2025-10-28 501 if (tclk_rate < 0)
c4e1a7fda7e52d Cosmin Tanislav 2025-10-28 502 return tclk_rate;
goto quit_resets;
c4e1a7fda7e52d Cosmin Tanislav 2025-10-28 503
8b61c8919dff08 Fabrizio Castro 2025-07-04 504 controller->min_speed_hz = rzv2h_rspi_calc_bitrate(tclk_rate,
8b61c8919dff08 Fabrizio Castro 2025-07-04 505 RSPI_SPBR_SPR_MAX,
8b61c8919dff08 Fabrizio Castro 2025-07-04 506 RSPI_SPCMD_BRDV_MAX);
c4e1a7fda7e52d Cosmin Tanislav 2025-10-28 507
c4e1a7fda7e52d Cosmin Tanislav 2025-10-28 508 tclk_rate = clk_round_rate(rspi->tclk, ULONG_MAX);
c4e1a7fda7e52d Cosmin Tanislav 2025-10-28 509 if (tclk_rate < 0)
c4e1a7fda7e52d Cosmin Tanislav 2025-10-28 @510 return tclk_rate;
goto quit_resets;
c4e1a7fda7e52d Cosmin Tanislav 2025-10-28 511
8b61c8919dff08 Fabrizio Castro 2025-07-04 512 controller->max_speed_hz = rzv2h_rspi_calc_bitrate(tclk_rate,
8b61c8919dff08 Fabrizio Castro 2025-07-04 513 RSPI_SPBR_SPR_MIN,
8b61c8919dff08 Fabrizio Castro 2025-07-04 514 RSPI_SPCMD_BRDV_MIN);
8b61c8919dff08 Fabrizio Castro 2025-07-04 515
8b61c8919dff08 Fabrizio Castro 2025-07-04 516 device_set_node(&controller->dev, dev_fwnode(dev));
8b61c8919dff08 Fabrizio Castro 2025-07-04 517
8b61c8919dff08 Fabrizio Castro 2025-07-04 518 ret = spi_register_controller(controller);
8b61c8919dff08 Fabrizio Castro 2025-07-04 519 if (ret) {
8b61c8919dff08 Fabrizio Castro 2025-07-04 520 dev_err(dev, "register controller failed\n");
8b61c8919dff08 Fabrizio Castro 2025-07-04 521 goto quit_resets;
8b61c8919dff08 Fabrizio Castro 2025-07-04 522 }
8b61c8919dff08 Fabrizio Castro 2025-07-04 523
8b61c8919dff08 Fabrizio Castro 2025-07-04 524 return 0;
8b61c8919dff08 Fabrizio Castro 2025-07-04 525
8b61c8919dff08 Fabrizio Castro 2025-07-04 526 quit_resets:
8b61c8919dff08 Fabrizio Castro 2025-07-04 527 reset_control_bulk_assert(RSPI_RESET_NUM, rspi->resets);
8b61c8919dff08 Fabrizio Castro 2025-07-04 528
8b61c8919dff08 Fabrizio Castro 2025-07-04 529 return ret;
8b61c8919dff08 Fabrizio Castro 2025-07-04 530 }
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2025-10-31 12:03 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-10-28 13:31 [PATCH 00/14] Add RSPI support for RZ/T2H and RZ/N2H Cosmin Tanislav
2025-10-28 13:31 ` [PATCH 01/14] clk: renesas: r9a09g077: add SPI module clocks Cosmin Tanislav
2025-10-28 13:39 ` Cosmin-Gabriel Tanislav
2025-10-28 13:31 ` [PATCH 02/14] spi: rzv2h-rspi: make resets optional Cosmin Tanislav
2025-10-28 13:31 ` [PATCH 03/14] spi: rzv2h-rspi: make FIFO size chip-specific Cosmin Tanislav
2025-10-28 13:31 ` [PATCH 04/14] spi: rzv2h-rspi: make clocks chip-specific Cosmin Tanislav
2025-10-28 13:31 ` [PATCH 05/14] spi: rzv2h-rspi: move register writes out of rzv2h_rspi_setup_clock() Cosmin Tanislav
2025-10-28 13:31 ` [PATCH 06/14] spi: rzv2h-rspi: avoid recomputing transfer frequency Cosmin Tanislav
2025-10-28 13:31 ` [PATCH 07/14] spi: rzv2h-rspi: make transfer clock rate finding chip-specific Cosmin Tanislav
2025-10-28 13:31 ` [PATCH 08/14] spi: rzv2h-rspi: add support for using PCLK for transfer clock Cosmin Tanislav
2025-10-28 13:31 ` [PATCH 09/14] spi: rzv2h-rspi: add support for variable " Cosmin Tanislav
2025-10-31 12:02 ` Dan Carpenter [this message]
2025-10-28 13:31 ` [PATCH 10/14] spi: rzv2h-rspi: add support for loopback mode Cosmin Tanislav
2025-10-28 13:31 ` [PATCH 11/14] dt-bindings: spi: renesas,rzv2h-rspi: document RZ/T2H and RZ/N2H Cosmin Tanislav
2025-10-29 17:58 ` Conor Dooley
2025-10-28 13:31 ` [PATCH 12/14] spi: rzv2h-rspi: add support for " Cosmin Tanislav
2025-10-28 13:31 ` [PATCH 13/14] arm64: dts: renesas: r9a09g077: Add SPIs support Cosmin Tanislav
2025-10-28 13:31 ` [PATCH 14/14] arm64: dts: renesas: r9a09g087: " Cosmin Tanislav
-- strict thread matches above, loose matches on Subject: below --
2025-11-05 9:13 [PATCH 00/14] Add RSPI support for RZ/T2H and RZ/N2H Cosmin Tanislav
2025-11-05 9:13 ` [PATCH 09/14] spi: rzv2h-rspi: add support for variable transfer clock Cosmin Tanislav
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=202510311957.SRNglGAw-lkp@intel.com \
--to=dan.carpenter@linaro.org \
--cc=broonie@kernel.org \
--cc=conor+dt@kernel.org \
--cc=cosmin-gabriel.tanislav.xa@renesas.com \
--cc=devicetree@vger.kernel.org \
--cc=fabrizio.castro.jz@renesas.com \
--cc=geert+renesas@glider.be \
--cc=krzk@kernel.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=linux-spi@vger.kernel.org \
--cc=lkp@intel.com \
--cc=magnus.damm@gmail.com \
--cc=mturquette@baylibre.com \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=oe-kbuild@lists.linux.dev \
--cc=p.zabel@pengutronix.de \
--cc=robh@kernel.org \
--cc=sboyd@kernel.org \
/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