NTB: switchtec_ntb: Add new line on appropriate printks

Trivial addition of "\n" to the dev_* prints where necessary

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Jon Mason <jdmason@kudzu.us>
This commit is contained in:
Jon Mason 2017-11-27 11:03:57 -05:00
parent c5ec8b451a
commit 2dd0f6a64a

View file

@ -172,7 +172,7 @@ static int switchtec_ntb_part_op(struct switchtec_ntb *sndev,
if (ps == status) { if (ps == status) {
dev_err(&sndev->stdev->dev, dev_err(&sndev->stdev->dev,
"Timed out while performing %s (%d). (%08x)", "Timed out while performing %s (%d). (%08x)\n",
op_text[op], op, op_text[op], op,
ioread32(&ctl->partition_status)); ioread32(&ctl->partition_status));
@ -306,7 +306,7 @@ static int switchtec_ntb_mw_set_trans(struct ntb_dev *ntb, int pidx, int widx,
if (pidx != NTB_DEF_PEER_IDX) if (pidx != NTB_DEF_PEER_IDX)
return -EINVAL; return -EINVAL;
dev_dbg(&sndev->stdev->dev, "MW %d: part %d addr %pad size %pap", dev_dbg(&sndev->stdev->dev, "MW %d: part %d addr %pad size %pap\n",
widx, pidx, &addr, &size); widx, pidx, &addr, &size);
if (widx >= switchtec_ntb_mw_count(ntb, pidx)) if (widx >= switchtec_ntb_mw_count(ntb, pidx))
@ -337,7 +337,7 @@ static int switchtec_ntb_mw_set_trans(struct ntb_dev *ntb, int pidx, int widx,
if (rc == -EIO) { if (rc == -EIO) {
dev_err(&sndev->stdev->dev, dev_err(&sndev->stdev->dev,
"Hardware reported an error configuring mw %d: %08x", "Hardware reported an error configuring mw %d: %08x\n",
widx, ioread32(&ctl->bar_error)); widx, ioread32(&ctl->bar_error));
if (widx < nr_direct_mw) if (widx < nr_direct_mw)
@ -491,7 +491,7 @@ static void switchtec_ntb_check_link(struct switchtec_ntb *sndev)
if (link_sta != old) { if (link_sta != old) {
switchtec_ntb_send_msg(sndev, LINK_MESSAGE, MSG_CHECK_LINK); switchtec_ntb_send_msg(sndev, LINK_MESSAGE, MSG_CHECK_LINK);
ntb_link_event(&sndev->ntb); ntb_link_event(&sndev->ntb);
dev_info(&sndev->stdev->dev, "ntb link %s", dev_info(&sndev->stdev->dev, "ntb link %s\n",
link_sta ? "up" : "down"); link_sta ? "up" : "down");
} }
} }
@ -523,7 +523,7 @@ static int switchtec_ntb_link_enable(struct ntb_dev *ntb,
{ {
struct switchtec_ntb *sndev = ntb_sndev(ntb); struct switchtec_ntb *sndev = ntb_sndev(ntb);
dev_dbg(&sndev->stdev->dev, "enabling link"); dev_dbg(&sndev->stdev->dev, "enabling link\n");
sndev->self_shared->link_sta = 1; sndev->self_shared->link_sta = 1;
switchtec_ntb_send_msg(sndev, LINK_MESSAGE, MSG_LINK_UP); switchtec_ntb_send_msg(sndev, LINK_MESSAGE, MSG_LINK_UP);
@ -537,7 +537,7 @@ static int switchtec_ntb_link_disable(struct ntb_dev *ntb)
{ {
struct switchtec_ntb *sndev = ntb_sndev(ntb); struct switchtec_ntb *sndev = ntb_sndev(ntb);
dev_dbg(&sndev->stdev->dev, "disabling link"); dev_dbg(&sndev->stdev->dev, "disabling link\n");
sndev->self_shared->link_sta = 0; sndev->self_shared->link_sta = 0;
switchtec_ntb_send_msg(sndev, LINK_MESSAGE, MSG_LINK_UP); switchtec_ntb_send_msg(sndev, LINK_MESSAGE, MSG_LINK_UP);
@ -792,7 +792,7 @@ static void switchtec_ntb_init_sndev(struct switchtec_ntb *sndev)
part_map &= ~(1 << sndev->self_partition); part_map &= ~(1 << sndev->self_partition);
sndev->peer_partition = ffs(part_map) - 1; sndev->peer_partition = ffs(part_map) - 1;
dev_dbg(&sndev->stdev->dev, "Partition ID %d of %d (%llx)", dev_dbg(&sndev->stdev->dev, "Partition ID %d of %d (%llx)\n",
sndev->self_partition, sndev->stdev->partition_count, sndev->self_partition, sndev->stdev->partition_count,
part_map); part_map);
@ -829,7 +829,7 @@ static void switchtec_ntb_init_mw(struct switchtec_ntb *sndev)
sndev->nr_lut_mw = ioread16(&sndev->mmio_self_ctrl->lut_table_entries); sndev->nr_lut_mw = ioread16(&sndev->mmio_self_ctrl->lut_table_entries);
sndev->nr_lut_mw = rounddown_pow_of_two(sndev->nr_lut_mw); sndev->nr_lut_mw = rounddown_pow_of_two(sndev->nr_lut_mw);
dev_dbg(&sndev->stdev->dev, "MWs: %d direct, %d lut", dev_dbg(&sndev->stdev->dev, "MWs: %d direct, %d lut\n",
sndev->nr_direct_mw, sndev->nr_lut_mw); sndev->nr_direct_mw, sndev->nr_lut_mw);
sndev->peer_nr_direct_mw = map_bars(sndev->peer_direct_mw_to_bar, sndev->peer_nr_direct_mw = map_bars(sndev->peer_direct_mw_to_bar,
@ -839,7 +839,7 @@ static void switchtec_ntb_init_mw(struct switchtec_ntb *sndev)
ioread16(&sndev->mmio_peer_ctrl->lut_table_entries); ioread16(&sndev->mmio_peer_ctrl->lut_table_entries);
sndev->peer_nr_lut_mw = rounddown_pow_of_two(sndev->peer_nr_lut_mw); sndev->peer_nr_lut_mw = rounddown_pow_of_two(sndev->peer_nr_lut_mw);
dev_dbg(&sndev->stdev->dev, "Peer MWs: %d direct, %d lut", dev_dbg(&sndev->stdev->dev, "Peer MWs: %d direct, %d lut\n",
sndev->peer_nr_direct_mw, sndev->peer_nr_lut_mw); sndev->peer_nr_direct_mw, sndev->peer_nr_lut_mw);
} }
@ -897,7 +897,7 @@ static int switchtec_ntb_init_req_id_table(struct switchtec_ntb *sndev)
if (ioread32(&sndev->mmio_self_ctrl->req_id_table_size) < 2) { if (ioread32(&sndev->mmio_self_ctrl->req_id_table_size) < 2) {
dev_err(&sndev->stdev->dev, dev_err(&sndev->stdev->dev,
"Not enough requester IDs available."); "Not enough requester IDs available\n");
return -EFAULT; return -EFAULT;
} }
@ -928,7 +928,7 @@ static int switchtec_ntb_init_req_id_table(struct switchtec_ntb *sndev)
if (rc == -EIO) { if (rc == -EIO) {
error = ioread32(&sndev->mmio_self_ctrl->req_id_error); error = ioread32(&sndev->mmio_self_ctrl->req_id_error);
dev_err(&sndev->stdev->dev, dev_err(&sndev->stdev->dev,
"Error setting up the requester ID table: %08x", "Error setting up the requester ID table: %08x\n",
error); error);
} }
@ -975,7 +975,7 @@ static int switchtec_ntb_init_shared_mw(struct switchtec_ntb *sndev)
GFP_KERNEL); GFP_KERNEL);
if (!sndev->self_shared) { if (!sndev->self_shared) {
dev_err(&sndev->stdev->dev, dev_err(&sndev->stdev->dev,
"unable to allocate memory for shared mw"); "unable to allocate memory for shared mw\n");
return -ENOMEM; return -ENOMEM;
} }
@ -1005,7 +1005,7 @@ static int switchtec_ntb_init_shared_mw(struct switchtec_ntb *sndev)
bar_error = ioread32(&ctl->bar_error); bar_error = ioread32(&ctl->bar_error);
lut_error = ioread32(&ctl->lut_error); lut_error = ioread32(&ctl->lut_error);
dev_err(&sndev->stdev->dev, dev_err(&sndev->stdev->dev,
"Error setting up shared MW: %08x / %08x", "Error setting up shared MW: %08x / %08x\n",
bar_error, lut_error); bar_error, lut_error);
goto unalloc_and_exit; goto unalloc_and_exit;
} }
@ -1016,7 +1016,7 @@ static int switchtec_ntb_init_shared_mw(struct switchtec_ntb *sndev)
goto unalloc_and_exit; goto unalloc_and_exit;
} }
dev_dbg(&sndev->stdev->dev, "Shared MW Ready"); dev_dbg(&sndev->stdev->dev, "Shared MW Ready\n");
return 0; return 0;
unalloc_and_exit: unalloc_and_exit:
@ -1057,8 +1057,8 @@ static irqreturn_t switchtec_ntb_message_isr(int irq, void *dev)
u64 msg = ioread64(&sndev->mmio_self_dbmsg->imsg[i]); u64 msg = ioread64(&sndev->mmio_self_dbmsg->imsg[i]);
if (msg & NTB_DBMSG_IMSG_STATUS) { if (msg & NTB_DBMSG_IMSG_STATUS) {
dev_dbg(&sndev->stdev->dev, "message: %d %08x\n", i, dev_dbg(&sndev->stdev->dev, "message: %d %08x\n",
(u32)msg); i, (u32)msg);
iowrite8(1, &sndev->mmio_self_dbmsg->imsg[i].status); iowrite8(1, &sndev->mmio_self_dbmsg->imsg[i].status);
if (i == LINK_MESSAGE) if (i == LINK_MESSAGE)
@ -1086,7 +1086,7 @@ static int switchtec_ntb_init_db_msg_irq(struct switchtec_ntb *sndev)
message_irq == event_irq) message_irq == event_irq)
message_irq++; message_irq++;
dev_dbg(&sndev->stdev->dev, "irqs - event: %d, db: %d, msgs: %d", dev_dbg(&sndev->stdev->dev, "irqs - event: %d, db: %d, msgs: %d\n",
event_irq, doorbell_irq, message_irq); event_irq, doorbell_irq, message_irq);
for (i = 0; i < idb_vecs - 4; i++) for (i = 0; i < idb_vecs - 4; i++)
@ -1136,7 +1136,7 @@ static int switchtec_ntb_add(struct device *dev,
return -ENODEV; return -ENODEV;
if (stdev->partition_count != 2) if (stdev->partition_count != 2)
dev_warn(dev, "ntb driver only supports 2 partitions"); dev_warn(dev, "ntb driver only supports 2 partitions\n");
sndev = kzalloc_node(sizeof(*sndev), GFP_KERNEL, dev_to_node(dev)); sndev = kzalloc_node(sizeof(*sndev), GFP_KERNEL, dev_to_node(dev));
if (!sndev) if (!sndev)
@ -1166,7 +1166,7 @@ static int switchtec_ntb_add(struct device *dev,
stdev->sndev = sndev; stdev->sndev = sndev;
stdev->link_notifier = switchtec_ntb_link_notification; stdev->link_notifier = switchtec_ntb_link_notification;
dev_info(dev, "NTB device registered"); dev_info(dev, "NTB device registered\n");
return 0; return 0;
@ -1176,7 +1176,7 @@ static int switchtec_ntb_add(struct device *dev,
switchtec_ntb_deinit_shared_mw(sndev); switchtec_ntb_deinit_shared_mw(sndev);
free_and_exit: free_and_exit:
kfree(sndev); kfree(sndev);
dev_err(dev, "failed to register ntb device: %d", rc); dev_err(dev, "failed to register ntb device: %d\n", rc);
return rc; return rc;
} }
@ -1195,7 +1195,7 @@ void switchtec_ntb_remove(struct device *dev,
switchtec_ntb_deinit_db_msg_irq(sndev); switchtec_ntb_deinit_db_msg_irq(sndev);
switchtec_ntb_deinit_shared_mw(sndev); switchtec_ntb_deinit_shared_mw(sndev);
kfree(sndev); kfree(sndev);
dev_info(dev, "ntb device unregistered"); dev_info(dev, "ntb device unregistered\n");
} }
static struct class_interface switchtec_interface = { static struct class_interface switchtec_interface = {