I just had a patch go in here yesterday so these don't apply any
more, sorry.

On Thu, May 30, 2013 at 10:14:56PM -0400, Sean MacLennan wrote:
> Cleanup the rt_stats structure. Many of the fields where unused and a
> lot of the fields where set but never referenced.
> 
> Signed-off-by: Sean MacLennan <[email protected]>
> ---
> diff --git a/drivers/staging/rtl8192e/rtl8192e/r8190P_def.h 
> b/drivers/staging/rtl8192e/rtl8192e/r8190P_def.h
> index b7bb71f..3c8e922 100644
> --- a/drivers/staging/rtl8192e/rtl8192e/r8190P_def.h
> +++ b/drivers/staging/rtl8192e/rtl8192e/r8190P_def.h
> @@ -269,12 +269,14 @@ struct log_int_8190 {
>       u32     nIMR_RXFOVW;
>  };
>  
> +#if 0
>  struct phy_ofdm_rx_status_rxsc_sgien_exintfflag {
>       u8                      reserved:4;
>       u8                      rxsc:2;
>       u8                      sgi_en:1;
>       u8                      ex_intf_flag:1;
>  };
> +#endif

Didn't you just want to delete this?  The patch ifdefs out some
other code below as well.

> @@ -288,57 +231,11 @@ static  void cmpk_handle_tx_status(struct net_device 
> *dev, u8 *pmsg)
>  {
>       struct cmpk_tx_status rx_tx_sts;        /* */
>  
> +     // SAM FIXME

Get on the ball, Sam!

>       memcpy((void *)&rx_tx_sts, (void *)pmsg, sizeof(struct cmpk_tx_status));
>       cmpk_count_tx_status(dev, &rx_tx_sts);
>  }
> @@ -1598,7 +1593,6 @@ static void rtl8192_query_rxphystatus(
>                       tmp_rxsnr = pofdm_buf->rxsnr_X[i];
>                       rx_snrX = (char)(tmp_rxsnr);
>                       rx_snrX /= 2;
> -                     priv->stats.rxSNRdB[i] = (long)rx_snrX;

The rx_snrX variable isn't needed now.

>  
>                       RSSI = rtl819x_query_rxpwrpercentage(rx_pwr[i]);
>                       if (priv->brfpath_rxenable[i])

regards,
dan carpenter
_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to