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 1D5B7C433EF for ; Tue, 26 Jul 2022 11:41:27 +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=Vjez+BVo23VsLkGGepXcN7aRg3XbvCvG5laIf3D+sH8=; b=ZIN3AvAJGwqN6m qaZZuuOqTwnw7NgsmpIlTAKo2e6DZ0JC61Mm1A+wWo+bF2kmnIcN/kEq0XEYzOVkmiGbKjYLZrWpo 3OxGAVdzFUoAV+nf6aKQnunu6xgUOUHUVmQYxZf6CQait8J4sWWOiwwE13lcZo0zWBIf3+RI3f2M+ 7++26r/V1HLmmmjvUOJUQpiNMPslT94gkmT7lmhogdK7ZGzNbuiHqE41KGYAgQg2lrkmuI4rQ18/8 4ErJMYSs7RvbtJjHesZAu2kjr1Qa+45FH/+WsD9bvkR59chE3cjwjNmexN4t19ehGuv3VJtWmhVrZ ywr4ZruQs0GfCMHoycEg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oGIvH-00GALI-E2; Tue, 26 Jul 2022 11:40:24 +0000 Received: from ams.source.kernel.org ([2604:1380:4601:e00::1]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oGIv6-00GA0l-Jo for linux-arm-kernel@lists.infradead.org; Tue, 26 Jul 2022 11:40:14 +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 B4876B81604; Tue, 26 Jul 2022 11:40:08 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 92010C341C8; Tue, 26 Jul 2022 11:40:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1658835607; bh=yL/84tqi9gVAqhM4QwwAMzi32hAb0stsqpRwvMm7GAM=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=ZHyDADn38pBiojVQRSYLMdqpCW60v82r3p9EwMoz2DEqz0PNDMiW8VJv7/LLzDCxD 7rEZhIWSy7QmYv6uVvObGFYHEyO0a44+J4SdqzqZaKQieEQNIyZerarBTf2qvoG3ej bJxHTjhpPU4gYt45/f27KlV6n0MYAOIqyGqjTze0HL89rvJ46LhQOYHw28ZL6O6ceu 1g6r5n4Dc37NsgF4odx6rOLuWSwtK4ONpTgSllXMil4Al7m9XuvP24d42nqKk+OhbH t0F2wMXPIjV9cJGyHfu0SJx4K3vMZT+kcLduE5LK327DDLnffIqAq3ou+PY1Wuh3kj UPKJefehBh9xA== Date: Tue, 26 Jul 2022 12:40:00 +0100 From: Will Deacon To: Jeremy Linton Cc: linux-pci@vger.kernel.org, bhelgaas@google.com, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, lpieralisi@kernel.org, kw@linux.com, mark.rutland@arm.com, sudeep.holla@arm.com, boqun.feng@gmail.com, catalin.marinas@arm.com, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, maz@kernel.org, jonmasters@google.com Subject: Re: [PATCH 0/4] PCI SMC conduit, now with DT support Message-ID: <20220726114000.GA21450@willie-the-truck> References: <20220725163905.2024437-1-jeremy.linton@arm.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20220725163905.2024437-1-jeremy.linton@arm.com> User-Agent: Mutt/1.10.1 (2018-07-13) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220726_044012_874007_F93E72AE X-CRM114-Status: GOOD ( 16.79 ) 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="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, Jul 25, 2022 at 11:39:01AM -0500, Jeremy Linton wrote: > This is a rebase of the later revisions of [1], but refactored > slightly to add a DT method as well. It has all the same advantages of > the ACPI method (putting HW quirks in the firmware rather than the > kernel) but now applied to a 'pci-host-smc-generic' compatible > property which extends the pci-host-generic logic to handle cases > where the PCI Config region isn't ECAM compliant. With this in place, > and firmware managed clock/phy/etc its possible to run the generic > driver on hardware that isn't what one would consider standards > compliant PCI root ports. I still think that hiding the code in firmware because the hardware is broken is absolutely the wrong way to tackle this problem and I thought the general idea from last time was that we were going to teach Linux about the broken hardware instead [1]. I'd rather have the junk where we can see it, reason about it and modify it. What's changed? In my mind, the main thing that's happened since we last discussed this is that Apple shipped arm64 client hardware with working ECAM. *Apple* for goodness sake: a company with basically no incentive to follow standards for their vertically integrated devices! Perhaps others need to raise their game instead of wasting everybody's time on firmware hacks; getting the hardware right obviously isn't as difficult as folks would lead us to believe. Will [1] https://lore.kernel.org/r/20210325131231.GA18590@e121166-lin.cambridge.arm.com _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel