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"))

Reply via email to