Commit cbd2c4c2 authored by Rafael J. Wysocki's avatar Rafael J. Wysocki
Browse files

intel_idle: Eliminate __setup_broadcast_timer() 



The __setup_broadcast_timer() static function is only called in one
place and "true" is passed to it as the argument in there, so
effectively it is a wrapper arround tick_broadcast_enable().

To simplify the code, call tick_broadcast_enable() directly instead
of __setup_broadcast_timer() and drop the latter.

No intentional functional impact.

Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent e6cf623b

drivers/idle/intel_idle.c

+1 −9
Original line number Diff line number Diff line
@@ -967,14 +967,6 @@ static bool intel_idle_verify_cstate(unsigned int mwait_hint) 
	return true;

}



static void __setup_broadcast_timer(bool on)

{

	if (on)

		tick_broadcast_enable();

	else

		tick_broadcast_disable();

}



static void auto_demotion_disable(void)

{

	unsigned long long msr_bits;
@@ -1624,7 +1616,7 @@ static int intel_idle_cpu_online(unsigned int cpu) 
	struct cpuidle_device *dev;



	if (lapic_timer_reliable_states != LAPIC_TIMER_ALWAYS_RELIABLE)

		__setup_broadcast_timer(true);

		tick_broadcast_enable();



	/*

	 * Some systems can hotplug a cpu at runtime after