On Sat, Jan 28, 2017 at 05:13:32PM -0500, Devesh Sharma wrote: > add_subdirectory(providers/ocrdma) > add_subdirectory(providers/qedr) > add_subdirectory(providers/vmw_pvrdma) > +add_subdirectory(providers/bnxtre) > endif() Keep list sorted > > add_subdirectory(providers/hfi1verbs) > diff --git a/MAINTAINERS b/MAINTAINERS > index 2ae504c..f52da1c 100644 > +++ b/MAINTAINERS > @@ -166,3 +166,8 @@ M: Adit Ranadive <aditr@xxxxxxxxxx> > L: pv-drivers@xxxxxxxxxx > S: Supported > F: providers/vmw_pvrdma/ > + > +BNXTRE USERSPACE PROVIDER (for bnxtre.ko) > +M: Devesh Sharma <Devesh.sharma@xxxxxxxxxxxx> > +S: Supported > +F: providers/bnxtre Keep list sorted > diff --git a/providers/bnxtre/bnxtre.driver b/providers/bnxtre/bnxtre.driver > new file mode 100644 > index 0000000..5ce796f > +++ b/providers/bnxtre/bnxtre.driver > @@ -0,0 +1 @@ > +driver bnxtre No, delete this file > +#if HAVE_CONFIG_H > +#include <config.h> > +#endif /* HAVE_CONFIG_H */ HAVE_CONFIG_H is not defined in our builds, do not use config.h if we don't need it. Ditto everywhere > +struct { > + unsigned int vendor; > + unsigned int device; > +} cna_table[] = { static const? Jason -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html