This is an automatic generated email to let you know that the following patch were queued at the http://git.linuxtv.org/cgit.cgi/v4l-utils.git tree:
Subject: cec-compliance: fix wrong event test Author: Hans Verkuil <hverkuil-ci...@xs4all.nl> Date: Mon May 20 15:46:05 2019 +0200 The cec-gpio driver generates more events than just the state change event. The adapter test didn't take that into account, which caused the compliance test to fail for the cec-gpio driver. Signed-off-by: Hans Verkuil <hverkuil-ci...@xs4all.nl> utils/cec-compliance/cec-test-adapter.cpp | 20 +++++++++++++++++++- 1 file changed, 19 insertions(+), 1 deletion(-) --- http://git.linuxtv.org/cgit.cgi/v4l-utils.git/commit/?id=51edf99b20d6bc2ff5e40ebd0691a65d60f9d1ed diff --git a/utils/cec-compliance/cec-test-adapter.cpp b/utils/cec-compliance/cec-test-adapter.cpp index 887cc2e12186..ba3c05f9fb54 100644 --- a/utils/cec-compliance/cec-test-adapter.cpp +++ b/utils/cec-compliance/cec-test-adapter.cpp @@ -189,7 +189,25 @@ int testAdapLogAddrs(struct node *node) if (!res) { struct timeval tv = { 0, 10000 }; // 10 ms - fail_on_test(ev.flags & CEC_EVENT_FL_INITIAL_STATE); + switch (ev.event) { + case CEC_EVENT_STATE_CHANGE: + fail_on_test(ev.flags & CEC_EVENT_FL_INITIAL_STATE); + break; + case CEC_EVENT_PIN_CEC_LOW: + case CEC_EVENT_PIN_CEC_HIGH: + case CEC_EVENT_PIN_HPD_LOW: + case CEC_EVENT_PIN_HPD_HIGH: + case CEC_EVENT_PIN_5V_LOW: + case CEC_EVENT_PIN_5V_HIGH: + fail_on_test(!(ev.flags & CEC_EVENT_FL_INITIAL_STATE)); + break; + case CEC_EVENT_LOST_MSGS: + fail("Unexpected event %d\n", ev.event); + break; + default: + fail("Unknown event %d\n", ev.event); + break; + } select(0, NULL, NULL, NULL, &tv); } } while (!res); _______________________________________________ linuxtv-commits mailing list linuxtv-commits@linuxtv.org https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits
