[PATCH v2 2/4] arm64: add basic DTS for i.MX8MQ

[A.s. Dong]

> -----Original Message-----
> From: Shawn Guo [mailto:shawnguo at kernel.org]
> Sent: Monday, February 26, 2018 3:10 PM
> To: A.s. Dong <aisheng.dong at nxp.com>
> Cc: Mark Rutland <mark.rutland at arm.com>; devicetree at vger.kernel.org;
> Baruch Siach <baruch at tkos.co.il>; Catalin Marinas
> <catalin.marinas at arm.com>; Will Deacon <will.deacon at arm.com>;
> patchwork-lst at pengutronix.de; Rob Herring <robh+dt at kernel.org>; dl-
> linux-imx <linux-imx at nxp.com>; kernel at pengutronix.de; Fabio Estevam
> <fabio.estevam at nxp.com>; linux-arm-kernel at lists.infradead.org; Lucas
> Stach <l.stach at pengutronix.de>
> Subject: Re: [PATCH v2 2/4] arm64: add basic DTS for i.MX8MQ
> 
> On Mon, Feb 26, 2018 at 06:52:15AM +0000, A.s. Dong wrote:
> > Hi Shawn,
> >
> > > -----Original Message-----
> > > From: Shawn Guo [mailto:shawnguo at kernel.org]
> > > Sent: Monday, February 26, 2018 11:17 AM
> > > To: A.s. Dong <aisheng.dong at nxp.com>
> > > Cc: Lucas Stach <l.stach at pengutronix.de>; Mark Rutland
> > > <mark.rutland at arm.com>; devicetree at vger.kernel.org; Baruch Siach
> > > <baruch at tkos.co.il>; Catalin Marinas <catalin.marinas at arm.com>; Will
> > > Deacon <will.deacon at arm.com>; patchwork-lst at pengutronix.de; Rob
> > > Herring <robh+dt at kernel.org>; dl-linux-imx <linux-imx at nxp.com>;
> > > kernel at pengutronix.de; Fabio Estevam <fabio.estevam at nxp.com>;
> linux-
> > > arm-kernel at lists.infradead.org
> > > Subject: Re: [PATCH v2 2/4] arm64: add basic DTS for i.MX8MQ
> > >
> > > On Sun, Feb 25, 2018 at 10:47:11AM +0000, A.s. Dong wrote:
> > > > >  Documentation/devicetree/bindings/arm/fsl.txt  |   4 +
> > > > >  arch/arm64/boot/dts/freescale/imx8mq-pinfunc.h | 623
> > > > > +++++++++++++++++++++++++
> > > >
> > > > Should this belongs to pinctrl part?
> > >
> > > No.  Those pinfunc defines are used by device tree rather than pinctrl
> driver.
> > >
> >
> > I had this question because we have below info in the patch
> > "dt-bindings: add binding for i.MX8MQ IOMUXC":
> >
> > diff --git
> > a/Documentation/devicetree/bindings/pinctrl/fsl,imx8mq-pinctrl.txt
> > b/Documentation/devicetree/bindings/pinctrl/fsl,imx8mq-pinctrl.txt
> > new file mode 100644
> > index 0000000..5c5d2d8
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/pinctrl/fsl,imx8mq-pinctrl.txt
> > @@ -0,0 +1,39 @@
> > ...
> > +       Refer to imx8mq-pinfunc.h in device tree source folder for all available
> > +       imx8mq PIN_FUNC_ID.
> 
> This is just a cross reference in document, and should be okay on different
> tree during development cycle.  If we have the pinfunc header go via pinctrl
> tree, we will have a more serious dependency between DT and pinctrl tree.
> 

Okay, understand now.

Regards
Dong Aisheng

> Shawn