Re: [PATCH v2] mtd: maps: l440gx: Avoid printing address to dmesg

From: Boris Brezillon
Date: Thu Oct 10 2019 - 09:31:42 EST


On Thu, 10 Oct 2019 16:40:19 +0800
Fuqian Huang <huangfq.daxian@xxxxxxxxx> wrote:

> Avoid printing the address of l440gx_map.virt every time l440gx init.
>
> Signed-off-by: Fuqian Huang <huangfq.daxian@xxxxxxxxx>
> ---
> drivers/mtd/maps/l440gx.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/mtd/maps/l440gx.c b/drivers/mtd/maps/l440gx.c
> index 876f12f40018..ebe37edc8e88 100644
> --- a/drivers/mtd/maps/l440gx.c
> +++ b/drivers/mtd/maps/l440gx.c
> @@ -86,7 +86,7 @@ static int __init init_l440gx(void)
> return -ENOMEM;
> }
> simple_map_init(&l440gx_map);
> - printk(KERN_NOTICE "window_addr = 0x%08lx\n", (unsigned long)l440gx_map.virt);
> + printk(KERN_DEBUG "window_addr = 0x%08lx\n", (unsigned long)l440gx_map.virt);

pr_debug() please, and use %p instead of this %lx with a cast.

>
> /* Setup the pm iobase resource
> * This code should move into some kind of generic bridge