[PATCH] clk: keystone: sci-clk: fix application of sizeof to pointer
Nishanth Menon
nm at ti.com
Fri May 8 10:07:51 PDT 2026
On 11:59-20260508, Andrew Davis wrote:
[...]
> > - Functionality: No impact (on Linux architectures), thus no Fixes/Stable tag.
>
> Not sure what "on Linux architectures" means here, this should be true for any
> sane system, right? You are going from the sizeof(pointer-to-pointer) to the
> sizeof(pointer), both should be equal. This is just to be more technically
Yep (also why i put it in diffstat notes ;) ) - any sane system
(which is what we support in Linux ;) ) - though C standard does'nt
explicitly call that out from what I could gather, it is practically
what all systems tend to do. i had the extreme paranoia about this
and got claude[1] to check it out using clang.. Results match the
expectation.
> correct and silence a Coccinelle warning. Anyway,
Yep, That is all this is.. patches keep popping up every few years.
>
> Reviewed-by: Andrew Davis <afd at ti.com>
>
Thanks.
[1] https://gist.githubusercontent.com/nmenon/afa9ac915e189334f048c177f16fe54f/raw/ccd08843ff7193090f9fbdca3f5c76d0a4366c85/quick%2520check%2520with%2520clang
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
https://ti.com/opensource
More information about the linux-arm-kernel
mailing list