[rtems commit] bsp/beatnik: Fix warnings

Sebastian Huber sebh at rtems.org
Fri Nov 9 14:08:27 UTC 2018


Module:    rtems
Branch:    master
Commit:    efdb4a767aeb0f47eb288b8a7abcf51808afa7a8
Changeset: http://git.rtems.org/rtems/commit/?id=efdb4a767aeb0f47eb288b8a7abcf51808afa7a8

Author:    Sebastian Huber <sebastian.huber at embedded-brains.de>
Date:      Fri Nov  9 09:37:53 2018 +0100

bsp/beatnik: Fix warnings

---

 bsps/powerpc/beatnik/include/bsp.h             |   3 +
 bsps/powerpc/beatnik/include/bsp/if_mve_pub.h  |   4 +
 bsps/powerpc/beatnik/net/if_em/if_em_hw.c      | 174 +++++++++++--------------
 bsps/powerpc/beatnik/net/if_gfe/if_gfe.c       |   6 +
 bsps/powerpc/beatnik/net/porting/rtemscompat.h |   2 -
 bsps/powerpc/shared/start/pgtbl_setup.c        |   1 +
 bsps/shared/net/if_dc.c                        |   7 +-
 7 files changed, 94 insertions(+), 103 deletions(-)

diff --git a/bsps/powerpc/beatnik/include/bsp.h b/bsps/powerpc/beatnik/include/bsp.h
index 4a475eb..fa29d37 100644
--- a/bsps/powerpc/beatnik/include/bsp.h
+++ b/bsps/powerpc/beatnik/include/bsp.h
@@ -157,6 +157,9 @@ int BSP_i2c_initialize(void);
 /* Networking; */
 #if defined(RTEMS_NETWORKING)
 #include <bsp/bsp_bsdnet_attach.h>
+int rtems_em_attach(struct rtems_bsdnet_ifconfig *, int);
+int rtems_dec21140_driver_attach(struct rtems_bsdnet_ifconfig *, int);
+int rtems_dc_driver_attach(struct rtems_bsdnet_ifconfig *, int);
 #endif
 
 /* NOT FOR PUBLIC USE BELOW HERE */
diff --git a/bsps/powerpc/beatnik/include/bsp/if_mve_pub.h b/bsps/powerpc/beatnik/include/bsp/if_mve_pub.h
index 0a84310..46cc4cc 100644
--- a/bsps/powerpc/beatnik/include/bsp/if_mve_pub.h
+++ b/bsps/powerpc/beatnik/include/bsp/if_mve_pub.h
@@ -243,6 +243,10 @@ BSP_mve_stop_hw(struct mveth_private *mp);
 int
 BSP_mve_detach(struct mveth_private *mp);
 
+int
+BSP_mve_send_buf_raw(struct mveth_private *mp, void *head_p, int h_len,
+    void *data_p, int d_len);
+
 /*
  * Enqueue a mbuf chain or a raw data buffer for transmission;
  * RETURN: #bytes sent or -1 if there are not enough free descriptors
diff --git a/bsps/powerpc/beatnik/net/if_em/if_em_hw.c b/bsps/powerpc/beatnik/net/if_em/if_em_hw.c
index 0dcf349..e200a6c 100644
--- a/bsps/powerpc/beatnik/net/if_em/if_em_hw.c
+++ b/bsps/powerpc/beatnik/net/if_em/if_em_hw.c
@@ -95,18 +95,6 @@ uint16_t em_igp_cable_length_table[IGP01E1000_AGC_LENGTH_TABLE_SIZE] =
       100, 100, 100, 100, 110, 110, 110, 110, 110, 110, 110, 110, 110, 110, 110, 110,
       110, 110, 110, 110, 110, 110, 120, 120, 120, 120, 120, 120, 120, 120, 120, 120};
 
-static const
-uint16_t em_igp_2_cable_length_table[IGP02E1000_AGC_LENGTH_TABLE_SIZE] =
-    { 8, 13, 17, 19, 21, 23, 25, 27, 29, 31, 33, 35, 37, 39, 41, 43,
-      22, 24, 27, 30, 32, 35, 37, 40, 42, 44, 47, 49, 51, 54, 56, 58,
-      32, 35, 38, 41, 44, 47, 50, 53, 55, 58, 61, 63, 66, 69, 71, 74,
-      43, 47, 51, 54, 58, 61, 64, 67, 71, 74, 77, 80, 82, 85, 88, 90,
-      57, 62, 66, 70, 74, 77, 81, 85, 88, 91, 94, 97, 100, 103, 106, 108,
-      73, 78, 82, 87, 91, 95, 98, 102, 105, 109, 112, 114, 117, 119, 122, 124,
-      91, 96, 101, 105, 109, 113, 116, 119, 122, 125, 127, 128, 128, 128, 128, 128,
-      108, 113, 117, 121, 124, 127, 128, 128, 128, 128, 128, 128, 128, 128, 128, 128};
-
-
 /******************************************************************************
  * Set the phy type member in the hw struct.
  *
@@ -153,7 +141,6 @@ em_set_phy_type(struct em_hw *hw)
 static void
 em_phy_init_script(struct em_hw *hw)
 {
-    uint32_t ret_val;
     uint16_t phy_saved_data;
 
     DEBUGFUNC("em_phy_init_script");
@@ -163,7 +150,7 @@ em_phy_init_script(struct em_hw *hw)
 
         /* Save off the current value of register 0x2F5B to be restored at
          * the end of this routine. */
-        ret_val = em_read_phy_reg(hw, 0x2F5B, &phy_saved_data);
+        em_read_phy_reg(hw, 0x2F5B, &phy_saved_data);
 
         /* Disabled the PHY transmitter */
         em_write_phy_reg(hw, 0x2F5B, 0x0003);
@@ -396,7 +383,6 @@ em_reset_hw(struct em_hw *hw)
 {
     uint32_t ctrl;
     uint32_t ctrl_ext;
-    uint32_t icr;
     uint32_t manc;
     uint32_t led_ctrl;
     uint32_t timeout;
@@ -565,7 +551,7 @@ em_reset_hw(struct em_hw *hw)
     E1000_WRITE_REG(hw, IMC, 0xffffffff);
 
     /* Clear any pending interrupt events. */
-    icr = E1000_READ_REG(hw, ICR);
+    E1000_READ_REG(hw, ICR);
 
     /* If MWI was previously enabled, reenable it. */
     if(hw->mac_type == em_82542_rev2_0) {
@@ -1273,9 +1259,9 @@ em_copper_link_mgp_setup(struct em_hw *hw)
     phy_data &= ~M88E1000_PSCR_POLARITY_REVERSAL;
     if(hw->disable_polarity_correction == 1)
         phy_data |= M88E1000_PSCR_POLARITY_REVERSAL;
-        ret_val = em_write_phy_reg(hw, M88E1000_PHY_SPEC_CTRL, phy_data);
-        if(ret_val)
-            return ret_val;
+    ret_val = em_write_phy_reg(hw, M88E1000_PHY_SPEC_CTRL, phy_data);
+    if(ret_val)
+        return ret_val;
 
     /* Force TX_CLK in the Extended PHY Specific Control Register
      * to 25MHz clock.
@@ -1507,10 +1493,10 @@ em_phy_setup_autoneg(struct em_hw *hw)
     if(ret_val)
         return ret_val;
 
-        /* Read the MII 1000Base-T Control Register (Address 9). */
-        ret_val = em_read_phy_reg(hw, PHY_1000T_CTRL, &mii_1000t_ctrl_reg);
-        if(ret_val)
-            return ret_val;
+    /* Read the MII 1000Base-T Control Register (Address 9). */
+    ret_val = em_read_phy_reg(hw, PHY_1000T_CTRL, &mii_1000t_ctrl_reg);
+    if(ret_val)
+        return ret_val;
 
     /* Need to parse both autoneg_advertised and fc and set up
      * the appropriate PHY registers.  First we will parse for
@@ -4918,84 +4904,82 @@ em_led_off(struct em_hw *hw)
 void
 em_clear_hw_cntrs(struct em_hw *hw)
 {
-    volatile uint32_t temp;
-
-    temp = E1000_READ_REG(hw, CRCERRS);
-    temp = E1000_READ_REG(hw, SYMERRS);
-    temp = E1000_READ_REG(hw, MPC);
-    temp = E1000_READ_REG(hw, SCC);
-    temp = E1000_READ_REG(hw, ECOL);
-    temp = E1000_READ_REG(hw, MCC);
-    temp = E1000_READ_REG(hw, LATECOL);
-    temp = E1000_READ_REG(hw, COLC);
-    temp = E1000_READ_REG(hw, DC);
-    temp = E1000_READ_REG(hw, SEC);
-    temp = E1000_READ_REG(hw, RLEC);
-    temp = E1000_READ_REG(hw, XONRXC);
-    temp = E1000_READ_REG(hw, XONTXC);
-    temp = E1000_READ_REG(hw, XOFFRXC);
-    temp = E1000_READ_REG(hw, XOFFTXC);
-    temp = E1000_READ_REG(hw, FCRUC);
-    temp = E1000_READ_REG(hw, PRC64);
-    temp = E1000_READ_REG(hw, PRC127);
-    temp = E1000_READ_REG(hw, PRC255);
-    temp = E1000_READ_REG(hw, PRC511);
-    temp = E1000_READ_REG(hw, PRC1023);
-    temp = E1000_READ_REG(hw, PRC1522);
-    temp = E1000_READ_REG(hw, GPRC);
-    temp = E1000_READ_REG(hw, BPRC);
-    temp = E1000_READ_REG(hw, MPRC);
-    temp = E1000_READ_REG(hw, GPTC);
-    temp = E1000_READ_REG(hw, GORCL);
-    temp = E1000_READ_REG(hw, GORCH);
-    temp = E1000_READ_REG(hw, GOTCL);
-    temp = E1000_READ_REG(hw, GOTCH);
-    temp = E1000_READ_REG(hw, RNBC);
-    temp = E1000_READ_REG(hw, RUC);
-    temp = E1000_READ_REG(hw, RFC);
-    temp = E1000_READ_REG(hw, ROC);
-    temp = E1000_READ_REG(hw, RJC);
-    temp = E1000_READ_REG(hw, TORL);
-    temp = E1000_READ_REG(hw, TORH);
-    temp = E1000_READ_REG(hw, TOTL);
-    temp = E1000_READ_REG(hw, TOTH);
-    temp = E1000_READ_REG(hw, TPR);
-    temp = E1000_READ_REG(hw, TPT);
-    temp = E1000_READ_REG(hw, PTC64);
-    temp = E1000_READ_REG(hw, PTC127);
-    temp = E1000_READ_REG(hw, PTC255);
-    temp = E1000_READ_REG(hw, PTC511);
-    temp = E1000_READ_REG(hw, PTC1023);
-    temp = E1000_READ_REG(hw, PTC1522);
-    temp = E1000_READ_REG(hw, MPTC);
-    temp = E1000_READ_REG(hw, BPTC);
+    E1000_READ_REG(hw, CRCERRS);
+    E1000_READ_REG(hw, SYMERRS);
+    E1000_READ_REG(hw, MPC);
+    E1000_READ_REG(hw, SCC);
+    E1000_READ_REG(hw, ECOL);
+    E1000_READ_REG(hw, MCC);
+    E1000_READ_REG(hw, LATECOL);
+    E1000_READ_REG(hw, COLC);
+    E1000_READ_REG(hw, DC);
+    E1000_READ_REG(hw, SEC);
+    E1000_READ_REG(hw, RLEC);
+    E1000_READ_REG(hw, XONRXC);
+    E1000_READ_REG(hw, XONTXC);
+    E1000_READ_REG(hw, XOFFRXC);
+    E1000_READ_REG(hw, XOFFTXC);
+    E1000_READ_REG(hw, FCRUC);
+    E1000_READ_REG(hw, PRC64);
+    E1000_READ_REG(hw, PRC127);
+    E1000_READ_REG(hw, PRC255);
+    E1000_READ_REG(hw, PRC511);
+    E1000_READ_REG(hw, PRC1023);
+    E1000_READ_REG(hw, PRC1522);
+    E1000_READ_REG(hw, GPRC);
+    E1000_READ_REG(hw, BPRC);
+    E1000_READ_REG(hw, MPRC);
+    E1000_READ_REG(hw, GPTC);
+    E1000_READ_REG(hw, GORCL);
+    E1000_READ_REG(hw, GORCH);
+    E1000_READ_REG(hw, GOTCL);
+    E1000_READ_REG(hw, GOTCH);
+    E1000_READ_REG(hw, RNBC);
+    E1000_READ_REG(hw, RUC);
+    E1000_READ_REG(hw, RFC);
+    E1000_READ_REG(hw, ROC);
+    E1000_READ_REG(hw, RJC);
+    E1000_READ_REG(hw, TORL);
+    E1000_READ_REG(hw, TORH);
+    E1000_READ_REG(hw, TOTL);
+    E1000_READ_REG(hw, TOTH);
+    E1000_READ_REG(hw, TPR);
+    E1000_READ_REG(hw, TPT);
+    E1000_READ_REG(hw, PTC64);
+    E1000_READ_REG(hw, PTC127);
+    E1000_READ_REG(hw, PTC255);
+    E1000_READ_REG(hw, PTC511);
+    E1000_READ_REG(hw, PTC1023);
+    E1000_READ_REG(hw, PTC1522);
+    E1000_READ_REG(hw, MPTC);
+    E1000_READ_REG(hw, BPTC);
 
     if(hw->mac_type < em_82543) return;
 
-    temp = E1000_READ_REG(hw, ALGNERRC);
-    temp = E1000_READ_REG(hw, RXERRC);
-    temp = E1000_READ_REG(hw, TNCRS);
-    temp = E1000_READ_REG(hw, CEXTERR);
-    temp = E1000_READ_REG(hw, TSCTC);
-    temp = E1000_READ_REG(hw, TSCTFC);
+    E1000_READ_REG(hw, ALGNERRC);
+    E1000_READ_REG(hw, RXERRC);
+    E1000_READ_REG(hw, TNCRS);
+    E1000_READ_REG(hw, CEXTERR);
+    E1000_READ_REG(hw, TSCTC);
+    E1000_READ_REG(hw, TSCTFC);
 
     if(hw->mac_type <= em_82544) return;
 
-    temp = E1000_READ_REG(hw, MGTPRC);
-    temp = E1000_READ_REG(hw, MGTPDC);
-    temp = E1000_READ_REG(hw, MGTPTC);
+    E1000_READ_REG(hw, MGTPRC);
+    E1000_READ_REG(hw, MGTPDC);
+    E1000_READ_REG(hw, MGTPTC);
 
     if(hw->mac_type <= em_82547_rev_2) return;
 
-    temp = E1000_READ_REG(hw, IAC);
-    temp = E1000_READ_REG(hw, ICRXOC);
-    temp = E1000_READ_REG(hw, ICRXPTC);
-    temp = E1000_READ_REG(hw, ICRXATC);
-    temp = E1000_READ_REG(hw, ICTXPTC);
-    temp = E1000_READ_REG(hw, ICTXATC);
-    temp = E1000_READ_REG(hw, ICTXQEC);
-    temp = E1000_READ_REG(hw, ICTXQMTC);
-    temp = E1000_READ_REG(hw, ICRXDMTC);
+    E1000_READ_REG(hw, IAC);
+    E1000_READ_REG(hw, ICRXOC);
+    E1000_READ_REG(hw, ICRXPTC);
+    E1000_READ_REG(hw, ICRXATC);
+    E1000_READ_REG(hw, ICTXPTC);
+    E1000_READ_REG(hw, ICTXATC);
+    E1000_READ_REG(hw, ICTXQEC);
+    E1000_READ_REG(hw, ICTXQMTC);
+    E1000_READ_REG(hw, ICRXDMTC);
 
 }
 
@@ -5832,9 +5816,9 @@ em_set_d0_lplu_state(struct em_hw *hw,
     if(hw->mac_type <= em_82547_rev_2)
         return E1000_SUCCESS;
 
-        ret_val = em_read_phy_reg(hw, IGP02E1000_PHY_POWER_MGMT, &phy_data);
-        if(ret_val)
-            return ret_val;
+    ret_val = em_read_phy_reg(hw, IGP02E1000_PHY_POWER_MGMT, &phy_data);
+    if(ret_val)
+        return ret_val;
 
     if (!active) {
             phy_data &= ~IGP02E1000_PM_D0_LPLU;
diff --git a/bsps/powerpc/beatnik/net/if_gfe/if_gfe.c b/bsps/powerpc/beatnik/net/if_gfe/if_gfe.c
index e642636..7b29717 100644
--- a/bsps/powerpc/beatnik/net/if_gfe/if_gfe.c
+++ b/bsps/powerpc/beatnik/net/if_gfe/if_gfe.c
@@ -229,6 +229,8 @@ STATIC void gfe_mii_statchg (struct device *);
 STATIC void gfe_tick(void *arg);
 
 STATIC void gfe_tx_restart(void *);
+STATIC void gfe_assign_desc(volatile struct gt_eth_desc *, struct mbuf *,
+	uint32_t);
 STATIC int gfe_tx_enqueue(struct gfe_softc *, enum gfe_txprio);
 STATIC uint32_t gfe_tx_done(struct gfe_softc *, enum gfe_txprio, uint32_t);
 STATIC void gfe_tx_cleanup(struct gfe_softc *, enum gfe_txprio, int);
@@ -2395,7 +2397,9 @@ gfe_hash_entry_op(struct gfe_softc *sc, enum gfe_hash_op op,
 	uint64_t *maybe_he_p = NULL;
 	int limit;
 	int hash;
+#ifndef __rtems__
 	int maybe_hash = 0;
+#endif /* __rtems__ */
 
 	GE_FUNC_ENTER(sc, "gfe_hash_entry_op");
 
@@ -2467,7 +2471,9 @@ gfe_hash_entry_op(struct gfe_softc *sc, enum gfe_hash_op op,
 		 */
 		if (maybe_he_p == NULL && (thishe & HSH_S)) {
 			maybe_he_p = he_p;
+#ifndef __rtems__
 			maybe_hash = hash;
+#endif /* __rtems__ */
 		}
 
 		hash = (hash + 1) & (sc->sc_hashmask / sizeof(he));
diff --git a/bsps/powerpc/beatnik/net/porting/rtemscompat.h b/bsps/powerpc/beatnik/net/porting/rtemscompat.h
index 57d4ffb..5a4bb6b 100644
--- a/bsps/powerpc/beatnik/net/porting/rtemscompat.h
+++ b/bsps/powerpc/beatnik/net/porting/rtemscompat.h
@@ -255,8 +255,6 @@ register u_int32_t rval;
 #define __FBSDID(arg)
 #endif
 
-#define _KERNEL
-
 #define device_printf(device,format,args...) printk(format,## args)
 
 static inline u_int8_t bus_space_do_read_1(u_long handle, unsigned reg)
diff --git a/bsps/powerpc/shared/start/pgtbl_setup.c b/bsps/powerpc/shared/start/pgtbl_setup.c
index 8a8407b..f2543f5 100644
--- a/bsps/powerpc/shared/start/pgtbl_setup.c
+++ b/bsps/powerpc/shared/start/pgtbl_setup.c
@@ -1,3 +1,4 @@
+#include <sys/param.h>
 #include <rtems.h>
 #include <libcpu/mmu.h>
 #include <rtems/bspIo.h>
diff --git a/bsps/shared/net/if_dc.c b/bsps/shared/net/if_dc.c
index 69e8494..0308d68 100644
--- a/bsps/shared/net/if_dc.c
+++ b/bsps/shared/net/if_dc.c
@@ -1913,7 +1913,7 @@ rtems_dc_driver_attach(struct rtems_bsdnet_ifconfig *config, int attaching)
 	struct ifnet		*ifp;
 	struct dc_type		*t;
 	uint32_t		revision;
-	int			error = 0, mac_offset;
+	int			mac_offset;
 	uint32_t		value;
 
 	/*
@@ -1976,13 +1976,11 @@ rtems_dc_driver_attach(struct rtems_bsdnet_ifconfig *config, int attaching)
 #ifdef DC_USEIOSPACE
 	if (!(command & PCI_COMMAND_IO)) {
 		printk("dc%d: failed to enable I/O ports!\n", sc->dc_unit);
-		error = ENXIO;
 		goto fail;
 	}
 #else
 	if (!(command & PCI_COMMAND_MEMORY)) {
 		printk("dc%d: failed to enable memory mapping!\n", sc->dc_unit);
-		error = ENXIO;
 		goto fail;
 	}
 #endif
@@ -1994,7 +1992,6 @@ rtems_dc_driver_attach(struct rtems_bsdnet_ifconfig *config, int attaching)
 
 	if (sc->dc_res == NULL) {
 		printk("dc%d: couldn't map ports/memory\n", unit);
-		error = ENXIO;
 		goto fail;
 	}
 	sc->dc_btag = rman_get_bustag(sc->dc_res);
@@ -2038,7 +2035,6 @@ rtems_dc_driver_attach(struct rtems_bsdnet_ifconfig *config, int attaching)
 	if (sc->dc_irq == NULL) {
 		printk("dc%d: couldn't map interrupt\n", unit);
 		bus_release_resource(dev, DC_RES, DC_RID, sc->dc_res);
-		error = ENXIO;
 		goto fail;
 	}
 
@@ -2270,7 +2266,6 @@ rtems_dc_driver_attach(struct rtems_bsdnet_ifconfig *config, int attaching)
 		bus_release_resource(dev, SYS_RES_IRQ, 0, sc->dc_irq);
 		bus_release_resource(dev, DC_RES, DC_RID, sc->dc_res);
 #endif
-		error = ENXIO;
 		goto fail;
 	}
 




More information about the vc mailing list