IB/ocrdma: pr_err() strings should end with newlines
authorArvind Yadav <arvind.yadav.cs@gmail.com>
Tue, 26 Sep 2017 07:29:51 +0000 (12:59 +0530)
committerDoug Ledford <dledford@redhat.com>
Wed, 27 Sep 2017 14:13:14 +0000 (10:13 -0400)
pr_err() messages should end with a new-line to avoid other messages
being concatenated.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
Reviewed-by: Yuval Shaia <yuval.shaia@oracle.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
drivers/infiniband/hw/ocrdma/ocrdma_hw.c
drivers/infiniband/hw/ocrdma/ocrdma_stats.c

index 65b166cc743748fde5f89153dd96ed1950abb762..8e2e9bcf1916f8716400521e7b33b78c61536269 100644 (file)
@@ -1093,7 +1093,7 @@ static int ocrdma_mbx_cmd(struct ocrdma_dev *dev, struct ocrdma_mqe *mqe)
                rsp = &mqe->u.rsp;
 
        if (cqe_status || ext_status) {
-               pr_err("%s() cqe_status=0x%x, ext_status=0x%x,",
+               pr_err("%s() cqe_status=0x%x, ext_status=0x%x,\n",
                       __func__, cqe_status, ext_status);
                if (rsp) {
                        /* This is for embedded cmds. */
index 66056f9a9700ab733b9f6ec24847d36427b9ec26..e528d7acb7f64787c31071ac7f4de8184c70c5a6 100644 (file)
@@ -658,7 +658,7 @@ static ssize_t ocrdma_dbgfs_ops_write(struct file *filp,
                if (reset) {
                        status = ocrdma_mbx_rdma_stats(dev, true);
                        if (status) {
-                               pr_err("Failed to reset stats = %d", status);
+                               pr_err("Failed to reset stats = %d\n", status);
                                goto err;
                        }
                }