On Thu, Apr 05, 2018 at 02:16:45PM -0700, Oliver Smith-Denny wrote: > These files contain module load and unload, global driver context, > PCI registration, PCI probe and remove, and definitions of > the fnic2 global context. > > Signed-off-by: Oliver Smith-Denny <[email protected]> > Signed-off-by: Sesidhar Baddela <[email protected]> > Signed-off-by: Anil Chintalapati <[email protected]> > Signed-off-by: Arulprabhu Ponnusamy <[email protected]> > Signed-off-by: Gian Carlo Boffa <[email protected]> > Co-Developed-by: Arulprabhu Ponnusamy <[email protected]> > Co-Developed-by: Gian Carlo Boffa <[email protected]> > Co-Developed-by: Oliver Smith-Denny <[email protected]> > --- > drivers/staging/fnic2/src/fnic2.h | 256 ++++++++++++ > drivers/staging/fnic2/src/fnic2_main.c | 711 > +++++++++++++++++++++++++++++++++ > 2 files changed, 967 insertions(+) > create mode 100644 drivers/staging/fnic2/src/fnic2.h > create mode 100644 drivers/staging/fnic2/src/fnic2_main.c
Why is this a drivers/staging/ driver at all? What is keeping you from getting this merged into the "proper" place in the kernel? If you have a staging driver, you have to have a TODO file in the directory listing what is keeping this in the staging section. Also, one tiny thing to fix up: > --- /dev/null > +++ b/drivers/staging/fnic2/src/fnic2.h > @@ -0,0 +1,256 @@ > +/* > + * SPDX-License-Identifier: GPL-2.0 Please read the documentation on how to properly use SPDX tags on kernel files. This needs to be the first line of the file. thanks, greg k-h
