Re: [PATCH] clk: tenstorrent: Assign .num before accessing .hws

[patchwork-bot+linux-riscv@kernel.org]

Hello:

This patch was applied to riscv/linux.git (fixes)
by Drew Fustini (Tenstorrent) <fustini@kernel.org>:

On Sat,  5 Sep 2026 21:48:35 +0100 you wrote:
> Commit f316cdff8d67 ("clk: Annotate struct clk_hw_onecell_data with
> __counted_by") annotated the hws member of 'struct clk_hw_onecell_data'
> with __counted_by, which informs the bounds sanitizer (UBSAN_BOUNDS)
> about the number of elements in .hws[], so that it can warn when .hws[]
> is accessed out of bounds. As noted in that change, the __counted_by
> member must be initialized with the number of elements before the first
> array access happens, otherwise there will be a warning from each access
> prior to the initialization because the number of elements is zero.
> This occurs in atlantis_prcm_clocks_register() due to .num being
> assigned only after every clock has been stored in .hws[]. With
> CONFIG_UBSAN_BOUNDS and a compiler that implements __counted_by (GCC
> 15.1+ or Clang 20.1+), this triggers an array-index-out-of-bounds report
> during probe, and with CONFIG_UBSAN_TRAP the first store traps so the
> clock provider never registers.
> 
> [...]

Here is the summary with links:
  - clk: tenstorrent: Assign .num before accessing .hws
    https://git.kernel.org/riscv/c/da9f1312b06e

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html