Re: [PATCH 1/1] media: Documentation: Mention CCS tools

[Laurent Pinchart <laurent.pinchart@ideasonboard.com>]

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@linux.intel.com>
> ---
>  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