drivers/media/dvb/frontends/stv0299.c : Use of the
Subsystems (2)
PATCH 1/1 drivers/media/dvb/frontends/stv0299.c : Use of the
Use of the time_after() macro, defined at linux/jiffies.h, which deal with wrapping correctly and are nicer to read.
Signed-off-by Marcelo Feitoza Parisi <[email protected]>
drivers/media/dvb/frontends/stv0299.c +3 -2
@@ -48,6 +48,7 @@#include <linux/moduleparam.h> #include <linux/string.h> #include <linux/slab.h> +#include <linux/jiffies.h> #include <asm/div64.h> #include "dvb_frontend.h"
@@ -190,7 +191,7 @@dprintk ("%s\n" , __FUNCTION__);while (stv0299_readreg (state,0x0a ) &1 ) { -if (jiffies - start > timeout) { +if (time_after (jiffies, start + timeout)) {dprintk ("%s: timeout!!\n" , __FUNCTION__);return -ETIMEDOUT ; }
@@ -207,7 +208,7 @@dprintk ("%s\n" , __FUNCTION__);while ((stv0299_readreg (state,0x0a ) &3 ) !=2 ) { -if (jiffies - start > timeout) { +if (time_after (jiffies, start + timeout)) {dprintk ("%s: timeout!!\n" , __FUNCTION__);return -ETIMEDOUT ; }
_______________________________________________
Kernel-janitors mailing list
[email protected]
https://lists.osdl.org/mailman/listinfo/kernel-janitors
PATCH 1/1 drivers/media/dvb/frontends/tda8083.c : Use of the
Use of the time_before() macro, defined at linux/jiffies.h, which deal with wrapping correctly and are nicer to read.
Signed-off-by Marcelo Feitoza Parisi <[email protected]>
drivers/media/dvb/frontends/tda8083.c +2 -1
@@ -30,6 +30,7 @@#include <linux/moduleparam.h> #include <linux/string.h> #include <linux/slab.h> +#include <linux/jiffies.h> #include "dvb_frontend.h" #include "tda8083.h"
@@ -172,7 +173,7 @@ {unsigned long start = jiffies; -while (jiffies - start < timeout && +while (time_before (jiffies, start + timeout) && !(tda8083_readreg (state,0x02 ) &0x80 )) {msleep (50 );
_______________________________________________
Kernel-janitors mailing list
[email protected]
https://lists.osdl.org/mailman/listinfo/kernel-janitors