diff mbox

[1/2] i2c: designware: Remove space between variable and its value in debug print

Message ID 1438944208-16276-1-git-send-email-jarkko.nikula@linux.intel.com
State Superseded
Headers show

Commit Message

Jarkko Nikula Aug. 7, 2015, 10:43 a.m. UTC 
Remove extra space from "enabled= 0x" to make debug print a bit more dense
and to be in sync with "stat=0x". While at it, use "%#x" for printing "0x"
prefixed hexadecimal values.

Signed-off-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
---
 drivers/i2c/busses/i2c-designware-core.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Wolfram Sang Aug. 7, 2015, 11:08 a.m. UTC | #1 
> +	dev_dbg(dev->dev, "%s:  %s enabled=%#x stat=%#x\n", __func__,

What about the double space between the %s?
diff mbox

Patch

diff --git a/drivers/i2c/busses/i2c-designware-core.c b/drivers/i2c/busses/i2c-designware-core.c
index 6f19a33773fe..7310051d4a50 100644
--- a/drivers/i2c/busses/i2c-designware-core.c
+++ b/drivers/i2c/busses/i2c-designware-core.c
@@ -777,7 +777,7 @@  irqreturn_t i2c_dw_isr(int this_irq, void *dev_id)
 
 	enabled = dw_readl(dev, DW_IC_ENABLE);
 	stat = dw_readl(dev, DW_IC_RAW_INTR_STAT);
-	dev_dbg(dev->dev, "%s:  %s enabled= 0x%x stat=0x%x\n", __func__,
+	dev_dbg(dev->dev, "%s:  %s enabled=%#x stat=%#x\n", __func__,
 		dev->adapter.name, enabled, stat);
 	if (!enabled || !(stat & ~DW_IC_INTR_ACTIVITY))
 		return IRQ_NONE;