i2c: rk3x: Don't print visible virtual mapping MMIO address
authorShawn Lin <shawn.lin@rock-chips.com>
Fri, 18 May 2018 00:45:30 +0000 (08:45 +0800)
committerWolfram Sang <wsa@the-dreams.de>
Thu, 24 May 2018 20:09:05 +0000 (22:09 +0200)
Now %p doesn't print visible pointer address unless the user
really want it. According to Documentation/core-api/printk-formats.rst,
%px should be used instead, otherwise we could see:

rk3x-i2c ff110000.i2c: Initialized RK3xxx I2C bus at (____ptrval____)
rk3x-i2c ff130000.i2c: Initialized RK3xxx I2C bus at (____ptrval____)
rk3x-i2c ff3c0000.i2c: Initialized RK3xxx I2C bus at (____ptrval____)
rk3x-i2c ff3d0000.i2c: Initialized RK3xxx I2C bus at (____ptrval____)

But I don't really understand why we need dump it in the first place!
Let's remove the whole pointless log.

Signed-off-by: Shawn Lin <shawn.lin@rock-chips.com>
Reviewed-by: Heiko Stuebner <heiko@sntech.de>
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
drivers/i2c/busses/i2c-rk3x.c

index e1a18d9..b8a2728 100644 (file)
@@ -1326,8 +1326,6 @@ static int rk3x_i2c_probe(struct platform_device *pdev)
        if (ret < 0)
                goto err_clk_notifier;
 
-       dev_info(&pdev->dev, "Initialized RK3xxx I2C bus at %p\n", i2c->regs);
-
        return 0;
 
 err_clk_notifier: