| Message ID | 20231012032856.2640-3-xueshuai@linux.alibaba.com (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Bjorn Helgaas |
| Headers | show |
| Series | drivers/perf: add Synopsys DesignWare PCIe PMU driver support | expand |
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 <xueshuai@linux.alibaba.com> Acked-by: Bjorn Helgaas <bhelgaas@google.com> # pci_ids.h > --- > 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 <linux/types.h> > > /* 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
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 <xueshuai@linux.alibaba.com> > > Acked-by: Bjorn Helgaas <bhelgaas@google.com> # 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 <linux/types.h> > > > > /* 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
On 2023/10/12 23:27, Bjorn Helgaas wrote: > 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 <xueshuai@linux.alibaba.com> >> >> Acked-by: Bjorn Helgaas <bhelgaas@google.com> # 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.) > Sorry, I forgot to pick up your acked-by tag. Will add it next version. Thank you. Cheers, Shuai
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 <linux/types.h> /* 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
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 <xueshuai@linux.alibaba.com> --- drivers/infiniband/hw/erdma/erdma_hw.h | 2 -- include/linux/pci_ids.h | 2 ++ 2 files changed, 2 insertions(+), 2 deletions(-)