dri-devel.lists.freedesktop.org archive mirror
 help / color / mirror / Atom feed
From: Jagan Teki <jagan@amarulasolutions.com>
To: Sam Ravnborg <sam@ravnborg.org>
Cc: Mark Rutland <mark.rutland@arm.com>,
	devicetree <devicetree@vger.kernel.org>,
	David Airlie <airlied@linux.ie>,
	linux-kernel <linux-kernel@vger.kernel.org>,
	dri-devel <dri-devel@lists.freedesktop.org>,
	Rob Herring <robh+dt@kernel.org>,
	Thierry Reding <thierry.reding@gmail.com>,
	linux-amarula <linux-amarula@amarulasolutions.com>
Subject: Re: [PATCH 0/6] dt-bindings: display: Update few panel bindings with YAML
Date: Mon, 6 Jan 2020 15:41:40 +0530	[thread overview]
Message-ID: <CAMty3ZDbDf6YovrEdG0pACQAwMQidjKr6BJvx-FPXqyT11G05w@mail.gmail.com> (raw)
In-Reply-To: <20200104151702.GC17768@ravnborg.org>

Hi Sam,

On Sat, Jan 4, 2020 at 8:47 PM Sam Ravnborg <sam@ravnborg.org> wrote:
>
> Hi Jagan.
> On Wed, Jan 01, 2020 at 04:54:38PM +0530, Jagan Teki wrote:
> > These panel bindings are owned by me, so updated all of them into
> > YAML DT schema.
> >
> > Any inputs?
> Thanks for doing the conversion.
>
> dt_binding_check was not happy:
> Documentation/devicetree/bindings/display/panel/rocktech,rk070er9427.example.dt.yaml: panel: 'backlight', 'port' do not match any of the regexes: 'pinctrl-[0-9]+'
>   DTC     Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.example.dt.yaml
>   CHECK   Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.example.dt.yaml
> Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.example.dt.yaml: panel: 'backlight', 'port' do not match any of the regexes: 'pinctrl-[0-9]+'
> Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.example.dt.yaml: panel: compatible: Additional items are not allowed ('simple-panel' was unexpected)
> Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.example.dt.yaml: panel: compatible: ['friendlyarm,hd702e', 'simple-panel'] is too long
>   DTC     Documentation/devicetree/bindings/display/panel/sitronix,st7701.example.dt.yaml
> Error: Documentation/devicetree/bindings/display/panel/sitronix,st7701.example.dts:22.42-43 syntax error
> FATAL ERROR: Unable to parse input tree
>
> Please fix and check the bindings using dt_binding_check before
> resubmit.
>
> I had to install libyaml-dev (as least I recall this was the name)
> before dt_binding_check worked OK for me.

I did check dt_binfing_check with this series. Here is the complete
build log and you can see the panels related to this series are
checked fine. Let me know if I miss anything here?

₹ make dt_binding_check
arch/x86/Makefile:147: CONFIG_X86_X32 enabled but no binutils support
  SCHEMA  Documentation/devicetree/bindings/processed-schema.yaml
/home/jagan/work/code/drm-misc/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml:
ignoring, error in schema: properties: reg: minItems
warning: no schema found in file:
Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml
/home/jagan/work/code/drm-misc/Documentation/devicetree/bindings/interrupt-controller/arm,gic-v3.yaml:
ignoring, error in schema: properties: ppi-partitions:
patternProperties: ^interrupt-partition-[0-9]+$
warning: no schema found in file:
Documentation/devicetree/bindings/interrupt-controller/arm,gic-v3.yaml
/home/jagan/work/code/drm-misc/Documentation/devicetree/bindings/net/snps,dwmac.yaml:
ignoring, error in schema: allOf: 2: then
warning: no schema found in file:
Documentation/devicetree/bindings/net/snps,dwmac.yaml
/home/jagan/work/code/drm-misc/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml:
ignoring, error in schema: properties: compatible: enum: 0
warning: no schema found in file:
Documentation/devicetree/bindings/regulator/fixed-regulator.yaml
/home/jagan/work/code/drm-misc/Documentation/devicetree/bindings/riscv/cpus.yaml:
ignoring, error in schema: properties: timebase-frequency: type
warning: no schema found in file:
Documentation/devicetree/bindings/riscv/cpus.yaml
  CHKDT   Documentation/devicetree/bindings/arm/pmu.yaml
  CHKDT   Documentation/devicetree/bindings/arm/rockchip.yaml
  CHKDT   Documentation/devicetree/bindings/arm/sirf.yaml
  CHKDT   Documentation/devicetree/bindings/arm/vt8500.yaml
  CHKDT   Documentation/devicetree/bindings/arm/calxeda.yaml
  CHKDT   Documentation/devicetree/bindings/arm/atmel-at91.yaml
  CHKDT   Documentation/devicetree/bindings/arm/sunxi.yaml
  CHKDT   Documentation/devicetree/bindings/arm/xilinx.yaml
  CHKDT   Documentation/devicetree/bindings/arm/moxart.yaml
  CHKDT   Documentation/devicetree/bindings/arm/stm32/stm32.yaml
  CHKDT   Documentation/devicetree/bindings/arm/spear.yaml
  CHKDT   Documentation/devicetree/bindings/arm/al,alpine.yaml
  CHKDT   Documentation/devicetree/bindings/arm/digicolor.yaml
  CHKDT   Documentation/devicetree/bindings/arm/altera/socfpga-clk-manager.yaml
  CHKDT   Documentation/devicetree/bindings/arm/altera.yaml
  CHKDT   Documentation/devicetree/bindings/arm/bitmain.yaml
  CHKDT   Documentation/devicetree/bindings/arm/nxp/lpc32xx.yaml
  CHKDT   Documentation/devicetree/bindings/arm/fsl.yaml
  CHKDT   Documentation/devicetree/bindings/arm/axxia.yaml
  CHKDT   Documentation/devicetree/bindings/arm/psci.yaml
  CHKDT   Documentation/devicetree/bindings/arm/qcom.yaml
  CHKDT   Documentation/devicetree/bindings/arm/zte.yaml
  CHKDT   Documentation/devicetree/bindings/arm/intel-ixp4xx.yaml
  CHKDT   Documentation/devicetree/bindings/arm/mediatek.yaml
  CHKDT   Documentation/devicetree/bindings/arm/socionext/milbeaut.yaml
  CHKDT   Documentation/devicetree/bindings/arm/ti/nspire.yaml
  CHKDT   Documentation/devicetree/bindings/arm/ti/ti,davinci.yaml
  CHKDT   Documentation/devicetree/bindings/arm/realtek.yaml
  CHKDT   Documentation/devicetree/bindings/arm/rda.yaml
  CHKDT   Documentation/devicetree/bindings/arm/actions.yaml
  CHKDT   Documentation/devicetree/bindings/arm/l2c2x0.yaml
  CHKDT   Documentation/devicetree/bindings/arm/renesas.yaml
  CHKDT   Documentation/devicetree/bindings/arm/tegra.yaml
  CHKDT   Documentation/devicetree/bindings/arm/amlogic.yaml
  CHKDT   Documentation/devicetree/bindings/arm/amlogic/amlogic,meson-gx-ao-secure.yaml
  CHKDT   Documentation/devicetree/bindings/arm/sti.yaml
  CHKDT   Documentation/devicetree/bindings/arm/primecell.yaml
  CHKDT   Documentation/devicetree/bindings/arm/cpus.yaml
  CHKDT   Documentation/devicetree/bindings/bus/allwinner,sun50i-a64-de2.yaml
  CHKDT   Documentation/devicetree/bindings/bus/allwinner,sun8i-a23-rsb.yaml
  CHKDT   Documentation/devicetree/bindings/clock/fixed-factor-clock.yaml
  CHKDT   Documentation/devicetree/bindings/clock/milbeaut-clock.yaml
  CHKDT   Documentation/devicetree/bindings/clock/allwinner,sun4i-a10-ccu.yaml
  CHKDT   Documentation/devicetree/bindings/clock/imx8mn-clock.yaml
  CHKDT   Documentation/devicetree/bindings/clock/fixed-clock.yaml
  CHKDT   Documentation/devicetree/bindings/crypto/allwinner,sun4i-a10-crypto.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/ampire,am-480272h3tmqw-t01h.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/innolux,ee101ia-01d.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/pda,91-00156-a0.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/sitronix,st7701.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/nec,nl8048hl11.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/friendlyarm,hd702e.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/bananapi,s070wv20-ct16.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/feiyang,fy07024di26a30d.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/dlc,dlc0700yzg-1.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/koe,tx31d200vm0baa.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/mitsubishi,aa121td01.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/sony,acx424akp.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/logicpd,type28.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/tfc,s9700rtwv43tr-01b.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/ti,nspire.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/tpo,tpg110.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/rocktech,rk070er9427.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/armadeus,st0700-adapt.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/raspberrypi,7inch-touchscreen.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/lvds.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/panel-common.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/sgd,gktw70sdae4se.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/mitsubishi,aa104xd12.yaml
  CHKDT   Documentation/devicetree/bindings/display/panel/ronbo,rb070d30.yaml
  CHKDT   Documentation/devicetree/bindings/display/amlogic,meson-vpu.yaml
  CHKDT   Documentation/devicetree/bindings/display/dsi-controller.yaml
  CHKDT   Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml
  CHKDT   Documentation/devicetree/bindings/display/amlogic,meson-dw-hdmi.yaml
  CHKDT   Documentation/devicetree/bindings/display/simple-framebuffer.yaml
  CHKDT   Documentation/devicetree/bindings/display/allwinner,sun6i-a31-mipi-dsi.yaml
  CHKDT   Documentation/devicetree/bindings/dma/dma-router.yaml
  CHKDT   Documentation/devicetree/bindings/dma/dma-controller.yaml
  CHKDT   Documentation/devicetree/bindings/dma/allwinner,sun50i-a64-dma.yaml
  CHKDT   Documentation/devicetree/bindings/dma/allwinner,sun4i-a10-dma.yaml
  CHKDT   Documentation/devicetree/bindings/dma/dma-common.yaml
  CHKDT   Documentation/devicetree/bindings/dma/allwinner,sun6i-a31-dma.yaml
  CHKDT   Documentation/devicetree/bindings/dsp/fsl,dsp.yaml
  CHKDT   Documentation/devicetree/bindings/example-schema.yaml
  CHKDT   Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml
/home/jagan/work/code/drm-misc/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml:
properties:reg:minItems: False schema does not allow 3
/home/jagan/work/code/drm-misc/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml:
properties:reg:maxItems: False schema does not allow 3
Documentation/devicetree/bindings/Makefile:12: recipe for target
'Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.example.dts'
failed
make[1]: *** [Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.example.dts]
Error 1
Makefile:1263: recipe for target 'dt_binding_check' failed
make: *** [dt_binding_check] Error 2
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

  reply	other threads:[~2020-01-06 10:11 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-01-01 11:24 [PATCH 0/6] dt-bindings: display: Update few panel bindings with YAML Jagan Teki
2020-01-01 11:24 ` [PATCH 1/6] dt-bindings: display: panel: Convert feiyang, fy07024di26a30d to DT schema Jagan Teki
2020-01-01 11:24 ` [PATCH 2/6] dt-bindings: display: panel: Convert sitronix, st7701 " Jagan Teki
2020-01-01 11:24 ` [PATCH 3/6] MAINTAINERS: Update feiyang, st7701 panel bindings converted as YAML Jagan Teki
2020-01-01 11:24 ` [PATCH 4/6] dt-bindings: display: panel: Convert friendlyarm, hd702e to DT schema Jagan Teki
2020-01-01 11:24 ` [PATCH 5/6] dt-bindings: display: panel: Convert rocktech, rk070er9427 " Jagan Teki
2020-01-01 11:24 ` [PATCH 6/6] dt-bindings: display: panel: Convert koe, tx31d200vm0baa " Jagan Teki
2020-01-04 15:17 ` [PATCH 0/6] dt-bindings: display: Update few panel bindings with YAML Sam Ravnborg
2020-01-06 10:11   ` Jagan Teki [this message]
2020-01-10 22:05     ` Sam Ravnborg

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAMty3ZDbDf6YovrEdG0pACQAwMQidjKr6BJvx-FPXqyT11G05w@mail.gmail.com \
    --to=jagan@amarulasolutions.com \
    --cc=airlied@linux.ie \
    --cc=devicetree@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=linux-amarula@amarulasolutions.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=robh+dt@kernel.org \
    --cc=sam@ravnborg.org \
    --cc=thierry.reding@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).