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 set_osd_string test
Author:  Hans Verkuil <hans.verk...@cisco.com>
Date:    Fri Sep 14 11:06:55 2018 +0200

Even though cec-follower implemented SET_OSD_STRING correctly,
cec-compliance would still give this warning:

warn: cec-test.cpp(332): The device is in an unsuitable state or cannot display 
the complete message.

This was due to an incorrect 'else' statement which should
have checked if the received message was a Feature Abort.

Signed-off-by: Hans Verkuil <hans.verk...@cisco.com>

 utils/cec-compliance/cec-test.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

---

http://git.linuxtv.org/cgit.cgi/v4l-utils.git/commit/?id=23a83f637f3c8777e798ab27b420333574330657
diff --git a/utils/cec-compliance/cec-test.cpp 
b/utils/cec-compliance/cec-test.cpp
index 501f9143c217..0744ba7176ba 100644
--- a/utils/cec-compliance/cec-test.cpp
+++ b/utils/cec-compliance/cec-test.cpp
@@ -328,7 +328,7 @@ static int osd_string_set_default(struct node *node, 
unsigned me, unsigned la, b
                return NOTSUPPORTED;
        else if (refused(&msg))
                return REFUSED;
-       else {
+       else if (cec_msg_status_is_abort(&msg)) {
                warn("The device is in an unsuitable state or cannot display 
the complete message.\n");
                unsuitable = true;
        }

_______________________________________________
linuxtv-commits mailing list
linuxtv-commits@linuxtv.org
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to