net: amd8111e: Remove duplicate definition of PCI_VENDOR_ID_AMD

[PATCH net-next] net: amd8111e: Remove duplicate definition of PCI_VENDOR_ID_AMD

Posted by Yazen Ghannam 1 month ago

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(-)

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 */
-- 
2.43.0

Re: [PATCH net-next] net: amd8111e: Remove duplicate definition of PCI_VENDOR_ID_AMD

Posted by Kalesh Anakkur Purayil 1 month ago

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>

-- 
Regards,
Kalesh A P