On Mon, Jan 09, 2023 at 10:04:53AM +0200, Alvaro Karsz wrote: > Add SolidRun vendor ID to pci_ids.h > > The vendor ID is used in 2 different source files, > the SNET vDPA driver and PCI quirks. Nits: wrap to fill 75 columns. > Signed-off-by: Alvaro Karsz <alvaro.karsz@xxxxxxxxxxxxx> > Acked-by: Bjorn Helgaas <bhelgaas@xxxxxxxxxx> > --- > include/linux/pci_ids.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h > index b362d90eb9b..9a3102e61db 100644 > --- a/include/linux/pci_ids.h > +++ b/include/linux/pci_ids.h > @@ -3092,6 +3092,8 @@ > > #define PCI_VENDOR_ID_3COM_2 0xa727 > > +#define PCI_VENDOR_ID_SOLIDRUN 0xd063 This should be indented with tabs instead of spaces so it matches the rest of the file. It's helpful if you can send the patches in a series so the individual patches are replies to the cover letter. That makes tools and archives work better: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/5.Posting.rst?id=v6.1#n342