Commit d01b7791 authored by Madhusudhan Chikkature's avatar Madhusudhan Chikkature Committed by Tony Lindgren

HDQ driver: remove unlikely calls

Remove the unneeded "unlikely" calls.
Signed-off-by: default avatarMadhusudhan Chikkature <madhu.cr@ti.com>
Acked-by: default avatarEvgeniy Polyakov <johnpol@2ka.mipt.ru>
Acked-by: default avatarFelipe Balbi <felipe.balbi@nokia.com>
Signed-off-by: default avatarTony Lindgren <tony@atomide.com>
parent bda8fa89
...@@ -138,7 +138,7 @@ static int hdq_wait_for_flag(struct hdq_data *hdq_data, u32 offset, ...@@ -138,7 +138,7 @@ static int hdq_wait_for_flag(struct hdq_data *hdq_data, u32 offset,
set_current_state(TASK_UNINTERRUPTIBLE); set_current_state(TASK_UNINTERRUPTIBLE);
schedule_timeout(1); schedule_timeout(1);
} }
if (unlikely(*status & flag)) if (*status & flag)
ret = -ETIMEDOUT; ret = -ETIMEDOUT;
} else if (flag_set == OMAP_HDQ_FLAG_SET) { } else if (flag_set == OMAP_HDQ_FLAG_SET) {
/* wait for the flag set */ /* wait for the flag set */
...@@ -147,7 +147,7 @@ static int hdq_wait_for_flag(struct hdq_data *hdq_data, u32 offset, ...@@ -147,7 +147,7 @@ static int hdq_wait_for_flag(struct hdq_data *hdq_data, u32 offset,
set_current_state(TASK_UNINTERRUPTIBLE); set_current_state(TASK_UNINTERRUPTIBLE);
schedule_timeout(1); schedule_timeout(1);
} }
if (unlikely(!(*status & flag))) if (!(*status & flag))
ret = -ETIMEDOUT; ret = -ETIMEDOUT;
} else } else
return -EINVAL; return -EINVAL;
...@@ -182,7 +182,7 @@ hdq_write_byte(struct hdq_data *hdq_data, u8 val, u8 *status) ...@@ -182,7 +182,7 @@ hdq_write_byte(struct hdq_data *hdq_data, u8 val, u8 *status)
/* wait for the TXCOMPLETE bit */ /* wait for the TXCOMPLETE bit */
ret = wait_event_interruptible_timeout(hdq_wait_queue, ret = wait_event_interruptible_timeout(hdq_wait_queue,
hdq_data->hdq_irqstatus, OMAP_HDQ_TIMEOUT); hdq_data->hdq_irqstatus, OMAP_HDQ_TIMEOUT);
if (unlikely(ret < 0)) { if (ret < 0) {
dev_dbg(hdq_data->dev, "wait interrupted"); dev_dbg(hdq_data->dev, "wait interrupted");
return -EINTR; return -EINTR;
} }
...@@ -333,7 +333,7 @@ omap_hdq_break(struct hdq_data *hdq_data) ...@@ -333,7 +333,7 @@ omap_hdq_break(struct hdq_data *hdq_data)
/* wait for the TIMEOUT bit */ /* wait for the TIMEOUT bit */
ret = wait_event_interruptible_timeout(hdq_wait_queue, ret = wait_event_interruptible_timeout(hdq_wait_queue,
hdq_data->hdq_irqstatus, OMAP_HDQ_TIMEOUT); hdq_data->hdq_irqstatus, OMAP_HDQ_TIMEOUT);
if (unlikely(ret < 0)) { if (ret < 0) {
dev_dbg(hdq_data->dev, "wait interrupted"); dev_dbg(hdq_data->dev, "wait interrupted");
up(&hdq_data->hdq_semlock); up(&hdq_data->hdq_semlock);
return -EINTR; return -EINTR;
......
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