Re: [PATCH v4 06/19] dt-bindings: media: i2c: max96717: add support for MAX96793

[Cosmin Tanislav]

On 6/24/25 7:14 PM, Martin Hecht wrote:
> Hi Cosmin,
>
> I'm preparing a test with Alvium + max96717 + max96716. I can't see away 
> to overwrite cfg settings to force pixel mode or tunnel mode. Any plans 
> to add this? Sometimes it is required to force the other mode than set 
> by pin-strapping.
>
> BR Martin

Hi Martin.

Pixel/tunnel mode is chosen automatically based on HW capabilities
routing, formats, etc.

I don't plan on adding a way to force it, since the presumption
is that the serdes framework will pick the best use case available, as
long as all the data can make its way to the PHY.

Forcing pixel/tunnel mode shouldn't be done via device-tree since it's
not related to the way hardware is wired up to the rest of the system.

> On 6/18/25 11:58, Cosmin Tanislav wrote:
> > MAX96793 is a newer variant of the MAX96717 which also supports GMSL3
> > links.
> >
> > Document this compatibility.
> >
> > Signed-off-by: Cosmin Tanislav <demonsingur@xxxxxxxxx>
> > Acked-by: Rob Herring (Arm) <robh@xxxxxxxxxx>
> > ---
> >   .../devicetree/bindings/media/i2c/maxim,max96717.yaml          | 3 +++
> >   1 file changed, 3 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/media/i2c/ 
> > maxim,max96717.yaml b/Documentation/devicetree/bindings/media/i2c/ 
> > maxim,max96717.yaml
> > index ab46a5f0bd7e..23f611177a87 100644
> > --- a/Documentation/devicetree/bindings/media/i2c/maxim,max96717.yaml
> > +++ b/Documentation/devicetree/bindings/media/i2c/maxim,max96717.yaml
> > @@ -30,6 +30,8 @@ description:
> >     MAX9295A only supports pixel mode.
> > +  MAX96793 also supports GMSL3 mode.
> > +
> >   properties:
> >     compatible:
> >       oneOf:
> > @@ -39,6 +41,7 @@ properties:
> >         - items:
> >             - enum:
> >                 - maxim,max96717
> > +              - maxim,max96793
> >             - const: maxim,max96717f
> >     '#gpio-cells':