media: vivid: use time_is_after_jiffies() instead of open coding it
authorWang Qing <wangqing@vivo.com>
Tue, 15 Feb 2022 01:55:50 +0000 (17:55 -0800)
committerHans Verkuil <hverkuil-cisco@xs4all.nl>
Tue, 22 Feb 2022 08:41:13 +0000 (09:41 +0100)
Use the helper function time_is_{before,after}_jiffies() to improve
code readability.

Signed-off-by: Wang Qing <wangqing@vivo.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
drivers/media/test-drivers/vivid/vivid-kthread-cap.c
drivers/media/test-drivers/vivid/vivid-kthread-out.c
drivers/media/test-drivers/vivid/vivid-kthread-touch.c
drivers/media/test-drivers/vivid/vivid-sdr-cap.c

index 6baa046c1ae390915d97ee2e1eb21a695085d60a..690daada7db4f60b9736bf8f74a88ec18bd833fb 100644 (file)
@@ -18,6 +18,7 @@
 #include <linux/freezer.h>
 #include <linux/random.h>
 #include <linux/v4l2-dv-timings.h>
+#include <linux/jiffies.h>
 #include <asm/div64.h>
 #include <media/videobuf2-vmalloc.h>
 #include <media/v4l2-dv-timings.h>
@@ -893,7 +894,7 @@ static int vivid_thread_vid_cap(void *data)
                        next_jiffies_since_start = jiffies_since_start;
 
                wait_jiffies = next_jiffies_since_start - jiffies_since_start;
-               while (jiffies - cur_jiffies < wait_jiffies &&
+               while (time_is_after_jiffies(cur_jiffies + wait_jiffies) &&
                       !kthread_should_stop())
                        schedule();
        }
index b6d43169e970b3bc5886308ac0bd6dc22062a53f..0833e021bb11dd724a6c5b5cbd8ad985919b7163 100644 (file)
@@ -18,6 +18,7 @@
 #include <linux/freezer.h>
 #include <linux/random.h>
 #include <linux/v4l2-dv-timings.h>
+#include <linux/jiffies.h>
 #include <asm/div64.h>
 #include <media/videobuf2-vmalloc.h>
 #include <media/v4l2-dv-timings.h>
@@ -234,7 +235,7 @@ static int vivid_thread_vid_out(void *data)
                        next_jiffies_since_start = jiffies_since_start;
 
                wait_jiffies = next_jiffies_since_start - jiffies_since_start;
-               while (jiffies - cur_jiffies < wait_jiffies &&
+               while (time_is_after_jiffies(cur_jiffies + wait_jiffies) &&
                       !kthread_should_stop())
                        schedule();
        }
index f065faae4988819a0b2b8e045a84b808f51e9b2f..fa711ee36a3fbca1bfe07a8806e241e134581e0b 100644 (file)
@@ -5,6 +5,7 @@
  */
 
 #include <linux/freezer.h>
+#include <linux/jiffies.h>
 #include "vivid-core.h"
 #include "vivid-kthread-touch.h"
 #include "vivid-touch-cap.h"
@@ -134,7 +135,7 @@ static int vivid_thread_touch_cap(void *data)
                        next_jiffies_since_start = jiffies_since_start;
 
                wait_jiffies = next_jiffies_since_start - jiffies_since_start;
-               while (jiffies - cur_jiffies < wait_jiffies &&
+               while (time_is_after_jiffies(cur_jiffies + wait_jiffies) &&
                       !kthread_should_stop())
                        schedule();
        }
index 59fd508e27c843d39ac2aeb8c9c7c3ee90d61b55..0ae5628b86c95b978b84a59b6883786733587193 100644 (file)
@@ -17,6 +17,7 @@
 #include <media/v4l2-event.h>
 #include <media/v4l2-dv-timings.h>
 #include <linux/fixp-arith.h>
+#include <linux/jiffies.h>
 
 #include "vivid-core.h"
 #include "vivid-ctrls.h"
@@ -205,7 +206,7 @@ static int vivid_thread_sdr_cap(void *data)
                        next_jiffies_since_start = jiffies_since_start;
 
                wait_jiffies = next_jiffies_since_start - jiffies_since_start;
-               while (jiffies - cur_jiffies < wait_jiffies &&
+               while (time_is_after_jiffies(cur_jiffies + wait_jiffies) &&
                       !kthread_should_stop())
                        schedule();
        }