git: 26439b57877c - main - e1000: Remove redundant EITR shift from igb

From: Kevin Bowling <kbowling_at_FreeBSD.org>
Date: Sat, 28 Sep 2024 03:36:11 UTC
The branch main has been updated by kbowling:

URL: https://cgit.FreeBSD.org/src/commit/?id=26439b57877ccae7a2404f7d179afaa5ddb2f64d

commit 26439b57877ccae7a2404f7d179afaa5ddb2f64d
Author:     Kevin Bowling <kbowling@FreeBSD.org>
AuthorDate: 2024-09-28 03:34:18 +0000
Commit:     Kevin Bowling <kbowling@FreeBSD.org>
CommitDate: 2024-09-28 03:36:00 +0000

    e1000: Remove redundant EITR shift from igb
    
    The E1000_EITR() macro is already multiplying by 0x4 which is the same
    as this shift, so we were shifting more than expected.
    
    MFC after:      6 days
    Sponsored by:   BBOX.io
---
 sys/dev/e1000/if_em.h | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/sys/dev/e1000/if_em.h b/sys/dev/e1000/if_em.h
index 7219dc57c333..83a0d4e96fc6 100644
--- a/sys/dev/e1000/if_em.h
+++ b/sys/dev/e1000/if_em.h
@@ -251,8 +251,7 @@
 #define IGB_ITR_DIVIDEND	1000000
 #define IGB_ITR_SHIFT		2
 #define IGB_QVECTOR_MASK	0x7FFC
-#define IGB_INTS_TO_EITR(i)	(((IGB_ITR_DIVIDEND/i) & IGB_QVECTOR_MASK) << \
-				    IGB_ITR_SHIFT)
+#define IGB_INTS_TO_EITR(i)	((IGB_ITR_DIVIDEND/i) & IGB_QVECTOR_MASK)
 
 #define IGB_LINK_ITR		2000
 #define I210_LINK_DELAY		1000