On Thu, 2017-02-23 at 13:40 +0300, Dan Carpenter wrote: > We want to return zero on success or negative error codes. The type > should be int and not u8. > > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > > diff --git a/drivers/infiniband/hw/ocrdma/ocrdma_verbs.c > b/drivers/infiniband/hw/ocrdma/ocrdma_verbs.c > index e06ad7250963..b1b075763eaa 100644 > --- a/drivers/infiniband/hw/ocrdma/ocrdma_verbs.c > +++ b/drivers/infiniband/hw/ocrdma/ocrdma_verbs.c > @@ -371,7 +371,7 @@ static int _ocrdma_pd_mgr_put_bitmap(struct > ocrdma_dev *dev, u16 pd_id, > return 0; > } > > -static u8 ocrdma_put_pd_num(struct ocrdma_dev *dev, u16 pd_id, > +static int ocrdma_put_pd_num(struct ocrdma_dev *dev, u16 pd_id, > bool dpp_pool) > { > int status; Thanks, applied for -rc. -- Doug Ledford <dledford@xxxxxxxxxx> GPG KeyID: B826A3330E572FDD Key fingerprint = AE6B 1BDA 122B 23B4 265B 1274 B826 A333 0E57 2FDD -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html