Re: [PATCH v4 1/3] dt-bindings: media: Add bindings for THine THP7312 ISP

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Laurent,

On Fri, Oct 27, 2023 at 03:46:34PM +0300, Laurent Pinchart wrote:
> > > > > +  port:
> > > > > +    $ref: /schemas/graph.yaml#/$defs/port-base
> > > > > +    additionalProperties: false
> > > > > +
> > > > > +    properties:
> > > > > +      endpoint:
> > > > > +        $ref: /schemas/media/video-interfaces.yaml#
> > > > > +        unevaluatedProperties: false
> > > > > +
> > > > > +        properties:
> > > > > +          data-lanes:
> > > > > +            description:
> > > > > +              This property is for lane reordering between the THP7312 and the
> > > > > +              SoC. The sensor supports either two-lane, or four-lane operation.
> > > > > +              If this property is omitted four-lane operation is assumed. For
> > > > > +              two-lane operation the property must be set to <1 2>.
> > > > 
> > > > How are link frequencies determined by the device?
> > > 
> > > The link frequency is not under direct control of the host. There are
> > > registers to select between mode presets, each of them producing a
> > > particular link frequency.
> > 
> > If they are known, one could only allow setting the modes with known-good
> > link requencies. This could be added later on, too, affecting systems with
> > link-frequencies set. I guess you could also add it now, as optional.
> 
> I vote for later :-)
> 
> > Where's the link frequency information available? In documentation or in
> > on the firmware API?
> 
> In the documentation (which is not public I'm afraid).

Could you add this information in the driver?

Co-incidentally, the driver is lacking the LINK_FREQ control. It should be
added.

-- 
Regards,

Sakari Ailus



[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux