# \[PATCH\] video: fbdev: omap2: omapfb: displays: panel-dsi-cm: Use time comparison kernel macro.

- Subsystems: FRAMEBUFFER LAYER, OMAP DISPLAY SUBSYSTEM and FRAMEBUFFER SUPPORT (DSS2), THE REST
- Maintainers: Helge Deller, Linus Torvalds
- List: linux-fbdev
- 2 messages, 2 authors
- Page: https://ratatoskr.run/linux-fbdev/2017/05/4996605/t

## \[PATCH\] video: fbdev: omap2: omapfb: displays: panel-dsi-cm: Use time comparison kernel macro.

- From: Karim Eshapa \<hidden\>
- Date: 2017-05-15 19:14:44 
- Message: https://ratatoskr.run/linux-fbdev/2017/05/4996605.md

```text
Use time_before_eq() kernel macro for
time comparison more safe.

Signed-off-by: Karim Eshapa <redacted>
---
 drivers/video/fbdev/omap2/omapfb/displays/panel-dsi-cm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/video/fbdev/omap2/omapfb/displays/panel-dsi-cm.c b/drivers/video/fbdev/omap2/omapfb/displays/panel-dsi-cm.c
index fd2b372d..7f20bc2 100644
--- a/drivers/video/fbdev/omap2/omapfb/displays/panel-dsi-cm.c
+++ b/drivers/video/fbdev/omap2/omapfb/displays/panel-dsi-cm.c
@@ -100,7 +100,7 @@ static void hw_guard_wait(struct panel_drv_data *ddata)
 {
 	unsigned long wait = ddata->hw_guard_end - jiffies;
 
-	if ((long)wait > 0 && wait <= ddata->hw_guard_wait) {
+	if ((long)wait > 0 && time_before_eq(wait, ddata->hw_guard_wait)) {
 		set_current_state(TASK_UNINTERRUPTIBLE);
 		schedule_timeout(wait);
 	}
-- 
2.7.4

```

## Re: \[PATCH\] video: fbdev: omap2: omapfb: displays: panel-dsi-cm: Use time comparison kernel macro.

- From: Bartlomiej Zolnierkiewicz \<hidden\>
- Date: 2017-06-14 13:48:29 
- Message: https://ratatoskr.run/linux-fbdev/2017/06/5020833.md

```text
On Monday, May 15, 2017 09:14:26 PM Karim Eshapa wrote:
> Use time_before_eq() kernel macro for
> time comparison more safe.
> 
> Signed-off-by: Karim Eshapa <redacted>

Patch queued for 4.13, thanks.

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics

```