On 6/19/20 4:50 PM, Philippe Mathieu-Daudé wrote:
> This series add trace events to better display LEDs changes.
> This helps me to work on a generic LED device, see:
> https://www.mail-archive.com/[email protected]/msg711917.html

FYI I'm going to send a v4, using the generic LED device.

> 
> Example when booting an obmc-phosphor-image, we can see the LED #14
> (front-power LED) starting to blink.
> 
> - ASCII LED bar view:
> 
>   $ qemu-system-arm -M witherspoon-bmc -trace pca9552_leds_status
>   [email protected]:pca9552_leds_status 0x55dde47807c0 LEDs 0-15 
> [*...............]
>   [email protected]:pca9552_leds_status 0x55dde47807c0 LEDs 0-15 
> [**..............]
>   [email protected]:pca9552_leds_status 0x55dde47807c0 LEDs 0-15 
> [***.............]
>   [email protected]:pca9552_leds_status 0x55dde47807c0 LEDs 0-15 
> [****............]
>   [email protected]:pca9552_leds_status 0x55dde47807c0 LEDs 0-15 
> [****.........*..]
>   [email protected]:pca9552_leds_status 0x55dde47807c0 LEDs 0-15 
> [****.........**.]
>   [email protected]:pca9552_leds_status 0x55dde47807c0 LEDs 0-15 
> [****.........***]
>   [email protected]:pca9552_leds_status 0x55dde47807c0 LEDs 0-15 
> [****.........*.*]
>   [email protected]:pca9552_leds_status 0x55dde47807c0 LEDs 0-15 
> [****.........***]
>   [email protected]:pca9552_leds_status 0x55dde47807c0 LEDs 0-15 
> [****.........*.*]
>   [email protected]:pca9552_leds_status 0x55dde47807c0 LEDs 0-15 
> [****.........***]
>   [email protected]:pca9552_leds_status 0x55dde47807c0 LEDs 0-15 
> [****.........*.*]
>   [email protected]:pca9552_leds_status 0x55dde47807c0 LEDs 0-15 
> [****.........***]
>   [email protected]:pca9552_leds_status 0x55dde47807c0 LEDs 0-15 
> [****.........*.*]
> 
> - Only display LEDs which status changes:
> 
>   $ qemu-system-arm -M witherspoon-bmc -trace pca9552_led_change
>   [email protected]:pca9552_led_change 0x557cb6896d80 LED id:0 status: 
> 0 -> 1
>   [email protected]:pca9552_led_change 0x557cb6896d80 LED id:1 status: 
> 0 -> 1
>   [email protected]:pca9552_led_change 0x557cb6896d80 LED id:2 status: 
> 0 -> 1
>   [email protected]:pca9552_led_change 0x557cb6896d80 LED id:3 status: 
> 0 -> 1
>   [email protected]:pca9552_led_change 0x557cb6896d80 LED id:13 status: 
> 0 -> 1
>   [email protected]:pca9552_led_change 0x557cb6896d80 LED id:14 status: 
> 0 -> 1
>   [email protected]:pca9552_led_change 0x557cb6896d80 LED id:15 status: 
> 0 -> 1
>   [email protected]:pca9552_led_change 0x557cb6896d80 LED id:14 status: 
> 1 -> 0
>   [email protected]:pca9552_led_change 0x557cb6896d80 LED id:14 status: 
> 0 -> 1
>   [email protected]:pca9552_led_change 0x557cb6896d80 LED id:14 status: 
> 1 -> 0
>   [email protected]:pca9552_led_change 0x557cb6896d80 LED id:14 status: 
> 0 -> 1
> 
> For information about how to test the obmc-phosphor-image, see:
> https://www.mail-archive.com/[email protected]/msg712911.html
> 
> Supersedes: <[email protected]>
> 
> Philippe Mathieu-Daudé (4):
>   hw/misc/pca9552: Replace magic value by PCA9552_LED_COUNT definition
>   hw/misc/pca9552: Add a PCA955X_LED_MAX definition
>   hw/misc/pca9552: Trace LED On/Off events
>   hw/misc/pca9552: Trace LED change events
> 
>  include/hw/misc/pca9552.h |  2 ++
>  hw/misc/pca9552.c         | 60 ++++++++++++++++++++++++++++++++++++++-
>  hw/misc/trace-events      |  4 +++
>  3 files changed, 65 insertions(+), 1 deletion(-)
> 

Reply via email to