Subject: Re: [PATCH net-next 3/6] octeontx2-vf: Virtual function driver dupport

From: Leon Romanovsky <[email-masked]>
Date: Wed, 11 Mar 2020 12:16:46 +0200

On Wed, Mar 11, 2020 at 12:44:00PM +0530, Sunil Kovvuri wrote:
> On Wed, Mar 11, 2020 at 3:13 AM Jakub Kicinski <[email-masked]> wrote:
> >
> > On Wed, 11 Mar 2020 00:17:22 +0530 [email-masked] wrote:
> > > +#define DRV_NAME     "octeontx2-nicvf"
> > > +#define DRV_STRING   "Marvell OcteonTX2 NIC Virtual Function Driver"
> > > +#define DRV_VERSION  "1.0"
> >
> > Please drop the driver version, kernel version should be used upstream.
> >
>
> Okay, will do.
>
> > > +
> > > +static const struct pci_device_id otx2_vf_id_table[] = {
> > > +     { PCI_DEVICE(PCI_VENDOR_ID_CAVIUM, PCI_DEVID_OCTEONTX2_RVU_AFVF) },
> > > +     { PCI_DEVICE(PCI_VENDOR_ID_CAVIUM, PCI_DEVID_OCTEONTX2_RVU_VF) },
> > > +     { }
> > > +};
> > > +
> > > +MODULE_AUTHOR("Marvell International Ltd.");
> >
> > Only people can be authors, please put your name here or remove this.
> >
>
> Just for my understanding, is this due to a decision taken in netdev recently ?
> I have searched through all drivers in netdev and there is a mix of
> organizations and individuals as AUTHORS.
> Here we used org name to avoid specifying multiple names.

It was always the case in all subsystems. The authorship belongs to the developer
and not to the company. In opposite to the copyright.

Thanks