On Fri, Jan 24, 2020 at 4:53 PM Leon Romanovsky <leon@xxxxxxxxxx> wrote: > > On Fri, Jan 24, 2020 at 12:52:39AM -0500, Devesh Sharma wrote: > > Restructuring the bnxt_re_create_qp function. Listing below > > the major changes: > > --Monolithic central part of create_qp where attributes are > > initialized is now enclosed in one function and this new > > function has few more sub-functions. > > --Top level qp limit checking code moved to a function. > > --GSI QP creation and GSI Shadow qp creation code is handled > > in a sub function. > > > > Signed-off-by: Naresh Kumar PBS <nareshkumar.pbs@xxxxxxxxxxxx> > > Signed-off-by: Selvin Xavier <selvin.xavier@xxxxxxxxxxxx> > > Signed-off-by: Devesh Sharma <devesh.sharma@xxxxxxxxxxxx> > > --- > > drivers/infiniband/hw/bnxt_re/bnxt_re.h | 13 +- > > drivers/infiniband/hw/bnxt_re/ib_verbs.c | 635 ++++++++++++++++++++----------- > > drivers/infiniband/hw/bnxt_re/main.c | 3 +- > > 3 files changed, 434 insertions(+), 217 deletions(-) > > > > Please remove dev_err/dev_dbg/dev_* prints from the driver code. Sure I can do that, are you suggesting to add one more patch in this series? I guess it should be okay to follow the hw/efa way to have debug msgs still on. > > > Thanks
