@@ -589,7 +589,7 @@ static int ov2680_enum_mbus_code(struct v4l2_subdev *sd,
{
struct ov2680_dev *sensor = to_ov2680_dev(sd);
- if (code->pad != 0 || code->index != 0)
+ if (code->index != 0)
return -EINVAL;
code->code = sensor->mode.fmt.code;
@@ -604,9 +604,6 @@ static int ov2680_get_fmt(struct v4l2_subdev *sd,
struct ov2680_dev *sensor = to_ov2680_dev(sd);
struct v4l2_mbus_framefmt *fmt;
- if (format->pad != 0)
- return -EINVAL;
-
fmt = __ov2680_get_pad_format(sensor, sd_state, format->pad,
format->which);
@@ -627,9 +624,6 @@ static int ov2680_set_fmt(struct v4l2_subdev *sd,
unsigned int width, height;
int ret = 0;
- if (format->pad != 0)
- return -EINVAL;
-
crop = __ov2680_get_pad_crop(sensor, sd_state, format->pad,
format->which);
Drop unnecessary pad checks in enum_mbus_code, get_fmt, set_fmt this is already checked by check_pad() from drivers/media/v4l2-core/v4l2-subdev.c. Signed-off-by: Hans de Goede <hdegoede@redhat.com> --- drivers/media/i2c/ov2680.c | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-)