linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: Joakim Zhang <qiangqing.zhang@nxp.com>
To: Rob Herring <robh@kernel.org>
Cc: "mark.rutland@arm.com" <mark.rutland@arm.com>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"festevam@gmail.com" <festevam@gmail.com>,
	Andy Duan <fugang.duan@nxp.com>,
	"jason@lakedaemon.net" <jason@lakedaemon.net>,
	"maz@kernel.org" <maz@kernel.org>,
	"s.hauer@pengutronix.de" <s.hauer@pengutronix.de>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"robh+dt@kernel.org" <robh+dt@kernel.org>,
	dl-linux-imx <linux-imx@nxp.com>,
	"kernel@pengutronix.de" <kernel@pengutronix.de>,
	"tglx@linutronix.de" <tglx@linutronix.de>,
	"shawnguo@kernel.org" <shawnguo@kernel.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>
Subject: RE: [PATCH V5 1/2] dt-bindings/irq: add binding for NXP INTMUX interrupt multiplexer
Date: Fri, 17 Jan 2020 05:45:34 +0000	[thread overview]
Message-ID: <DB7PR04MB461896F81D346CC88505F2C1E6310@DB7PR04MB4618.eurprd04.prod.outlook.com> (raw)
In-Reply-To: <20200116192928.GA1014@bogus>


> -----Original Message-----
> From: Rob Herring <robh@kernel.org>
> Sent: 2020年1月17日 3:29
> To: Joakim Zhang <qiangqing.zhang@nxp.com>
> Cc: maz@kernel.org; jason@lakedaemon.net; tglx@linutronix.de;
> robh+dt@kernel.org; mark.rutland@arm.com; shawnguo@kernel.org;
> s.hauer@pengutronix.de; kernel@pengutronix.de; festevam@gmail.com;
> linux-kernel@vger.kernel.org; devicetree@vger.kernel.org;
> linux-arm-kernel@lists.infradead.org; dl-linux-imx <linux-imx@nxp.com>; Andy
> Duan <fugang.duan@nxp.com>; Joakim Zhang <qiangqing.zhang@nxp.com>
> Subject: Re: [PATCH V5 1/2] dt-bindings/irq: add binding for NXP INTMUX
> interrupt multiplexer
> 
> On Wed, 15 Jan 2020 13:04:23 +0800, Joakim Zhang wrote:
> > This patch adds the DT bindings for the NXP INTMUX interrupt
> > multiplexer for i.MX8 family SoCs.
> >
> > Signed-off-by: Joakim Zhang <qiangqing.zhang@nxp.com>
> > ---
> >  .../interrupt-controller/fsl,intmux.yaml      | 67 +++++++++++++++++++
> >  1 file changed, 67 insertions(+)
> >  create mode 100644
> > Documentation/devicetree/bindings/interrupt-controller/fsl,intmux.yaml
> >
> 
> My bot found errors running 'make dt_binding_check' on your patch:
> 
> Documentation/devicetree/bindings/display/simple-framebuffer.example.dts:2
> 1.16-37.11: Warning (chosen_node_is_root): /example-0/chosen: chosen node
> must be at root node
> Error:
> Documentation/devicetree/bindings/interrupt-controller/fsl,intmux.example.dt
> s:20.27-28 syntax error FATAL ERROR: Unable to parse input tree
> scripts/Makefile.lib:300: recipe for target
> 'Documentation/devicetree/bindings/interrupt-controller/fsl,intmux.example.d
> t.yaml' failed
> make[1]: ***
> [Documentation/devicetree/bindings/interrupt-controller/fsl,intmux.example.d
> t.yaml] Error 1
> Makefile:1263: recipe for target 'dt_binding_check' failed
> make: *** [dt_binding_check] Error 2
> 
> See
> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatchw
> ork.ozlabs.org%2Fpatch%2F1223172&amp;data=02%7C01%7Cqiangqing.zhan
> g%40nxp.com%7Caade42e256f248dc2ef708d79aba6941%7C686ea1d3bc2b4c6
> fa92cd99c5c301635%7C0%7C0%7C637147997726888265&amp;sdata=7opsZO
> V%2FKzIJBl4MoEESCYZa%2BjlzKKofG6m84SsfAHg%3D&amp;reserved=0
> Please check and re-submit.
Hi Rob,

make dt_binding_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/interrupt-controller/fsl,intmux.yaml
  CHKDT   Documentation/devicetree/bindings/interrupt-controller/fsl,intmux.yaml
  SCHEMA  Documentation/devicetree/bindings/processed-schema.yaml
  DTC     Documentation/devicetree/bindings/interrupt-controller/fsl,intmux.example.dt.yaml
  CHECK   Documentation/devicetree/bindings/interrupt-controller/fsl,intmux.example.dt.yaml

I don't know DTC also need pass the check, sorry for that. I will re-send the patch set.

Best Regards,
Joakim Zhang
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2020-01-17  5:45 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-01-15  5:04 [PATCH V5 0/2] irqchip: add NXP INTMUX interrupt controller Joakim Zhang
2020-01-15  5:04 ` [PATCH V5 1/2] dt-bindings/irq: add binding for NXP INTMUX interrupt multiplexer Joakim Zhang
2020-01-16 19:29   ` Rob Herring
2020-01-17  5:45     ` Joakim Zhang [this message]
2020-01-15  5:04 ` [PATCH V5 2/2] drivers/irqchip: add NXP INTMUX interrupt multiplexer support Joakim Zhang

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=DB7PR04MB461896F81D346CC88505F2C1E6310@DB7PR04MB4618.eurprd04.prod.outlook.com \
    --to=qiangqing.zhang@nxp.com \
    --cc=devicetree@vger.kernel.org \
    --cc=festevam@gmail.com \
    --cc=fugang.duan@nxp.com \
    --cc=jason@lakedaemon.net \
    --cc=kernel@pengutronix.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-imx@nxp.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=maz@kernel.org \
    --cc=robh+dt@kernel.org \
    --cc=robh@kernel.org \
    --cc=s.hauer@pengutronix.de \
    --cc=shawnguo@kernel.org \
    --cc=tglx@linutronix.de \
    /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).