diff mbox series

[linux,dev-4.10] fsi: core: Remove more spam FSI prints to console

Message ID 20170907192103.36929-1-cbostic@linux.vnet.ibm.com
State Accepted, archived
Headers show
Series [linux,dev-4.10] fsi: core: Remove more spam FSI prints to console | expand

Commit Message

Christopher Bostic Sept. 7, 2017, 7:21 p.m. UTC
Move the most noisy fsi prints to dev_dbg() to unclutter the console
output.

Example:
[  484.610000]  slave@00:00: status: 0x84000500, sisc: 0x00040108
[  484.610000]  fsi1: can't read slave 00:00 -5

Signed-off-by: Christopher Bostic <cbostic@linux.vnet.ibm.com>
---
 drivers/fsi/fsi-core.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Joel Stanley Sept. 12, 2017, 7:58 a.m. UTC | #1
On Fri, Sep 8, 2017 at 5:21 AM, Christopher Bostic
<cbostic@linux.vnet.ibm.com> wrote:
> Move the most noisy fsi prints to dev_dbg() to unclutter the console
> output.
>
> Example:
> [  484.610000]  slave@00:00: status: 0x84000500, sisc: 0x00040108
> [  484.610000]  fsi1: can't read slave 00:00 -5
>
> Signed-off-by: Christopher Bostic <cbostic@linux.vnet.ibm.com>

This appears to be the last of the dev_info's in the core. Good work.

Applied to dev-4.10.

Cheers,

Joel
diff mbox series

Patch

diff --git a/drivers/fsi/fsi-core.c b/drivers/fsi/fsi-core.c
index 0f19bd3..8a17176 100644
--- a/drivers/fsi/fsi-core.c
+++ b/drivers/fsi/fsi-core.c
@@ -214,7 +214,7 @@  int fsi_slave_report_and_clear_errors(struct fsi_slave *slave)
 	if (rc)
 		return rc;
 
-	dev_info(&slave->dev, "status: 0x%08x, sisc: 0x%08x\n",
+	dev_dbg(&slave->dev, "status: 0x%08x, sisc: 0x%08x\n",
 			be32_to_cpu(stat), be32_to_cpu(irq));
 
 	/* clear interrupts */
@@ -677,7 +677,7 @@  static int fsi_slave_init(struct fsi_master *master, int link, uint8_t id)
 
 	rc = fsi_master_read(master, link, id, 0, &chip_id, sizeof(chip_id));
 	if (rc) {
-		dev_warn(&master->dev, "can't read slave %02x:%02x %d\n",
+		dev_dbg(&master->dev, "can't read slave %02x:%02x %d\n",
 				link, id, rc);
 		return -ENODEV;
 	}