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 C61AAC433F5 for ; Wed, 2 Mar 2022 13:16:57 +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:MIME-Version:References:In-Reply-To: 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=7NfEdnugWG3lMCsLvB8iLB5ca+m+5YzMTcKD/kXSiQc=; b=zrH+7rTvfXhWqC VRK9ppgyQGOv77VznHPtBfzJIHXtYZZomQzZeFwFawfLLFIlj6ftCWhLY3nb0fSQyXOamUiU59pvl PPY6Nrmn+H4dVDElQGU6qRlQ70LyKkx1GpG72QkCu2n3p4+JT2I6WUUcNqIpQnIW3rju6FluuyBsR OFAU/2rm8K0eZfmqHzfQTJhYwohH1Ln3RvlxFLSnpt5m1jdguxuXz9br1vc+KL+1TUwBl4EDbwkxu SP5hqHOqN/jU+r0fPX0xQiUae3d+u8NoHrseyqA9LzJSK6wJqVoCDiTlWgGEqMc4ZOf/6tTcDeKE+ ukUz9MqXrwUDjXu0cGEw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nPOpC-002jXi-LP; Wed, 02 Mar 2022 13:15:26 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nPOp8-002jWV-FJ for linux-arm-kernel@lists.infradead.org; Wed, 02 Mar 2022 13:15:24 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 774D86195A; Wed, 2 Mar 2022 13:15:21 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id D5E57C004E1; Wed, 2 Mar 2022 13:15:19 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1646226920; bh=Q4H0riTTWP2jL+cmFpHpBq4jbdqrA2hs5QRCXpPCZlY=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=ZzSCnDmB0RPLAN3jOn4eOtyld8lVAahCFIxPQZemiwSkmgsHs8PulyOqkC5mFBGjh Fg7MesUv2wL6EvccPozn7977rZ++P6NBj1TlBn+3qJaFtQLK5SdxxYY8eATUAKGnKh 95gw5hlLaqAv8cUskRcQO73ZqerceqaA64k3eeULUrZny6KZT2CpLGP0BVKpB8leqz A8M/HAoxgbbuR1zeI10PIb7tvRH77OqGcJ2oVtkzBdyZ75MtemD7SUFx7wPafNynwF U1pa5I1mSRDgXyiOmhMs06SdkDopr/7gJrqWh56LJnymBPm208we7lUbhcoauuFWQt rYaka5pjdBQSQ== Date: Wed, 2 Mar 2022 14:15:15 +0100 From: Marek =?UTF-8?B?QmVow7pu?= To: Andrew Lunn Cc: Pali =?UTF-8?B?Um9ow6Fy?= , Gregory CLEMENT , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] arm64: dts: marvell: armada-37xx: Remap IO space to bus address 0x0 Message-ID: <20220302141515.51340cab@dellmb> In-Reply-To: References: <20220218212526.16021-1-pali@kernel.org> <87o82r0zjh.fsf@BL-laptop> <875yoz0wpw.fsf@BL-laptop> <20220301092539.lru7hsaqxrjqz32r@pali> X-Mailer: Claws Mail 3.18.0 (GTK+ 2.24.33; x86_64-pc-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220302_051522_616621_687FA2D6 X-CRM114-Status: GOOD ( 28.83 ) 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 T24gV2VkLCAyIE1hciAyMDIyIDE0OjA2OjAxICswMTAwCkFuZHJldyBMdW5uIDxhbmRyZXdAbHVu bi5jaD4gd3JvdGU6Cgo+IE9uIFR1ZSwgTWFyIDAxLCAyMDIyIGF0IDEwOjI1OjM5QU0gKzAxMDAs IFBhbGkgUm9ow6FyIHdyb3RlOgo+ID4gT24gTW9uZGF5IDI4IEZlYnJ1YXJ5IDIwMjIgMTc6NDI6 MDMgR3JlZ29yeSBDTEVNRU5UIHdyb3RlOiAgCj4gPiA+ID4gSGVsbG8gUGFsaSwKPiA+ID4gPiAg Cj4gPiA+ID4+IFJlbWFwIFBDSSBJL08gc3BhY2UgdG8gdGhlIGJ1cyBhZGRyZXNzIDB4MCBpbiB0 aGUgQXJtYWRhIDM3eHgKPiA+ID4gPj4gZGV2aWNlLXRyZWUgaW4gb3JkZXIgdG8gc3VwcG9ydCBs ZWdhY3kgSS9PIHBvcnQgYmFzZWQgY2FyZHMgd2hpY2ggaGF2ZQo+ID4gPiA+PiBoYXJkY29kZWQg SS9PIHBvcnRzIGluIGxvdyBhZGRyZXNzIHNwYWNlLgo+ID4gPiA+Pgo+ID4gPiA+PiBTb21lIGxl Z2FjeSBQQ0kgSS9PIGJhc2VkIGNhcmRzIGRvIG5vdCBzdXBwb3J0IDMyLWJpdCBJL08gYWRkcmVz c2luZy4KPiA+ID4gPj4KPiA+ID4gPj4gU2luY2UgY29tbWl0IDY0ZjE2MGUxOWU5MiAoIlBDSTog YWFyZHZhcms6IENvbmZpZ3VyZSBQQ0llIHJlc291cmNlcyBmcm9tCj4gPiA+ID4+ICdyYW5nZXMn IERUIHByb3BlcnR5IikgdGhpcyBkcml2ZXIgY2FuIHdvcmsgd2l0aCBJL08gd2luZG93cyB3aGlj aAo+ID4gPiA+PiBoYXZlICAKPiA+ID4gPgo+ID4gPiA+IFNob3VsZCB3ZSBhZGQgYSAiRml4ZXM6 IDY0ZjE2MGUxOWU5MiAoIlBDSTogYWFyZHZhcms6IENvbmZpZ3VyZSBQQ0llCj4gPiA+ID4gcmVz b3VyY2VzIGZyb20gJ3JhbmdlcycgRFQgcHJvcGVydHkiKSIgdGFnID8gIAo+ID4gPiAKPiA+ID4g V2FpdGluZyBmb3IgeW91ciBjb25maXJtYXRpb24gSSB0cmllZCB0byBhcHBsaWVkIGl0IGJ1dCBp dCBmYWlsZWQuCj4gPiA+IAo+ID4gPiBEaWQgeW91IGJhc2UgdGhpcyBwYXRjaCBvbiB2NS4xNy1y YzEgPwo+ID4gPiAKPiA+ID4gR3JlZ29yeSAgCj4gPiAKPiA+IEhlbGxvISBUaGlzIGNoYW5nZSBp cyBicmVha2luZyBib290aW5nIG9mIFR1cnJpcyBNb3gga2VybmVsIHdpdGggb2xkZXIKPiA+IGJv b3Rsb2FkZXIgZHVlIHRvIGJ1Z3MgaW4gYm9vdGxvYWRlci4gIAo+IAo+IERvIHlvdSBrbm93IHdo YXQgYWN0dWFsbHkgZ29lcyB3cm9uZz8KPiAKPiBJJ3ZlIG5vdCBiZWVuIGludm9sdmVkIGluIHRo ZSBkaXNjdXNzaW9uLCBidXQgbG9va2luZyBhdCB0aGUgY29tbWVudHMKPiBhYm92ZSwgbm90IGNo YW5naW5nIHRoZSBzcGFjZSBjYW4gcmVzdWx0IGluIG5vbi13b3JraW5nIGNhcmRzLiBTbyBpdAo+ IGRvZXMgc291bmQgbGlrZSBzb21ldGhpbmcgd2hpY2ggaW4gZ2VuZXJhbCB3ZSB3YW50IHRvIGRv LiBEb2VzIHRoZQo+IGN1cnJlbnQgY29kZSBhc3N1bWUgdGhlIGJvb3Rsb2FkZXIgaGFzIGluaXRp YWxpemVkIHNvbWUgcmVnaXN0ZXJzIHdpdGgKPiBzcGVjaWZpYyB2YWx1ZXM/IENhbiB0aGF0IGJl IG1vdmVkIGludG8gdGhlIGRyaXZlciBzbyBpdCBhbHNvIHdvcmtzCj4gd2l0aCBvbGRlciBib290 bG9hZGVycz8KCk5vLiBURi1BIG1heSByZW1hcCBDUFUgUENJZSB3aW5kb3csIGFuZCBzbyBVLUJv b3QgZml4ZXMgdGhlc2UgYWRkcmVzc2VzCmluIGRldmljZS10cmVlLiBCdXQgdGhlIGZpeHVwIGZ1 bmN0aW9uIHdhcyBhdCBmaXJzdCB3cml0dGVuIGluIHN1Y2ggYQp3YXkgdGhhdCBpdCBhc3N1bWVz IHRoYXQgdGhlIHJhbmdlcyBwcm9wcmV0aWVzIGNvbnRhaW5zIHNwZWNpZmljCnZhbHVlcy4gVGhl IHByb3Bvc2VkIERUIGNoYW5nZSwgdG9nZXRoZXIgd2l0aCB0aGUgZml4dXAgZnVuY3Rpb24gaW4K b2xkZXIgVS1Cb290LCB3aWxsIGJyZWFrIHJhbmdlcyBwcm9wZXJ0eSB0byBub24tZnVuY3Rpb25h bCBzdGF0ZS4KClNlZSBjb3JyZXNwb25kaW5nIFUtQm9vdCBwYXRjaGVzCgpodHRwczovL3BhdGNo d29yay5vemxhYnMub3JnL3Byb2plY3QvdWJvb3QvcGF0Y2gvMjAyMDA0MDgxNzI1MjIuMTg5NDEt NS1tYXJlay5iZWh1bkBuaWMuY3ovCmh0dHBzOi8vcGF0Y2h3b3JrLm96bGFicy5vcmcvcHJvamVj dC91Ym9vdC9wYXRjaC8yMDIxMDUyNjE1NTk0MC4yNjE0MS01LXBhbGlAa2VybmVsLm9yZy8KaHR0 cHM6Ly9wYXRjaHdvcmsub3psYWJzLm9yZy9wcm9qZWN0L3Vib290L3BhdGNoLzIwMjIwMjIzMTI1 MjMyLjc5NzQtMS1rYWJlbEBrZXJuZWwub3JnLwoKVGhlIGxhc3QgcGF0Y2ggaXMgbm90IG1lcmdl ZCB5ZXQuCgpNYXJlawoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX18KbGludXgtYXJtLWtlcm5lbCBtYWlsaW5nIGxpc3QKbGludXgtYXJtLWtlcm5lbEBsaXN0 cy5pbmZyYWRlYWQub3JnCmh0dHA6Ly9saXN0cy5pbmZyYWRlYWQub3JnL21haWxtYW4vbGlzdGlu Zm8vbGludXgtYXJtLWtlcm5lbAo= 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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 89EA4C433F5 for ; Wed, 2 Mar 2022 13:15:28 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S242044AbiCBNQJ (ORCPT ); Wed, 2 Mar 2022 08:16:09 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34398 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S241604AbiCBNQH (ORCPT ); Wed, 2 Mar 2022 08:16:07 -0500 Received: from ams.source.kernel.org (ams.source.kernel.org [IPv6:2604:1380:4601:e00::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 87EE0C3C2C for ; Wed, 2 Mar 2022 05:15:24 -0800 (PST) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 36F13B81F16 for ; Wed, 2 Mar 2022 13:15:22 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id D5E57C004E1; Wed, 2 Mar 2022 13:15:19 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1646226920; bh=Q4H0riTTWP2jL+cmFpHpBq4jbdqrA2hs5QRCXpPCZlY=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=ZzSCnDmB0RPLAN3jOn4eOtyld8lVAahCFIxPQZemiwSkmgsHs8PulyOqkC5mFBGjh Fg7MesUv2wL6EvccPozn7977rZ++P6NBj1TlBn+3qJaFtQLK5SdxxYY8eATUAKGnKh 95gw5hlLaqAv8cUskRcQO73ZqerceqaA64k3eeULUrZny6KZT2CpLGP0BVKpB8leqz A8M/HAoxgbbuR1zeI10PIb7tvRH77OqGcJ2oVtkzBdyZ75MtemD7SUFx7wPafNynwF U1pa5I1mSRDgXyiOmhMs06SdkDopr/7gJrqWh56LJnymBPm208we7lUbhcoauuFWQt rYaka5pjdBQSQ== Date: Wed, 2 Mar 2022 14:15:15 +0100 From: Marek =?UTF-8?B?QmVow7pu?= To: Andrew Lunn Cc: Pali =?UTF-8?B?Um9ow6Fy?= , Gregory CLEMENT , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] arm64: dts: marvell: armada-37xx: Remap IO space to bus address 0x0 Message-ID: <20220302141515.51340cab@dellmb> In-Reply-To: References: <20220218212526.16021-1-pali@kernel.org> <87o82r0zjh.fsf@BL-laptop> <875yoz0wpw.fsf@BL-laptop> <20220301092539.lru7hsaqxrjqz32r@pali> X-Mailer: Claws Mail 3.18.0 (GTK+ 2.24.33; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 2 Mar 2022 14:06:01 +0100 Andrew Lunn wrote: > On Tue, Mar 01, 2022 at 10:25:39AM +0100, Pali Roh=C3=A1r wrote: > > On Monday 28 February 2022 17:42:03 Gregory CLEMENT wrote: =20 > > > > Hello Pali, > > > > =20 > > > >> Remap PCI I/O space to the bus address 0x0 in the Armada 37xx > > > >> device-tree in order to support legacy I/O port based cards which = have > > > >> hardcoded I/O ports in low address space. > > > >> > > > >> Some legacy PCI I/O based cards do not support 32-bit I/O addressi= ng. > > > >> > > > >> Since commit 64f160e19e92 ("PCI: aardvark: Configure PCIe resource= s from > > > >> 'ranges' DT property") this driver can work with I/O windows which > > > >> have =20 > > > > > > > > Should we add a "Fixes: 64f160e19e92 ("PCI: aardvark: Configure PCIe > > > > resources from 'ranges' DT property")" tag ? =20 > > >=20 > > > Waiting for your confirmation I tried to applied it but it failed. > > >=20 > > > Did you base this patch on v5.17-rc1 ? > > >=20 > > > Gregory =20 > >=20 > > Hello! This change is breaking booting of Turris Mox kernel with older > > bootloader due to bugs in bootloader. =20 >=20 > Do you know what actually goes wrong? >=20 > I've not been involved in the discussion, but looking at the comments > above, not changing the space can result in non-working cards. So it > does sound like something which in general we want to do. Does the > current code assume the bootloader has initialized some registers with > specific values? Can that be moved into the driver so it also works > with older bootloaders? No. TF-A may remap CPU PCIe window, and so U-Boot fixes these addresses in device-tree. But the fixup function was at first written in such a way that it assumes that the ranges propreties contains specific values. The proposed DT change, together with the fixup function in older U-Boot, will break ranges property to non-functional state. See corresponding U-Boot patches https://patchwork.ozlabs.org/project/uboot/patch/20200408172522.18941-5-mar= ek.behun@nic.cz/ https://patchwork.ozlabs.org/project/uboot/patch/20210526155940.26141-5-pal= i@kernel.org/ https://patchwork.ozlabs.org/project/uboot/patch/20220223125232.7974-1-kabe= l@kernel.org/ The last patch is not merged yet. Marek