Re: [PATCH] powerpc/32: add missing \n at end of printk warning message

[Christophe Leroy]

Le 12/09/2016 Ã 12:12, Colin King a Ãcrit :
> From: Colin Ian King <colin.king@xxxxxxxxxxxxx>
>
> The message is missing a \n, add it.
>
> Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx>
> ---
>  arch/powerpc/mm/init_32.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/powerpc/mm/init_32.c b/arch/powerpc/mm/init_32.c
> index 448685f..b532dd9 100644
> --- a/arch/powerpc/mm/init_32.c
> +++ b/arch/powerpc/mm/init_32.c
> @@ -137,7 +137,7 @@ void __init MMU_init(void)
>  	if (memblock.memory.cnt > 1) {
>  #ifndef CONFIG_WII
>  		memblock_enforce_memory_limit(memblock.memory.regions[0].size);
> -		printk(KERN_WARNING "Only using first contiguous memory region");
> +		printk(KERN_WARNING "Only using first contiguous memory region\n");

This line was already 81 characters long. It is now 83.
I would suggest to use pr_warn() instead in order to shorten the line.

Christophe

>  #else
>  		wii_memory_fixups();
>  #endif