public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
* [patch 1/2] [media] ds3000: using logical && instead of bitwise &
@ 2012-01-17  7:28 Dan Carpenter
  0 siblings, 0 replies; only message in thread
From: Dan Carpenter @ 2012-01-17  7:28 UTC (permalink / raw)
  To: Mauro Carvalho Chehab; +Cc: Igor M. Liplianin, linux-media, kernel-janitors

The intent here was to test if the FE_HAS_LOCK was set.  The current
test is equivalent to "if (status) { ..."

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>

diff --git a/drivers/media/dvb/frontends/ds3000.c b/drivers/media/dvb/frontends/ds3000.c
index 9387770..af65d01 100644
--- a/drivers/media/dvb/frontends/ds3000.c
+++ b/drivers/media/dvb/frontends/ds3000.c
@@ -1195,7 +1195,7 @@ static int ds3000_set_frontend(struct dvb_frontend *fe)
 
 	for (i = 0; i < 30 ; i++) {
 		ds3000_read_status(fe, &status);
-		if (status && FE_HAS_LOCK)
+		if (status & FE_HAS_LOCK)
 			break;
 
 		msleep(10);

^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2012-01-17  7:28 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-01-17  7:28 [patch 1/2] [media] ds3000: using logical && instead of bitwise & Dan Carpenter

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox