state-slave: avoid some redundant calculation

Please note that timeout_init is not resetting timeouts, it's just saving the values for later settings. Signed-off-by: Alessandro Rubini <rubini@gnudd.com>

state-slave: avoid some redundant calculation
Please note that timeout_init is not resetting timeouts, it's just saving the values for later settings. Signed-off-by: Alessandro Rubini <rubini@gnudd.com>
a16a3460 · Alessandro Rubini · 9903624c · a16a3460
Commit a16a3460 authored Sep 05, 2016 by Alessandro Rubini
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 3 deletions

state-slave.c proto-standard/state-slave.c +7 -3

No files found.
--- a/proto-standard/state-slave.c
+++ b/proto-standard/state-slave.c
@@ -81,9 +81,13 @@ int pp_slave(struct pp_instance *ppi, unsigned char *pkt, int plen)
 			if (e)
 				goto out;

-			DSPOR(ppi)->logMinDelayReqInterval =
-				hdr->logMessageInterval;
-			pp_timeout_init(ppi); /* new value for logMin */
+			if (DSPOR(ppi)->logMinDelayReqInterval !=
+			    hdr->logMessageInterval) {
+				DSPOR(ppi)->logMinDelayReqInterval =
+					hdr->logMessageInterval;
+				/* new value for logMin */
+				pp_timeout_init(ppi);
+			}
 		} else {
 			pp_diag(ppi, frames, 2, "pp_slave : "
 			     "Delay Resp doesn't match Delay Req\n");