diff mbox series

[5/6] drivers/i2c: Remove caller signal_pending branch predictions

Message ID 20181116002713.8474-6-dave@stgolabs.net
State Accepted
Headers show
Series None | expand

Commit Message

Davidlohr Bueso Nov. 16, 2018, 12:27 a.m. UTC
This is already done for us internally by the signal machinery.

Cc: linux-i2c@vger.kernel.org
Cc: peda@axentia.se
Signed-off-by: Davidlohr Bueso <dave@stgolabs.net>
---
 drivers/i2c/busses/i2c-ibm_iic.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Wolfram Sang Nov. 27, 2018, 12:01 p.m. UTC | #1
On Thu, Nov 15, 2018 at 04:27:12PM -0800, Davidlohr Bueso wrote:
> This is already done for us internally by the signal machinery.
> 
> Cc: linux-i2c@vger.kernel.org
> Cc: peda@axentia.se
> Signed-off-by: Davidlohr Bueso <dave@stgolabs.net>

Applied to for-next, thanks!

Would have been nice if the cover letter was also sent to the
i2c-mailing list.
diff mbox series

Patch

diff --git a/drivers/i2c/busses/i2c-ibm_iic.c b/drivers/i2c/busses/i2c-ibm_iic.c
index 6f6e1dfe7cce..d78023d42a35 100644
--- a/drivers/i2c/busses/i2c-ibm_iic.c
+++ b/drivers/i2c/busses/i2c-ibm_iic.c
@@ -437,7 +437,7 @@  static int iic_wait_for_tc(struct ibm_iic_private* dev){
 				break;
 			}
 
-			if (unlikely(signal_pending(current))){
+			if (signal_pending(current)){
 				DBG("%d: poll interrupted\n", dev->idx);
 				ret = -ERESTARTSYS;
 				break;