Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 31 Dec 2011 19:55:19 +0000 (UTC)
From:      Matthew D Fleming <mdf@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-8@freebsd.org
Subject:   svn commit: r229147 - stable/8/sys/dev/e1000
Message-ID:  <201112311955.pBVJtJDp080426@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: mdf
Date: Sat Dec 31 19:55:19 2011
New Revision: 229147
URL: http://svn.freebsd.org/changeset/base/229147

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/8/sys/dev/e1000/e1000_hw.h
  stable/8/sys/dev/e1000/e1000_osdep.h
  stable/8/sys/dev/e1000/if_igb.c
Directory Properties:
  stable/8/sys/   (props changed)

Modified: stable/8/sys/dev/e1000/e1000_hw.h
==============================================================================
--- stable/8/sys/dev/e1000/e1000_hw.h	Sat Dec 31 19:53:00 2011	(r229146)
+++ stable/8/sys/dev/e1000/e1000_hw.h	Sat Dec 31 19:55:19 2011	(r229147)
@@ -889,13 +889,13 @@ 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 {
 	bool sgmii_active;
 	bool global_device_reset;
-	bool eee_disable;
+	int eee_disable;
 };
 
 struct e1000_dev_spec_vf {

Modified: stable/8/sys/dev/e1000/e1000_osdep.h
==============================================================================
--- stable/8/sys/dev/e1000/e1000_osdep.h	Sat Dec 31 19:53:00 2011	(r229146)
+++ stable/8/sys/dev/e1000/e1000_osdep.h	Sat Dec 31 19:55:19 2011	(r229147)
@@ -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/8/sys/dev/e1000/if_igb.c
==============================================================================
--- stable/8/sys/dev/e1000/if_igb.c	Sat Dec 31 19:53:00 2011	(r229146)
+++ stable/8/sys/dev/e1000/if_igb.c	Sat Dec 31 19:55:19 2011	(r229147)
@@ -224,7 +224,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 *);
@@ -334,7 +334,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);
 
 /*
@@ -3361,7 +3361,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;



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201112311955.pBVJtJDp080426>