2019-10-10 09:04:29

by Fuqian Huang

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

Avoid printing the address of l440gx_map.virt every time l440gx init.

Signed-off-by: Fuqian Huang <[email protected]>
---
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);

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

2019-10-10 13:32:21

by Boris Brezillon

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

On Thu, 10 Oct 2019 16:40:19 +0800
Fuqian Huang <[email protected]> wrote:

> Avoid printing the address of l440gx_map.virt every time l440gx init.
>
> Signed-off-by: Fuqian Huang <[email protected]>
> ---
> 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