From d7e2dc844666cf3f488fd69409640f1f8a9844d4 Mon Sep 17 00:00:00 2001 From: Ian Moffett Date: Fri, 21 Feb 2025 13:15:10 -0500 Subject: kernel: Update kernel log style Kernel logs must now all be lowercase for consistency. A new style called Peripheral Description Notation (PDN) has also been introduced to describe devices. Signed-off-by: Ian Moffett --- sys/dev/ic/ahci.c | 10 +++++----- sys/dev/ic/nvme.c | 20 ++++++++++---------- 2 files changed, 15 insertions(+), 15 deletions(-) (limited to 'sys/dev/ic') diff --git a/sys/dev/ic/ahci.c b/sys/dev/ic/ahci.c index ec7e68a..5c85013 100644 --- a/sys/dev/ic/ahci.c +++ b/sys/dev/ic/ahci.c @@ -131,7 +131,7 @@ ahci_hba_init(struct ahci_hba *hba) return error; } - pr_trace("Successfully performed a hard reset.\n"); + pr_trace("successfully performed a hard reset\n"); /* * The HBA provides backwards compatibility with @@ -166,25 +166,25 @@ ahci_init(void) * devices to the PCI bus, acting as an interface * between them. */ - pr_trace("Detected AHCI HBA (%x:%x.%x, slot=%d)\n", + pr_trace("IDE storage ctrl \n", ahci_dev->bus, ahci_dev->device_id, ahci_dev->func, ahci_dev->slot); /* Try to request a general purpose timer */ if (req_timer(TIMER_GP, &tmr) != TMRR_SUCCESS) { - pr_error("Failed to fetch general purpose timer\n"); + pr_error("failed to fetch general purpose timer\n"); return -ENODEV; } /* Ensure it has get_time_usec() */ if (tmr.get_time_usec == NULL) { - pr_error("General purpose timer has no get_time_usec()\n"); + pr_error("general purpose timer has no get_time_usec()\n"); return -ENODEV; } /* We also need msleep() */ if (tmr.msleep == NULL) { - pr_error("General purpose timer has no msleep()\n"); + pr_error("general purpose timer has no msleep()\n"); return -ENODEV; } diff --git a/sys/dev/ic/nvme.c b/sys/dev/ic/nvme.c index 4e1a487..749ac93 100644 --- a/sys/dev/ic/nvme.c +++ b/sys/dev/ic/nvme.c @@ -219,7 +219,7 @@ nvme_stop_ctrl(struct nvme_bar *bar) mmio_write32(&bar->config, config); if (nvme_poll_reg(bar, &bar->status, STATUS_RDY, false) < 0) { - pr_error("Controller reset timeout\n"); + pr_error("controller reset timeout\n"); return -ETIME; } @@ -246,7 +246,7 @@ nvme_start_ctrl(struct nvme_bar *bar) mmio_write32(&bar->config, config); if (nvme_poll_reg(bar, &bar->status, STATUS_RDY, true) < 0) { - pr_error("Controller startup timeout\n"); + pr_error("controller startup timeout\n"); return -ETIME; } @@ -292,7 +292,7 @@ nvme_poll_submit_cmd(struct nvme_queue *q, struct nvme_cmd cmd) /* Check for timeout */ if (spins > 5) { - pr_error("Hang while polling phase bit, giving up\n"); + pr_error("hang while polling phase bit, giving up\n"); return -ETIME; } @@ -350,9 +350,9 @@ nvme_log_ctrl_id(struct nvme_id *id) sn[i] = id->sn[i]; } - pr_trace("Model number: %s\n", mn); - pr_trace("Serial number: %s\n", sn); - pr_trace("Firmware revision: %s\n", fr); + pr_trace("model number: %s\n", mn); + pr_trace("serial number: %s\n", sn); + pr_trace("firmware revision: %s\n", fr); } /* @@ -601,7 +601,7 @@ nvme_init_ctrl(struct nvme_bar *bar) } if (nvme_init_ns(&ctrl, nsids[i]) != 0) { - pr_error("Failed to initialize NSID %d\n", nsids[i]); + pr_error("failed to initialize NSID %d\n", nsids[i]); } } @@ -627,19 +627,19 @@ nvme_init(void) /* Try to request a general purpose timer */ if (req_timer(TIMER_GP, &tmr) != TMRR_SUCCESS) { - pr_error("Failed to fetch general purpose timer\n"); + pr_error("failed to fetch general purpose timer\n"); return -ENODEV; } /* Ensure it has get_time_usec() */ if (tmr.get_time_usec == NULL) { - pr_error("General purpose timer has no get_time_usec()\n"); + pr_error("general purpose timer has no get_time_usec()\n"); return -ENODEV; } /* We also need msleep() */ if (tmr.msleep == NULL) { - pr_error("General purpose timer has no msleep()\n"); + pr_error("general purpose timer has no msleep()\n"); return -ENODEV; } -- cgit v1.2.3