From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from metis.ext.pengutronix.de ([92.198.50.35]:60629 "EHLO metis.ext.pengutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932248AbaKMKUU (ORCPT ); Thu, 13 Nov 2014 05:20:20 -0500 Message-ID: <1415874015.2420.5.camel@pengutronix.de> Subject: Re: [PATCH v2] PCI: designware: Add support 4 ATUs assignment From: Lucas Stach To: Lian Minghuan-B31939 Cc: Srikanth Thokala , Minghuan Lian , "linux-pci@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , Zang Roy-R61911 , Hu Mingkai-B21284 , Bjorn Helgaas Date: Thu, 13 Nov 2014 11:20:15 +0100 In-Reply-To: <546481A3.5050700@freescale.com> References: <1415682444-24911-1-git-send-email-Minghuan.Lian@freescale.com> <546308DD.40109@freescale.com> <546331E7.5060003@freescale.com> <1415809929.2876.5.camel@pengutronix.de> <546481A3.5050700@freescale.com> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Sender: linux-pci-owner@vger.kernel.org List-ID: Am Donnerstag, den 13.11.2014, 18:02 +0800 schrieb Lian Minghuan-B31939: > Hi Lucas, > > Please see my comments inline. > > Thanks, > Minghuan > > On 2014年11月13日 00:32, Lucas Stach wrote: > > Am Mittwoch, den 12.11.2014, 21:53 +0530 schrieb Srikanth Thokala: > >> Hi Minghuan, [...] > > Using a smaller type complicates the DT for little to no benefit. I > > think it's ok to use u32 here, which is a common standard for integer > > values in DT. > > > > Though this discussion lead me to the question if we even need to have > > this property in the DT at all. Isn't this a property that is fixed for > > a specific silicon implementation of the DW core? In that case we could > > just infer the number of ATUs from the DT compatible, so this should > > probably just be added to struct pcie_port and properly initialized by > > the SoC glue drivers. > [Minghuan] As far as I know, exynos implements only 2 ATUs, this is why > pcie-designware only supports 2 ATU. iMX implements 4 ATUs and LS1021A > implements 6 ATUs. > Right so we don't need an additional property in the DT at all. The number of ATUs is fixed for a specific core compatible and can be passed in by the respective exynos, imx and ls1021 glue drivers. You may ask the Keystone and Spear maintainers to get the correct number of ATUs for those implementations. Regards, Lucas -- Pengutronix e.K. | Lucas Stach | Industrial Linux Solutions | http://www.pengutronix.de/ |