Message ID | 20230129-ov2685-improvements-v1-2-f281bd49399c@z3ntu.xyz (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | Improvements for OmniVision OV2685 driver | expand |
Hi Luca On Sun, Jan 29, 2023 at 10:42:36AM +0100, Luca Weiss wrote: > If the sensor doens't power up correctly, for example due to incorrect > devicetree description, the power up i2c writes will fail. > > Add an error print for this situation. > > Signed-off-by: Luca Weiss <luca@z3ntu.xyz> > --- > drivers/media/i2c/ov2685.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/media/i2c/ov2685.c b/drivers/media/i2c/ov2685.c > index a422f4c8a2eb..844a91dbc8e5 100644 > --- a/drivers/media/i2c/ov2685.c > +++ b/drivers/media/i2c/ov2685.c > @@ -419,8 +419,10 @@ static int __ov2685_power_on(struct ov2685 *ov2685) > * writing register before .s_stream() as a workaround > */ > ret = ov2685_write_array(ov2685->client, ov2685->cur_mode->reg_list); > - if (ret) > + if (ret) { > + dev_err(dev, "Failed to set regs for power on\n"); > goto disable_supplies; > + } This is fine. I would also consider if it's worth to fail loud in ov2685_write_reg(). Reviewed-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com> > > return 0; > > > -- > 2.39.1 >
diff --git a/drivers/media/i2c/ov2685.c b/drivers/media/i2c/ov2685.c index a422f4c8a2eb..844a91dbc8e5 100644 --- a/drivers/media/i2c/ov2685.c +++ b/drivers/media/i2c/ov2685.c @@ -419,8 +419,10 @@ static int __ov2685_power_on(struct ov2685 *ov2685) * writing register before .s_stream() as a workaround */ ret = ov2685_write_array(ov2685->client, ov2685->cur_mode->reg_list); - if (ret) + if (ret) { + dev_err(dev, "Failed to set regs for power on\n"); goto disable_supplies; + } return 0;
If the sensor doens't power up correctly, for example due to incorrect devicetree description, the power up i2c writes will fail. Add an error print for this situation. Signed-off-by: Luca Weiss <luca@z3ntu.xyz> --- drivers/media/i2c/ov2685.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)