From: Chihau Chau <[email protected]>

This fixes some code style issues detected with the checkpatch.pl
script, like not necessary braces {} in some if and else statements and
include a KERN_INFO facility level in a printk() function.

Signed-off-by: Chihau Chau <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
---
 drivers/staging/dream/smd/smd_qmi.c |   15 ++++++---------
 1 files changed, 6 insertions(+), 9 deletions(-)

diff --git a/drivers/staging/dream/smd/smd_qmi.c 
b/drivers/staging/dream/smd/smd_qmi.c
index 9e9067f..76fce51 100644
--- a/drivers/staging/dream/smd/smd_qmi.c
+++ b/drivers/staging/dream/smd/smd_qmi.c
@@ -211,11 +211,10 @@ static int qmi_send(struct qmi_ctxt *ctxt, struct qmi_msg 
*msg)
 
        qmi_dump_msg(msg, "send");
 
-       if (msg->service == QMI_CTL) {
+       if (msg->service == QMI_CTL)
                hlen = QMUX_HEADER - 1;
-       } else {
+       else
                hlen = QMUX_HEADER;
-       }
 
        /* QMUX length is total header + total payload - IFC selector */
        len = hlen + msg->size - 1;
@@ -248,11 +247,10 @@ static int qmi_send(struct qmi_ctxt *ctxt, struct qmi_msg 
*msg)
        /* len + 1 takes the interface selector into account */
        r = smd_write(ctxt->ch, msg->tlv - hlen, len + 1);
 
-       if (r != len) {
+       if (r != len)
                return -1;
-       } else {
+       else
                return 0;
-       }
 }
 
 static void qmi_process_ctl_msg(struct qmi_ctxt *ctxt, struct qmi_msg *msg)
@@ -376,7 +374,7 @@ static void qmi_process_broadcast_wds_msg(struct qmi_ctxt 
*ctxt,
 static void qmi_process_wds_msg(struct qmi_ctxt *ctxt,
                                struct qmi_msg *msg)
 {
-       printk("wds: %04x @ %02x\n", msg->type, msg->client_id);
+       printk(KERN_INFO "wds: %04x @ %02x\n", msg->type, msg->client_id);
        if (msg->client_id == ctxt->wds_client_id) {
                qmi_process_unicast_wds_msg(ctxt, msg);
        } else if (msg->client_id == 0xff) {
@@ -504,9 +502,8 @@ static void qmi_notify(void *priv, unsigned event)
        case SMD_EVENT_DATA: {
                int sz;
                sz = smd_cur_packet_size(ctxt->ch);
-               if ((sz > 0) && (sz <= smd_read_avail(ctxt->ch))) {
+               if ((sz > 0) && (sz <= smd_read_avail(ctxt->ch)))
                        queue_work(qmi_wq, &ctxt->read_work);
-               }
                break;
        }
        case SMD_EVENT_OPEN:
-- 
1.7.0.3

_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to