Re: [PATCH 1/2] dt-bindings: display/panel: Add Innolux EJ030NA

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Sam, thanks for the review.

Le sam., juil. 10 2021 at 08:14:43 +0200, Sam Ravnborg <sam@xxxxxxxxxxxx> a écrit :
Hi Paul,

On Fri, Jun 25, 2021 at 01:10:44PM +0100, Paul Cercueil wrote:
Add binding for the Innolux EJ030NA panel, which is a 320x480 3.0" 4:3
 24-bit TFT LCD panel with non-square pixels and a delta-RGB 8-bit
 interface.

 Signed-off-by: Paul Cercueil <paul@xxxxxxxxxxxxxxx>
 ---
.../display/panel/innolux,ej030na.yaml | 62 +++++++++++++++++++
  1 file changed, 62 insertions(+)
create mode 100644 Documentation/devicetree/bindings/display/panel/innolux,ej030na.yaml

diff --git a/Documentation/devicetree/bindings/display/panel/innolux,ej030na.yaml b/Documentation/devicetree/bindings/display/panel/innolux,ej030na.yaml
 new file mode 100644
 index 000000000000..cda36c04e85c
 --- /dev/null
+++ b/Documentation/devicetree/bindings/display/panel/innolux,ej030na.yaml
 @@ -0,0 +1,62 @@
 +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
 +%YAML 1.2
 +---
+$id: http://devicetree.org/schemas/display/panel/innolux,ej030na.yaml#
 +$schema: http://devicetree.org/meta-schemas/core.yaml#
 +
 +title: Innolux EJ030NA 3.0" (320x480 pixels) 24-bit TFT LCD panel
 +
 +description: |
+ The panel must obey the rules for a SPI slave device as specified in
 +  spi/spi-controller.yaml
 +
 +maintainers:
 +  - Paul Cercueil <paul@xxxxxxxxxxxxxxx>
 +
 +allOf:
 +  - $ref: panel-common.yaml#
 +
 +properties:
 +  compatible:
 +    const: innolux,ej030na
 +
 +  backlight: true
 +  port: true
 +  power-supply: true
 +  reg: true
 +  reset-gpios: true
 +
 +required:
 +  - compatible
 +  - reg
 +  - power-supply
 +  - reset-gpios
 +
 +unevaluatedProperties: false
I had expected:
additionalProperties: false

With this fixed:
Reviewed-by: Sam Ravnborg <sam@xxxxxxxxxxxx>

I am not sure; the doc states that this (additionalProperties: false) "can't be used in case where another schema is referenced", which is the case here, as we include "panel-common.yaml".

Cheers,
-Paul

 +
 +examples:
 +  - |
 +    #include <dt-bindings/gpio/gpio.h>
 +
 +    spi {
 +        #address-cells = <1>;
 +        #size-cells = <0>;
 +
 +        panel@0 {
 +            compatible = "innolux,ej030na";
 +            reg = <0>;
 +
 +            spi-max-frequency = <10000000>;
 +
 +            reset-gpios = <&gpe 4 GPIO_ACTIVE_LOW>;
 +            power-supply = <&lcd_power>;
 +
 +            backlight = <&backlight>;
 +
 +            port {
 +                panel_input: endpoint {
 +                    remote-endpoint = <&panel_output>;
 +                };
 +            };
 +        };
 +    };
 --
 2.30.2





[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux