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 5407BC433EF for ; Tue, 30 Nov 2021 12:13:12 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S240941AbhK3MQ3 (ORCPT ); Tue, 30 Nov 2021 07:16:29 -0500 Received: from foss.arm.com ([217.140.110.172]:36262 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234091AbhK3MQA (ORCPT ); Tue, 30 Nov 2021 07:16:00 -0500 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 397EF1042; Tue, 30 Nov 2021 04:12:41 -0800 (PST) Received: from lpieralisi (e121166-lin.cambridge.arm.com [10.1.196.255]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id DD0393F766; Tue, 30 Nov 2021 04:12:39 -0800 (PST) Date: Tue, 30 Nov 2021 12:12:37 +0000 From: Lorenzo Pieralisi To: Marc Zyngier Cc: Bjorn Helgaas , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-pci@vger.kernel.org, Pali =?iso-8859-1?Q?Roh=E1r?= , Alyssa Rosenzweig , Mark Kettenis , Luca Ceresoli , kernel-team@android.com Subject: Re: [PATCH v3 0/3] PCI: apple: Assorted #PERST fixes Message-ID: <20211130121237.GB3355@lpieralisi> References: <20211123180636.80558-1-maz@kernel.org> <20211130115632.GA3355@lpieralisi> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.9.4 (2018-02-28) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Nov 30, 2021 at 11:59:32AM +0000, Marc Zyngier wrote: > Hi Lorenzo, Bjorn, > > On 2021-11-30 11:56, Lorenzo Pieralisi wrote: > > On Tue, Nov 23, 2021 at 06:06:33PM +0000, Marc Zyngier wrote: > > > Apologies for the rapid fire (I tend to be much more conservative when > > > resending series), but given that this series has a direct impact on > > > other projects (such as u-boot), I'm trying to converge as quickly as > > > possible. > > > > > > This series aims at fixing a number of issues for the recently merged > > > Apple PCIe driver, all revolving around the mishandling of #PERST: > > > > > > - we didn't properly drive #PERST, and we didn't follow the specified > > > timings > > > > > > - the DT had the wrong polarity, which has impacts on the driver > > > itself > > > > > > Hopefully, this should address all the issues reported so far. > > > > > > * From v2: > > > - Fixed DT > > > - Fixed #PERST polarity in the driver > > > - Collected Pali's ack on patch #1 > > > > > > [1] https://lore.kernel.org/r/20211122104156.518063-1-maz@kernel.org > > > > > > Marc Zyngier (3): > > > PCI: apple: Follow the PCIe specifications when resetting the port > > > arm64: dts: apple: t8103: Fix PCIe #PERST polarity > > > PCI: apple: Fix #PERST polarity > > > > > > arch/arm64/boot/dts/apple/t8103.dtsi | 7 ++++--- > > > drivers/pci/controller/pcie-apple.c | 12 +++++++++++- > > > 2 files changed, 15 insertions(+), 4 deletions(-) > > > > Hi Bjorn, > > > > this series is v5.16-rcX material for PCI fixes, can you pick patches > > 1,3 > > up please ? > > Do you mind picking patch #2 as well? Or shall I route it somewhere else? We were told that we should not pick up dts changes, they would normally go via the ARM SOC team, not sure whether the fixes policy is different though but I suspect that's not the case. Thanks, Lorenzo