| Message ID | 20240101161601.2232247-6-dario.binacchi@amarulasolutions.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | Add display support for stm32f769-disco board | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success | |
| robh/patch-applied | success | |
| robh/dtbs-check | warning | build log |
| robh/dt-meta-schema | success |
On Mon, Jan 1, 2024 at 5:16 PM Dario Binacchi <dario.binacchi@amarulasolutions.com> 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@amarulasolutions.com> Reviewed-by: Linus Walleij <linus.walleij@linaro.org> Yours, Linus Walleij
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@amarulasolutions.com> > > --- > > 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
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 description: This indicates the panel manufacturer of the panel that is in turn using the NT35510 panel driver. The compatible string determines how the NT35510 panel driver shall be configured
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@amarulasolutions.com> --- 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(-)