Re: [PATCH 1/2] dt-bindings: Add ch7322 as a trivial device

[Rob Herring <robh@kernel.org>]

On Fri, Apr 24, 2020 at 01:38:18AM -0400, Jeff Chase wrote:
> The ch7322 is a Chrontel CEC controller.
> 
> Signed-off-by: Jeff Chase <jnchase@google.com>
> ---
>  Documentation/devicetree/bindings/trivial-devices.yaml | 2 ++
>  Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++
>  2 files changed, 4 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/trivial-devices.yaml b/Documentation/devicetree/bindings/trivial-devices.yaml
> index 4165352a590a..ec2ddc6cdf9a 100644
> --- a/Documentation/devicetree/bindings/trivial-devices.yaml
> +++ b/Documentation/devicetree/bindings/trivial-devices.yaml
> @@ -48,6 +48,8 @@ properties:
>            - capella,cm32181
>              # CM3232: Ambient Light Sensor
>            - capella,cm3232
> +            # CH7322: HDMI-CEC Controller
> +          - chrontel,ch7322

I don't think this qualifies as a trivial device. It has HPDI, OE and 
reset signals all likely hooked up to GPIOs. You might not have those 
hooked up for s/w control, but someone will.

And I'd assume if you had multiple instances, they will need to be 
associated with each connector.

Rob