diff mbox series

[v2,1/3] dt-bindings: media: tegra-cec: Document Tegra186 and Tegra194

Message ID 20181211094841.16027-1-thierry.reding@gmail.com
State Deferred
Headers show
Series [v2,1/3] dt-bindings: media: tegra-cec: Document Tegra186 and Tegra194 | expand

Commit Message

Thierry Reding Dec. 11, 2018, 9:48 a.m. UTC
From: Thierry Reding <treding@nvidia.com>

The Tegra186 and Tegra194 contain a CEC controller that is identical to
that found in earlier generations. Document the compatible strings for
these newer chips.

Signed-off-by: Thierry Reding <treding@nvidia.com>
---
Changes in v2:
- new patch adding missing compatible strings

 Documentation/devicetree/bindings/media/tegra-cec.txt | 2 ++
 1 file changed, 2 insertions(+)

Comments

Rob Herring Dec. 18, 2018, 4:42 p.m. UTC | #1
On Tue, Dec 11, 2018 at 10:48:39AM +0100, Thierry Reding wrote:
> From: Thierry Reding <treding@nvidia.com>
> 
> The Tegra186 and Tegra194 contain a CEC controller that is identical to
> that found in earlier generations. Document the compatible strings for
> these newer chips.

If identical, why don't you have a fallback compatible?

> 
> Signed-off-by: Thierry Reding <treding@nvidia.com>
> ---
> Changes in v2:
> - new patch adding missing compatible strings
> 
>  Documentation/devicetree/bindings/media/tegra-cec.txt | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/media/tegra-cec.txt b/Documentation/devicetree/bindings/media/tegra-cec.txt
> index c503f06f3b84..da3590f0b7a1 100644
> --- a/Documentation/devicetree/bindings/media/tegra-cec.txt
> +++ b/Documentation/devicetree/bindings/media/tegra-cec.txt
> @@ -8,6 +8,8 @@ Required properties:
>  	"nvidia,tegra114-cec"
>  	"nvidia,tegra124-cec"
>  	"nvidia,tegra210-cec"
> +	"nvidia,tegra186-cec"
> +	"nvidia,tegra194-cec"
>    - reg : Physical base address of the IP registers and length of memory
>  	  mapped region.
>    - interrupts : HDMI CEC interrupt number to the CPU.
> -- 
> 2.19.1
>
Thierry Reding Dec. 18, 2018, 9:32 p.m. UTC | #2
On Tue, Dec 18, 2018 at 10:42:30AM -0600, Rob Herring wrote:
> On Tue, Dec 11, 2018 at 10:48:39AM +0100, Thierry Reding wrote:
> > From: Thierry Reding <treding@nvidia.com>
> > 
> > The Tegra186 and Tegra194 contain a CEC controller that is identical to
> > that found in earlier generations. Document the compatible strings for
> > these newer chips.
> 
> If identical, why don't you have a fallback compatible?

That's a good point. I think it would be fine to always fall back to the
nvidia,tegra20-cec compatible.

Thierry
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/media/tegra-cec.txt b/Documentation/devicetree/bindings/media/tegra-cec.txt
index c503f06f3b84..da3590f0b7a1 100644
--- a/Documentation/devicetree/bindings/media/tegra-cec.txt
+++ b/Documentation/devicetree/bindings/media/tegra-cec.txt
@@ -8,6 +8,8 @@  Required properties:
 	"nvidia,tegra114-cec"
 	"nvidia,tegra124-cec"
 	"nvidia,tegra210-cec"
+	"nvidia,tegra186-cec"
+	"nvidia,tegra194-cec"
   - reg : Physical base address of the IP registers and length of memory
 	  mapped region.
   - interrupts : HDMI CEC interrupt number to the CPU.