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 44B93C3DA63 for ; Wed, 24 Jul 2024 16:24:35 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=BmmCsrEKtkT6QYIrH14E6ieUaFyLJiDeKFWKx3Gt5fE=; b=SkFM4zz3cQ2xUNEX6Yi4kvqjsD X7GwOe4ideVvQrfuEvOhzXKIwKDsmVhdY4gRkU7T9fT3cwDpSqCmgL4t56u9z30UsOCsNTwoOzasm X5XwVoDAfbh4a0iKilRkW7M7HBMt2/rLC4OeQrRJYZyPHyGWlG9ifjumPQFB75fOviJ41X+oSLY4u 57sqaJcIFmqPneckiF7qzKWiFSoiADhoGw36pAtGaesi/E6nv8DrWNNWzyh+Hw/GWp2BTi2IsvcrY Vp62xksPrqXwFo9CRaRiZauRtdrHr+UAotSnLtKvAnnbSB0qzSRynxdAKhhwQoABFrMi414uO4Sfj yJWabCBg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sWemt-0000000FvO2-1Zut; Wed, 24 Jul 2024 16:24:23 +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 1sWemU-0000000FvGC-3EtA for linux-arm-kernel@lists.infradead.org; Wed, 24 Jul 2024 16:24:00 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sin.source.kernel.org (Postfix) with ESMTP id D50BFCE1151; Wed, 24 Jul 2024 16:23:55 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 9AD62C32781; Wed, 24 Jul 2024 16:23:52 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1721838235; bh=BmmCsrEKtkT6QYIrH14E6ieUaFyLJiDeKFWKx3Gt5fE=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=nkKFNMpCfiWbCFG+Pp1FtO3+vTnuz38B1up8r8fzu7d5Dl6bm23c2H2h8fjMQ4GSh +hEsR0rLfmXoMerAoRvTQbyufA/ReMluaaN3r3cBL3QEStg2UiIrEqZmEMU9xUQvne eVe0Vrm2hv0E5OCV8+m7U86bTebvMZT0aVsT6R/LFKiimOfb89jxuEK9Hf/46Osbz7 KNL+ZMRlGr0S6GOAAAdcdTTLQDS3frbcVyxwOOvFQvplmU1lv5rtZuo0MkmLX2LHJn TRS06EDojNbNQinzoQdPt0B5iR4bDUH2lb0SbksHD7ef3fxZA2s67sl9ZD1fqKj6uk ITh+s4US5DuJw== Date: Wed, 24 Jul 2024 17:23:50 +0100 From: Conor Dooley To: Krzysztof Kozlowski Cc: Hongxing Zhu , "robh@kernel.org" , "krzk+dt@kernel.org" , "conor+dt@kernel.org" , "shawnguo@kernel.org" , "l.stach@pengutronix.de" , "devicetree@vger.kernel.org" , "linux-pci@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "kernel@pengutronix.de" , "imx@lists.linux.dev" Subject: Re: [PATCH v2 1/4] dt-bindings: imx6q-pcie: Add reg-name "dbi2" and "atu" for i.MX8M PCIe Endpoint Message-ID: <20240724-among-citric-cb3084658ae5@spud> References: <1721790236-3966-1-git-send-email-hongxing.zhu@nxp.com> <1721790236-3966-2-git-send-email-hongxing.zhu@nxp.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="q2WNMnQ6djTo/dXQ" Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240724_092359_178480_1B4A0199 X-CRM114-Status: GOOD ( 19.92 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --q2WNMnQ6djTo/dXQ Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Jul 24, 2024 at 08:32:34AM +0200, Krzysztof Kozlowski wrote: > On 24/07/2024 08:26, Hongxing Zhu wrote: > >> -----Original Message----- > >> From: Krzysztof Kozlowski > >> Sent: 2024=E5=B9=B47=E6=9C=8824=E6=97=A5 14:07 > >> To: Hongxing Zhu ; robh@kernel.org; > >> krzk+dt@kernel.org; conor+dt@kernel.org; shawnguo@kernel.org; > >> l.stach@pengutronix.de > >> Cc: devicetree@vger.kernel.org; linux-pci@vger.kernel.org; > >> linux-arm-kernel@lists.infradead.org; linux-kernel@vger.kernel.org; > >> kernel@pengutronix.de; imx@lists.linux.dev > >> Subject: Re: [PATCH v2 1/4] dt-bindings: imx6q-pcie: Add reg-name "dbi= 2" and > >> "atu" for i.MX8M PCIe Endpoint > >> > >> On 24/07/2024 05:03, Richard Zhu wrote: > >>> Add reg-name: "dbi2", "atu" for i.MX8M PCIe Endpoint. > >> > >> This we see in the diff. What I do not see is why? Hardware changed? H= ow come? > >> > > For i.MX8M PCIe EP, the dbi2 and atu address are pre-defined in the dri= ver. > > This method is not good. > > In commit b7d67c6130ee ("PCI: imx6: Add iMX95 Endpoint (EP) support"), > > Frank suggests to fetch the dbi2 and atu from DT directly. > > This series is preparation to do that for i.MX8M PCIe EP. >=20 > This all must be explained in commit msg. >=20 > Anyway, this will be an ABI break, so explain exactly why it is OK to > break the ABI. And the driver needs to be written in such a way that if only two reg properties are provided, it falls back to the old method of acquiring the two new reg regions. I didn't see a driver patch on v1, so I missed that this is what was actually the plan - I thought that this was just adding two missing regions. --q2WNMnQ6djTo/dXQ Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZqEqlgAKCRB4tDGHoIJi 0mScAQDrCdN9N+hytVE/UlNUGEVVwiDGq1y4l4IPkDsDAQMKVgEA0Z07x7tn8/PX 6RpN5hGJjqrWdBYkWTFj3QNupP3kHQc= =PRJV -----END PGP SIGNATURE----- --q2WNMnQ6djTo/dXQ--