Hi Sakari On Tue, Dec 10, 2024 at 09:59:04AM +0200, Sakari Ailus wrote: > Add the get_mbus_config() as the means for conveying the link frequency > towards the receiver drivers. > > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > --- > Documentation/driver-api/media/tx-rx.rst | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/Documentation/driver-api/media/tx-rx.rst b/Documentation/driver-api/media/tx-rx.rst > index dd09484df1d3..1430c330fd52 100644 > --- a/Documentation/driver-api/media/tx-rx.rst > +++ b/Documentation/driver-api/media/tx-rx.rst > @@ -49,6 +49,10 @@ Link frequency > The :ref:`V4L2_CID_LINK_FREQ <v4l2-cid-link-freq>` control is used to tell the > receiver the frequency of the bus (i.e. it is not the same as the symbol rate). > > +On devices where the link frequency isn't configurable, the link_freq field of > +struct v4l2_mbus_config is recommended over controls for conveying the link > +frequency to the downstream driver in the pipeline. When read in its entirety, the paragraphs Link frequency ^^^^^^^^^^^^^^ The :ref:`V4L2_CID_LINK_FREQ <v4l2-cid-link-freq>` control is used to tell the receiver the frequency of the bus (i.e. it is not the same as the symbol rate). On devices where the link frequency isn't configurable, the link_freq field of struct v4l2_mbus_config is recommended over controls for conveying the link frequency. Sounds simpler without the last 7 words. A nit and just tastes maybe I like where this is going! Reviewed-by: Jacopo Mondi <jacopo.mondi@xxxxxxxxxxxxxxxx> > + > ``.enable_streams()`` and ``.disable_streams()`` callbacks > ^^^^^^^^^^^^^^^^^^^^^^^^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > > -- > 2.39.5 > >
