DIV_ROUND_CLOSEST is faster if the compiler knows it will only be dealing with unsigned dividends. This optimization rips 32 bytes of binary code on x86_64.
Signed-off-by: Jean Delvare <khali at linux-fr.org> Cc: Guenter Roeck <linux at roeck-us.net> Cc: Andrew Morton <akpm at linux-foundation.org> Cc: Daniel Vetter <daniel.vetter at ffwll.ch> Cc: David Airlie <airlied at linux.ie> --- Already sent on: 2012-09-03. Daniel, I think we can safely assume ia_freq can't be negative? drivers/gpu/drm/i915/intel_pm.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) --- linux-3.7-rc5.orig/drivers/gpu/drm/i915/intel_pm.c 2012-11-12 09:30:56.796836818 +0100 +++ linux-3.7-rc5/drivers/gpu/drm/i915/intel_pm.c 2012-11-12 10:49:38.241676096 +0100 @@ -2547,7 +2547,8 @@ static void gen6_update_ring_freq(struct { struct drm_i915_private *dev_priv = dev->dev_private; int min_freq = 15; - int gpu_freq, ia_freq, max_ia_freq; + int gpu_freq; + unsigned int ia_freq, max_ia_freq; int scaling_factor = 180; WARN_ON(!mutex_is_locked(&dev->struct_mutex)); -- Jean Delvare