diff options
author | Karthikeyan Ramasubramanian <kramasub@codeaurora.org> | 2015-12-17 17:41:34 -0700 |
---|---|---|
committer | David Keitel <dkeitel@codeaurora.org> | 2016-03-23 21:16:52 -0700 |
commit | 9daf459220e197f8d470e55fc31b4f449c614f42 (patch) | |
tree | 68f7da2c5a9f4dbb56f01b363d6547a4eee6efe3 /drivers/soc | |
parent | b7244df4e04e5ce7c4ef1e47f8ebf42934abaa1e (diff) |
soc: qcom: glink_smd_xprt: Remove unnecessary mutual exclusion
Receive command mutual exclusion is required only for the channel open
& close related operations and is not required for any operations relating
to data exchange.
Remove receive command mutual exclusion from the location where it is not
required.
Change-Id: Icebab0b36d71165d895e0fed2e9cfe2c624056b9
Signed-off-by: Karthikeyan Ramasubramanian <kramasub@codeaurora.org>
Diffstat (limited to 'drivers/soc')
-rw-r--r-- | drivers/soc/qcom/glink_smd_xprt.c | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/drivers/soc/qcom/glink_smd_xprt.c b/drivers/soc/qcom/glink_smd_xprt.c index f3d868d8cef3..bfe6bfe77a4f 100644 --- a/drivers/soc/qcom/glink_smd_xprt.c +++ b/drivers/soc/qcom/glink_smd_xprt.c @@ -559,12 +559,10 @@ static void process_tx_done(struct work_struct *work) riid = ch_work->iid; einfo = ch->edge; kfree(ch_work); - mutex_lock(&einfo->rx_cmd_lock); einfo->xprt_if.glink_core_if_ptr->rx_cmd_tx_done(&einfo->xprt_if, ch->rcid, riid, false); - mutex_unlock(&einfo->rx_cmd_lock); } /** @@ -657,11 +655,9 @@ static void process_status_event(struct work_struct *work) if (set & TIOCM_RI) sigs |= SMD_RI_SIG; - mutex_lock(&einfo->rx_cmd_lock); einfo->xprt_if.glink_core_if_ptr->rx_cmd_remote_sigs(&einfo->xprt_if, ch->rcid, sigs); - mutex_unlock(&einfo->rx_cmd_lock); } /** @@ -747,13 +743,11 @@ static void process_data_event(struct work_struct *work) __func__, ch->name, ch->lcid, ch->rcid); ch->intent_req = true; - mutex_lock(&einfo->rx_cmd_lock); einfo->xprt_if.glink_core_if_ptr-> rx_cmd_remote_rx_intent_req( &einfo->xprt_if, ch->rcid, pkt_remaining); - mutex_unlock(&einfo->rx_cmd_lock); return; } } @@ -1075,14 +1069,12 @@ static void tx_cmd_version(struct glink_transport_if *if_ptr, uint32_t version, struct edge_info *einfo; einfo = container_of(if_ptr, struct edge_info, xprt_if); - mutex_lock(&einfo->rx_cmd_lock); einfo->xprt_if.glink_core_if_ptr->rx_cmd_version_ack(&einfo->xprt_if, version, features); einfo->xprt_if.glink_core_if_ptr->rx_cmd_version(&einfo->xprt_if, version, features); - mutex_unlock(&einfo->rx_cmd_lock); } /** @@ -1758,7 +1750,6 @@ static int tx_cmd_rx_intent_req(struct glink_transport_if *if_ptr, break; } spin_unlock_irqrestore(&einfo->channels_lock, flags); - mutex_lock(&einfo->rx_cmd_lock); einfo->xprt_if.glink_core_if_ptr->rx_cmd_rx_intent_req_ack( &einfo->xprt_if, ch->rcid, @@ -1768,7 +1759,6 @@ static int tx_cmd_rx_intent_req(struct glink_transport_if *if_ptr, ch->rcid, ch->next_intent_id++, size); - mutex_unlock(&einfo->rx_cmd_lock); return 0; } |