Hi Sakari, Thank you for the patch. On Fri, Sep 15, 2023 at 01:02:35AM +0300, Sakari Ailus wrote: > CCS tools is a useful set of programs and libraries for working with CCS > static data. Do mention that in context of the CCS driver. > > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > --- > Documentation/driver-api/media/drivers/ccs/ccs.rst | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/Documentation/driver-api/media/drivers/ccs/ccs.rst b/Documentation/driver-api/media/drivers/ccs/ccs.rst > index 7389204afcb8..07249f7d7863 100644 > --- a/Documentation/driver-api/media/drivers/ccs/ccs.rst > +++ b/Documentation/driver-api/media/drivers/ccs/ccs.rst > @@ -114,4 +114,12 @@ The PLL model implemented by the PLL calculator corresponds to MIPI CCS 1.1. > > .. kernel-doc:: drivers/media/i2c/ccs-pll.h > > +CCS tools > +========= > + > +`CCS tools <https://github.com/MIPI-Alliance/ccs-tools/>`_ is a set of > +tools for working with CCS static data files. CCS tools includes a > +definition of the human-readable CCS static data YAML format and includes a > +program to convert it to a binary. The text looks good to me, but how about moving it to the "CCS static data" section above ? > + > **Copyright** |copy| 2020 Intel Corporation -- Regards, Laurent Pinchart