On Tue, Jan 02, 2018 at 11:43:27AM +0200, Yuval Shaia wrote:
> On Tue, Dec 26, 2017 at 11:16:29AM +0200, Leon Romanovsky wrote:
> > From: Moni Shoua <monis@xxxxxxxxxxxx>
> >
> > Add a missing command interface to work with a DCT. It includes: creating,
> > destroying and get events for.
> >
> > Signed-off-by: Moni Shoua <monis@xxxxxxxxxxxx>
> > Reviewed-by: Yishai Hadas <yishaih@xxxxxxxxxxxx>
> > Signed-off-by: Leon Romanovsky <leon@xxxxxxxxxx>
> > ---
> > drivers/net/ethernet/mellanox/mlx5/core/eq.c | 9 +-
> > drivers/net/ethernet/mellanox/mlx5/core/qp.c | 128 ++++++++++++++++++++++++---
> > include/linux/mlx5/device.h | 9 ++
> > include/linux/mlx5/driver.h | 8 ++
> > include/linux/mlx5/qp.h | 12 +++
> > 5 files changed, 153 insertions(+), 13 deletions(-)
> >
> > diff --git a/drivers/net/ethernet/mellanox/mlx5/core/eq.c b/drivers/net/ethernet/mellanox/mlx5/core/eq.c
> > index 60771865c99c..7d3d503fa675 100644
> > --- a/drivers/net/ethernet/mellanox/mlx5/core/eq.c
> > +++ b/drivers/net/ethernet/mellanox/mlx5/core/eq.c
> >
> > +static u64 dct_allowed_event_types(void)
> > +{
> > + u64 mask;
> > +
> > + mask = BIT(MLX5_EVENT_TYPE_DCT_DRAINED);
> > +
> > + return mask;
>
> We don't really need this "mask" variable, right?
I removed it in v1 and it won't be in v2 too.
Thanks
>
> > +}
> > +
Attachment:
signature.asc
Description: PGP signature
