diff options
| author | Mike J. Chen <mjchen@google.com> | 2012-04-05 18:46:30 -0700 |
|---|---|---|
| committer | Mike J. Chen <mjchen@google.com> | 2012-04-05 18:46:30 -0700 |
| commit | d51d3e92d4aa0121e42f6002188c7b5c5869e618 (patch) | |
| tree | 9eeb18f2e19f4b591ad311444cdd332f2c5e290e | |
| parent | 9c533c3299d6ab2b647e65b4c0614bb37759d2b6 (diff) | |
| download | uboot-d51d3e92d4aa0121e42f6002188c7b5c5869e618.tar.gz | |
ARMV7: tungsten: Remove unneeded clearing of leds in avr detect
Bug 6299538
This was some old code from the kernel which maintained a cache
of the led state of the AVR. Bootloader has no state cache
so really has no business setting the leds during init.
Change-Id: Ic379f284c3e596907f749d9987a44c1bc8ce1ba8
Signed-off-by: Mike J. Chen <mjchen@google.com>
| -rw-r--r-- | board/google/tungsten/steelhead_avr.c | 13 |
1 files changed, 0 insertions, 13 deletions
diff --git a/board/google/tungsten/steelhead_avr.c b/board/google/tungsten/steelhead_avr.c index 4c19fb393..76cdc5625 100644 --- a/board/google/tungsten/steelhead_avr.c +++ b/board/google/tungsten/steelhead_avr.c @@ -230,7 +230,6 @@ static int avr_read_event_fifo(u8 *next_event) int detect_avr(void) { int rc; - struct avr_led_rgb_vals clear_led_req; printf("%s\n", __func__); @@ -263,18 +262,6 @@ int detect_avr(void) goto error; } - /* Set the LED state to all off in order to match the internal state we - * just established. - */ - clear_led_req.rgb[0] = 0x00; - clear_led_req.rgb[1] = 0x00; - clear_led_req.rgb[2] = 0x00; - rc = avr_led_set_all(&clear_led_req); - if (rc) { - printf("Failed to clear LEDs on AVR (rc = %d)\n", rc); - goto error; - } - printf("Steelhead AVR detected and initialized\n"); state.avr_detected = 1; |