[lkml]   [2017]   [Jul]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH 2/5] powernv:idle: Change return type of pnv_probe_idle_states to int
On Wed,  5 Jul 2017 22:08:13 +0530
"Gautham R. Shenoy" <> wrote:

> From: "Gautham R. Shenoy" <>
> In the current idle initialization code, if there are failures in
> pnv_probe_idle_states, then no platform idle state is
> enabled. However, since the error is not propagated to the top-level
> function pnv_init_idle_states, we continue initialization in this
> top-level function even though this will never be used.
> Hence change the the return type of pnv_probe_idle_states from void to
> int and in case of failures, bail out early on in
> pnv_init_idle_states.
> Signed-off-by: Gautham R. Shenoy <>

Looks good to me.

Reviewed-by: Nicholas Piggin <>

I wonder if the warnings are strong enough here to let people know
idle won't be used so power consumption will be high and performance
significantly reduced on SMT machines?

 \ /
  Last update: 2017-07-06 17:03    [W:0.142 / U:6.112 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site