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 E43F8CDB46E for ; Thu, 12 Oct 2023 15:27:42 +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: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:References: List-Owner; bh=LILVLmiGifHFry0weJx74Vf8LRwe+SzukWqb4cV97+c=; b=tHwI2v8IpWzXz1 t5XtarCxDWcuIrbXPs9fOaMAtvHRWj1funkoZGbTTtuq0EdwT1yuhLYB5xt6+8O4ZgDHbFafl/Itd laXGOiuoX5TTLw33IM8BVlTWaQdRMir6CzRn2G9nnW1JnPMc+/VMYkKX3vIp2QNjbyZUKpl/PN3Oe 1Bvm7csp4ujgU67SOnCI0LA0u85KxpteZMbtdL45tW5Kr5VyxiXQQgfjRyS4FhwweHZRQFornL6np wl8Yh5YbvvlonuZjWq2nYneBFTs3GhWBueP3t6YwPHSlxG7JrovTsFw0n50bGbihOfzxf1x7RUW8Q YDbxRBqtXS4wlch6rsSw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qqxak-001IC5-1Y; Thu, 12 Oct 2023 15:27:14 +0000 Received: from sin.source.kernel.org ([145.40.73.55]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qqxai-001IBQ-00 for linux-arm-kernel@lists.infradead.org; Thu, 12 Oct 2023 15:27:13 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sin.source.kernel.org (Postfix) with ESMTP id 3E2DCCE291E; Thu, 12 Oct 2023 15:27:10 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 11FEDC433C7; Thu, 12 Oct 2023 15:27:07 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1697124428; bh=2H8uGr4Sh1OiavbH2JgHBR1t5d/J1kkWJxFqreXacg0=; h=Date:From:To:Cc:Subject:In-Reply-To:From; b=bhzWOIrpuMZoiq7IdctJDzTQtMCE8yHFiZ3kY8XAPqEyyLH81t/fU18359sKdvOIj uoKgIWPcSAiWMo52UUFt5+2zHdgtd1SPKUbjpL/BhFXQjyzZFtwxaqjmNFao8AaT1H +baVPnJQC/38rsOJZ+7CzRAcGSaR0qiDUUUnFL55Fy9YTBryvjb4b33iQln0RzFddT MHK94ipJ+h2DC0EbZJ3eexQdfoxyN49BeG6fy5PUTeko7tdY/dwac67KiVCgh646ME Ws6pCtpHgaYvqqpfiod96tjTbIGsQXT+XSs3BO26ndwnU/k1U8OpKajzeZDEBKeeSx hRpq/cpqM12GQ== Date: Thu, 12 Oct 2023 10:27:05 -0500 From: Bjorn Helgaas To: Shuai Xue Cc: chengyou@linux.alibaba.com, kaishen@linux.alibaba.com, yangyicong@huawei.com, will@kernel.org, Jonathan.Cameron@huawei.com, baolin.wang@linux.alibaba.com, robin.murphy@arm.com, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-pci@vger.kernel.org, rdunlap@infradead.org, mark.rutland@arm.com, zhuo.song@linux.alibaba.com, renyu.zj@linux.alibaba.com Subject: Re: [PATCH v7 2/4] PCI: Add Alibaba Vendor ID to linux/pci_ids.h Message-ID: <20231012152705.GA1070955@bhelgaas> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20231012145940.GA1069329@bhelgaas> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231012_082712_238387_681AFC3B X-CRM114-Status: GOOD ( 20.04 ) 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 Thu, Oct 12, 2023 at 09:59:40AM -0500, Bjorn Helgaas wrote: > On Thu, Oct 12, 2023 at 11:28:54AM +0800, Shuai Xue wrote: > > The Alibaba Vendor ID (0x1ded) is now used by Alibaba elasticRDMA ("erdma") > > and will be shared with the upcoming PCIe PMU ("dwc_pcie_pmu"). Move the > > Vendor ID to linux/pci_ids.h so that it can shared by several drivers > > later. > > > > Signed-off-by: Shuai Xue > > Acked-by: Bjorn Helgaas # pci_ids.h Hehe, just noticed that I acked this previously: https://lore.kernel.org/r/20230606153143.GA1124867@bhelgaas You can pick up acks like that and include them when you post future versions so people don't have to ack them again. (Drop the acks if you make significant changes to the patch, of course.) > > --- > > drivers/infiniband/hw/erdma/erdma_hw.h | 2 -- > > include/linux/pci_ids.h | 2 ++ > > 2 files changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/infiniband/hw/erdma/erdma_hw.h b/drivers/infiniband/hw/erdma/erdma_hw.h > > index 9d316fdc6f9a..a155519a862f 100644 > > --- a/drivers/infiniband/hw/erdma/erdma_hw.h > > +++ b/drivers/infiniband/hw/erdma/erdma_hw.h > > @@ -11,8 +11,6 @@ > > #include > > > > /* PCIe device related definition. */ > > -#define PCI_VENDOR_ID_ALIBABA 0x1ded > > - > > #define ERDMA_PCI_WIDTH 64 > > #define ERDMA_FUNC_BAR 0 > > #define ERDMA_MISX_BAR 2 > > diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h > > index 5fb3d4c393a9..d8760daf9e5a 100644 > > --- a/include/linux/pci_ids.h > > +++ b/include/linux/pci_ids.h > > @@ -2601,6 +2601,8 @@ > > #define PCI_VENDOR_ID_TEKRAM 0x1de1 > > #define PCI_DEVICE_ID_TEKRAM_DC290 0xdc29 > > > > +#define PCI_VENDOR_ID_ALIBABA 0x1ded > > + > > #define PCI_VENDOR_ID_TEHUTI 0x1fc9 > > #define PCI_DEVICE_ID_TEHUTI_3009 0x3009 > > #define PCI_DEVICE_ID_TEHUTI_3010 0x3010 > > -- > > 2.39.3 > > > > > > _______________________________________________ > > linux-arm-kernel mailing list > > linux-arm-kernel@lists.infradead.org > > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel