1
0
Files
linux/drivers
Justin Madru 8c172dde41 Staging: s5k3e2fx.c: simplify complexity by factoring
the code was looping, setting s_move[i] to the following calculations

if (actual_step>= 0)
         s_move[i] = ((((i + 1) * gain + 0x200) - (i * gain + 0x200)) / 0x400);
else
         s_move[i] = ((((i + 1) * gain - 0x200) - (i * gain - 0x200)) / 0x400);

but, this code reduces to the expression
	s_move[i] = gain>>  10;

The reason for the complexity was to generate a step function with
integer division and rounding to land on specific values. But these calculations
can be simplified to the following code:

	gain = ((actual_step<<  10) / 5)>>  10;
	for (i = 0; i<= 4; i++)
		s_move[i] = gain;

Signed-off-by: Justin Madru<jdm64@gawab.com>
Reviewed-by: Ray Lee<ray-lk@madrabbit.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2010-03-03 16:42:33 -08:00
..
2010-02-23 00:39:00 -05:00
2010-02-10 12:07:28 -07:00
2010-01-29 21:03:09 +01:00
2010-02-11 13:59:42 -08:00
2010-02-02 18:11:21 -08:00
2010-02-03 16:55:20 -05:00
2010-02-16 15:11:10 -08:00
2010-02-22 19:50:34 -08:00
2010-01-13 10:01:35 +00:00