Commit 2de6bb97 authored by Jacopo Mondi's avatar Jacopo Mondi Committed by Mauro Carvalho Chehab
Browse files

media: ov5640: Provide timings accessor 



Provide a function to shortcut access to the correct timings definition
to avoid repeating the same pattern when accessing the sensor timings.

Signed-off-by: default avatarJacopo Mondi <jacopo@jmondi.org>
Reviewed-by: default avatarLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: default avatarSakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@kernel.org>
parent 5113d5b3

drivers/media/i2c/ov5640.c

+11 −5
Original line number Diff line number Diff line
@@ -1123,6 +1123,16 @@ static const struct ov5640_mode_info ov5640_mode_data[OV5640_NUM_MODES] = { 
	},

};



static const struct ov5640_timings *

ov5640_timings(const struct ov5640_dev *sensor,

	       const struct ov5640_mode_info *mode)

{

	if (ov5640_is_csi2(sensor))

		return &mode->csi2_timings;



	return &mode->dvp_timings;

}



static int ov5640_init_slave_id(struct ov5640_dev *sensor)

{

	struct i2c_client *client = sensor->i2c_client;
@@ -1635,11 +1645,7 @@ static int ov5640_set_timings(struct ov5640_dev *sensor, 
			return ret;

	}



	if (ov5640_is_csi2(sensor))

		timings = &mode->csi2_timings;

	else

		timings = &mode->dvp_timings;



	timings = ov5640_timings(sensor, mode);

	analog_crop = &timings->analog_crop;

	crop = &timings->crop;