media: adv7604: try all infoframe types
authorHans Verkuil <hverkuil-cisco@xs4all.nl>
Thu, 12 May 2022 09:36:20 +0000 (10:36 +0100)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Mon, 20 Jun 2022 09:30:30 +0000 (10:30 +0100)
When logging the infoframes the adv7604 driver stopped reading
infoframes at the first missing infoframe type. Instead it should
try all supported infoframe types.

Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
drivers/media/i2c/adv7604.c

index bb0c8fc..497419a 100644 (file)
@@ -2505,9 +2505,8 @@ static void adv76xx_log_infoframes(struct v4l2_subdev *sd)
                union hdmi_infoframe frame;
                struct i2c_client *client = v4l2_get_subdevdata(sd);
 
-               if (adv76xx_read_infoframe(sd, i, &frame))
-                       return;
-               hdmi_infoframe_log(KERN_INFO, &client->dev, &frame);
+               if (!adv76xx_read_infoframe(sd, i, &frame))
+                       hdmi_infoframe_log(KERN_INFO, &client->dev, &frame);
        }
 }