From mboxrd@z Thu Jan 1 00:00:00 1970 From: Li Yang Subject: Re: [PATCH v2] dts/ls2080a: Update PCIe compatible Date: Mon, 18 Jan 2016 16:09:22 +0800 Message-ID: References: <1449224417-20753-1-git-send-email-Mingkai.Hu@freescale.com> <20151204162243.GA12415@rob-hp-laptop> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Hu Vincent Cc: Rob Herring , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "arm-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org" , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" , "Lian M.H." List-Id: devicetree@vger.kernel.org On Mon, Dec 7, 2015 at 4:44 PM, Hu Vincent wrote: > > >> -----Original Message----- >> From: Rob Herring [mailto:robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org] >> Sent: Saturday, December 05, 2015 12:23 AM >> To: Hu Mingkai-B21284 >> Cc: arm-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org; devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; Li Yang-Leo-R58472; >> linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org; Lian Minghuan-B31939 >> Subject: Re: [PATCH v2] dts/ls2080a: Update PCIe compatible >> >> On Fri, Dec 04, 2015 at 06:20:17PM +0800, Mingkai Hu wrote: >> > From: Minghuan Lian >> > >> > The patch adds LS2085a to PCIe compatible to fix the compatibility >> > issue when using firmware with LS2085a compatible property. >> > >> > Signed-off-by: Minghuan Lian >> > Signed-off-by: Mingkai Hu >> > --- >> > v2 change: >> > - Rebased to latest kernel base >> >> How about implementing my comments on v1, too? >> > > Hi Rob, > > Do you mean to change the combinations in layerscape-pci.txt as follows? > >> --- a/Documentation/devicetree/bindings/pci/layerscape-pci.txt >> +++ b/Documentation/devicetree/bindings/pci/layerscape-pci.txt >> @@ -15,6 +15,7 @@ Required properties: >> - compatible: should contain the platform identifier such as: >> "fsl,ls1021a-pcie", "snps,dw-pcie" >> - "fsl,ls2080a-pcie", "snps,dw-pcie" >> + "fsl,ls2080a-pcie", "fsl,ls2085a-pcie", "snps,dw-pcie" > > > And the ls2085a was released first, but the dts file is for ls2080a (fsl-ls2080a.dtsi), > So I prefer to put the ls2080a as the first compatible string. How about your suggestion? I think this is what Rob suggested in v1 to make the combination in document align with real dts. And it would be good to also have an entry for ls2085: "fsl,ls2085a-pcie", "snps,dw-pcie". Can you send an updated version? Regards, Leo -- 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