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 38CE4C433FE for ; Thu, 10 Mar 2022 10:24:51 +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:Message-ID:Date:References :In-Reply-To:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=UBBBcFjXg/ssWdkjIvcqCq2Lo7+aC45Xzw4T+MaFHLA=; b=U5r0djtMoWqG7e pJca1Nr0WO9IB1E6UUPQRW0aunZUb3pDtzzfJP//tWO9t9wiAAlnojJJ9v1PS4cruNX4cw1uye6jV gF3MOGSiR+mKSQ8ubbECUBdflrogPz96W45kcuv8N1n5cj8HAq8LldMnsB8osKWT0hx9tplAPDzcw Dkp+dT3KB0uv0tvO2zVtFW/2w2Xz4LZmLSEw2pPJdJClWgyQVhOBOaFvinOm0kd46FGtLPqHNsDIP oqW87uZ0wk7rW9NtQ0eCXog9WyoNCMNYvi6BjIpU1uQB9A2CHgD3Z0PJi8tKzN5cXFXcotI+42e2I RkWHFl3MOIkOL8EhvZPw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nSFxM-00CN56-Hb; Thu, 10 Mar 2022 10:23:40 +0000 Received: from relay1-d.mail.gandi.net ([2001:4b98:dc4:8::221]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nSFxI-00CN2p-7A for linux-arm-kernel@lists.infradead.org; Thu, 10 Mar 2022 10:23:38 +0000 Received: (Authenticated sender: gregory.clement@bootlin.com) by mail.gandi.net (Postfix) with ESMTPSA id A546E24000D; Thu, 10 Mar 2022 10:23:31 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1646907813; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=5QlkG0Yk6rlOSIS9fyw+RpNZ0vcoYZG+oYxg92O6ZcU=; b=XGcWQfkJ+iTF9exGdtsYJ3vgGz6AtM7oL6mDLCg0ezqySnzV3E7896d99zAVW0ZXk8VLGw WezI84Tl3X7L85zfsFwkBq3ZTl7gVxlXbZHIqnLuDKXpM57ZS1PvaD+pCLB8zEyPYdXfkc FasLA3bfaaVxdPQWaI99uvgH/JyFUHfrO5GXBGphYxQ1Y89c+ndAPbLml7n4qocxfqD2oF e4+5n/NGN34Fj7qkGxjW6fKfYqoBahA0oDVy1YFn+5bCVcQSazWCeTpN76nBtnaV/ZEkVn NBmCKNndXQxb6XJzd2cKO1AJBLXnNt+tiYhLgJUC1HZqI9PiBQzN/m2mG01bbA== From: Gregory CLEMENT To: Pali =?utf-8?Q?Roh=C3=A1r?= Cc: Andrew Lunn , Marek =?utf-8?Q?Beh=C3=BAn?= , Arnd Bergmann , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] arm64: dts: marvell: armada-37xx: Remap IO space to bus address 0x0 In-Reply-To: <20220310100943.m33wsynnvexw7dts@pali> References: <20220218212526.16021-1-pali@kernel.org> <20220304163027.29357-1-pali@kernel.org> <87k0d2i0mr.fsf@BL-laptop> <20220310100943.m33wsynnvexw7dts@pali> Date: Thu, 10 Mar 2022 11:23:26 +0100 Message-ID: <87h786hzs1.fsf@BL-laptop> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220310_022336_583947_C55EF28B X-CRM114-Status: GOOD ( 31.93 ) 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 UGFsaSBSb2jDoXIgPHBhbGlAa2VybmVsLm9yZz4gd3JpdGVzOgoKPiBPbiBUaHVyc2RheSAxMCBN YXJjaCAyMDIyIDExOjA1OjAwIEdyZWdvcnkgQ0xFTUVOVCB3cm90ZToKPj4gSGVsbG8gUGFsaSwK Pj4gCj4+ID4gTGVnYWN5IGFuZCBvbGQgUENJIEkvTyBiYXNlZCBjYXJkcyBkbyBub3Qgc3VwcG9y dCAzMi1iaXQgSS9PIGFkZHJlc3NpbmcuCj4+ID4KPj4gPiBTaW5jZSBjb21taXQgNjRmMTYwZTE5 ZTkyICgiUENJOiBhYXJkdmFyazogQ29uZmlndXJlIFBDSWUgcmVzb3VyY2VzIGZyb20KPj4gPiAn cmFuZ2VzJyBEVCBwcm9wZXJ0eSIpIGtlcm5lbCBjYW4gc2V0IGRpZmZlcmVudCBQQ0llIGFkZHJl c3Mgb24gQ1BVIGFuZAo+PiA+IGRpZmZlcmVudCBvbiB0aGUgYnVzIGZvciB0aGUgb25lIEEzN3h4 IGFkZHJlc3MgbWFwcGluZyB3aXRob3V0IGFueSBmaXJtd2FyZQo+PiA+IHN1cHBvcnQgaW4gY2Fz ZSB0aGUgYnVzIGFkZHJlc3MgZG9lcyBub3QgY29uZmxpY3Qgd2l0aCBvdGhlciBBMzd4eCBtYXBw aW5nLgo+PiA+Cj4+ID4gU28gcmVtYXAgSS9PIHNwYWNlIHRvIHRoZSBidXMgYWRkcmVzcyAweDAg dG8gZW5hYmxlIHN1cHBvcnQgZm9yIG9sZCBsZWdhY3kKPj4gPiBJL08gcG9ydCBiYXNlZCBjYXJk cyB3aGljaCBoYXZlIGhhcmRjb2RlZCBJL08gcG9ydHMgaW4gbG93IGFkZHJlc3Mgc3BhY2UuCj4+ ID4KPj4gPiBOb3RlIHRoYXQgRERSIG9uIEEzN3h4IGlzIG1hcHBlZCB0byBidXMgYWRkcmVzcyAw eDAuIEFuZCBtYXBwaW5nIG9mIEkvTwo+PiA+IHNwYWNlIGNhbiBiZSBzZXQgdG8gYWRkcmVzcyAw eDAgdG9vIGJlY2F1c2UgTUVNIHNwYWNlIGFuZCBJL08gc3BhY2UgYXJlCj4+ID4gc2VwYXJhdGUg YW5kIHNvIGRvIG5vdCBjb25mbGljdC4KPj4gPgo+PiA+IFJlbWFwcGluZyBJTyBzcGFjZSBvbiBU dXJyaXMgTW94IHRvIGRpZmZlcmVudCBhZGRyZXNzIGlzIG5vdCBwb3NzaWJsZSB0bwo+PiA+IGR1 ZSBib290bG9hZGVyIGJ1Zy4KPj4gPgo+PiA+IFNpZ25lZC1vZmYtYnk6IFBhbGkgUm9ow6FyIDxw YWxpQGtlcm5lbC5vcmc+Cj4+ID4gUmVwb3J0ZWQtYnk6IEFybmQgQmVyZ21hbm4gPGFybmRAYXJu ZGIuZGU+Cj4+ID4gRml4ZXM6IDc2ZjYzODZiMjVjYyAoImFybTY0OiBkdHM6IG1hcnZlbGw6IEFk ZCBBYXJkdmFyayBQQ0llIHN1cHBvcnQgZm9yIEFybWFkYSAzNzAwIikKPj4gPiBDYzogc3RhYmxl QHZnZXIua2VybmVsLm9yZyAjIDY0ZjE2MGUxOWU5MiAoIlBDSTogYWFyZHZhcms6IENvbmZpZ3Vy ZSBQQ0llIHJlc291cmNlcyBmcm9tICdyYW5nZXMnIERUIHByb3BlcnR5IikKPj4gPiBDYzogc3Rh YmxlQHZnZXIua2VybmVsLm9yZyAjIDUxNGVmMWU2MmQ2NSAoImFybTY0OiBkdHM6IG1hcnZlbGw6 IGFybWFkYS0zN3h4OiBFeHRlbmQgUENJZSBNRU0gc3BhY2UiKQo+PiA+Cj4+IENjOiBzdGFibGVA dmdlci5rZXJuZWwub3JnICMgPz8/Pz8/Pz8/Pz8/ICgiYXJtNjQ6IGR0czogbWFydmVsbDogYXJt YWRhLTM3eHg6IEluY3JlYXNlIFBDSWUgSU8gc2l6ZSBmcm9tIDY0IEtpQiB0byAxIE1pQiIpCj4+ IAo+PiBUaGlzIHBhdGNoIGhhcyBiZWVuIHJlZnVzZWQgYnkgQXJuZCBzbyBJIHJlbW92ZWQgaXQg ZnJvbSB0aGUgbXZlYnUvZml4ZXMKPj4gYnJhbmNoIHNvIHlvdSBzaG91bGQgbm90IGFwcGx5IGFu eXRoaW5nIG9uIHRvcCBvZiBpdC4KPgo+IE9rLCBzbyB3aGF0IGlzIHdyb25nIHdpdGggYSBjaGFu Z2Ugd2hpY2ggaW5jcmVhc2Ugc2l6ZSBvZiBJTyBzcGFjZSB0byAxIE1CPwoKaHR0cHM6Ly9sb3Jl Lmtlcm5lbC5vcmcvbGludXgtYXJtLWtlcm5lbC9DQUs4UDNhMkQ4WXYrS3BNNE5KeVA5bW9zaWVx YmhIaDA4PW1kRXkrT0E4NFZ4NkZWQ1FAbWFpbC5nbWFpbC5jb20vCgo+Cj4+IEFjdHVhbGx5IEkg c3RpbGwgdHJ5IHRvIGZpcnN0IGFwcGx5IHRoZSBvbGQgcGF0Y2ggYW5kIHRoZW4gdGhpcyBvbmUg YnV0Cj4+IGl0IHN0aWxsIGZhaWwuIEFuZCBpdCBpcyBhbHNvIGZhaWxlZCB3aGVuIEkgYXBwbGll ZCB0aGlzIG9uZSBvbiBhCj4+IHY1LjE3LXJjMSwgc28gSSB3b25kZXJlZCBvbiB3aGljaCBkaWQg Y3JlYXRlIHRoaXMgcGF0Y2guCj4KPiBPaywgYXQgd2hpY2ggYnJhbmNoIC8gY29tbWl0IHNob3Vs ZCBJIHJlYmFzZSBpdD8KClBsZWFzZSBjcmVhdGUgb25seSBvbmUgc2luZ2xlIHBhdGNoIG9uIHRv cCBvZiB2NS4xNy1yYzEuCgpUaGFua3MhCgpHcmVnb3J5Cgo+Cj4+IEdyw6lnb3J5Cj4+IAo+PiA+ IC0tLQo+PiA+IENoYW5nZXMgaW4gdjI6Cj4+ID4gKiBEbyBub3QgcmVtYXAgSU8gc3BhY2Ugb24g VHVycmlzIE1veAo+PiA+IC0tLQo+PiA+ICBhcmNoL2FybTY0L2Jvb3QvZHRzL21hcnZlbGwvYXJt YWRhLTM3MjAtdHVycmlzLW1veC5kdHMgfCA3ICsrKysrKy0KPj4gPiAgYXJjaC9hcm02NC9ib290 L2R0cy9tYXJ2ZWxsL2FybWFkYS0zN3h4LmR0c2kgICAgICAgICAgIHwgMiArLQo+PiA+ICAyIGZp bGVzIGNoYW5nZWQsIDcgaW5zZXJ0aW9ucygrKSwgMiBkZWxldGlvbnMoLSkKPj4gPgo+PiA+IGRp ZmYgLS1naXQgYS9hcmNoL2FybTY0L2Jvb3QvZHRzL21hcnZlbGwvYXJtYWRhLTM3MjAtdHVycmlz LW1veC5kdHMgYi9hcmNoL2FybTY0L2Jvb3QvZHRzL21hcnZlbGwvYXJtYWRhLTM3MjAtdHVycmlz LW1veC5kdHMKPj4gPiBpbmRleCA2NTgxMDkyYzJjOTAuLjI4MzhlM2Y2NWFkYSAxMDA2NDQKPj4g PiAtLS0gYS9hcmNoL2FybTY0L2Jvb3QvZHRzL21hcnZlbGwvYXJtYWRhLTM3MjAtdHVycmlzLW1v eC5kdHMKPj4gPiArKysgYi9hcmNoL2FybTY0L2Jvb3QvZHRzL21hcnZlbGwvYXJtYWRhLTM3MjAt dHVycmlzLW1veC5kdHMKPj4gPiBAQCAtMTUwLDE3ICsxNTAsMjIgQEAKPj4gPiAgCXNsb3QtcG93 ZXItbGltaXQgPSA8MTAwMDA+Owo+PiA+ICAJLyoKPj4gPiAgCSAqIFUtQm9vdCBwb3J0IGZvciBU dXJyaXMgTW94IGhhcyBhIGJ1ZyB3aGljaCBhbHdheXMgZXhwZWN0cyB0aGF0ICJyYW5nZXMiIERU IHByb3BlcnR5Cj4+ID4gIAkgKiBjb250YWlucyBleGFjdGx5IDIgcmFuZ2VzIHdpdGggMyAoY2hp bGQpIGFkZHJlc3MgY2VsbHMsIDIgKHBhcmVudCkgYWRkcmVzcyBjZWxscyBhbmQKPj4gPiAtCSAq IDIgc2l6ZSBjZWxscyBhbmQgYWxzbyBleHBlY3RzIHRoYXQgdGhlIHNlY29uZCByYW5nZSBzdGFy dHMgYXQgMTYgTUIgb2Zmc2V0LiBJZiB0aGVzZQo+PiA+ICsJICogMiBzaXplIGNlbGxzIGFuZCBh bHNvIGV4cGVjdHMgdGhhdCB0aGUgc2Vjb25kIHJhbmdlIHN0YXJ0cyBhdCAxNiBNQiBvZmZzZXQu IEFsc28gaXQKPj4gPiArCSAqIGV4cGVjdHMgdGhhdCBmaXJzdCByYW5nZSB1c2VzIHNhbWUgYWRk cmVzcyBmb3IgUENJIChjaGlsZCkgYW5kIENQVSAocGFyZW50KSBjZWxscyAoc28KPj4gPiArCSAq IG5vIHJlbWFwcGluZykgYW5kIHRoYXQgdGhpcyBhZGRyZXNzIGlzIHRoZSBsb3dlc3QgZnJvbSBh bGwgc3BlY2lmaWVkIHJhbmdlcy4gSWYgdGhlc2UKPj4gPiAgCSAqIGNvbmRpdGlvbnMgYXJlIG5v dCBtZXQgdGhlbiBVLUJvb3QgY3Jhc2hlcyBkdXJpbmcgbG9hZGluZyBrZXJuZWwgRFRCIGZpbGUu IFBDSWUgYWRkcmVzcwo+PiA+ICAJICogc3BhY2UgaXMgMTI4IE1CIGxvbmcsIHNvIHRoZSBiZXN0 IHNwbGl0IGJldHdlZW4gTUVNIGFuZCBJTyBpcyB0byB1c2UgZml4ZWQgMTYgTUIgd2luZG93Cj4+ ID4gIAkgKiBmb3IgSU8gYW5kIHRoZSByZXN0IDExMiBNQiAoNjQrMzIrMTYpIGZvciBNRU0uIENv bnRyb2xsZXIgc3VwcG9ydHMgMzItYml0IElPIG1hcHBpbmcuCj4+ID4gIAkgKiBUaGlzIGJ1ZyBp cyBub3QgcHJlc2VudCBpbiBVLUJvb3QgcG9ydHMgZm9yIG90aGVyIEFybWFkYSAzNzAwIGRldmlj ZXMgYW5kIGlzIGZpeGVkIGluCj4+ID4gIAkgKiBVLUJvb3QgdmVyc2lvbiAyMDIxLjA3LiBTZWUg cmVsZXZhbnQgVS1Cb290IGNvbW1pdHMgKHRoZSBsYXN0IG9uZSBjb250YWlucyBmaXgpOgo+PiA+ ICAJICogaHR0cHM6Ly9zb3VyY2UuZGVueC5kZS91LWJvb3QvdS1ib290Ly0vY29tbWl0L2NiMmRk YjI5MWVlNmZjYmRkZDZkOGY0ZmY0OTA4OWRmZTU4MGY1ZDcKPj4gPiAgCSAqIGh0dHBzOi8vc291 cmNlLmRlbnguZGUvdS1ib290L3UtYm9vdC8tL2NvbW1pdC9jNjRhYzNiMzE4NWFlYjM4NDYyOTdh ZDczOTFmYzZkZjhlY2Q3M2JmCj4+ID4gIAkgKiBodHRwczovL3NvdXJjZS5kZW54LmRlL3UtYm9v dC91LWJvb3QvLS9jb21taXQvNGE4MmZjYThlMzMwMTU3MDgxZmMxMzJhNTkxZWJkOTliYTAyZWUz Mwo+PiA+ICsJICogQnVnIHJlbGF0ZWQgdG8gcmVxdWlyZW1lbnQgb2Ygc2FtZSBjaGlsZCBhbmQg cGFyZW50IGFkZHJlc3NlcyBmb3IgZmlyc3QgcmFuZ2UgaXMgZml4ZWQKPj4gPiArCSAqIGluIFUt Qm9vdCB2ZXJzaW9uIDIwMjIuMDQgYnkgZm9sbG93aW5nIGNvbW1pdDoKPj4gPiArCSAqIGh0dHBz Oi8vc291cmNlLmRlbnguZGUvdS1ib290L3UtYm9vdC8tL2NvbW1pdC8xZmQ1NDI1M2JjYTdkNDNk MDQ2YmJhNDg1M2ZlNWZhZmQwMzRiYzE3Cj4+ID4gIAkgKi8KPj4gPiAgCSNhZGRyZXNzLWNlbGxz ID0gPDM+Owo+PiA+ICAJI3NpemUtY2VsbHMgPSA8Mj47Cj4+ID4gIAlyYW5nZXMgPSA8MHg4MTAw MDAwMCAwIDB4ZTgwMDAwMDAgICAwIDB4ZTgwMDAwMDAgICAwIDB4MDEwMDAwMDAgICAvKiBQb3J0 IDAgSU8gKi8KPj4gPiBkaWZmIC0tZ2l0IGEvYXJjaC9hcm02NC9ib290L2R0cy9tYXJ2ZWxsL2Fy bWFkYS0zN3h4LmR0c2kgYi9hcmNoL2FybTY0L2Jvb3QvZHRzL21hcnZlbGwvYXJtYWRhLTM3eHgu ZHRzaQo+PiA+IGluZGV4IDU0OWMzZjdjNWIyNy4uYTA5OWI3Nzg3NDI5IDEwMDY0NAo+PiA+IC0t LSBhL2FyY2gvYXJtNjQvYm9vdC9kdHMvbWFydmVsbC9hcm1hZGEtMzd4eC5kdHNpCj4+ID4gKysr IGIvYXJjaC9hcm02NC9ib290L2R0cy9tYXJ2ZWxsL2FybWFkYS0zN3h4LmR0c2kKPj4gPiBAQCAt NTE0LDkgKzUxNCw5IEBACj4+ID4gIAkJCSAqIElPIGF0IHRoZSBlbmQgYW5kIHRoZSByZW1haW5p bmcgc2V2ZW4gd2luZG93cwo+PiA+ICAJCQkgKiAodG90YWxpbmcgMTI3IE1pQikgZm9yIE1FTS4K Pj4gPiAgCQkJICovCj4+ID4gIAkJCXJhbmdlcyA9IDwweDgyMDAwMDAwIDAgMHhlODAwMDAwMCAg IDAgMHhlODAwMDAwMCAgIDAgMHgwN2YwMDAwMCAgIC8qIFBvcnQgMCBNRU0gKi8KPj4gPiAtCQkJ CSAgMHg4MTAwMDAwMCAwIDB4ZWZmMDAwMDAgICAwIDB4ZWZmMDAwMDAgICAwIDB4MDAxMDAwMDA+ OyAvKiBQb3J0IDAgSU8qLwo+PiA+ICsJCQkJICAweDgxMDAwMDAwIDAgMHgwMDAwMDAwMCAgIDAg MHhlZmYwMDAwMCAgIDAgMHgwMDEwMDAwMD47IC8qIFBvcnQgMCBJTyAqLwo+PiA+ICAJCQlpbnRl cnJ1cHQtbWFwLW1hc2sgPSA8MCAwIDAgNz47Cj4+ID4gIAkJCWludGVycnVwdC1tYXAgPSA8MCAw IDAgMSAmcGNpZV9pbnRjIDA+LAo+PiA+ICAJCQkJCTwwIDAgMCAyICZwY2llX2ludGMgMT4sCj4+ ID4gIAkJCQkJPDAgMCAwIDMgJnBjaWVfaW50YyAyPiwKPj4gPiAtLSAKPj4gPiAyLjIwLjEKPj4g Pgo+PiAKPj4gLS0gCj4+IEdyZWdvcnkgQ2xlbWVudCwgQm9vdGxpbgo+PiBFbWJlZGRlZCBMaW51 eCBhbmQgS2VybmVsIGVuZ2luZWVyaW5nCj4+IGh0dHA6Ly9ib290bGluLmNvbQoKLS0gCkdyZWdv cnkgQ2xlbWVudCwgQm9vdGxpbgpFbWJlZGRlZCBMaW51eCBhbmQgS2VybmVsIGVuZ2luZWVyaW5n Cmh0dHA6Ly9ib290bGluLmNvbQoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX18KbGludXgtYXJtLWtlcm5lbCBtYWlsaW5nIGxpc3QKbGludXgtYXJtLWtlcm5l bEBsaXN0cy5pbmZyYWRlYWQub3JnCmh0dHA6Ly9saXN0cy5pbmZyYWRlYWQub3JnL21haWxtYW4v bGlzdGluZm8vbGludXgtYXJtLWtlcm5lbAo= 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 D369FC433F5 for ; Thu, 10 Mar 2022 10:23:41 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S241179AbiCJKYk (ORCPT ); Thu, 10 Mar 2022 05:24:40 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:40732 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234060AbiCJKYi (ORCPT ); Thu, 10 Mar 2022 05:24:38 -0500 Received: from relay1-d.mail.gandi.net (relay1-d.mail.gandi.net [217.70.183.193]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D209AEDF25 for ; Thu, 10 Mar 2022 02:23:34 -0800 (PST) Received: (Authenticated sender: gregory.clement@bootlin.com) by mail.gandi.net (Postfix) with ESMTPSA id A546E24000D; Thu, 10 Mar 2022 10:23:31 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1646907813; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=5QlkG0Yk6rlOSIS9fyw+RpNZ0vcoYZG+oYxg92O6ZcU=; b=XGcWQfkJ+iTF9exGdtsYJ3vgGz6AtM7oL6mDLCg0ezqySnzV3E7896d99zAVW0ZXk8VLGw WezI84Tl3X7L85zfsFwkBq3ZTl7gVxlXbZHIqnLuDKXpM57ZS1PvaD+pCLB8zEyPYdXfkc FasLA3bfaaVxdPQWaI99uvgH/JyFUHfrO5GXBGphYxQ1Y89c+ndAPbLml7n4qocxfqD2oF e4+5n/NGN34Fj7qkGxjW6fKfYqoBahA0oDVy1YFn+5bCVcQSazWCeTpN76nBtnaV/ZEkVn NBmCKNndXQxb6XJzd2cKO1AJBLXnNt+tiYhLgJUC1HZqI9PiBQzN/m2mG01bbA== From: Gregory CLEMENT To: Pali =?utf-8?Q?Roh=C3=A1r?= Cc: Andrew Lunn , Marek =?utf-8?Q?Beh=C3=BAn?= , Arnd Bergmann , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] arm64: dts: marvell: armada-37xx: Remap IO space to bus address 0x0 In-Reply-To: <20220310100943.m33wsynnvexw7dts@pali> References: <20220218212526.16021-1-pali@kernel.org> <20220304163027.29357-1-pali@kernel.org> <87k0d2i0mr.fsf@BL-laptop> <20220310100943.m33wsynnvexw7dts@pali> Date: Thu, 10 Mar 2022 11:23:26 +0100 Message-ID: <87h786hzs1.fsf@BL-laptop> 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 Pali Roh=C3=A1r writes: > On Thursday 10 March 2022 11:05:00 Gregory CLEMENT wrote: >> Hello Pali, >>=20 >> > Legacy and old PCI I/O based cards do not support 32-bit I/O addressin= g. >> > >> > Since commit 64f160e19e92 ("PCI: aardvark: Configure PCIe resources fr= om >> > 'ranges' DT property") kernel can set different PCIe address on CPU and >> > different on the bus for the one A37xx address mapping without any fir= mware >> > support in case the bus address does not conflict with other A37xx map= ping. >> > >> > So remap I/O space to the bus address 0x0 to enable support for old le= gacy >> > I/O port based cards which have hardcoded I/O ports in low address spa= ce. >> > >> > Note that DDR on A37xx is mapped to bus address 0x0. And mapping of I/O >> > space can be set to address 0x0 too because MEM space and I/O space are >> > separate and so do not conflict. >> > >> > Remapping IO space on Turris Mox to different address is not possible = to >> > due bootloader bug. >> > >> > Signed-off-by: Pali Roh=C3=A1r >> > Reported-by: Arnd Bergmann >> > Fixes: 76f6386b25cc ("arm64: dts: marvell: Add Aardvark PCIe support f= or Armada 3700") >> > Cc: stable@vger.kernel.org # 64f160e19e92 ("PCI: aardvark: Configure P= CIe resources from 'ranges' DT property") >> > Cc: stable@vger.kernel.org # 514ef1e62d65 ("arm64: dts: marvell: armad= a-37xx: Extend PCIe MEM space") >> > >> Cc: stable@vger.kernel.org # ???????????? ("arm64: dts: marvell: armada-= 37xx: Increase PCIe IO size from 64 KiB to 1 MiB") >>=20 >> This patch has been refused by Arnd so I removed it from the mvebu/fixes >> branch so you should not apply anything on top of it. > > Ok, so what is wrong with a change which increase size of IO space to 1 M= B? https://lore.kernel.org/linux-arm-kernel/CAK8P3a2D8Yv+KpM4NJyP9mosieqbhHh08= =3DmdEy+OA84Vx6FVCQ@mail.gmail.com/ > >> Actually I still try to first apply the old patch and then this one but >> it still fail. And it is also failed when I applied this one on a >> v5.17-rc1, so I wondered on which did create this patch. > > Ok, at which branch / commit should I rebase it? Please create only one single patch on top of v5.17-rc1. Thanks! Gregory > >> Gr=C3=A9gory >>=20 >> > --- >> > Changes in v2: >> > * Do not remap IO space on Turris Mox >> > --- >> > arch/arm64/boot/dts/marvell/armada-3720-turris-mox.dts | 7 ++++++- >> > arch/arm64/boot/dts/marvell/armada-37xx.dtsi | 2 +- >> > 2 files changed, 7 insertions(+), 2 deletions(-) >> > >> > diff --git a/arch/arm64/boot/dts/marvell/armada-3720-turris-mox.dts b/= arch/arm64/boot/dts/marvell/armada-3720-turris-mox.dts >> > index 6581092c2c90..2838e3f65ada 100644 >> > --- a/arch/arm64/boot/dts/marvell/armada-3720-turris-mox.dts >> > +++ b/arch/arm64/boot/dts/marvell/armada-3720-turris-mox.dts >> > @@ -150,17 +150,22 @@ >> > slot-power-limit =3D <10000>; >> > /* >> > * U-Boot port for Turris Mox has a bug which always expects that "r= anges" DT property >> > * contains exactly 2 ranges with 3 (child) address cells, 2 (parent= ) address cells and >> > - * 2 size cells and also expects that the second range starts at 16 = MB offset. If these >> > + * 2 size cells and also expects that the second range starts at 16 = MB offset. Also it >> > + * expects that first range uses same address for PCI (child) and CP= U (parent) cells (so >> > + * no remapping) and that this address is the lowest from all specif= ied ranges. If these >> > * conditions are not met then U-Boot crashes during loading kernel = DTB file. PCIe address >> > * space is 128 MB long, so the best split between MEM and IO is to = use fixed 16 MB window >> > * for IO and the rest 112 MB (64+32+16) for MEM. Controller support= s 32-bit IO mapping. >> > * This bug is not present in U-Boot ports for other Armada 3700 dev= ices and is fixed in >> > * U-Boot version 2021.07. See relevant U-Boot commits (the last one= contains fix): >> > * https://source.denx.de/u-boot/u-boot/-/commit/cb2ddb291ee6fcbddd6= d8f4ff49089dfe580f5d7 >> > * https://source.denx.de/u-boot/u-boot/-/commit/c64ac3b3185aeb38462= 97ad7391fc6df8ecd73bf >> > * https://source.denx.de/u-boot/u-boot/-/commit/4a82fca8e330157081f= c132a591ebd99ba02ee33 >> > + * Bug related to requirement of same child and parent addresses for= first range is fixed >> > + * in U-Boot version 2022.04 by following commit: >> > + * https://source.denx.de/u-boot/u-boot/-/commit/1fd54253bca7d43d046= bba4853fe5fafd034bc17 >> > */ >> > #address-cells =3D <3>; >> > #size-cells =3D <2>; >> > ranges =3D <0x81000000 0 0xe8000000 0 0xe8000000 0 0x01000000 = /* Port 0 IO */ >> > diff --git a/arch/arm64/boot/dts/marvell/armada-37xx.dtsi b/arch/arm64= /boot/dts/marvell/armada-37xx.dtsi >> > index 549c3f7c5b27..a099b7787429 100644 >> > --- a/arch/arm64/boot/dts/marvell/armada-37xx.dtsi >> > +++ b/arch/arm64/boot/dts/marvell/armada-37xx.dtsi >> > @@ -514,9 +514,9 @@ >> > * IO at the end and the remaining seven windows >> > * (totaling 127 MiB) for MEM. >> > */ >> > ranges =3D <0x82000000 0 0xe8000000 0 0xe8000000 0 0x07f00000 = /* Port 0 MEM */ >> > - 0x81000000 0 0xeff00000 0 0xeff00000 0 0x00100000>; /* Port= 0 IO*/ >> > + 0x81000000 0 0x00000000 0 0xeff00000 0 0x00100000>; /* Port= 0 IO */ >> > interrupt-map-mask =3D <0 0 0 7>; >> > interrupt-map =3D <0 0 0 1 &pcie_intc 0>, >> > <0 0 0 2 &pcie_intc 1>, >> > <0 0 0 3 &pcie_intc 2>, >> > --=20 >> > 2.20.1 >> > >>=20 >> --=20 >> Gregory Clement, Bootlin >> Embedded Linux and Kernel engineering >> http://bootlin.com --=20 Gregory Clement, Bootlin Embedded Linux and Kernel engineering http://bootlin.com