From: Martin Hecht <mhecht73@gmail.com>
To: Sakari Ailus <sakari.ailus@linux.intel.com>, dumitru.ceclan@analog.com
Cc: "Tomi Valkeinen" <tomi.valkeinen+renesas@ideasonboard.com>,
"Mauro Carvalho Chehab" <mchehab@kernel.org>,
"Laurent Pinchart" <laurent.pinchart@ideasonboard.com>,
"Julien Massot" <julien.massot@collabora.com>,
"Rob Herring" <robh@kernel.org>,
"Niklas Söderlund" <niklas.soderlund@ragnatech.se>,
"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
"Cosmin Tanislav" <cosmin.tanislav@analog.com>,
mitrutzceclan@gmail.com, linux-media@vger.kernel.org,
linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
linux-staging@lists.linux.dev, linux-gpio@vger.kernel.org,
"Niklas Söderlund" <niklas.soderlund+renesas@ragnatech.se>,
"Martin Hecht" <Martin.Hecht@avnet.eu>,
"Tomi Valkeinen" <tomi.valkeinen@ideasonboard.com>,
"Cosmin Tanislav" <demonsingur@gmail.com>,
"Cory Keitz" <ckeitz@amazon.com>
Subject: Re: [PATCH v9 00/21] media: i2c: add Maxim GMSL2/3 serializer and deserializer drivers
Date: Fri, 13 Mar 2026 09:32:18 +0100 [thread overview]
Message-ID: <2bcc0442-cdd7-4093-831c-0bd4652edf56@gmail.com> (raw)
In-Reply-To: <abEgijQAqW27i5fU@kekkonen.localdomain>
Hi Sakari, hi Dumitru,
I'm testing v8 already and will migrate next week to that version. Below
some further comments.
On 3/11/26 08:58, Sakari Ailus wrote:
> Hi Dumitru,
>
> On Wed, Mar 11, 2026 at 09:17:12AM +0200, Dumitru Ceclan via B4 Relay wrote:
>> The following deserializers are supported:
>> * MAX96712 (already exists in staging)
>> * MAX96714 (already exists)
>> * MAX96714F (already exists)
>> * MAX96714R (GMSL2)
>> * MAX96716 (GMSL2)
>> * MAX96724 (already exists as part of existing MAX96712 driver)
>> * MAX96724F (GMSL2)
>> * MAX96724R (GMSL2)
>> * MAX9296A (GMSL2)
>> * MAX96792A (GMSL3)
>>
>> The following serializers are supported:
>> * MAX96717 (already exists)
>> * MAX9295A (GMSL2)
>> * MAX96793 (GMSL3)
>
> It'd be nice to have more reviews as well as comments from the users of the
> existing drivers especially on how this works for them.
>
> Are there differences in functionality or UAPI compared to the drivers
> already in upstream?
Compared with the current mainline drivers from Julian (what offere a
good basic functionality) there are a lot or more features enabled. One
ov them is support for routing the channels as well as i2c-atr instead
of i2c-gate only. Both helps to setup up more complex and reconfigurable
routing of the video channels in complex systems with many
sensors/cameras/streams.
Additionally a lot more and never devices are supported by that patchset
including the new GMSL3 devices what are also backward compatible to GMSL2.
Both link types are supported now. Generation of test patterns in pixel
mode has been added.
Also log-status returns a lot of very useful information about the link
mode (tunnel mode versus pixel mode) and potential errors / packet
losses over cable by presenting the counter registers. That helps to
monitor the quality and reliability of the GMSL links as well as CSI links.
>
> ...
>
>> 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?
>
BR Martin
next prev parent reply other threads:[~2026-03-13 8:32 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-03-11 7:17 [PATCH v9 00/21] media: i2c: add Maxim GMSL2/3 serializer and deserializer drivers Dumitru Ceclan via B4 Relay
2026-03-11 7:17 ` [PATCH v9 01/21] media: mc: Add INTERNAL pad flag Dumitru Ceclan via B4 Relay
2026-03-11 7:17 ` [PATCH v9 02/21] dt-bindings: media: i2c: max96717: add support for I2C ATR Dumitru Ceclan via B4 Relay
2026-03-11 7:17 ` [PATCH v9 03/21] dt-bindings: media: i2c: max96717: add support for pinctrl/pinconf Dumitru Ceclan via B4 Relay
2026-03-11 7:17 ` [PATCH v9 04/21] dt-bindings: media: i2c: max96717: add support for MAX9295A Dumitru Ceclan via B4 Relay
2026-03-11 7:17 ` [PATCH v9 05/21] dt-bindings: media: i2c: max96717: add support for MAX96793 Dumitru Ceclan via B4 Relay
2026-03-11 7:17 ` [PATCH v9 06/21] dt-bindings: media: i2c: max96712: use pattern properties for ports Dumitru Ceclan via B4 Relay
2026-03-11 7:17 ` [PATCH v9 07/21] dt-bindings: media: i2c: max96712: add support for I2C ATR Dumitru Ceclan via B4 Relay
2026-03-11 7:17 ` [PATCH v9 08/21] dt-bindings: media: i2c: max96712: add support for POC supplies Dumitru Ceclan via B4 Relay
2026-03-11 7:17 ` [PATCH v9 09/21] dt-bindings: media: i2c: max96712: add support for MAX96724F/R Dumitru Ceclan via B4 Relay
2026-03-11 7:17 ` [PATCH v9 10/21] dt-bindings: media: i2c: max96714: add support for MAX96714R Dumitru Ceclan via B4 Relay
2026-03-11 7:17 ` [PATCH v9 11/21] dt-bindings: media: i2c: add MAX9296A, MAX96716A, MAX96792A Dumitru Ceclan via B4 Relay
2026-03-11 7:17 ` [PATCH v9 12/21] media: i2c: add Maxim GMSL2/3 serializer and deserializer framework Dumitru Ceclan via B4 Relay
2026-03-11 7:17 ` [PATCH v9 13/21] media: i2c: add Maxim GMSL2/3 serializer framework Dumitru Ceclan via B4 Relay
2026-03-11 7:17 ` [PATCH v9 14/21] media: i2c: add Maxim GMSL2/3 deserializer framework Dumitru Ceclan via B4 Relay
2026-03-11 7:17 ` [PATCH v9 15/21] media: i2c: maxim-serdes: add MAX96717 driver Dumitru Ceclan via B4 Relay
2026-03-11 7:17 ` [PATCH v9 16/21] media: i2c: maxim-serdes: add MAX96724 driver Dumitru Ceclan via B4 Relay
2026-03-11 7:17 ` [PATCH v9 17/21] media: i2c: maxim-serdes: add MAX9296A driver Dumitru Ceclan via B4 Relay
2026-03-11 7:17 ` [PATCH v9 18/21] arm64: defconfig: disable deprecated MAX96712 driver Dumitru Ceclan via B4 Relay
2026-03-11 7:17 ` [PATCH v9 19/21] staging: media: remove " Dumitru Ceclan via B4 Relay
2026-03-11 7:17 ` [PATCH v9 20/21] media: i2c: remove MAX96717 driver Dumitru Ceclan via B4 Relay
2026-03-13 13:56 ` Julien Massot
2026-03-11 7:17 ` [PATCH v9 21/21] media: i2c: remove MAX96714 driver Dumitru Ceclan via B4 Relay
2026-03-13 13:58 ` Julien Massot
2026-03-11 7:58 ` [PATCH v9 00/21] media: i2c: add Maxim GMSL2/3 serializer and deserializer drivers Sakari Ailus
2026-03-13 8:32 ` Martin Hecht [this message]
2026-03-30 7:15 ` Ceclan Dumitru
2026-04-01 6:47 ` Tomi Valkeinen
2026-04-01 7:46 ` Tomi Valkeinen
2026-03-26 17:00 ` Dayananda, Vivekananda
2026-03-30 7:54 ` Ceclan Dumitru
2026-03-31 17:43 ` Dayananda, Vivekananda
2026-04-01 5:52 ` Ceclan Dumitru
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=2bcc0442-cdd7-4093-831c-0bd4652edf56@gmail.com \
--to=mhecht73@gmail.com \
--cc=Martin.Hecht@avnet.eu \
--cc=ckeitz@amazon.com \
--cc=cosmin.tanislav@analog.com \
--cc=demonsingur@gmail.com \
--cc=devicetree@vger.kernel.org \
--cc=dumitru.ceclan@analog.com \
--cc=gregkh@linuxfoundation.org \
--cc=julien.massot@collabora.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-staging@lists.linux.dev \
--cc=mchehab@kernel.org \
--cc=mitrutzceclan@gmail.com \
--cc=niklas.soderlund+renesas@ragnatech.se \
--cc=niklas.soderlund@ragnatech.se \
--cc=robh@kernel.org \
--cc=sakari.ailus@linux.intel.com \
--cc=tomi.valkeinen+renesas@ideasonboard.com \
--cc=tomi.valkeinen@ideasonboard.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).