Skip to contentSkip to conversation

stv0900: remove an unneeded check

PATCH In review Dan Carpenter 2 participants 2 messages Feb 6, 2014 – Feb 20, 2014

Subsystems (2)
Media (V4L/DVB) Kernel janitors
lore
Share & tools
lore

Dan Carpenter

author
#0 lore raw

PATCH 1/1 stv0900: remove an unneeded check

No need to check "lock" twice here.

Signed-off-by Dan Carpenter <[email protected]>
drivers/media/dvb-frontends/stv0900_sw.c +1 -1
index 0a40edfad739..4ce1d260b3eb 100644
@@ -1081,7 +1081,7 @@ static int stv0900_wait_for_lock(struct stv0900_internal *intp,
 	lock = stv0900_get_demod_lock(intp, demod, dmd_timeout);
 
 	if (lock)
-		lock = lock && stv0900_get_fec_lock(intp, demod, fec_timeout);
+		lock = stv0900_get_fec_lock(intp, demod, fec_timeout);
 
 	if (lock) {
 		lock = 0;
Acked-by on v1 1/1
#1 lore raw
Quoted 3 lines · Dan Carpenter

No need to check "lock" twice here.

Signed-off-by Dan Carpenter <[email protected]>
Acked-by Hans Verkuil <[email protected]>

Regards,

Hans

Quoted 19 lines
drivers/media/dvb-frontends/stv0900_sw.c +1 -2
index 0a40edfad739..4ce1d260b3eb 100644
@@ -1081,7 +1081,7 @@ static int stv0900_wait_for_lock(struct stv0900_internal *intp,
 	lock = stv0900_get_demod_lock(intp, demod, dmd_timeout);
 
 	if (lock)
-		lock = lock && stv0900_get_fec_lock(intp, demod, fec_timeout);
+		lock = stv0900_get_fec_lock(intp, demod, fec_timeout);
 
 	if (lock) {
 		lock = 0;
--

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