From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jingoo Han Subject: Re: [PATCH v4 3/3] PCI: imx6: Add support for i.MX6 PCIe controller Date: Tue, 17 Sep 2013 07:53:19 +0900 Message-ID: <003901ceb32f$89941060$9cbc3120$%han@samsung.com> References: <1379065222-7275-1-git-send-email-xobs@kosagi.com> <1379065222-7275-4-git-send-email-xobs@kosagi.com> <20130916060631.GF31147@S2101-09.ap.freescale.net> <5236AB28.5060708@kosagi.com> <20130916090146.GL31147@S2101-09.ap.freescale.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Return-path: In-reply-to: <20130916090146.GL31147-rvtDTF3kK1ictlrPMvKcciBecyulp+rMXqFh9Ls21Oc@public.gmane.org> Content-language: ko Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: 'Shawn Guo' Cc: 'Sean Cross' , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-pci-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, 'Sascha Hauer' , 'Zhu Richard-R65037' , 'Jingoo Han' List-Id: devicetree@vger.kernel.org On Monday, September 16, 2013 6:02 PM, Shawn Guo wrote: > On Mon, Sep 16, 2013 at 02:54:32PM +0800, Sean Cross wrote: > > >>@@ -116,6 +116,22 @@ > > >> arm,data-latency = <4 2 3>; > > >> }; > > >>+ pcie: pcie@0x01000000 { > > >>+ compatible = "fsl,imx6-pcie", "snps,dw-pcie"; > > >We generally use particular SoC name to specify the compatible string. > > >So "fsl,imx6q-pcie" will be more appropriate. > > Any reason to specify imx6q in particular? It's also present on the > > imx6d, imx6dl, and imx6s (basically, everything but the imx6sl). > > The compatible naming is not about coverage but versioning. We > generally encode the name of the SoC that firstly integrates the block > to specify the version. > Yes, you're right. :-) The name of the first SoC would be used. Best regards, Jingoo Han -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html