media: cec: improve debugging

cec_transmit_msg_fh() first checked the message for errors, and only
after the message was found to be valid did it log the message contents.

However, that makes it hard to associate an error in the kernel log with
the message since the message contents was never logged in that case.

So swap the order: first log the message (once some very basic checks
are done), and only after that check for errors.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>

diff --git a/drivers/media/cec/cec-adap.c b/drivers/media/cec/cec-adap.c
index 768f7d70b55c1f8cb9e9678825d1ea65b01c752c..cf6dc3f3a17e471e0bc82bdbf40e3e41ad2c3a1f 100644 (file)
--- a/drivers/media/cec/cec-adap.c
+++ b/drivers/media/cec/cec-adap.c
@@ -699,16 +699,31 @@ int cec_transmit_msg_fh(struct cec_adapter *adap, struct cec_msg *msg,
        else
                msg->flags = 0;
 
+       if (msg->len > 1 && msg->msg[1] == CEC_MSG_CDC_MESSAGE) {
+               msg->msg[2] = adap->phys_addr >> 8;
+               msg->msg[3] = adap->phys_addr & 0xff;
+       }
+
        /* Sanity checks */
        if (msg->len == 0 || msg->len > CEC_MAX_MSG_SIZE) {
                dprintk(1, "%s: invalid length %d\n", __func__, msg->len);
                return -EINVAL;
        }
+
+       memset(msg->msg + msg->len, 0, sizeof(msg->msg) - msg->len);
+
+       if (msg->timeout)
+               dprintk(2, "%s: %*ph (wait for 0x%02x%s)\n",
+                       __func__, msg->len, msg->msg, msg->reply,
+                       !block ? ", nb" : "");
+       else
+               dprintk(2, "%s: %*ph%s\n",
+                       __func__, msg->len, msg->msg, !block ? " (nb)" : "");
+
        if (msg->timeout && msg->len == 1) {
-               dprintk(1, "%s: can't reply for poll msg\n", __func__);
+               dprintk(1, "%s: can't reply to poll msg\n", __func__);
                return -EINVAL;
        }
-       memset(msg->msg + msg->len, 0, sizeof(msg->msg) - msg->len);
        if (msg->len == 1) {
                if (cec_msg_destination(msg) == 0xf) {
                        dprintk(1, "%s: invalid poll message\n", __func__);
@@ -768,19 +783,6 @@ int cec_transmit_msg_fh(struct cec_adapter *adap, struct cec_msg *msg,
        if (!msg->sequence)
                msg->sequence = ++adap->sequence;
 
-       if (msg->len > 1 && msg->msg[1] == CEC_MSG_CDC_MESSAGE) {
-               msg->msg[2] = adap->phys_addr >> 8;
-               msg->msg[3] = adap->phys_addr & 0xff;
-       }
-
-       if (msg->timeout)
-               dprintk(2, "%s: %*ph (wait for 0x%02x%s)\n",
-                       __func__, msg->len, msg->msg, msg->reply,
-                       !block ? ", nb" : "");
-       else
-               dprintk(2, "%s: %*ph%s\n",
-                       __func__, msg->len, msg->msg, !block ? " (nb)" : "");
-
        data->msg = *msg;
        data->fh = fh;
        data->adap = adap;