Thread (111 messages) 111 messages, 13 authors, 2024-10-19

Re: [PATCH 08/11] misc: rp1: RaspberryPi RP1 misc driver

From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Date: 2024-08-24 05:17:30
Also in: linux-arch, linux-clk, linux-devicetree, linux-gpio, linux-pci, lkml, netdev

On Tue, Aug 20, 2024 at 04:36:10PM +0200, Andrea della Porta wrote:
quoted hunk ↗ jump to hunk
--- a/include/linux/pci_ids.h
+++ b/include/linux/pci_ids.h
@@ -2610,6 +2610,9 @@
 #define PCI_VENDOR_ID_TEKRAM		0x1de1
 #define PCI_DEVICE_ID_TEKRAM_DC290	0xdc29
 
+#define PCI_VENDOR_ID_RPI		0x1de4
+#define PCI_DEVICE_ID_RP1_C0		0x0001
Minor thing, but please read the top of this file.  As you aren't using
these values anywhere outside of this one driver, there's no need to add
these values to pci_ids.h.  Just keep them local to the .c file itself.

thanks,

greg k-h
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help