On Mon, Dec 10, 2018 at 11:40:27AM -0700, Jason Gunthorpe wrote: > On Mon, Dec 10, 2018 at 08:25:23PM +0200, Kamal Heib wrote: > > On Mon, Dec 10, 2018 at 11:01:44AM -0700, Jason Gunthorpe wrote: > > > On Mon, Dec 10, 2018 at 04:32:21PM +0200, Kamal Heib wrote: > > > > Initialize ib_device_ops with the supported operations using > > > > ib_set_device_ops(). > > > > > > > > Signed-off-by: Kamal Heib <kamalheib1@xxxxxxxxx> > > > > drivers/infiniband/hw/bnxt_re/main.c | 96 +++++++++++++--------------- > > > > 1 file changed, 45 insertions(+), 51 deletions(-) > > > > > > I'm sure 0-day wil eventually get here, but this doesn't compile. > > > > > > Every patch must compile. > > > > > > Perhaps a patch is missing? > > > > > > Jason > > > > Yes, Sorry about that, I was trying to avoid including the commit [1], > > which was already including in for-rc and this patch set depend on it, > > but by mistake I dropped also the first patch... :/ > > > > Do you think that I should include commit [1] anyway and when merging > > the for-next with for-rc you will drop it? > > You should merge rc and next in your own tree and use the merge as the > baseline for submission, noting this in the cover letter > > Jason I'll fix that soon. Thanks, Kamal