svn commit: r229150 - stable/7/sys/dev/e1000
Matthew D Fleming
mdf at FreeBSD.org
Sat Dec 31 20:01:10 UTC 2011
Author: mdf
Date: Sat Dec 31 20:01:09 2011
New Revision: 229150
URL: http://svn.freebsd.org/changeset/base/229150
Log:
MFC r228441:
Consistently use types in e1000 driver code:
- Two struct members eee_disable are used in a function that expects
an int *, so declare them int, not bool.
- igb_tx_ctx_setup() returns a boolean value, so declare it bool, not int.
- igb_header_split is passed to TUNABLE_INT, so delcare it int, not bool.
- igb_tso_setup() returns a bool, so declare it bool, not boolean_t.
- Do not re-define bool/true/false if the symbols already exist.
Modified:
stable/7/sys/dev/e1000/e1000_hw.h
stable/7/sys/dev/e1000/e1000_osdep.h
stable/7/sys/dev/e1000/if_igb.c
Directory Properties:
stable/7/sys/ (props changed)
Modified: stable/7/sys/dev/e1000/e1000_hw.h
==============================================================================
--- stable/7/sys/dev/e1000/e1000_hw.h Sat Dec 31 19:58:25 2011 (r229149)
+++ stable/7/sys/dev/e1000/e1000_hw.h Sat Dec 31 20:01:09 2011 (r229150)
@@ -880,7 +880,7 @@ struct e1000_dev_spec_ich8lan {
E1000_MUTEX nvm_mutex;
E1000_MUTEX swflag_mutex;
bool nvm_k1_enabled;
- bool eee_disable;
+ int eee_disable;
};
struct e1000_dev_spec_82575 {
Modified: stable/7/sys/dev/e1000/e1000_osdep.h
==============================================================================
--- stable/7/sys/dev/e1000/e1000_osdep.h Sat Dec 31 19:58:25 2011 (r229149)
+++ stable/7/sys/dev/e1000/e1000_osdep.h Sat Dec 31 20:01:09 2011 (r229150)
@@ -73,9 +73,11 @@
#define STATIC static
#define FALSE 0
-#define false FALSE
#define TRUE 1
+#ifndef __bool_true_false_are_defined
+#define false FALSE
#define true TRUE
+#endif
#define CMD_MEM_WRT_INVALIDATE 0x0010 /* BIT_4 */
#define PCI_COMMAND_REGISTER PCIR_COMMAND
@@ -96,7 +98,9 @@ typedef int64_t s64;
typedef int32_t s32;
typedef int16_t s16;
typedef int8_t s8;
+#ifndef __bool_true_false_are_defined
typedef boolean_t bool;
+#endif
#define __le16 u16
#define __le32 u32
Modified: stable/7/sys/dev/e1000/if_igb.c
==============================================================================
--- stable/7/sys/dev/e1000/if_igb.c Sat Dec 31 19:58:25 2011 (r229149)
+++ stable/7/sys/dev/e1000/if_igb.c Sat Dec 31 20:01:09 2011 (r229150)
@@ -205,7 +205,7 @@ static __inline void igb_rx_input(struct
static bool igb_rxeof(struct igb_queue *, int, int *);
static void igb_rx_checksum(u32, struct mbuf *, u32);
-static int igb_tx_ctx_setup(struct tx_ring *, struct mbuf *);
+static bool igb_tx_ctx_setup(struct tx_ring *, struct mbuf *);
static bool igb_tso_setup(struct tx_ring *, struct mbuf *, u32 *);
static void igb_set_promisc(struct adapter *);
static void igb_disable_promisc(struct adapter *);
@@ -315,7 +315,7 @@ TUNABLE_INT("hw.igb.max_interrupt_rate",
** into the header and thus use no cluster. Its
** a very workload dependent type feature.
*/
-static bool igb_header_split = FALSE;
+static int igb_header_split = FALSE;
TUNABLE_INT("hw.igb.hdr_split", &igb_header_split);
/*
@@ -3107,7 +3107,7 @@ igb_free_transmit_buffers(struct tx_ring
* Setup work for hardware segmentation offload (TSO)
*
**********************************************************************/
-static boolean_t
+static bool
igb_tso_setup(struct tx_ring *txr, struct mbuf *mp, u32 *hdrlen)
{
struct adapter *adapter = txr->adapter;
More information about the svn-src-stable-7
mailing list