On Thu, Oct 27, 2022 at 07:51:23PM +0800, Slark Xiao wrote: > In MHI driver, there are some companies product still > do not have their own PCI vendor macro. So we add it > here to make the code neat. Ref ID could be found in > link https://pcisig.com/membership/member-companies. Rewrap to fill 75 columns. Post along with the patch that uses the definitions. > Signed-off-by: Slark Xiao <slark_xiao@xxxxxxx> > --- > include/linux/pci_ids.h | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h > index b362d90eb9b0..e3bfea06b90e 100644 > --- a/include/linux/pci_ids.h > +++ b/include/linux/pci_ids.h > @@ -172,6 +172,10 @@ > #define PCI_DEVICE_ID_BERKOM_A4T 0xffa4 > #define PCI_DEVICE_ID_BERKOM_SCITEL_QUADRO 0xffa8 > > +#define PCI_VENDOR_ID_CINTERION 0x1269 The URL above says 0x1269 belongs to Thales. Maybe add a comment to that effect, similar to the one for PCI_VENDOR_ID_CB. > +#define PCI_VENDOR_ID_QUECTEL 0x1eac Both new definitions look like they're dropped in a random location. Please move them so the vendor IDs in the file remain sorted by numeric value. > #define PCI_VENDOR_ID_COMPAQ 0x0e11 > #define PCI_DEVICE_ID_COMPAQ_TOKENRING 0x0508 > #define PCI_DEVICE_ID_COMPAQ_TACHYON 0xa0fc > -- > 2.34.1 >