| Message ID | 20170109201216.GA3055@mwanda (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
On Mon, Jan 09, 2017 at 11:12:16PM +0300, Dan Carpenter wrote: > The debug printk was indented more than it should have been and we > can remove an unnecessary line break. > > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> > --- > v2: I accidentally had a tab instead of a space. Thank you for the patch Dan. Acked-by: Shiraz Saleem <shiraz.saleem@intel.com> -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Mon, Jan 09, 2017 at 11:12:16PM +0300, Dan Carpenter wrote: > The debug printk was indented more than it should have been and we > can remove an unnecessary line break. > > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> > --- > v2: I accidentally had a tab instead of a space. Also for this one: Reviewed-by: Yuval Shaia <yuval.shaia@oracle.com> > > diff --git a/drivers/infiniband/hw/i40iw/i40iw_ctrl.c b/drivers/infiniband/hw/i40iw/i40iw_ctrl.c > index 98923a8cf86d..b9dc2c540946 100644 > --- a/drivers/infiniband/hw/i40iw/i40iw_ctrl.c > +++ b/drivers/infiniband/hw/i40iw/i40iw_ctrl.c > @@ -4498,9 +4498,9 @@ void i40iw_sc_vsi_init(struct i40iw_sc_vsi *vsi, struct i40iw_vsi_init_info *inf > i40iw_fill_qos_list(info->params->qs_handle_list); > > for (i = 0; i < I40IW_MAX_USER_PRIORITY; i++) { > - vsi->qos[i].qs_handle = > - info->params->qs_handle_list[i]; > - i40iw_debug(vsi->dev, I40IW_DEBUG_DCB, "qset[%d]: %d\n", i, vsi->qos[i].qs_handle); > + vsi->qos[i].qs_handle = info->params->qs_handle_list[i]; > + i40iw_debug(vsi->dev, I40IW_DEBUG_DCB, "qset[%d]: %d\n", i, > + vsi->qos[i].qs_handle); > spin_lock_init(&vsi->qos[i].lock); > INIT_LIST_HEAD(&vsi->qos[i].qplist); > } > -- > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Mon, 2017-01-09 at 14:31 -0600, Shiraz Saleem wrote: > On Mon, Jan 09, 2017 at 11:12:16PM +0300, Dan Carpenter wrote: > > > > The debug printk was indented more than it should have been and we > > can remove an unnecessary line break. > > > > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> > > --- > > v2: I accidentally had a tab instead of a space. > > Thank you for the patch Dan. > > Acked-by: Shiraz Saleem <shiraz.saleem@intel.com> Applied v2 of the patch, thanks.
diff --git a/drivers/infiniband/hw/i40iw/i40iw_ctrl.c b/drivers/infiniband/hw/i40iw/i40iw_ctrl.c index 98923a8cf86d..b9dc2c540946 100644 --- a/drivers/infiniband/hw/i40iw/i40iw_ctrl.c +++ b/drivers/infiniband/hw/i40iw/i40iw_ctrl.c @@ -4498,9 +4498,9 @@ void i40iw_sc_vsi_init(struct i40iw_sc_vsi *vsi, struct i40iw_vsi_init_info *inf i40iw_fill_qos_list(info->params->qs_handle_list); for (i = 0; i < I40IW_MAX_USER_PRIORITY; i++) { - vsi->qos[i].qs_handle = - info->params->qs_handle_list[i]; - i40iw_debug(vsi->dev, I40IW_DEBUG_DCB, "qset[%d]: %d\n", i, vsi->qos[i].qs_handle); + vsi->qos[i].qs_handle = info->params->qs_handle_list[i]; + i40iw_debug(vsi->dev, I40IW_DEBUG_DCB, "qset[%d]: %d\n", i, + vsi->qos[i].qs_handle); spin_lock_init(&vsi->qos[i].lock); INIT_LIST_HEAD(&vsi->qos[i].qplist); }
The debug printk was indented more than it should have been and we can remove an unnecessary line break. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> --- v2: I accidentally had a tab instead of a space. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html