Re: [PATCH v2 01/11] dt-bindings: update the binding for Allwinner H3 TVE support

Maxime Ripard
2017-06-09T16:49:52Z

[-- Attachment #1: Type: text/plain, Size: 747 bytes --]

On Wed, Jun 07, 2017 at 04:48:50PM +0800, Icenowy Zheng wrote:
> >> @@ -189,6 +211,8 @@ supported.
> >> Required properties:
> >> - compatible: value must be one of:
> >> * allwinner,sun8i-v3s-de2-mixer
> >> + * allwinner,sun8i-h3-de2-mixer0
> >> + * allwinner,sun8i-h3-de2-mixer1
> >
> >Again, please explain why we need to have different compatibles
> >here. If it's only about the number of planes, this should be dealt
> >with a property, not a compatible.
>
> Only mixer0 has "VEP" and write-back support, at least on H3.

What is that VEP? writeback support can also be expressed by a
property.

Maxime

--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]