Re: [PATCH 1/8] dt-bindings: media: renesas, fcp: Convert binding to YAML

Hi Laurent,

On Fri, May 15, 2020 at 3:42 AM Laurent Pinchart laurent.pinchart+renesas@ideasonboard.com wrote:

Convert the Renesas R-Car FCP text binding to YAML.

Signed-off-by: Laurent Pinchart laurent.pinchart+renesas@ideasonboard.com

Reviewed-by: Geert Uytterhoeven geert+renesas@glider.be

--- /dev/null +++ b/Documentation/devicetree/bindings/media/renesas,fcp.yaml @@ -0,0 +1,56 @@ +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/media/renesas,fcp.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml#

+title: Renesas R-Car Frame Compression Processor (FCP)

+maintainers:

• • Laurent Pinchart laurent.pinchart@ideasonboard.com

+description: |

• The FCP is a companion module of video processing modules in the Renesas
• R-Car Gen3 and RZ/G2 SoCs. It provides data compression and decompression,
• data caching, and conversion of AXI transactions in order to reduce the
• memory bandwidth.
• There are three types of FCP: FCP for Codec (FCPC), FCP for VSP (FCPV) and
• FCP for FDP (FCPF). Their configuration and behaviour depend on the module
• they are paired with. These DT bindings currently support the FCPV and FCPF.

+properties:

• compatible:
• enum:

•  - renesas,fcpv # for 'FCP for VSP'
  

•  - renesas,fcpf # for 'FCP for FDP'
  

I would drop the "for" part, and just write e.g. "# FCP for VSP".

Gr{oetje,eeting}s,

Geert