On Wed Apr 30 11:27:36 2025 +0300, Dan Carpenter wrote: > The "ret" variable is not initialized on the success path. > > Fixes: a05744749600 ("media: i2c: ds90ub9xx: Set serializer temperature ramp") > Cc: sta...@vger.kernel.org > Signed-off-by: Dan Carpenter <dan.carpen...@linaro.org> > Reviewed-by: Jai Luthra <jai.lut...@ideasonboard.com> > Signed-off-by: Sakari Ailus <sakari.ai...@linux.intel.com> > Signed-off-by: Hans Verkuil <hverk...@xs4all.nl>
Patch committed. Thanks, Hans Verkuil drivers/media/i2c/ds90ub960.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- diff --git a/drivers/media/i2c/ds90ub960.c b/drivers/media/i2c/ds90ub960.c index 1877eb735cc7..665efd661882 100644 --- a/drivers/media/i2c/ds90ub960.c +++ b/drivers/media/i2c/ds90ub960.c @@ -2051,7 +2051,7 @@ static int ub960_serializer_temp_ramp(struct ub960_rxport *rxport) u8 temp_dynamic_cfg; u8 nport = rxport->nport; u8 ser_temp_code; - int ret; + int ret = 0; /* Configure temp ramp only on UB953 */ if (!fwnode_device_is_compatible(rxport->ser.fwnode, "ti,ds90ub953-q1"))