diff options
| author | Eric Caruso <ejcaruso@chromium.org> | 2014-09-29 18:03:38 -0700 |
|---|---|---|
| committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | 2014-09-30 23:04:29 +0000 |
| commit | 85d2d1f503ae8ad43094672910130df18e4a7d13 (patch) | |
| tree | 27e98b953b5923443fd3e3ac44bc58881fcdb512 | |
| parent | d747c44dda978acc58458fd5ae84c64054515f22 (diff) | |
| download | chrome-ec-85d2d1f503ae8ad43094672910130df18e4a7d13.tar.gz | |
lightbar: remove erroneous multiplications by FP_SCALE
Not sure why these were around. They were making RAMP_ONCE and
CYCLE_ONCE do obviously incorrect things.
BUG=None
BRANCH=ToT
TEST=Manual, visual inspection
Change-Id: I618078e8bef8824f5449d5b5453924beb36d50e1
Signed-off-by: Eric Caruso <ejcaruso@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/220463
Reviewed-by: Bill Richardson <wfrichar@chromium.org>
| -rw-r--r-- | common/lightbar.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/common/lightbar.c b/common/lightbar.c index 12ff8baa88..6d6e62be20 100644 --- a/common/lightbar.c +++ b/common/lightbar.c @@ -1152,7 +1152,7 @@ static uint32_t lightbyte_RAMP_ONCE(void) } for (w = 0; w < 128; w++) { - f = cycle_010(w) * FP_SCALE; + f = cycle_010(w); for (i = 0; i < NUM_LEDS; i++) { r = get_interp_value(i, LB_COL_RED, f); g = get_interp_value(i, LB_COL_GREEN, f); @@ -1188,7 +1188,7 @@ static uint32_t lightbyte_CYCLE_ONCE(void) } for (w = 0; w < 256; w++) { - f = cycle_010(w) * FP_SCALE; + f = cycle_010(w); for (i = 0; i < NUM_LEDS; i++) { r = get_interp_value(i, LB_COL_RED, f); g = get_interp_value(i, LB_COL_GREEN, f); |