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 X-Spam-Level: X-Spam-Status: No, score=-2.3 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,USER_AGENT_SANE_2 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id DF22CC43603 for ; Wed, 18 Dec 2019 10:23:30 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id A1876218AC for ; Wed, 18 Dec 2019 10:23:30 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="ZNCqtLoX" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org A1876218AC Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=arm.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Subject: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=Bzc/Ct3PcXXIuGFYn3fEf5rEVqYEdaF2YVc+ky4uR2I=; b=ZNCqtLoXo2U4rN DItsb05oPQtwn/l3RoQ3afgL+gv+NvcjsaGweBBHLSDKqybG4ZU9Tk6+BzhXcxEf/Ms+WTwnd/+0V n3VaRGLS+Mm/u4AULXiulTYo8vIRpzsgbQhIhsZjPQ/mrqMWTKU4WZWaWy1qdWROon80ERlZVInb6 4CHwlMCXG48sMXkjg2I7GZrZuGEmh4JoHXAukBNLl8mB2Ol/HC1jiluG399T2nXOgnoQP2rtzeAFG /CGbex59kvQD3iL2Az1HM2fCNEkXMXAs9jH76PwNnZz4x+1P5Uw3XbWfs+9Up1W4uF4tWHp6QS/T8 Ty4VR16TFpEZ/jrKGQSg==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1ihWUF-0001y0-Sk; Wed, 18 Dec 2019 10:23:23 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1ihWT4-0000xl-TF for linux-arm-kernel@lists.infradead.org; Wed, 18 Dec 2019 10:22:13 +0000 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 8A5FC30E; Wed, 18 Dec 2019 02:22:07 -0800 (PST) Received: from donnerap.cambridge.arm.com (usa-sjc-imap-foss1.foss.arm.com [10.121.207.14]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id F01C93F6CF; Wed, 18 Dec 2019 02:22:05 -0800 (PST) Date: Wed, 18 Dec 2019 10:22:03 +0000 From: Andre Przywara To: Jon Masters Subject: Re: [PATCH] pcie: Add quirk for the Arm Neoverse N1SDP platform Message-ID: <20191218102203.4078b011@donnerap.cambridge.arm.com> In-Reply-To: References: <20191209160638.141431-1-andre.przywara@arm.com> <20191209162645.GA7489@willie-the-truck> Organization: ARM X-Mailer: Claws Mail 3.17.3 (GTK+ 2.24.32; aarch64-unknown-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20191218_022211_013678_C2DE4498 X-CRM114-Status: GOOD ( 18.42 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Lorenzo Pieralisi , Catalin Marinas , "Rafael J . Wysocki" , linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org, linux-pci@vger.kernel.org, Bjorn Helgaas , Andrew Murray , Will Deacon , linux-arm-kernel@lists.infradead.org, Len Brown Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue, 17 Dec 2019 21:21:17 -0500 Jon Masters wrote: Hi Jon, > On 12/9/19 11:26 AM, Will Deacon wrote: > > On Mon, Dec 09, 2019 at 04:06:38PM +0000, Andre Przywara wrote: > >> From: Deepak Pandey > >> > >> The Arm N1SDP SoC suffers from some PCIe integration issues, most > >> prominently config space accesses to not existing BDFs being answered > >> with a bus abort, resulting in an SError. > > > > "Do as I say, not as I do"? > > In my former role I asked nicely that these patches not be posted > upstream, but I see that they ended up being posted anyway. Hacking up > upstream Linux to cover for the fact that a (reference) platform is > non-standard is not only not good form but it actively harms the community. Please keep in mind that this platform was designed to be standards compliant, it is just due to an integration problem that this is not the case with this silicon. So we end up with the usual hardware errata, which the kernel can fix up. I agree it's not nice, and I also want it fixed in hardware, but I guess that's the usual software guy's pipe dream. > You'll have people consume this platform and not realize that it's > broken, IP won't get fixed, and generally it'll be a mess. I don't see how yet another ACPI quirk in the ACPI quirk framework(!) will make a mess. Actually the rest of the system is standards compliant (it even uses ACPI from the very beginning ;-), so it's just this problem that prevents us from using the system in the proper, standards compliant way. Effectively we are back to the embedded times with compiling your own kernel and somehow getting a root filesystem on the hard drive. If there would be mainline kernel support, all of this would go away and would could use standard distro installers (given they backport the patch). > Yes, it's > unfortunate, but so was taping out that platform without working PCI. We > all know what should have happened, and what the right move ahead is. That may come as a surprise to some, but Arm Ltd. is actually not really in the business of *producing silicon*, so a respin of the chip was and is not an option. I too wish it would be different, but that's how it is. Cheers, Andre. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel