[PATCH] powerpc/64s: Fix misleading SPR and timebase information

Zhangshaokun zhangshaokun at hisilicon.com
Wed Jun 12 19:54:22 AEST 2019


Hi Michael,

A gentle ping.

On 2019/5/29 17:21, Shaokun Zhang wrote:
> pr_info shows SPR and timebase as a decimal value with a '0x'
> prefix, which is somewhat misleading.
> 
> Fix it to print hexadecimal, as was intended.
> 
> Fixes: 10d91611f426 ("powerpc/64s: Reimplement book3s idle code in C")
> Cc: Michael Ellerman <mpe at ellerman.id.au>
> Cc: Nicholas Piggin <npiggin at gmail.com>
> Signed-off-by: Shaokun Zhang <zhangshaokun at hisilicon.com>
> ---
>  arch/powerpc/platforms/powernv/idle.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/powerpc/platforms/powernv/idle.c b/arch/powerpc/platforms/powernv/idle.c
> index c9133f7908ca..77f2e0a4ee37 100644
> --- a/arch/powerpc/platforms/powernv/idle.c
> +++ b/arch/powerpc/platforms/powernv/idle.c
> @@ -1159,10 +1159,10 @@ static void __init pnv_power9_idle_init(void)
>  			pnv_deepest_stop_psscr_mask);
>  	}
>  
> -	pr_info("cpuidle-powernv: First stop level that may lose SPRs = 0x%lld\n",
> +	pr_info("cpuidle-powernv: First stop level that may lose SPRs = 0x%llx\n",
>  		pnv_first_spr_loss_level);
>  
> -	pr_info("cpuidle-powernv: First stop level that may lose timebase = 0x%lld\n",
> +	pr_info("cpuidle-powernv: First stop level that may lose timebase = 0x%llx\n",
>  		pnv_first_tb_loss_level);
>  }
>  
> 



More information about the Linuxppc-dev mailing list