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 20EC8C4167B for ; Fri, 1 Dec 2023 15:53:27 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1379497AbjLAPxS (ORCPT ); Fri, 1 Dec 2023 10:53:18 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58216 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1379471AbjLAPxR (ORCPT ); Fri, 1 Dec 2023 10:53:17 -0500 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5F00419E for ; Fri, 1 Dec 2023 07:53:24 -0800 (PST) Received: by smtp.kernel.org (Postfix) with ESMTPSA id CB7D5C433C7; Fri, 1 Dec 2023 15:53:23 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1701446004; bh=Z0b5BVslyfrqgtbZ4YRuZ38WQuFzRRb7gFv+mS/ELXI=; h=Date:From:To:Cc:Subject:In-Reply-To:From; b=Of7zrhHZR3zdxbtwFj8w34us/XZGonkVZ/fs9bneTntO9uhsXiCFrRvxdW/Rg5s03 GOo5jrIiVMBqPXHNhL6acynkOhY5UcwdFkn+DZM+fADd70DPOhi3ptTbRF6rJQ1VMD ge/NiHq+q1zm2REY4ynj2v8//wjdsIZJiXoYwkDKZZfWgmG+eJGfWc937bbhpP73dH 83T0HaWcXr0paSmHw5rv03VsTRcWA3+4me/W1Z132BiR9abcAEoHjP/OAMmCBF+Eqq fyX2H72S63kQvmHc/rIVlpUBkQQXx5UhaGPMHBxqWOVD3gtTU5gCiXb8KLH3LaJYwR a7OXWIciSztsw== Date: Fri, 1 Dec 2023 09:53:20 -0600 From: Bjorn Helgaas To: LeoLiu-oc Cc: bhelgaas@google.com, linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, CobeChen@zhaoxin.com, TonyWWang@zhaoxin.com, YeeLi@zhaoxin.com, Leoliu@zhaoxin.com Subject: Re: [PATCH] PCI: Extend PCI root port device IDs for Zhaoxin platforms Message-ID: <20231201155320.GA518898@bhelgaas> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20231201120942.680075-1-LeoLiu-oc@zhaoxin.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Dec 01, 2023 at 08:09:42PM +0800, LeoLiu-oc wrote: > From: leoliu-oc > > Add more PCI root port device IDs to the > pci_quirk_zhaoxin_pcie_ports_acs() for some new Zhaoxin platforms. Can you please add a note about the plan to deal with this for future devices, e.g., something like "future Zhaoxin devices now in development will advertise an ACS Capability as described in the PCIe spec"? The point of quirks is to work around hardware that is broken or doesn't conform to the spec in some way. We have to add quirks when broken hardware is already in the field, but we should have a plan to fix newer devices so they don't require quirks. > Signed-off-by: leoliu-oc > --- > drivers/pci/quirks.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c > index ea476252280a..db74f8f07096 100644 > --- a/drivers/pci/quirks.c > +++ b/drivers/pci/quirks.c > @@ -4709,7 +4709,7 @@ static int pci_quirk_zhaoxin_pcie_ports_acs(struct pci_dev *dev, u16 acs_flags) > switch (dev->device) { > case 0x0710 ... 0x071e: > case 0x0721: > - case 0x0723 ... 0x0732: > + case 0x0723 ... 0x073b: > return pci_acs_ctrl_enabled(acs_flags, > PCI_ACS_SV | PCI_ACS_RR | PCI_ACS_CR | PCI_ACS_UF); > } > -- > 2.34.1 >