Re: [PATCH V2] clk: keystone: sci-clk: fix application of sizeof to pointer

[Brian Masney]

On Tue, May 12, 2026 at 06:00:28AM -0500, Nishanth Menon wrote:
> From: Jing Yangyang <jing.yangyang@xxxxxxxxxx>
> 
> Coccinelle (scripts/coccinelle/misc/noderef.cocci) reports:
> 
>   drivers/clk/keystone/sci-clk.c:391:8-14: ERROR: application of
>   sizeof to pointer
> 
> In sci_clk_get(), 'clk' is declared as 'struct sci_clk **', so
> sizeof(clk) is sizeof(struct sci_clk **) which is the size of a
> pointer rather than the size of an array element. provider->clocks
> is an array of 'struct sci_clk *', so the canonical size argument
> to bsearch() is sizeof(*clk) (i.e. sizeof(struct sci_clk *)).
> 
> The two values are equal on every supported architecture, so this
> is correctness/idiom, not a runtime fix, but the new form matches
> the rest of the bsearch() callers in the tree and silences the
> Coccinelle warning the script flagged.
> 
> Reported-by: Zeal Robot <zealci@xxxxxxxxxx>
> Closes: https://lore.kernel.org/all/84a6ba16686347099a3dab2e5161a930e792eb6e.1629198281.git.jing.yangyang@xxxxxxxxxx/
> Reported-by: kernel test robot <lkp@xxxxxxxxx>
> Reported-by: Julia Lawall <julia.lawall@xxxxxxxx>
> Closes: https://lore.kernel.org/all/202512040525.zrHSDl5h-lkp@xxxxxxxxx/
> Link: https://lore.kernel.org/linux-clk/20211012021931.176727-1-davidcomponentone@xxxxxxxxx/
> Reviewed-by: Stepan Ionichev <sozdayvek@xxxxxxxxx>
> Reviewed-by: Andrew Davis <afd@xxxxxx>
> Signed-off-by: Jing Yangyang <jing.yangyang@xxxxxxxxxx>
> Signed-off-by: David Yang <davidcomponentone@xxxxxxxxx>
> [nm@xxxxxx: Improved commit message]
> Signed-off-by: Nishanth Menon <nm@xxxxxx>

Reviewed-by: Brian Masney <bmasney@xxxxxxxxxx>