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

From: [email-masked] (Shawn Guo)
Date: Mon, 26 Feb 2018 15:09:58 +0800

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 <[email-masked]>
> > Cc: Lucas Stach <[email-masked]>; Mark Rutland
> > <[email-masked]>; devicetree at vger.kernel.org; Baruch Siach
> > <[email-masked]>; Catalin Marinas <[email-masked]>; Will
> > Deacon <[email-masked]>; patchwork-lst at pengutronix.de; Rob
> > Herring <[email-masked]>; dl-linux-imx <[email-masked]>;
> > kernel at pengutronix.de; Fabio Estevam <[email-masked]>; 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.

Shawn