> Am 02.03.2020 um 20:19 schrieb Paul Cercueil <paul@xxxxxxxxxxxxxxx>: > > Hi Nikolaus, > > > Le ven., févr. 28, 2020 at 19:19, H. Nikolaus Schaller <hns@xxxxxxxxxxxxx> a écrit : >> From: Sam Ravnborg <sam@xxxxxxxxxxxx> >> Add DT bindings for the hdmi driver for the Ingenic JZ4780 SoC. >> Based on .txt binding from Zubair Lutfullah Kakakhel >> Signed-off-by: Sam Ravnborg <sam@xxxxxxxxxxxx> >> Cc: Zubair Lutfullah Kakakhel <Zubair.Kakakhel@xxxxxxxxxx> >> Cc: H. Nikolaus Schaller <hns@xxxxxxxxxxxxx> >> Cc: Rob Herring <robh@xxxxxxxxxx> >> Cc: devicetree@xxxxxxxxxxxxxxx >> --- >> .../bindings/display/ingenic-jz4780-hdmi.yaml | 83 +++++++++++++++++++ >> 1 file changed, 83 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/display/ingenic-jz4780-hdmi.yaml >> diff --git a/Documentation/devicetree/bindings/display/ingenic-jz4780-hdmi.yaml b/Documentation/devicetree/bindings/display/ingenic-jz4780-hdmi.yaml >> new file mode 100644 >> index 000000000000..9b71c427bd69 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/display/ingenic-jz4780-hdmi.yaml >> @@ -0,0 +1,83 @@ >> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/display/ingenic-jz4780-hdmi.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: Bindings for Ingenic JZ4780 HDMI Transmitter >> + >> +maintainers: >> + - Zubair Lutfullah Kakakhel <Zubair.Kakakhel@xxxxxxxxxx> >> + - H. Nikolaus Schaller <hns@xxxxxxxxxxxxx> > > Did Zubair write this glue driver? He's been MIA for a while, doesn't work at ImgTec anymore, and this email doesn't work. He did write the original .txt version of [RFC v1 2/8] and Sam simply added him. > >> + >> +description: | >> + The HDMI Transmitter in the Ingenic JZ4780 is a Synopsys DesignWare HDMI 1.4 >> + TX controller IP with accompanying PHY IP. >> + >> +allOf: >> + - $ref: panel/panel-common.yaml# >> + >> +properties: >> + compatible: >> + items: >> + - const: ingenic,jz4780-hdmi >> + >> + reg: >> + maxItems: 1 >> + description: the address & size of the LCD controller registers > > Remove the description here, > >> + >> + reg-io-width: >> + const: 4 >> + >> + interrupts: >> + maxItems: 1 >> + description: Specifies the interrupt provided by parent > > and here. > > The rule is that if there is only one "reg", "interrupts" or "clocks" entry then a description is not needed as it's pretty obvious what it's for. > >> + >> + clocks: >> + maxItems: 2 >> + description: Clock specifiers for isrf and iahb clocks > > You need two 'description:' like this: > > clocks: > items: > - description: ISRF clock > - description: IAHB clock Ok. BR and thanks, Nikolaus _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel