From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7CA7415FCF0; Tue, 7 May 2024 14:55:59 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1715093759; cv=none; b=dAZTa9DOKlFyuhL06s1l8DQ1t6tcl7DyJknJSeRosuoHLb45LJItkdGO6dNKux+FmA9fl21mOoYkvVkBp1bU1ZBkEupRrZzwGx/CWSZDKrRxz0+cx67ZcFmOGXXNEd+a7ShAYJThlu686zqzSdCaM8+W4VLRre3xoYNFtaL0ID0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1715093759; c=relaxed/simple; bh=hDoeAt0aCGqvaRLZUhPUSNs/Q6T/LZp1Y/m0zwy/8VA=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=a16QLWM4Kw+jewiYEsmvgn+xeFZKHse80UmdICkCcIYugzSg7SuxmdvzOunDavYptLV6DM5K26QFQ7xNtwnd/j5600A+cC9I3zFUPVVkrkmni3SaCy9PJe0+/a61JOp9eWxI0i9CJ1rBkdE0z/enr247UrRDQgGVCJfiB5LGHaU= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=QEUq8giv; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="QEUq8giv" Received: by smtp.kernel.org (Postfix) with ESMTPSA id D7414C2BBFC; Tue, 7 May 2024 14:55:58 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1715093759; bh=hDoeAt0aCGqvaRLZUhPUSNs/Q6T/LZp1Y/m0zwy/8VA=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=QEUq8givoAZQpiA+7kCVzFHDZ7o4/uRf2Ar8D35gCBskdc8ctAAiedeU/2XcNdmDT /jJ4W9KC09jl5eJwW1gWoAcrkfDbCHuF/ztqlKUSN5B07z8NBbUEr8A/F2GuvPAsUD VzzyajXyTrylJteeF95fmW9DZRu9p88ZJe+A4v5sw67LOfTOr/KgAnubAfaJ9VZhJ+ VIucFJJmVs9jM0KQ5vMNZz9383IvHLtkceQlByPzxQFXKNG+gwncdHKxoDGqs8QXbs AWNngkoj6WDB2F6S0p3I1G2z1cg9AjAkW8wYzfsuCo+i4vnruH75rqVzsiQgCLLSIt P59VCWQBxvNhQ== Date: Tue, 7 May 2024 09:55:57 -0500 From: Rob Herring To: Frank Li Cc: Richard Zhu , Lucas Stach , Lorenzo Pieralisi , Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= , Bjorn Helgaas , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Philipp Zabel , Liam Girdwood , Mark Brown , Manivannan Sadhasivam , Krzysztof Kozlowski , Conor Dooley , linux-pci@vger.kernel.org, imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, bpf@vger.kernel.org, devicetree@vger.kernel.org Subject: Re: [PATCH v3 10/11] dt-bindings: imx6q-pcie: Add i.MX8Q pcie compatible string Message-ID: <20240507145557.GA461201-robh@kernel.org> References: <20240402-pci2_upstream-v3-0-803414bdb430@nxp.com> <20240402-pci2_upstream-v3-10-803414bdb430@nxp.com> <20240429154823.GD1709920-robh@kernel.org> Precedence: bulk X-Mailing-List: imx@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: On Mon, Apr 29, 2024 at 05:23:23PM -0400, Frank Li wrote: > On Mon, Apr 29, 2024 at 10:48:23AM -0500, Rob Herring wrote: > > On Tue, Apr 02, 2024 at 10:33:46AM -0400, Frank Li wrote: > > > From: Richard Zhu > > > > > > Add i.MX8Q PCIe "fsl,imx8q-pcie" compatible strings. > > > > > > Add "fsl,local-address" property for i.MX8Q platforms. fsl,local-address > > > is address of PCIe module in high speed io (HSIO)subsystem bus fabric. HSIO > > > bus fabric convert the incoming address base to this local-address. Two > > > instances of PCI have difference local address. > > > > This is just some intermediate bus address? We really should be able to > > describe this with standard ranges properties. > > Yes, Maybe dwc's implement have some problem. After read below doc again > https://elinux.org/Device_Tree_Usage#PCI_Address_Translation > > ┌──────┐ ┌──────────┐ > ┌────┐0x18001000 │ │ │ │ > │CPU ├───────────►│ ├──┤ Others │ > └────┘ │ │ │ │ > │ │ └──────────┘ > │ │ > │ │ ┌─────────┐ > │ │ │ │ ┌───────────┐ > │ ├──►│ HSIO │ 0xB8001000 ├───────────┤ > │ │ │ Fabric ├───────────►│Bar0 │ TLP mem 0xB8001000 > │ │ │ │ │0xB8000000 ├───────► > └──────┘ └─────────┘ │ │ Note the 0xB8xxxxxxx address on the right is a PCI address which could be anything though folks often make it 1:1. > Main Fabric ├───────────┤ > │ │ > │ │ > │ │ > │ │ > │ │ > │ │ > │ DWC │ > │ PCIe │ > │ Controller│ > │ │ > │ │ > └───────────┘ > > > dts should be > > ranges = <0x82000000 0 0xB8000000 0x18000000 0 0x07f00000> > ^^^^ And HSIO needs a node with ranges = <0xb8000000 0x18000000 size>; Rob From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 66AD5C25B4F for ; Tue, 7 May 2024 14:56:21 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=nXmI8hzqO56+6OjGnQ89MxbPFLAP39QTfglbNuSeNNY=; b=gUWRwHVv3oeG08 I36tOvfpZgNJt1s0rGxqJ42Llnq1FHq2/ZvVJtXNplnLKgQ+qt/lAqw+gwDTDAG5I7e0BIAdsl6M1 6TKP85mpxHHQSWC6nkVN6ubXKWww1CdHQ25ALnQUMb1rwY33EMYOyG7v0FHxjlyTEPAVIpyXNTRef Pl6++5rrl6RHRuGZzK9JvwO05VoiZmBXpJjEWprZPhNeDPVByDexYVD7v2pZgCQmnXuTxLmTCIh6V fN1gn6h/Vnfy7AeLRtL1DItTg6L1E+xFnxZ0BUwNd0CJk2nuFI9vtW3HBf+55QGYNKoJWyjt4BrLa r3c+r65Mlz4Ad8F/6Zvg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1s4MEe-0000000BcJc-1F8n; Tue, 07 May 2024 14:56:04 +0000 Received: from sin.source.kernel.org ([2604:1380:40e1:4800::1]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1s4MEb-0000000BcHT-2q3Q for linux-arm-kernel@lists.infradead.org; Tue, 07 May 2024 14:56:03 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sin.source.kernel.org (Postfix) with ESMTP id E6A56CE119B; Tue, 7 May 2024 14:55:59 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id D7414C2BBFC; Tue, 7 May 2024 14:55:58 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1715093759; bh=hDoeAt0aCGqvaRLZUhPUSNs/Q6T/LZp1Y/m0zwy/8VA=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=QEUq8givoAZQpiA+7kCVzFHDZ7o4/uRf2Ar8D35gCBskdc8ctAAiedeU/2XcNdmDT /jJ4W9KC09jl5eJwW1gWoAcrkfDbCHuF/ztqlKUSN5B07z8NBbUEr8A/F2GuvPAsUD VzzyajXyTrylJteeF95fmW9DZRu9p88ZJe+A4v5sw67LOfTOr/KgAnubAfaJ9VZhJ+ VIucFJJmVs9jM0KQ5vMNZz9383IvHLtkceQlByPzxQFXKNG+gwncdHKxoDGqs8QXbs AWNngkoj6WDB2F6S0p3I1G2z1cg9AjAkW8wYzfsuCo+i4vnruH75rqVzsiQgCLLSIt P59VCWQBxvNhQ== Date: Tue, 7 May 2024 09:55:57 -0500 From: Rob Herring To: Frank Li Cc: Richard Zhu , Lucas Stach , Lorenzo Pieralisi , Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= , Bjorn Helgaas , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Philipp Zabel , Liam Girdwood , Mark Brown , Manivannan Sadhasivam , Krzysztof Kozlowski , Conor Dooley , linux-pci@vger.kernel.org, imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, bpf@vger.kernel.org, devicetree@vger.kernel.org Subject: Re: [PATCH v3 10/11] dt-bindings: imx6q-pcie: Add i.MX8Q pcie compatible string Message-ID: <20240507145557.GA461201-robh@kernel.org> References: <20240402-pci2_upstream-v3-0-803414bdb430@nxp.com> <20240402-pci2_upstream-v3-10-803414bdb430@nxp.com> <20240429154823.GD1709920-robh@kernel.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240507_075602_101433_283C7DDA X-CRM114-Status: GOOD ( 17.09 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org T24gTW9uLCBBcHIgMjksIDIwMjQgYXQgMDU6MjM6MjNQTSAtMDQwMCwgRnJhbmsgTGkgd3JvdGU6 Cj4gT24gTW9uLCBBcHIgMjksIDIwMjQgYXQgMTA6NDg6MjNBTSAtMDUwMCwgUm9iIEhlcnJpbmcg d3JvdGU6Cj4gPiBPbiBUdWUsIEFwciAwMiwgMjAyNCBhdCAxMDozMzo0NkFNIC0wNDAwLCBGcmFu ayBMaSB3cm90ZToKPiA+ID4gRnJvbTogUmljaGFyZCBaaHUgPGhvbmd4aW5nLnpodUBueHAuY29t Pgo+ID4gPiAKPiA+ID4gQWRkIGkuTVg4USBQQ0llICJmc2wsaW14OHEtcGNpZSIgY29tcGF0aWJs ZSBzdHJpbmdzLgo+ID4gPiAKPiA+ID4gQWRkICJmc2wsbG9jYWwtYWRkcmVzcyIgcHJvcGVydHkg Zm9yIGkuTVg4USBwbGF0Zm9ybXMuIGZzbCxsb2NhbC1hZGRyZXNzCj4gPiA+IGlzIGFkZHJlc3Mg b2YgUENJZSBtb2R1bGUgaW4gaGlnaCBzcGVlZCBpbyAoSFNJTylzdWJzeXN0ZW0gYnVzIGZhYnJp Yy4gSFNJTwo+ID4gPiBidXMgZmFicmljIGNvbnZlcnQgdGhlIGluY29taW5nIGFkZHJlc3MgYmFz ZSB0byB0aGlzIGxvY2FsLWFkZHJlc3MuIFR3bwo+ID4gPiBpbnN0YW5jZXMgb2YgUENJIGhhdmUg ZGlmZmVyZW5jZSBsb2NhbCBhZGRyZXNzLgo+ID4gCj4gPiBUaGlzIGlzIGp1c3Qgc29tZSBpbnRl cm1lZGlhdGUgYnVzIGFkZHJlc3M/IFdlIHJlYWxseSBzaG91bGQgYmUgYWJsZSB0byAKPiA+IGRl c2NyaWJlIHRoaXMgd2l0aCBzdGFuZGFyZCByYW5nZXMgcHJvcGVydGllcy4KPiAKPiBZZXMsIE1h eWJlIGR3YydzIGltcGxlbWVudCBoYXZlIHNvbWUgcHJvYmxlbS4gQWZ0ZXIgcmVhZCBiZWxvdyBk b2MgYWdhaW4KPiBodHRwczovL2VsaW51eC5vcmcvRGV2aWNlX1RyZWVfVXNhZ2UjUENJX0FkZHJl c3NfVHJhbnNsYXRpb24KPiAKPiAgICAgICAgICAgICAgICAgICDilIzilIDilIDilIDilIDilIDi lIDilJAgIOKUjOKUgOKUgOKUgOKUgOKUgOKUgOKUgOKUgOKUgOKUgOKUkCAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgIAo+IOKUjOKUgOKUgOKUgOKUgOKUkDB4MTgwMDEwMDAgIOKUgiAg ICAgIOKUgiAg4pSCICAgICAgICAgIOKUgiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg IAo+IOKUgkNQVSDilJzilIDilIDilIDilIDilIDilIDilIDilIDilIDilIDilIDilrrilIIgICAg ICDilJzilIDilIDilKQgIE90aGVycyAg4pSCICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgCj4g4pSU4pSA4pSA4pSA4pSA4pSYICAgICAgICAgICAg4pSCICAgICAg4pSCICDilIIgICAg ICAgICAg4pSCICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgCj4gICAgICAgICAgICAg ICAgICAg4pSCICAgICAg4pSCICDilJTilIDilIDilIDilIDilIDilIDilIDilIDilIDilIDilJgg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAKPiAgICAgICAgICAgICAgICAgICDilIIg ICAgICDilIIgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIAo+ ICAgICAgICAgICAgICAgICAgIOKUgiAgICAgIOKUgiAgIOKUjOKUgOKUgOKUgOKUgOKUgOKUgOKU gOKUgOKUgOKUkCAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIAo+ICAgICAgICAgICAg ICAgICAgIOKUgiAgICAgIOKUgiAgIOKUgiAgICAgICAgIOKUgiAgICAgICAgICAgIOKUjOKUgOKU gOKUgOKUgOKUgOKUgOKUgOKUgOKUgOKUgOKUgOKUkCAgICAgICAgCj4gICAgICAgICAgICAgICAg ICAg4pSCICAgICAg4pSc4pSA4pSA4pa64pSCIEhTSU8gICAg4pSCIDB4QjgwMDEwMDAg4pSc4pSA 4pSA4pSA4pSA4pSA4pSA4pSA4pSA4pSA4pSA4pSA4pSkICAgICAgICAKPiAgICAgICAgICAgICAg ICAgICDilIIgICAgICDilIIgICDilIIgRmFicmljICDilJzilIDilIDilIDilIDilIDilIDilIDi lIDilIDilIDilIDilrrilIJCYXIwICAgICAgIOKUgiBUTFAgbWVtIDB4QjgwMDEwMDAgICAKPiAg ICAgICAgICAgICAgICAgICDilIIgICAgICDilIIgICDilIIgICAgICAgICDilIIgICAgICAgICAg ICDilIIweEI4MDAwMDAwIOKUnOKUgOKUgOKUgOKUgOKUgOKUgOKUgOKWugo+ICAgICAgICAgICAg ICAgICAgIOKUlOKUgOKUgOKUgOKUgOKUgOKUgOKUmCAgIOKUlOKUgOKUgOKUgOKUgOKUgOKUgOKU gOKUgOKUgOKUmCAgICAgICAgICAgIOKUgiAgICAgICAgICAg4pSCICAgICAgICAKCk5vdGUgdGhl IDB4Qjh4eHh4eHh4IGFkZHJlc3Mgb24gdGhlIHJpZ2h0IGlzIGEgUENJIGFkZHJlc3Mgd2hpY2gg Y291bGQgCmJlIGFueXRoaW5nIHRob3VnaCBmb2xrcyBvZnRlbiBtYWtlIGl0IDE6MS4KCj4gICAg ICAgICAgICAgICAgICAgTWFpbiBGYWJyaWMgICAgICAgICAgICAgICAgICAgICAgIOKUnOKUgOKU gOKUgOKUgOKUgOKUgOKUgOKUgOKUgOKUgOKUgOKUpCAgICAgICAgCj4gICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIOKUgiAgICAgICAgICAg4pSCICAg ICAgICAKPiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAg4pSCICAgICAgICAgICDilIIgICAgICAgIAo+ICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICDilIIgICAgICAgICAgIOKUgiAgICAgICAgCj4gICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIOKUgiAgICAg ICAgICAg4pSCICAgICAgICAKPiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAg4pSCICAgICAgICAgICDilIIgICAgICAgIAo+ICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICDilIIgICAgICAgICAgIOKUgiAg ICAgICAgCj4gICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgIOKUgiBEV0MgICAgICAg4pSCICAgICAgICAKPiAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAg4pSCIFBDSWUgICAgICDilIIgICAgICAgIAo+ICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICDilIIgQ29u dHJvbGxlcuKUgiAgICAgICAgCj4gICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgIOKUgiAgICAgICAgICAg4pSCICAgICAgICAKPiAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg4pSCICAgICAgICAgICDilIIg ICAgICAgIAo+ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICDilJTilIDilIDilIDilIDilIDilIDilIDilIDilIDilIDilIDilJggICAgICAgIAo+IAo+ IAo+IGR0cyBzaG91bGQgYmUKPiAKPiByYW5nZXMgPSA8MHg4MjAwMDAwMCAwIDB4QjgwMDAwMDAg MHgxODAwMDAwMCAwIDB4MDdmMDAwMDA+Cj4gCQkgICAgICAgXl5eXgoKQW5kIEhTSU8gbmVlZHMg YSBub2RlIHdpdGggCgpyYW5nZXMgPSA8MHhiODAwMDAwMCAweDE4MDAwMDAwIHNpemU+OwoKUm9i CgoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KbGludXgt YXJtLWtlcm5lbCBtYWlsaW5nIGxpc3QKbGludXgtYXJtLWtlcm5lbEBsaXN0cy5pbmZyYWRlYWQu b3JnCmh0dHA6Ly9saXN0cy5pbmZyYWRlYWQub3JnL21haWxtYW4vbGlzdGluZm8vbGludXgtYXJt LWtlcm5lbAo=