Re: [PATCH v9 00/21] media: i2c: add Maxim GMSL2/3 serializer and deserializer drivers
From: Tomi Valkeinen
Date: Wed Apr 01 2026 - 04:06:57 EST
On 01/04/2026 09:47, Tomi Valkeinen wrote:
> Hi,
>
> On 30/03/2026 10:15, Ceclan Dumitru wrote:
>>
>>
>> On 3/11/26 9:58 AM, Sakari Ailus wrote:
>>> Hi Dumitru,
>>>
>>> On Wed, Mar 11, 2026 at 09:17:12AM +0200, Dumitru Ceclan via B4 Relay wrote:
>>
>> ...
>>
>>>
>>> ...
>>>
>>>> The following v4l2-compliance test still fails:
>>>> fail: v4l2-test-subdevs.cpp(371): fmt.code == 0 || fmt.code == ~0U
>>>> fail: v4l2-test-subdevs.cpp(418): checkMBusFrameFmt(node, fmt.format)
>>>> test Active VIDIOC_SUBDEV_G/S_FMT: FAIL
>>>
>>> Could you post the full report here, please?
>>>
> So why do these failures happen? It wasn't obvious to me from the full
> report either.
I made a quick test with FPD-Link, and I see something similar. It's
caused (for fpdlink, at least) by the serializer accepting any format,
as it's a pass-through device.
As a hack test, can you try adding something like this to the set_fmt op:
if (format->format.code == ~0)
format->format.code = MEDIA_BUS_FMT_UYVY8_1X16;
If all the compliance errors go away, then it's the same case for you.
Tomi