@@ -75,7 +75,7 @@ void usnic_ib_log_vf(struct usnic_ib_vf *vf)
{
char buf[1000];
usnic_ib_dump_vf(vf, buf, sizeof(buf));
- usnic_dbg(buf);
+ usnic_dbg("%s\n", buf);
}
/* Start of netdev section */
@@ -372,8 +372,7 @@ static void usnic_ib_undiscover_pf(struct kref *kref)
}
}
- if (!found)
- WARN("Failed to remove PF %s\n", pci_name(dev));
+ WARN(!found, "Failed to remove PF %s\n", pci_name(dev));
mutex_unlock(&usnic_ib_ibdev_list_lock);
}
@@ -432,7 +432,7 @@ static void log_spec(const struct usnic_vnic_res_spec
{
char buf[512];
usnic_vnic_spec_dump(buf, sizeof(buf), res_spec);
- usnic_dbg(buf);
+ usnic_dbg("%s\n", buf);
}
struct usnic_ib_qp_grp *usnic_ib_qp_grp_create(struct usnic_fwd_dev *ufdev,
@@ -182,8 +182,6 @@ int usnic_vnic_spec_dump(char *buf, int buf_sz, const struct usnic_vnic_res_spec
res_cnt);
}
- offset += scnprintf(buf + offset, buf_sz - offset,
- "\n");
return offset;
}
usNIC calls printk w/o string literal in some instances. Change these instances so the call is made with string literal. Signed-off-by: Upinder Malhi <umalhi@cisco.com> --- drivers/infiniband/hw/usnic/usnic_ib_main.c | 5 ++--- drivers/infiniband/hw/usnic/usnic_ib_qp_grp.c | 2 +- drivers/infiniband/hw/usnic/usnic_vnic.c | 2 -- 3 files changed, 3 insertions(+), 6 deletions(-)