From: Kieran Bingham <kieran.bingham+renesas@xxxxxxxxxxxxxxxx>
Disallow setting a format on the source link, but enable link validation
by returning the format of the first bound source when getting the
format of the source pad.
Signed-off-by: Kieran Bingham <kieran.bingham+renesas@xxxxxxxxxxxxxxxx>
---
drivers/media/i2c/max9286.c | 19 +++++++++++++++----
1 file changed, 15 insertions(+), 4 deletions(-)
diff --git a/drivers/media/i2c/max9286.c b/drivers/media/i2c/max9286.c
index ef824d2b26b8..7e59391a5736 100644
--- a/drivers/media/i2c/max9286.c
+++ b/drivers/media/i2c/max9286.c
@@ -711,7 +711,11 @@ static int max9286_set_fmt(struct v4l2_subdev *sd,
struct max9286_priv *priv = sd_to_max9286(sd);
struct v4l2_mbus_framefmt *cfg_fmt;
- if (format->pad >= MAX9286_SRC_PAD)
+ /* TODO:
+ * Multiplexed Stream Support: Prevent setting the format on the shared
+ * multiplexed bus.
+ */
+ if (format->pad == MAX9286_SRC_PAD)
return -EINVAL;
/* Refuse non YUV422 formats as we hardcode DT to 8 bit YUV422 */
@@ -743,11 +747,18 @@ static int max9286_get_fmt(struct v4l2_subdev *sd,
{
struct max9286_priv *priv = sd_to_max9286(sd);
struct v4l2_mbus_framefmt *cfg_fmt;
+ unsigned int pad = format->pad;
- if (format->pad >= MAX9286_SRC_PAD)
- return -EINVAL;
+ /* TODO:
+ * Multiplexed Stream Support: Support link validation by returning the
+ * format of the first bound link. All links must have the same format,
+ * as we do not support mixing, and matching of cameras connected to
+ * the max9286.
+ */
+ if (pad == MAX9286_SRC_PAD)
+ pad = __ffs(priv->bound_sources);
- cfg_fmt = max9286_get_pad_format(priv, cfg, format->pad, format->which);
+ cfg_fmt = max9286_get_pad_format(priv, cfg, pad, format->which);
if (!cfg_fmt)
return -EINVAL;
--
2.25.1
