Commit 41f8b96e authored by Chihau Chau's avatar Chihau Chau Committed by Greg Kroah-Hartman

Staging: dream: smd: smd_qmi: fix code style issues

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: default avatarChihau Chau <chihau@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent eb450e89
...@@ -211,11 +211,10 @@ static int qmi_send(struct qmi_ctxt *ctxt, struct qmi_msg *msg) ...@@ -211,11 +211,10 @@ static int qmi_send(struct qmi_ctxt *ctxt, struct qmi_msg *msg)
qmi_dump_msg(msg, "send"); qmi_dump_msg(msg, "send");
if (msg->service == QMI_CTL) { if (msg->service == QMI_CTL)
hlen = QMUX_HEADER - 1; hlen = QMUX_HEADER - 1;
} else { else
hlen = QMUX_HEADER; hlen = QMUX_HEADER;
}
/* QMUX length is total header + total payload - IFC selector */ /* QMUX length is total header + total payload - IFC selector */
len = hlen + msg->size - 1; len = hlen + msg->size - 1;
...@@ -248,11 +247,10 @@ static int qmi_send(struct qmi_ctxt *ctxt, struct qmi_msg *msg) ...@@ -248,11 +247,10 @@ static int qmi_send(struct qmi_ctxt *ctxt, struct qmi_msg *msg)
/* len + 1 takes the interface selector into account */ /* len + 1 takes the interface selector into account */
r = smd_write(ctxt->ch, msg->tlv - hlen, len + 1); r = smd_write(ctxt->ch, msg->tlv - hlen, len + 1);
if (r != len) { if (r != len)
return -1; return -1;
} else { else
return 0; return 0;
}
} }
static void qmi_process_ctl_msg(struct qmi_ctxt *ctxt, struct qmi_msg *msg) 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, ...@@ -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, static void qmi_process_wds_msg(struct qmi_ctxt *ctxt,
struct qmi_msg *msg) 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) { if (msg->client_id == ctxt->wds_client_id) {
qmi_process_unicast_wds_msg(ctxt, msg); qmi_process_unicast_wds_msg(ctxt, msg);
} else if (msg->client_id == 0xff) { } else if (msg->client_id == 0xff) {
...@@ -504,9 +502,8 @@ static void qmi_notify(void *priv, unsigned event) ...@@ -504,9 +502,8 @@ static void qmi_notify(void *priv, unsigned event)
case SMD_EVENT_DATA: { case SMD_EVENT_DATA: {
int sz; int sz;
sz = smd_cur_packet_size(ctxt->ch); 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); queue_work(qmi_wq, &ctxt->read_work);
}
break; break;
} }
case SMD_EVENT_OPEN: case SMD_EVENT_OPEN:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment