Commit 86822310 authored by Steven Davis's avatar Steven Davis Committed by Greg Kroah-Hartman

staging: nvec: Capitalize outputs to match the rest of the driver

Noticing that some messages were capitalized and some weren't, I
capitalized them to match the rest. This makes the messages relatively
easier to understand for an end user, and reduces confusion about
capitalization. A comment was also capitalized.
Signed-off-by: default avatarSteven Davis <goldside000@outlook.com>
Link: https://lore.kernel.org/r/SJ2P223MB102626B10E837EF5A93ED1F1F7B02@SJ2P223MB1026.NAMP223.PROD.OUTLOOK.COMSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 13dda5c5
...@@ -175,7 +175,7 @@ static struct nvec_msg *nvec_msg_alloc(struct nvec_chip *nvec, ...@@ -175,7 +175,7 @@ static struct nvec_msg *nvec_msg_alloc(struct nvec_chip *nvec,
} }
} }
dev_err(nvec->dev, "could not allocate %s buffer\n", dev_err(nvec->dev, "Could not allocate %s buffer\n",
(category == NVEC_MSG_TX) ? "TX" : "RX"); (category == NVEC_MSG_TX) ? "TX" : "RX");
return NULL; return NULL;
...@@ -315,7 +315,7 @@ int nvec_write_sync(struct nvec_chip *nvec, ...@@ -315,7 +315,7 @@ int nvec_write_sync(struct nvec_chip *nvec,
if (!(wait_for_completion_timeout(&nvec->sync_write, if (!(wait_for_completion_timeout(&nvec->sync_write,
msecs_to_jiffies(2000)))) { msecs_to_jiffies(2000)))) {
dev_warn(nvec->dev, dev_warn(nvec->dev,
"timeout waiting for sync write to complete\n"); "Timeout waiting for sync write to complete\n");
mutex_unlock(&nvec->sync_write_mutex); mutex_unlock(&nvec->sync_write_mutex);
return -ETIMEDOUT; return -ETIMEDOUT;
} }
...@@ -392,7 +392,7 @@ static void nvec_request_master(struct work_struct *work) ...@@ -392,7 +392,7 @@ static void nvec_request_master(struct work_struct *work)
msecs_to_jiffies(5000)); msecs_to_jiffies(5000));
if (err == 0) { if (err == 0) {
dev_warn(nvec->dev, "timeout waiting for ec transfer\n"); dev_warn(nvec->dev, "Timeout waiting for ec transfer\n");
nvec_gpio_set_value(nvec, 1); nvec_gpio_set_value(nvec, 1);
msg->pos = 0; msg->pos = 0;
} }
...@@ -454,7 +454,7 @@ static void nvec_dispatch(struct work_struct *work) ...@@ -454,7 +454,7 @@ static void nvec_dispatch(struct work_struct *work)
if (nvec->sync_write_pending == if (nvec->sync_write_pending ==
(msg->data[2] << 8) + msg->data[0]) { (msg->data[2] << 8) + msg->data[0]) {
dev_dbg(nvec->dev, "sync write completed!\n"); dev_dbg(nvec->dev, "Sync write completed!\n");
nvec->sync_write_pending = 0; nvec->sync_write_pending = 0;
nvec->last_sync_msg = msg; nvec->last_sync_msg = msg;
complete(&nvec->sync_write); complete(&nvec->sync_write);
...@@ -477,7 +477,7 @@ static void nvec_tx_completed(struct nvec_chip *nvec) ...@@ -477,7 +477,7 @@ static void nvec_tx_completed(struct nvec_chip *nvec)
{ {
/* We got an END_TRANS, let's skip this, maybe there's an event */ /* We got an END_TRANS, let's skip this, maybe there's an event */
if (nvec->tx->pos != nvec->tx->size) { if (nvec->tx->pos != nvec->tx->size) {
dev_err(nvec->dev, "premature END_TRANS, resending\n"); dev_err(nvec->dev, "Premature END_TRANS, resending\n");
nvec->tx->pos = 0; nvec->tx->pos = 0;
nvec_gpio_set_value(nvec, 0); nvec_gpio_set_value(nvec, 0);
} else { } else {
...@@ -608,7 +608,7 @@ static irqreturn_t nvec_interrupt(int irq, void *dev) ...@@ -608,7 +608,7 @@ static irqreturn_t nvec_interrupt(int irq, void *dev)
/* Filter out some errors */ /* Filter out some errors */
if ((status & irq_mask) == 0 && (status & ~irq_mask) != 0) { if ((status & irq_mask) == 0 && (status & ~irq_mask) != 0) {
dev_err(nvec->dev, "unexpected irq mask %lx\n", status); dev_err(nvec->dev, "Unexpected irq mask %lx\n", status);
return IRQ_HANDLED; return IRQ_HANDLED;
} }
if ((status & I2C_SL_IRQ) == 0) { if ((status & I2C_SL_IRQ) == 0) {
...@@ -631,7 +631,7 @@ static irqreturn_t nvec_interrupt(int irq, void *dev) ...@@ -631,7 +631,7 @@ static irqreturn_t nvec_interrupt(int irq, void *dev)
if (status != (I2C_SL_IRQ | RCVD)) if (status != (I2C_SL_IRQ | RCVD))
nvec_invalid_flags(nvec, status, false); nvec_invalid_flags(nvec, status, false);
break; break;
case 1: /* command byte */ case 1: /* Command byte */
if (status != I2C_SL_IRQ) { if (status != I2C_SL_IRQ) {
nvec_invalid_flags(nvec, status, true); nvec_invalid_flags(nvec, status, true);
} else { } else {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment