[PATCH] ethernet: micrel: use time_after_eq macro

Subsystems: networking drivers, the rest

STALE4079d

3 messages, 3 authors, 2015-06-05 · open the first message on its own page

[PATCH] ethernet: micrel: use time_after_eq macro

From: Antonio Murdaca <hidden>
Date: 2015-06-05 14:41:28

use time_after_eq macro for time comparison

Signed-off-by: Antonio Murdaca <redacted>
---
 drivers/net/ethernet/micrel/ksz884x.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/micrel/ksz884x.c b/drivers/net/ethernet/micrel/ksz884x.c
index 48d2aec..9bce30c 100644
--- a/drivers/net/ethernet/micrel/ksz884x.c
+++ b/drivers/net/ethernet/micrel/ksz884x.c
@@ -6689,7 +6689,7 @@ static void mib_monitor(unsigned long ptr)
 
 	/* This is used to verify Wake-on-LAN is working. */
 	if (hw_priv->pme_wait) {
-		if (hw_priv->pme_wait <= jiffies) {
+		if (time_after_eq(jiffies, hw_priv->pme_wait)) {
 			hw_clr_wol_pme_status(&hw_priv->hw);
 			hw_priv->pme_wait = 0;
 		}
-- 
2.4.2

Re: [PATCH] ethernet: micrel: use time_after_eq macro

From: Julia Lawall <hidden>
Date: 2015-06-05 15:46:00

On Fri, 5 Jun 2015, Antonio Murdaca wrote:
quoted hunk
use time_after_eq macro for time comparison

Signed-off-by: Antonio Murdaca <redacted>
---
 drivers/net/ethernet/micrel/ksz884x.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/micrel/ksz884x.c b/drivers/net/ethernet/micrel/ksz884x.c
index 48d2aec..9bce30c 100644
--- a/drivers/net/ethernet/micrel/ksz884x.c
+++ b/drivers/net/ethernet/micrel/ksz884x.c
@@ -6689,7 +6689,7 @@ static void mib_monitor(unsigned long ptr)

 	/* This is used to verify Wake-on-LAN is working. */
 	if (hw_priv->pme_wait) {
-		if (hw_priv->pme_wait <= jiffies) {
+		if (time_after_eq(jiffies, hw_priv->pme_wait)) {
There is also time_is_before_eq_jiffies.

julia
 			hw_clr_wol_pme_status(&hw_priv->hw);
 			hw_priv->pme_wait = 0;
 		}
--
2.4.2

Re: [PATCH] ethernet: micrel: use time_after_eq macro

From: Antonio Murdaca <hidden>
Date: 2015-06-05 16:54:19

I'll use that then and resubmit this patch

Thanks

2015-06-05 17:45 GMT+02:00 Julia Lawall [off-list ref]:
On Fri, 5 Jun 2015, Antonio Murdaca wrote:
quoted
use time_after_eq macro for time comparison

Signed-off-by: Antonio Murdaca <redacted>
---
 drivers/net/ethernet/micrel/ksz884x.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/micrel/ksz884x.c b/drivers/net/ethernet/micrel/ksz884x.c
index 48d2aec..9bce30c 100644
--- a/drivers/net/ethernet/micrel/ksz884x.c
+++ b/drivers/net/ethernet/micrel/ksz884x.c
@@ -6689,7 +6689,7 @@ static void mib_monitor(unsigned long ptr)

      /* This is used to verify Wake-on-LAN is working. */
      if (hw_priv->pme_wait) {
-             if (hw_priv->pme_wait <= jiffies) {
+             if (time_after_eq(jiffies, hw_priv->pme_wait)) {
There is also time_is_before_eq_jiffies.

julia
quoted
                      hw_clr_wol_pme_status(&hw_priv->hw);
                      hw_priv->pme_wait = 0;
              }
--
2.4.2


-- 
Antonio Murdaca
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help