Message ID | 20241021153825.2536819-1-yazen.ghannam@amd.com (mailing list archive) |
---|---|
State | Accepted |
Commit | 9f6cb31979739e46b4f863cf13736f8298cbd21e |
Delegated to: | Netdev Maintainers |
Headers | show |
Series | [net-next] net: amd8111e: Remove duplicate definition of PCI_VENDOR_ID_AMD | expand |
On Mon, Oct 21, 2024 at 9:08 PM Yazen Ghannam <yazen.ghannam@amd.com> wrote: > > The AMD PCI vendor ID is already defined in <linux/pci_ids.h>. > > Remove this local definition as it is not needed. > > Signed-off-by: Yazen Ghannam <yazen.ghannam@amd.com> LGTM Reviewed-by: Kalesh AP <kalesh-anakkur.purayil@broadcom.com>
Hello: This patch was applied to netdev/net-next.git (main) by Jakub Kicinski <kuba@kernel.org>: On Mon, 21 Oct 2024 15:38:25 +0000 you wrote: > The AMD PCI vendor ID is already defined in <linux/pci_ids.h>. > > Remove this local definition as it is not needed. > > Signed-off-by: Yazen Ghannam <yazen.ghannam@amd.com> > --- > drivers/net/ethernet/amd/amd8111e.h | 1 - > 1 file changed, 1 deletion(-) Here is the summary with links: - [net-next] net: amd8111e: Remove duplicate definition of PCI_VENDOR_ID_AMD https://git.kernel.org/netdev/net-next/c/9f6cb3197973 You are awesome, thank you!
diff --git a/drivers/net/ethernet/amd/amd8111e.h b/drivers/net/ethernet/amd/amd8111e.h index 305232f5476d..e4ee4c28800c 100644 --- a/drivers/net/ethernet/amd/amd8111e.h +++ b/drivers/net/ethernet/amd/amd8111e.h @@ -550,7 +550,6 @@ typedef enum { /* Driver definitions */ -#define PCI_VENDOR_ID_AMD 0x1022 #define PCI_DEVICE_ID_AMD8111E_7462 0x7462 #define MAX_UNITS 8 /* Maximum number of devices possible */
The AMD PCI vendor ID is already defined in <linux/pci_ids.h>. Remove this local definition as it is not needed. Signed-off-by: Yazen Ghannam <yazen.ghannam@amd.com> --- drivers/net/ethernet/amd/amd8111e.h | 1 - 1 file changed, 1 deletion(-)