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 3AB2CC433EF for ; Tue, 22 Feb 2022 16:19:01 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233981AbiBVQTZ (ORCPT ); Tue, 22 Feb 2022 11:19:25 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:45814 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232772AbiBVQTY (ORCPT ); Tue, 22 Feb 2022 11:19:24 -0500 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8AD1C166A50; Tue, 22 Feb 2022 08:18:58 -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 dfw.source.kernel.org (Postfix) with ESMTPS id 2489C61665; Tue, 22 Feb 2022 16:18:58 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 2F537C340E8; Tue, 22 Feb 2022 16:18:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1645546737; bh=IzYQgPVIrzdZrtx5q+tqnEui3obxWSTUv594dzI//Qk=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=WKV8OLQMZMHYyR3NO+/mjQki2fzbApuXnlQCu4xDAlF7LZQqC/T/OV/5CpSTohQ7R 6fCggzwEt931z8/DaVJ0kX61MEl+fizf1beaiN4UhjJ9Dk6Fe5ItuFifMrT0dnZlij IIxHvYEWb2EIjNLQBz3A9dp0JGIayHgXUvvnu0n6p1MlUJPgBSlZzUmpzmkbxi0QFs 1oqycRDr7vH+048HwrIURCyhWw5MG40icPvHcjPNkofomaXXbOelugkKOuhFlLO+qy w8LZclUEQOlTTVAtuCz34V5AmhkM3Xpx13PMA1Ks9lGnGsaw7cDkXzt9XDN3UMrUAt /WWSD1DmhiHBQ== Received: by pali.im (Postfix) id CD1F4FDB; Tue, 22 Feb 2022 17:18:54 +0100 (CET) Date: Tue, 22 Feb 2022 17:18:54 +0100 From: Pali =?utf-8?B?Um9ow6Fy?= To: Gregory CLEMENT Cc: Lorenzo Pieralisi , Russell King , Bjorn Helgaas , Thomas Petazzoni , Marek =?utf-8?B?QmVow7pu?= , Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= , Rob Herring , linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: (subset) [PATCH v4 00/12] PCI: mvebu: subsystem ids, AER and INTx Message-ID: <20220222161854.7oot5v6xlw2wzmsc@pali> References: <20220222155030.988-1-pali@kernel.org> <164554589988.5595.5091384618177225445.b4-ty@arm.com> <20220222161143.6ryghgtfmhnmhpmz@pali> <20220222161539.GA20114@lpieralisi> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20220222161539.GA20114@lpieralisi> User-Agent: NeoMutt/20180716 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org + Gregory On Tuesday 22 February 2022 16:15:39 Lorenzo Pieralisi wrote: > On Tue, Feb 22, 2022 at 05:11:43PM +0100, Pali Rohár wrote: > > On Tuesday 22 February 2022 16:06:20 Lorenzo Pieralisi wrote: > > > On Tue, 22 Feb 2022 16:50:18 +0100, Pali Rohár wrote: > > > > This patch series extends pci-bridge-emul.c driver to emulate PCI Subsystem > > > > Vendor ID capability and PCIe extended capabilities. And then implement > > > > in pci-mvebu.c driver support for PCI Subsystem Vendor IDs, PCIe AER > > > > registers, support for legacy INTx interrupts, configuration for X1/X4 > > > > mode and usage of new PCI child_ops API. > > > > > > > > Changes in v4: > > > > * rebased on c3bd7dc553eea5a3595ca3aa0adee9bf83622a1f > > > > > > > > [...] > > > > > > I can't apply dts changes, patch 12 should go via the arm-soc tree. > > > > Gregory already wrote about this dts change: > > https://lore.kernel.org/linux-pci/87tud1jwpr.fsf@BL-laptop/ > > "So the easier is to let merge it through the PCI subsystem with the > > other patches from this series." > > > > Are there any issues with applying this dts change via pci tree? > > I don't usually take dts changes through the PCI tree since they > can conflict with arm-soc, that's the issue - dts changes should > be managed by platform maintainers. Gregory, could you please take patch 12/12? https://lore.kernel.org/linux-pci/20220222155030.988-13-pali@kernel.org/ You have already Acked-by (see above previous link) > Thanks, > Lorenzo > > > > Applied the others to pci/mvebu, thanks. > > > > > > [01/12] PCI: pci-bridge-emul: Re-arrange register tests > > > https://git.kernel.org/lpieralisi/pci/c/c453bf6f9b > > > [02/12] PCI: pci-bridge-emul: Add support for PCIe extended capabilities > > > https://git.kernel.org/lpieralisi/pci/c/c0bd419732 > > > [03/12] PCI: pci-bridge-emul: Add support for PCI Bridge Subsystem Vendor ID capability > > > https://git.kernel.org/lpieralisi/pci/c/3767a90242 > > > [04/12] dt-bindings: PCI: mvebu: Add num-lanes property > > > https://git.kernel.org/lpieralisi/pci/c/26b982ca83 > > > [05/12] PCI: mvebu: Correctly configure x1/x4 mode > > > https://git.kernel.org/lpieralisi/pci/c/2a81dd9fd9 > > > [06/12] PCI: mvebu: Add support for PCI Bridge Subsystem Vendor ID on emulated bridge > > > https://git.kernel.org/lpieralisi/pci/c/e3e13c9135 > > > [07/12] PCI: mvebu: Add support for Advanced Error Reporting registers on emulated bridge > > > https://git.kernel.org/lpieralisi/pci/c/2b6ee04c0a > > > [08/12] PCI: mvebu: Use child_ops API > > > https://git.kernel.org/lpieralisi/pci/c/c099c2a761 > > > [09/12] dt-bindings: PCI: mvebu: Update information about intx interrupts > > > https://git.kernel.org/lpieralisi/pci/c/0124989220 > > > [10/12] PCI: mvebu: Fix macro names and comments about legacy interrupts > > > https://git.kernel.org/lpieralisi/pci/c/d00ea94e62 > > > [11/12] PCI: mvebu: Implement support for legacy INTx interrupts > > > https://git.kernel.org/lpieralisi/pci/c/ec07526264 > > > > > > Thanks, > > > Lorenzo 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 3A8EBC433F5 for ; Tue, 22 Feb 2022 16:20:19 +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=U7oZATodbShbxgzISsyf0VQA/fXMFkUxdFO+XMtICqo=; b=twaw6P1CtbWtvP Cs41Kc4bSkLtrC3e1bMPiUtxjj9rx5TPgQVezk3QwdG529sSCKziWur9fqZ8ZhRvULLL7pyB5ePlC odAPKbYlGjoeKTF/IX/eueilk57LpBlMQNQI/ptIh/+8FYibrvf54k/am+AS/CSL+ximZHU93bRCV Y7EtuA+cDvA7ngcl043iITP5oZGBrLgBeIEyikXhfgvWkcXZMPxR8pCk5Op6l+e7g4hDQqNpP643C kGI+nigPQbsEGi9TQ13dC/JqLEdclDQTo7ClnN5e1l0nvW4MQMrqp+R/gVeDn9CTt924L5avCYmhY CnOto7lA4+qKbUX1hu9A==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nMXsY-00Ab9Y-Tq; Tue, 22 Feb 2022 16:19:07 +0000 Received: from ams.source.kernel.org ([145.40.68.75]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nMXsT-00Ab7j-Kz for linux-arm-kernel@lists.infradead.org; Tue, 22 Feb 2022 16:19:03 +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 ams.source.kernel.org (Postfix) with ESMTPS id 2DDF5B81B73; Tue, 22 Feb 2022 16:19:00 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 2F537C340E8; Tue, 22 Feb 2022 16:18:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1645546737; bh=IzYQgPVIrzdZrtx5q+tqnEui3obxWSTUv594dzI//Qk=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=WKV8OLQMZMHYyR3NO+/mjQki2fzbApuXnlQCu4xDAlF7LZQqC/T/OV/5CpSTohQ7R 6fCggzwEt931z8/DaVJ0kX61MEl+fizf1beaiN4UhjJ9Dk6Fe5ItuFifMrT0dnZlij IIxHvYEWb2EIjNLQBz3A9dp0JGIayHgXUvvnu0n6p1MlUJPgBSlZzUmpzmkbxi0QFs 1oqycRDr7vH+048HwrIURCyhWw5MG40icPvHcjPNkofomaXXbOelugkKOuhFlLO+qy w8LZclUEQOlTTVAtuCz34V5AmhkM3Xpx13PMA1Ks9lGnGsaw7cDkXzt9XDN3UMrUAt /WWSD1DmhiHBQ== Received: by pali.im (Postfix) id CD1F4FDB; Tue, 22 Feb 2022 17:18:54 +0100 (CET) Date: Tue, 22 Feb 2022 17:18:54 +0100 From: Pali =?utf-8?B?Um9ow6Fy?= To: Gregory CLEMENT Cc: Lorenzo Pieralisi , Russell King , Bjorn Helgaas , Thomas Petazzoni , Marek =?utf-8?B?QmVow7pu?= , Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= , Rob Herring , linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: (subset) [PATCH v4 00/12] PCI: mvebu: subsystem ids, AER and INTx Message-ID: <20220222161854.7oot5v6xlw2wzmsc@pali> References: <20220222155030.988-1-pali@kernel.org> <164554589988.5595.5091384618177225445.b4-ty@arm.com> <20220222161143.6ryghgtfmhnmhpmz@pali> <20220222161539.GA20114@lpieralisi> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20220222161539.GA20114@lpieralisi> User-Agent: NeoMutt/20180716 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220222_081902_037647_34E89424 X-CRM114-Status: GOOD ( 23.42 ) 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 KyBHcmVnb3J5CgpPbiBUdWVzZGF5IDIyIEZlYnJ1YXJ5IDIwMjIgMTY6MTU6MzkgTG9yZW56byBQ aWVyYWxpc2kgd3JvdGU6Cj4gT24gVHVlLCBGZWIgMjIsIDIwMjIgYXQgMDU6MTE6NDNQTSArMDEw MCwgUGFsaSBSb2jDoXIgd3JvdGU6Cj4gPiBPbiBUdWVzZGF5IDIyIEZlYnJ1YXJ5IDIwMjIgMTY6 MDY6MjAgTG9yZW56byBQaWVyYWxpc2kgd3JvdGU6Cj4gPiA+IE9uIFR1ZSwgMjIgRmViIDIwMjIg MTY6NTA6MTggKzAxMDAsIFBhbGkgUm9ow6FyIHdyb3RlOgo+ID4gPiA+IFRoaXMgcGF0Y2ggc2Vy aWVzIGV4dGVuZHMgcGNpLWJyaWRnZS1lbXVsLmMgZHJpdmVyIHRvIGVtdWxhdGUgUENJIFN1YnN5 c3RlbQo+ID4gPiA+IFZlbmRvciBJRCBjYXBhYmlsaXR5IGFuZCBQQ0llIGV4dGVuZGVkIGNhcGFi aWxpdGllcy4gQW5kIHRoZW4gaW1wbGVtZW50Cj4gPiA+ID4gaW4gcGNpLW12ZWJ1LmMgZHJpdmVy IHN1cHBvcnQgZm9yIFBDSSBTdWJzeXN0ZW0gVmVuZG9yIElEcywgUENJZSBBRVIKPiA+ID4gPiBy ZWdpc3RlcnMsIHN1cHBvcnQgZm9yIGxlZ2FjeSBJTlR4IGludGVycnVwdHMsIGNvbmZpZ3VyYXRp b24gZm9yIFgxL1g0Cj4gPiA+ID4gbW9kZSBhbmQgdXNhZ2Ugb2YgbmV3IFBDSSBjaGlsZF9vcHMg QVBJLgo+ID4gPiA+IAo+ID4gPiA+IENoYW5nZXMgaW4gdjQ6Cj4gPiA+ID4gKiByZWJhc2VkIG9u IGMzYmQ3ZGM1NTNlZWE1YTM1OTVjYTNhYTBhZGVlOWJmODM2MjJhMWYKPiA+ID4gPiAKPiA+ID4g PiBbLi4uXQo+ID4gPiAKPiA+ID4gSSBjYW4ndCBhcHBseSBkdHMgY2hhbmdlcywgcGF0Y2ggMTIg c2hvdWxkIGdvIHZpYSB0aGUgYXJtLXNvYyB0cmVlLgo+ID4gCj4gPiBHcmVnb3J5IGFscmVhZHkg d3JvdGUgYWJvdXQgdGhpcyBkdHMgY2hhbmdlOgo+ID4gaHR0cHM6Ly9sb3JlLmtlcm5lbC5vcmcv bGludXgtcGNpLzg3dHVkMWp3cHIuZnNmQEJMLWxhcHRvcC8KPiA+ICJTbyB0aGUgZWFzaWVyIGlz IHRvIGxldCBtZXJnZSBpdCB0aHJvdWdoIHRoZSBQQ0kgc3Vic3lzdGVtIHdpdGggdGhlCj4gPiBv dGhlciBwYXRjaGVzIGZyb20gdGhpcyBzZXJpZXMuIgo+ID4gCj4gPiBBcmUgdGhlcmUgYW55IGlz c3VlcyB3aXRoIGFwcGx5aW5nIHRoaXMgZHRzIGNoYW5nZSB2aWEgcGNpIHRyZWU/Cj4gCj4gSSBk b24ndCB1c3VhbGx5IHRha2UgZHRzIGNoYW5nZXMgdGhyb3VnaCB0aGUgUENJIHRyZWUgc2luY2Ug dGhleQo+IGNhbiBjb25mbGljdCB3aXRoIGFybS1zb2MsIHRoYXQncyB0aGUgaXNzdWUgLSBkdHMg Y2hhbmdlcyBzaG91bGQKPiBiZSBtYW5hZ2VkIGJ5IHBsYXRmb3JtIG1haW50YWluZXJzLgoKR3Jl Z29yeSwgY291bGQgeW91IHBsZWFzZSB0YWtlIHBhdGNoIDEyLzEyPwpodHRwczovL2xvcmUua2Vy bmVsLm9yZy9saW51eC1wY2kvMjAyMjAyMjIxNTUwMzAuOTg4LTEzLXBhbGlAa2VybmVsLm9yZy8K WW91IGhhdmUgYWxyZWFkeSBBY2tlZC1ieSAoc2VlIGFib3ZlIHByZXZpb3VzIGxpbmspCgo+IFRo YW5rcywKPiBMb3JlbnpvCj4gCj4gPiA+IEFwcGxpZWQgdGhlIG90aGVycyB0byBwY2kvbXZlYnUs IHRoYW5rcy4KPiA+ID4gCj4gPiA+IFswMS8xMl0gUENJOiBwY2ktYnJpZGdlLWVtdWw6IFJlLWFy cmFuZ2UgcmVnaXN0ZXIgdGVzdHMKPiA+ID4gICAgICAgICBodHRwczovL2dpdC5rZXJuZWwub3Jn L2xwaWVyYWxpc2kvcGNpL2MvYzQ1M2JmNmY5Ygo+ID4gPiBbMDIvMTJdIFBDSTogcGNpLWJyaWRn ZS1lbXVsOiBBZGQgc3VwcG9ydCBmb3IgUENJZSBleHRlbmRlZCBjYXBhYmlsaXRpZXMKPiA+ID4g ICAgICAgICBodHRwczovL2dpdC5rZXJuZWwub3JnL2xwaWVyYWxpc2kvcGNpL2MvYzBiZDQxOTcz Mgo+ID4gPiBbMDMvMTJdIFBDSTogcGNpLWJyaWRnZS1lbXVsOiBBZGQgc3VwcG9ydCBmb3IgUENJ IEJyaWRnZSBTdWJzeXN0ZW0gVmVuZG9yIElEIGNhcGFiaWxpdHkKPiA+ID4gICAgICAgICBodHRw czovL2dpdC5rZXJuZWwub3JnL2xwaWVyYWxpc2kvcGNpL2MvMzc2N2E5MDI0Mgo+ID4gPiBbMDQv MTJdIGR0LWJpbmRpbmdzOiBQQ0k6IG12ZWJ1OiBBZGQgbnVtLWxhbmVzIHByb3BlcnR5Cj4gPiA+ ICAgICAgICAgaHR0cHM6Ly9naXQua2VybmVsLm9yZy9scGllcmFsaXNpL3BjaS9jLzI2Yjk4MmNh ODMKPiA+ID4gWzA1LzEyXSBQQ0k6IG12ZWJ1OiBDb3JyZWN0bHkgY29uZmlndXJlIHgxL3g0IG1v ZGUKPiA+ID4gICAgICAgICBodHRwczovL2dpdC5rZXJuZWwub3JnL2xwaWVyYWxpc2kvcGNpL2Mv MmE4MWRkOWZkOQo+ID4gPiBbMDYvMTJdIFBDSTogbXZlYnU6IEFkZCBzdXBwb3J0IGZvciBQQ0kg QnJpZGdlIFN1YnN5c3RlbSBWZW5kb3IgSUQgb24gZW11bGF0ZWQgYnJpZGdlCj4gPiA+ICAgICAg ICAgaHR0cHM6Ly9naXQua2VybmVsLm9yZy9scGllcmFsaXNpL3BjaS9jL2UzZTEzYzkxMzUKPiA+ ID4gWzA3LzEyXSBQQ0k6IG12ZWJ1OiBBZGQgc3VwcG9ydCBmb3IgQWR2YW5jZWQgRXJyb3IgUmVw b3J0aW5nIHJlZ2lzdGVycyBvbiBlbXVsYXRlZCBicmlkZ2UKPiA+ID4gICAgICAgICBodHRwczov L2dpdC5rZXJuZWwub3JnL2xwaWVyYWxpc2kvcGNpL2MvMmI2ZWUwNGMwYQo+ID4gPiBbMDgvMTJd IFBDSTogbXZlYnU6IFVzZSBjaGlsZF9vcHMgQVBJCj4gPiA+ICAgICAgICAgaHR0cHM6Ly9naXQu a2VybmVsLm9yZy9scGllcmFsaXNpL3BjaS9jL2MwOTljMmE3NjEKPiA+ID4gWzA5LzEyXSBkdC1i aW5kaW5nczogUENJOiBtdmVidTogVXBkYXRlIGluZm9ybWF0aW9uIGFib3V0IGludHggaW50ZXJy dXB0cwo+ID4gPiAgICAgICAgIGh0dHBzOi8vZ2l0Lmtlcm5lbC5vcmcvbHBpZXJhbGlzaS9wY2kv Yy8wMTI0OTg5MjIwCj4gPiA+IFsxMC8xMl0gUENJOiBtdmVidTogRml4IG1hY3JvIG5hbWVzIGFu ZCBjb21tZW50cyBhYm91dCBsZWdhY3kgaW50ZXJydXB0cwo+ID4gPiAgICAgICAgIGh0dHBzOi8v Z2l0Lmtlcm5lbC5vcmcvbHBpZXJhbGlzaS9wY2kvYy9kMDBlYTk0ZTYyCj4gPiA+IFsxMS8xMl0g UENJOiBtdmVidTogSW1wbGVtZW50IHN1cHBvcnQgZm9yIGxlZ2FjeSBJTlR4IGludGVycnVwdHMK PiA+ID4gICAgICAgICBodHRwczovL2dpdC5rZXJuZWwub3JnL2xwaWVyYWxpc2kvcGNpL2MvZWMw NzUyNjI2NAo+ID4gPiAKPiA+ID4gVGhhbmtzLAo+ID4gPiBMb3JlbnpvCgpfX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpsaW51eC1hcm0ta2VybmVsIG1haWxp bmcgbGlzdApsaW51eC1hcm0ta2VybmVsQGxpc3RzLmluZnJhZGVhZC5vcmcKaHR0cDovL2xpc3Rz LmluZnJhZGVhZC5vcmcvbWFpbG1hbi9saXN0aW5mby9saW51eC1hcm0ta2VybmVsCg==