Re: [PATCH 04/19] dt-bindings: bridge: simple: Add corpro,gm7123 compatible

Laurent Pinchart
2021-01-15T07:18:26Z

Hi Maxime,

Thank you for the patch.

On Thu, Jan 14, 2021 at 12:35:23PM +0100, Maxime Ripard wrote:
> The corpro,gm7123 was in use in a DT but was never properly documented,
> let's add it.
>
> Cc: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> Cc: Sam Ravnborg <sam@ravnborg.org>
> Signed-off-by: Maxime Ripard <maxime@cerno.tech>
> ---
> .../devicetree/bindings/display/bridge/simple-bridge.yaml | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/display/bridge/simple-bridge.yaml b/Documentation/devicetree/bindings/display/bridge/simple-bridge.yaml
> index 64e8a1c24b40..97ca07b56cbc 100644
> --- a/Documentation/devicetree/bindings/display/bridge/simple-bridge.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/simple-bridge.yaml
> @@ -22,6 +22,10 @@ properties:
> - ti,ths8134a
> - ti,ths8134b
> - const: ti,ths8134
> + - items:
> + - const: corpro,gm7123
> + - const: adi,adv7123

Do we need adi,adv7123 ?

> + - const: dumb-vga-dac
> - enum:
> - adi,adv7123
> - dumb-vga-dac

--
Regards,

Laurent Pinchart