Messages in this thread | | | Date | Mon, 14 Apr 2014 10:59:11 +0200 | From | Peter Zijlstra <> | Subject | Re: [RFC][PATCH 8/8] sched,idle: Reflow cpuidle_idle_call() |
| |
On Sun, Apr 13, 2014 at 05:36:46PM -0400, Nicolas Pitre wrote: > > @@ -102,90 +102,64 @@ static int cpuidle_idle_call(void) > > * Check if the cpuidle framework is ready, otherwise fallback > > * to the default arch specific idle method > > */ > > - ret = cpuidle_enabled(drv, dev); > > + if (cpuidle_enabled(drv, dev)) > > + goto use_default; > > Why not using braces here and moving the use_default code block inside > it instead?
I didn't because there's two goto use_default sites, but sure, I can do that.
| |