diff options
| author | Hans Verkuil <hverkuil-cisco@xs4all.nl> | 2021-12-13 16:10:34 +0100 |
|---|---|---|
| committer | Hans Verkuil <hverkuil-cisco@xs4all.nl> | 2021-12-13 16:10:34 +0100 |
| commit | 7acbf135af9c2fdc0ff27aecdd16659d1cd342b0 (patch) | |
| tree | 3959c9624891648a6540d12d4987278ee8192d04 | |
| parent | b22fc9e53051a7da903b3906826a527ee57ba92c (diff) | |
| download | v4l-utils-7acbf135af9c2fdc0ff27aecdd16659d1cd342b0.tar.gz | |
cec-ctl: show timestamp for events
Events (HPD/5V) didn't report the timestamp, unless --verbose was
added. But this is very useful to know, so change it to always report
the timestamp.
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
| -rw-r--r-- | utils/cec-ctl/cec-ctl.cpp | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/utils/cec-ctl/cec-ctl.cpp b/utils/cec-ctl/cec-ctl.cpp index 4bed81a9..867f3083 100644 --- a/utils/cec-ctl/cec-ctl.cpp +++ b/utils/cec-ctl/cec-ctl.cpp @@ -470,6 +470,7 @@ static const char *event2s(__u32 event) static void log_event(struct cec_event &ev, bool show) { bool is_high = ev.event == CEC_EVENT_PIN_CEC_HIGH; + bool is_initial = ev.flags & CEC_EVENT_FL_INITIAL_STATE; __u16 pa; if (ev.event != CEC_EVENT_PIN_CEC_LOW && ev.event != CEC_EVENT_PIN_CEC_HIGH && @@ -478,8 +479,13 @@ static void log_event(struct cec_event &ev, bool show) printf("\n"); if ((ev.flags & CEC_EVENT_FL_DROPPED_EVENTS) && show) printf("(warn: %s events were lost)\n", event2s(ev.event)); - if ((ev.flags & CEC_EVENT_FL_INITIAL_STATE) && show) - printf("Initial "); + if (show) { + if (is_initial) + printf("Initial "); + else if (ev.event != CEC_EVENT_PIN_CEC_LOW && ev.event != CEC_EVENT_PIN_CEC_HIGH) + printf("%s: ", ts2s(ev.ts).c_str()); + } + switch (ev.event) { case CEC_EVENT_STATE_CHANGE: pa = ev.state_change.phys_addr; @@ -518,8 +524,6 @@ static void log_event(struct cec_event &ev, bool show) printf("Event: Unknown (0x%x)\n", ev.event); break; } - if (verbose && show) - printf("\tTimestamp: %s\n", ts2s(ev.ts).c_str()); } /* |