[PATCH v6 09/16] media: v4l2-subdev: Always call get_fmt() if set_fmt() is unavailable

[Sakari Ailus <sakari.ailus@linux.intel.com>]

Many drivers implement set_fmt() pad op that simply returns the format
just as get_fmt() would do, usually because the driver only supports a
single one. As a driver should always implement set_fmt() either on its
own or via get_fmt(), call get_fmt() in case set_fmt() isn't implemented
by the driver. This allows removing a fair amount of boilerplate code as
well as ensures that set_fmt() works whenever get_fmt() does.

Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
---
 drivers/media/v4l2-core/v4l2-subdev.c | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/drivers/media/v4l2-core/v4l2-subdev.c b/drivers/media/v4l2-core/v4l2-subdev.c
index 86be4d51c9a5..c2be59e4c4a7 100644
--- a/drivers/media/v4l2-core/v4l2-subdev.c
+++ b/drivers/media/v4l2-core/v4l2-subdev.c
@@ -260,8 +260,11 @@ static int call_set_fmt(struct v4l2_subdev *sd,
 			struct v4l2_subdev_state *state,
 			struct v4l2_subdev_format *format)
 {
-	return do_subdev_call(sd, check_format(sd, state, format), pad, set_fmt,
-			      state, format);
+	return check_format(sd, state, format) ? :
+		sd->ops->pad->set_fmt ?
+		sd->ops->pad->set_fmt(sd, state, format) :
+		sd->ops->pad->get_fmt ?
+		sd->ops->pad->get_fmt(sd, state, format) : -ENOIOCTLCMD;
 }
 
 static int call_enum_mbus_code(struct v4l2_subdev *sd,
-- 
2.47.3