On Thu, 2019-05-16 at 08:16 -0300, Jason Gunthorpe wrote: > On Thu, May 16, 2019 at 01:53:08PM +0300, Kamal Heib wrote: > > For RoCE ports the call for ib_modify_port is not meaningful, so > > simplify the providers of RoCE by return OK in ib_core. > > > > Signed-off-by: Kamal Heib <kamalheib1@xxxxxxxxx> > > --- > > drivers/infiniband/core/device.c | 23 ++++++----- > > drivers/infiniband/hw/hns/hns_roce_main.c | 7 ---- > > drivers/infiniband/hw/mlx4/main.c | 8 ---- > > drivers/infiniband/hw/mlx5/main.c | 6 --- > > drivers/infiniband/hw/ocrdma/ocrdma_main.c | 1 - > > drivers/infiniband/hw/ocrdma/ocrdma_verbs.c | 6 --- > > drivers/infiniband/hw/ocrdma/ocrdma_verbs.h | 2 - > > drivers/infiniband/hw/qedr/main.c | 1 - > > drivers/infiniband/hw/qedr/verbs.c | 6 --- > > drivers/infiniband/hw/qedr/verbs.h | 2 - > > .../infiniband/hw/vmw_pvrdma/pvrdma_main.c | 1 - > > .../infiniband/hw/vmw_pvrdma/pvrdma_verbs.c | 38 ------------- > > ------ > > .../infiniband/hw/vmw_pvrdma/pvrdma_verbs.h | 2 - > > drivers/infiniband/sw/rxe/rxe_verbs.c | 18 --------- > > 14 files changed, 14 insertions(+), 107 deletions(-) > > We have more roce only drivers than this, why isn't everything > changed? > > Jason Not all of them implements modify_port(). Thanks, Kamal
