From: Jun Li <jun.li@nxp.com> To: Krzysztof Kozlowski <krzk@kernel.org> Cc: "robh+dt@kernel.org" <robh+dt@kernel.org>, "shawnguo@kernel.org" <shawnguo@kernel.org>, "balbi@kernel.org" <balbi@kernel.org>, "mathias.nyman@intel.com" <mathias.nyman@intel.com>, "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>, "s.hauer@pengutronix.de" <s.hauer@pengutronix.de>, "kernel@pengutronix.de" <kernel@pengutronix.de>, "festevam@gmail.com" <festevam@gmail.com>, dl-linux-imx <linux-imx@nxp.com>, Anson Huang <anson.huang@nxp.com>, Aisheng Dong <aisheng.dong@nxp.com>, Peng Fan <peng.fan@nxp.com>, Andy Duan <fugang.duan@nxp.com>, Joakim Zhang <qiangqing.zhang@nxp.com>, Horia Geanta <horia.geanta@nxp.com>, "linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org> Subject: RE: [PATCH v3 3/6] dt-bindings: usb: dwc3-imx8mp: add imx8mp dwc3 glue bindings Date: Tue, 22 Sep 2020 16:45:27 +0000 [thread overview] Message-ID: <VE1PR04MB6528C9AF51263B92EA687BEB893B0@VE1PR04MB6528.eurprd04.prod.outlook.com> (raw) In-Reply-To: <CAJKOXPfocQZY-5Xmn5q=oetmS1doe0zXsQH=uXx0+CxGaESgUg@mail.gmail.com> > -----Original Message----- > From: Krzysztof Kozlowski <krzk@kernel.org> > Sent: Tuesday, September 22, 2020 10:33 PM > To: Jun Li <jun.li@nxp.com> > Cc: robh+dt@kernel.org; shawnguo@kernel.org; balbi@kernel.org; > mathias.nyman@intel.com; gregkh@linuxfoundation.org; > s.hauer@pengutronix.de; kernel@pengutronix.de; festevam@gmail.com; > dl-linux-imx <linux-imx@nxp.com>; Anson Huang <anson.huang@nxp.com>; > Aisheng Dong <aisheng.dong@nxp.com>; Peng Fan <peng.fan@nxp.com>; Andy Duan > <fugang.duan@nxp.com>; Joakim Zhang <qiangqing.zhang@nxp.com>; Horia > Geanta <horia.geanta@nxp.com>; linux-usb@vger.kernel.org; > devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org > Subject: Re: [PATCH v3 3/6] dt-bindings: usb: dwc3-imx8mp: add imx8mp dwc3 > glue bindings > > On Tue, 22 Sep 2020 at 12:56, Li Jun <jun.li@nxp.com> wrote: > > > > NXP imx8mp integrates 2 dwc3 3.30b IP and add some wakeup logic to > > support low power mode, the glue layer is for this wakeup > > functionality, which has a separated interrupt, can support wakeup > > from U3 and connect events for host, and vbus wakeup for device. > > > > Signed-off-by: Li Jun <jun.li@nxp.com> > > --- > > .../devicetree/bindings/usb/fsl,imx8mp-dwc3.yaml | 99 > ++++++++++++++++++++++ > > 1 file changed, 99 insertions(+) > > > > diff --git > > a/Documentation/devicetree/bindings/usb/fsl,imx8mp-dwc3.yaml > > b/Documentation/devicetree/bindings/usb/fsl,imx8mp-dwc3.yaml > > new file mode 100644 > > index 0000000..a5d5566 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/usb/fsl,imx8mp-dwc3.yaml > > @@ -0,0 +1,99 @@ > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) # Copyright > > +(c) 2020 NXP %YAML 1.2 > > +--- > > +$id: > > > +https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdevi > > > +cetree.org%2Fschemas%2Fusb%2Ffsl%2Cimx8mp-dwc3.yaml%23&data=02%7C > > > +01%7Cjun.li%40nxp.com%7Ceaedeb64ec15472183d108d85f046841%7C686ea1d3bc > > > +2b4c6fa92cd99c5c301635%7C0%7C0%7C637363819819667936&sdata=IvFQ8l0 > > +7VXUMbG8riyITSTZb1P50oIXs%2BkAV%2FeX8ntk%3D&reserved=0 > > +$schema: > > > +https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdevi > > > +cetree.org%2Fmeta-schemas%2Fcore.yaml%23&data=02%7C01%7Cjun.li%40 > > > +nxp.com%7Ceaedeb64ec15472183d108d85f046841%7C686ea1d3bc2b4c6fa92cd99c > > > +5c301635%7C0%7C0%7C637363819819677934&sdata=H3%2BcwgiLKBynHv4N%2F > > +dogBi3SwaizVHmqieJXVrErv0I%3D&reserved=0 > > + > > +title: NXP iMX8MP Soc USB Controller > > + > > +maintainers: > > + - Li Jun <jun.li@nxp.com> > > + > > +properties: > > + compatible: > > + items: > > Items are not necessary > > > > + - const: fsl,imx8mp-dwc3 > > + > > + reg: > > + maxItems: 1 > > + description: Address and length of the register set for the wrapper > of > > + dwc3 core on the SOC. > > + > > + "#address-cells": > > + enum: [ 1, 2 ] > > + > > + "#size-cells": > > + enum: [ 1, 2 ] > > + > > + ranges: true > > + > > + interrupts: > > + maxItems: 1 > > + description: The interrupt that is asserted when a wakeup event is > > + received. > > + > > + clocks: > > + description: > > + A list of phandle and clock-specifier pairs for the clocks > > + listed in clock-names. > > + items: > > + - description: system hsio root clock. > > + - description: suspend clock, used for usb wakeup logic. > > + > > + clock-names: > > + items: > > + - const: hsio > > + - const: suspend > > + > > +# Required child node: > > + > > +patternProperties: > > + "^dwc3@[0-9a-f]+$": > > + type: object > > + description: > > + A child node must exist to represent the core DWC3 IP block > > + The content of the node is defined in dwc3.txt. > > + > > +required: > > + - compatible > > + - reg > > + - "#address-cells" > > + - "#size-cells" > > + - ranges > > + - clocks > > + - clock-names > > + - interrupts > > + > > +additionalProperties: false > > Do you actually validate your DTS changes with this schema? They do not look > like passing the validation... I did run dt_binding_check, maybe I missed some errors as there are always some other failures, I will re-run to double check. Li Jun > > Best regards, > Krzysztof
WARNING: multiple messages have this Message-ID (diff)
From: Jun Li <jun.li@nxp.com> To: Krzysztof Kozlowski <krzk@kernel.org> Cc: Aisheng Dong <aisheng.dong@nxp.com>, "balbi@kernel.org" <balbi@kernel.org>, Peng Fan <peng.fan@nxp.com>, "festevam@gmail.com" <festevam@gmail.com>, Andy Duan <fugang.duan@nxp.com>, Anson Huang <anson.huang@nxp.com>, "mathias.nyman@intel.com" <mathias.nyman@intel.com>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>, "s.hauer@pengutronix.de" <s.hauer@pengutronix.de>, "linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>, Joakim Zhang <qiangqing.zhang@nxp.com>, "robh+dt@kernel.org" <robh+dt@kernel.org>, dl-linux-imx <linux-imx@nxp.com>, "kernel@pengutronix.de" <kernel@pengutronix.de>, Horia Geanta <horia.geanta@nxp.com>, "shawnguo@kernel.org" <shawnguo@kernel.org>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org> Subject: RE: [PATCH v3 3/6] dt-bindings: usb: dwc3-imx8mp: add imx8mp dwc3 glue bindings Date: Tue, 22 Sep 2020 16:45:27 +0000 [thread overview] Message-ID: <VE1PR04MB6528C9AF51263B92EA687BEB893B0@VE1PR04MB6528.eurprd04.prod.outlook.com> (raw) In-Reply-To: <CAJKOXPfocQZY-5Xmn5q=oetmS1doe0zXsQH=uXx0+CxGaESgUg@mail.gmail.com> > -----Original Message----- > From: Krzysztof Kozlowski <krzk@kernel.org> > Sent: Tuesday, September 22, 2020 10:33 PM > To: Jun Li <jun.li@nxp.com> > Cc: robh+dt@kernel.org; shawnguo@kernel.org; balbi@kernel.org; > mathias.nyman@intel.com; gregkh@linuxfoundation.org; > s.hauer@pengutronix.de; kernel@pengutronix.de; festevam@gmail.com; > dl-linux-imx <linux-imx@nxp.com>; Anson Huang <anson.huang@nxp.com>; > Aisheng Dong <aisheng.dong@nxp.com>; Peng Fan <peng.fan@nxp.com>; Andy Duan > <fugang.duan@nxp.com>; Joakim Zhang <qiangqing.zhang@nxp.com>; Horia > Geanta <horia.geanta@nxp.com>; linux-usb@vger.kernel.org; > devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org > Subject: Re: [PATCH v3 3/6] dt-bindings: usb: dwc3-imx8mp: add imx8mp dwc3 > glue bindings > > On Tue, 22 Sep 2020 at 12:56, Li Jun <jun.li@nxp.com> wrote: > > > > NXP imx8mp integrates 2 dwc3 3.30b IP and add some wakeup logic to > > support low power mode, the glue layer is for this wakeup > > functionality, which has a separated interrupt, can support wakeup > > from U3 and connect events for host, and vbus wakeup for device. > > > > Signed-off-by: Li Jun <jun.li@nxp.com> > > --- > > .../devicetree/bindings/usb/fsl,imx8mp-dwc3.yaml | 99 > ++++++++++++++++++++++ > > 1 file changed, 99 insertions(+) > > > > diff --git > > a/Documentation/devicetree/bindings/usb/fsl,imx8mp-dwc3.yaml > > b/Documentation/devicetree/bindings/usb/fsl,imx8mp-dwc3.yaml > > new file mode 100644 > > index 0000000..a5d5566 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/usb/fsl,imx8mp-dwc3.yaml > > @@ -0,0 +1,99 @@ > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) # Copyright > > +(c) 2020 NXP %YAML 1.2 > > +--- > > +$id: > > > +https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdevi > > > +cetree.org%2Fschemas%2Fusb%2Ffsl%2Cimx8mp-dwc3.yaml%23&data=02%7C > > > +01%7Cjun.li%40nxp.com%7Ceaedeb64ec15472183d108d85f046841%7C686ea1d3bc > > > +2b4c6fa92cd99c5c301635%7C0%7C0%7C637363819819667936&sdata=IvFQ8l0 > > +7VXUMbG8riyITSTZb1P50oIXs%2BkAV%2FeX8ntk%3D&reserved=0 > > +$schema: > > > +https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdevi > > > +cetree.org%2Fmeta-schemas%2Fcore.yaml%23&data=02%7C01%7Cjun.li%40 > > > +nxp.com%7Ceaedeb64ec15472183d108d85f046841%7C686ea1d3bc2b4c6fa92cd99c > > > +5c301635%7C0%7C0%7C637363819819677934&sdata=H3%2BcwgiLKBynHv4N%2F > > +dogBi3SwaizVHmqieJXVrErv0I%3D&reserved=0 > > + > > +title: NXP iMX8MP Soc USB Controller > > + > > +maintainers: > > + - Li Jun <jun.li@nxp.com> > > + > > +properties: > > + compatible: > > + items: > > Items are not necessary > > > > + - const: fsl,imx8mp-dwc3 > > + > > + reg: > > + maxItems: 1 > > + description: Address and length of the register set for the wrapper > of > > + dwc3 core on the SOC. > > + > > + "#address-cells": > > + enum: [ 1, 2 ] > > + > > + "#size-cells": > > + enum: [ 1, 2 ] > > + > > + ranges: true > > + > > + interrupts: > > + maxItems: 1 > > + description: The interrupt that is asserted when a wakeup event is > > + received. > > + > > + clocks: > > + description: > > + A list of phandle and clock-specifier pairs for the clocks > > + listed in clock-names. > > + items: > > + - description: system hsio root clock. > > + - description: suspend clock, used for usb wakeup logic. > > + > > + clock-names: > > + items: > > + - const: hsio > > + - const: suspend > > + > > +# Required child node: > > + > > +patternProperties: > > + "^dwc3@[0-9a-f]+$": > > + type: object > > + description: > > + A child node must exist to represent the core DWC3 IP block > > + The content of the node is defined in dwc3.txt. > > + > > +required: > > + - compatible > > + - reg > > + - "#address-cells" > > + - "#size-cells" > > + - ranges > > + - clocks > > + - clock-names > > + - interrupts > > + > > +additionalProperties: false > > Do you actually validate your DTS changes with this schema? They do not look > like passing the validation... I did run dt_binding_check, maybe I missed some errors as there are always some other failures, I will re-run to double check. Li Jun > > Best regards, > Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-09-22 16:45 UTC|newest] Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-09-22 10:46 [PATCH v3 0/6] add NXP imx8mp usb support Li Jun 2020-09-22 10:46 ` Li Jun 2020-09-22 10:46 ` [PATCH v3 1/6] dt-bindings: usb: xhci: add property to disable xhci 64bit support Li Jun 2020-09-22 10:46 ` Li Jun 2020-09-29 18:11 ` Rob Herring 2020-09-29 18:11 ` Rob Herring 2020-10-02 16:39 ` Jun Li 2020-10-02 16:39 ` Jun Li 2020-09-22 10:46 ` [PATCH v3 2/6] usb: host: xhci-plat: add support of XHCI_NO_64BIT_SUPPORT via property Li Jun 2020-09-22 10:46 ` Li Jun 2020-09-22 10:46 ` [PATCH v3 3/6] dt-bindings: usb: dwc3-imx8mp: add imx8mp dwc3 glue bindings Li Jun 2020-09-22 10:46 ` Li Jun 2020-09-22 14:32 ` Krzysztof Kozlowski 2020-09-22 14:32 ` Krzysztof Kozlowski 2020-09-22 16:45 ` Jun Li [this message] 2020-09-22 16:45 ` Jun Li 2020-09-22 18:22 ` Krzysztof Kozlowski 2020-09-22 18:22 ` Krzysztof Kozlowski 2020-09-22 10:46 ` [PATCH v3 4/6] usb: dwc3: add imx8mp dwc3 glue layer driver Li Jun 2020-09-22 10:46 ` Li Jun 2020-09-22 10:46 ` [PATCH v3 5/6] arm64: dtsi: imx8mp: add usb nodes Li Jun 2020-09-22 10:46 ` Li Jun 2020-09-22 14:39 ` Krzysztof Kozlowski 2020-09-22 14:39 ` Krzysztof Kozlowski 2020-09-23 7:50 ` Sascha Hauer 2020-09-23 7:50 ` Sascha Hauer 2020-09-23 8:49 ` Jun Li 2020-09-23 8:49 ` Jun Li 2020-09-22 10:46 ` [PATCH v3 6/6] arm64: dts: imx8mp-evk: enable usb1 as host mode Li Jun 2020-09-22 10:46 ` Li Jun
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=VE1PR04MB6528C9AF51263B92EA687BEB893B0@VE1PR04MB6528.eurprd04.prod.outlook.com \ --to=jun.li@nxp.com \ --cc=aisheng.dong@nxp.com \ --cc=anson.huang@nxp.com \ --cc=balbi@kernel.org \ --cc=devicetree@vger.kernel.org \ --cc=festevam@gmail.com \ --cc=fugang.duan@nxp.com \ --cc=gregkh@linuxfoundation.org \ --cc=horia.geanta@nxp.com \ --cc=kernel@pengutronix.de \ --cc=krzk@kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-imx@nxp.com \ --cc=linux-usb@vger.kernel.org \ --cc=mathias.nyman@intel.com \ --cc=peng.fan@nxp.com \ --cc=qiangqing.zhang@nxp.com \ --cc=robh+dt@kernel.org \ --cc=s.hauer@pengutronix.de \ --cc=shawnguo@kernel.org \ /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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.