media: i2c: dw9714: Fix occasional probe errors

Ondrej Jirman · Hans Verkuil · commit 9d00ccabfbb5 · 2024-02-23T14:33:32.000+01:00
The powerup delay was not observed during probe, leading to occasional
I2C communication failures in RPM suspend callback. Power delay is
properly observed in resume callback already.

Signed-off-by: Ondrej Jirman &lt;megi@xff.cz&gt;
Signed-off-by: Sakari Ailus &lt;sakari.ailus@linux.intel.com&gt;
Signed-off-by: Hans Verkuil &lt;hverkuil-cisco@xs4all.nl&gt;
diff --git a/drivers/media/i2c/dw9714.c b/drivers/media/i2c/dw9714.c
@@ -157,6 +157,8 @@ static int dw9714_probe(struct i2c_client *client)
 		return rval;
 	}
 
+	usleep_range(1000, 2000);
+
 	v4l2_i2c_subdev_init(&dw9714_dev->sd, client, &dw9714_ops);
 	dw9714_dev->sd.flags |= V4L2_SUBDEV_FL_HAS_DEVNODE |
 				V4L2_SUBDEV_FL_HAS_EVENTS;

Original file line number	Diff line number	Diff line change
`@@ -157,6 +157,8 @@ static int dw9714_probe(struct i2c_client *client)`
`157`	`157`	`return rval;`
`158`	`158`	`}`
`159`	`159`
	`160`	`+ usleep_range(1000, 2000);`
	`161`	`+`
`160`	`162`	`v4l2_i2c_subdev_init(&dw9714_dev->sd, client, &dw9714_ops);`
`161`	`163`	`dw9714_dev->sd.flags \|= V4L2_SUBDEV_FL_HAS_DEVNODE \|`
`162`	`164`	`V4L2_SUBDEV_FL_HAS_EVENTS;`