Re: [PATCH v2 5/8] dt-bindings: nt35510: add compatible for FRIDA FRD400B25025-A-CTK

From: Krzysztof Kozlowski
Date: Tue Jan 02 2024 - 16:15:51 EST


On 01/01/2024 17:15, Dario Binacchi wrote:
> The patch adds the FRIDA FRD400B25025-A-CTK panel, which belongs to the
> Novatek NT35510-based panel family.
>
> Signed-off-by: Dario Binacchi <dario.binacchi@xxxxxxxxxxxxxxxxxxxx>
>
> ---
>
> Changes in v2:
> - Add a dash in front of each "items:"
>
> .../bindings/display/panel/novatek,nt35510.yaml | 10 +++++++---
> 1 file changed, 7 insertions(+), 3 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml b/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
> index bc92928c805b..8e69446e00e0 100644
> --- a/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
> @@ -14,9 +14,13 @@ allOf:
>
> properties:
> compatible:
> - items:
> - - const: hydis,hva40wv1
> - - const: novatek,nt35510
> + oneOf:
> + - items:
> + - const: hydis,hva40wv1
> + - const: novatek,nt35510
> + - items:
> + - const: frida,frd400b25025
> + - const: novatek,nt35510

If fallback compatibles are the same, just make the first item as enum.
Less code.

Best regards,
Krzysztof