Hi Marco. On Thu, Jan 03, 2019 at 07:26:57PM +0100, Marco Felsch wrote: > Add support for the Evervision VG804821 800x480 5.0" LCD TFT parallel > panel to DRM simple panel driver. > > Signed-off-by: Marco Felsch <m.felsch@xxxxxxxxxxxxxx> > --- > .../bindings/display/panel/evervision,vgg804821.txt | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > create mode 100644 Documentation/devicetree/bindings/display/panel/evervision,vgg804821.txt > > diff --git a/Documentation/devicetree/bindings/display/panel/evervision,vgg804821.txt b/Documentation/devicetree/bindings/display/panel/evervision,vgg804821.txt > new file mode 100644 > index 000000000000..82d22e191ac3 > --- /dev/null > +++ b/Documentation/devicetree/bindings/display/panel/evervision,vgg804821.txt > @@ -0,0 +1,12 @@ > +Evervision Electronics Co. Ltd. VGG804821 5.0" WVGA TFT LCD Panel > + > +Required properties: > +- compatible: should be "evervision,vgg804821" > +- power-supply: See simple-panel.txt > + > +Optional properties: > +- backlight: See simple-panel.txt > +- enable-gpios: See simple-panel.txt > + > +This binding is compatible with the simple-panel binding, which is specified > +in simple-panel.txt in this directory. Other bindings just have the compatible listed and the above two lines and do not repeat the required/optional properties. If there is not special reason to add these they I suggest to simplify the binding. Sam
