Yes, Please merge this Mauro.
Thank you :-)
Acked-by: Michael Krufky <[email protected]>
Strange, I didn't get the original mail, but I got Mauro's replies...
Oh well.
Cheers,
Mike
Mauro Carvalho Chehab wrote:
Patch resent to the proper ML, in order to be handled by patchwork.
Michael,
The patch seems ok to my eyes. Please ack if ok for you to apply it.
Forwarded message:
Date: Wed, 18 Feb 2009 10:11:10 +0100
From: Roel Kluin <[email protected]>
To: Michael Krufky <[email protected]>
Cc: Mauro Carvalho Chehab <[email protected]>, [email protected],
Andrew Morton <[email protected]>
Subject: [PATCH] V4L: missing parentheses?
Please review.
--------------------------->8-------------8<------------------------------
Add missing parentheses
Signed-off-by: Roel Kluin <[email protected]>
---
diff --git a/drivers/media/common/tuners/tda18271-common.c
b/drivers/media/common/tuners/tda18271-common.c
index 6fb5b45..fc76c30 100644
--- a/drivers/media/common/tuners/tda18271-common.c
+++ b/drivers/media/common/tuners/tda18271-common.c
@@ -490,9 +490,9 @@ int tda18271_set_standby_mode(struct dvb_frontend *fe,
tda_dbg("sm = %d, sm_lt = %d, sm_xt = %d\n", sm, sm_lt, sm_xt);
regs[R_EP3] &= ~0xe0; /* clear sm, sm_lt, sm_xt */
- regs[R_EP3] |= sm ? (1 << 7) : 0 |
- sm_lt ? (1 << 6) : 0 |
- sm_xt ? (1 << 5) : 0;
+ regs[R_EP3] |= (sm ? (1 << 7) : 0) |
+ (sm_lt ? (1 << 6) : 0) |
+ (sm_xt ? (1 << 5) : 0);
return tda18271_write_regs(fe, R_EP3, 1);
}
Cheers,
Mauro
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
