]> www.infradead.org Git - users/jedix/linux-maple.git/commit
i2c: k1: Initialize variable before use
authorAndi Shyti <andi.shyti@kernel.org>
Thu, 20 Mar 2025 11:35:21 +0000 (12:35 +0100)
committerAndi Shyti <andi.shyti@kernel.org>
Sat, 22 Mar 2025 12:11:52 +0000 (13:11 +0100)
commit088b1ca970ba6cac141f684b7592ce56bd25e7ea
tree601ad7ade227c079c10e7917437b7326a0ee8732
parent5ea558473fa31f4eeb5c76d58277a7ab68fd501d
i2c: k1: Initialize variable before use

Commit 95a8ca229032 ("i2c: spacemit: add support for SpacemiT K1
SoC") introduced a check in the probe function to warn the user
if the DTS has incorrect frequency settings. In such cases, the
driver falls back to default values.

However, the return value of of_property_read_u32() was not
stored, making the check ineffective. Fix this by storing the
return value in 'ret' and evaluating it properly.

Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202503200928.eBWfwnHG-lkp@intel.com/
Cc: Troy Mitchell <troymitchell988@gmail.com>
Link: https://lore.kernel.org/r/20250320113521.3966762-1-andi.shyti@kernel.org
Signed-off-by: Andi Shyti <andi.shyti@kernel.org>
drivers/i2c/busses/i2c-k1.c