[netdevice, Makefile, 802.11] Revert print_status callback changes
authorJoshua Oreman <oremanj@xenon.get-linux.org>
Thu, 18 Jun 2009 22:42:24 +0000 (15:42 -0700)
committerJoshua Oreman <oremanj@xenon.get-linux.org>
Thu, 18 Jun 2009 22:42:24 +0000 (15:42 -0700)
Upon consultation with mcb30 my method for link-layer status displays
is not the best way of doing things for gPXE; it conflates user-ish and
kernel-ish code. A better method will be shortly forthcoming.

src/Makefile.housekeeping
src/include/gpxe/netdevice.h
src/net/net80211.c
src/usr/ifmgmt.c

index f14e926..cf7a9c7 100644 (file)
@@ -327,7 +327,7 @@ CFLAGS              += -Os
 CFLAGS         += -g
 ifeq ($(CCTYPE),gcc)
 CFLAGS         += -ffreestanding
-CFLAGS         += -Wall -W
+CFLAGS         += -Wall -W -Wformat-nonliteral
 endif
 ifeq ($(CCTYPE),icc)
 CFLAGS         += -fno-builtin
index a846dae..f9d56bc 100644 (file)
@@ -284,17 +284,6 @@ struct net_device {
        /** Configuration settings applicable to this device */
        struct generic_settings settings;
 
-       /** ifstat callback
-        *
-        * @v netdev    Network device
-        *
-        * If non-NULL, this should be a function that prints one line
-        * containing information about the link-layer-specific state
-        * of the network device, terminated by a newline. It will be
-        * called when @c ifstat is invoked on the network device.
-        */
-       void ( * print_status ) ( struct net_device *netdev );
-
        /** Driver private data */
        void *priv;
 };
index a27dd93..20e9936 100644 (file)
@@ -128,7 +128,6 @@ static int net80211_process_ie ( struct net80211_device *dev,
 static union ieee80211_ie *
 net80211_marshal_request_info ( struct net80211_device *dev,
                                union ieee80211_ie *ie );
-static void net80211_print_status ( struct net_device *netdev );
 
 static void net80211_step_associate ( struct process *proc );
 static void net80211_set_rtscts_rate ( struct net80211_device *dev );
@@ -660,7 +659,6 @@ struct net80211_device * net80211_alloc ( size_t priv_size )
 
        netdev->ll_protocol = &net80211_ll_protocol;
        netdev->max_pkt_len = IEEE80211_MAX_DATA_LEN;
-       netdev->print_status = net80211_print_status;
        netdev_init ( netdev, &net80211_netdev_ops );
 
        dev = netdev->priv;
@@ -1040,40 +1038,6 @@ net80211_marshal_request_info ( struct net80211_device *dev,
        return ie;
 }
 
-/** Format strings for signal strength types */
-static const char *net80211_signal_strings[] = {
-       [NET80211_SIGNAL_NONE] = "unk",
-       [NET80211_SIGNAL_ARBITRARY] = "%d/%d",
-       [NET80211_SIGNAL_DB] = "%d/%d dB",
-       [NET80211_SIGNAL_DBM] = "%d dBm",
-};
-
-/**
- * Print status of 802.11 device
- *
- * @v dev      Wrapping network device
- *
- * Prints a line like
- *   [802.11 SSID gPXEtest, Ch:1 Sig:14/50, Rate:11 MBps]
- */
-void net80211_print_status ( struct net_device *netdev )
-{
-       struct net80211_device *dev = netdev->priv;
-
-       printf ( "  [802.11 SSID %s, ", dev->state & NET80211_ASSOCIATED ?
-                dev->essid : "<not associated>" );
-       if ( dev->channel < dev->nr_channels && dev->rate < dev->nr_rates ) {
-               printf ( "Ch:%d Sig:",
-                        dev->channels[dev->channel].channel_nr );
-               printf ( net80211_signal_strings[dev->hw->signal_type],
-                        dev->last_signal, dev->hw->signal_max );
-               printf ( ", Rate:%d Mbps]\n",
-                        NET80211_RATE_VALUE ( dev->rates[dev->rate] ) / 10 );
-       } else {
-               printf ( "antenna off]\n" );
-       }
-}
-
 /** Seconds to always take when probing, to gather better signal strengths */
 #define NET80211_PROBE_GATHER    2
 
index 2fd8155..9c82503 100644 (file)
@@ -94,8 +94,6 @@ void ifstat ( struct net_device *netdev ) {
                 ( netdev_link_ok ( netdev ) ? "up" : "down" ),
                 netdev->tx_stats.good, netdev->tx_stats.bad,
                 netdev->rx_stats.good, netdev->rx_stats.bad );
-       if ( netdev->print_status )
-               netdev->print_status ( netdev );
        ifstat_errors ( &netdev->tx_stats, "TXE" );
        ifstat_errors ( &netdev->rx_stats, "RXE" );
 }