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-ctl: fix destination for non-standard messages
Author:  Hans Verkuil <hans.verk...@cisco.com>
Date:    Thu Oct 4 10:16:55 2018 +0200

The code that allows for changing the destination between messages
broke non-standard messages (poll, vendor messages): they now always
used destination 0.

Fix this by moving up the code that sets the destination instead of
doing setting it in just the default case.

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

 utils/cec-ctl/cec-ctl.cpp | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

---

http://git.linuxtv.org/cgit.cgi/v4l-utils.git/commit/?id=71806d6cbfad009e2b5f5b98ae75dff9eda91bf0
diff --git a/utils/cec-ctl/cec-ctl.cpp b/utils/cec-ctl/cec-ctl.cpp
index 9b289cb4c836..10c5b27aeb94 100644
--- a/utils/cec-ctl/cec-ctl.cpp
+++ b/utils/cec-ctl/cec-ctl.cpp
@@ -1797,8 +1797,8 @@ int main(int argc, char **argv)
                if (ch == -1)
                        break;
 
-               if (ch > OptMessages || ch == OptPoll)
-                       cec_msg_init(&msg, 0, 0);
+               cec_msg_init(&msg, 0, 0);
+               msg.msg[0] = cec_msg_is_broadcast(&msg) ? 0xf : (options[OptTo] 
? to : 0xf0);
                options[(int)ch] = 1;
 
                switch (ch) {
@@ -2135,7 +2135,6 @@ int main(int argc, char **argv)
                                break;
                        opt = opt2message[ch - OptMessages];
                        parse_msg_args(msg, reply, opt, ch);
-                       msg.msg[0] = cec_msg_is_broadcast(&msg) ? 0xf : 
(options[OptTo] ? to : 0xf0);
                        msgs.push_back(msg);
                        break;
                }

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

Reply via email to