[PATCH] libertas: Fix ethtool statistics

[Sebastian Siewior]

* Dan Williams | 2008-04-17 14:02:43 [-0400]:

>On Thu, 2008-04-17 at 17:13 +0100, David Woodhouse wrote:
>> We removed CMD_MESH_ACCESS from the big case statements and provided
>> lbs_mesh_access() instead.... but we missed one user. Fix it.
>> 
>> Signed-off-by: David Woodhouse <dwmw2 at infradead.org>
>
>Acked-by: Dan Williams <dcbw at redhat.com>
Hmmm. This patch actually reminds me of something I was going to do. I
don't really see the big difference between this and [1]. I have to
exert myself to get on your auto-ack list...


[1]
http://lists.infradead.org/pipermail/libertas-dev/2008-March/001281.html

>> 
>> diff --git a/drivers/net/wireless/libertas/ethtool.c b/drivers/net/wireless/libertas/ethtool.c
>> index 21e6f98..9460300 100644
>> --- a/drivers/net/wireless/libertas/ethtool.c
>> +++ b/drivers/net/wireless/libertas/ethtool.c
>> @@ -106,8 +106,8 @@ done:
>>          return ret;
>>  }
>>  
>> -static void lbs_ethtool_get_stats(struct net_device * dev,
>> -				struct ethtool_stats * stats, u64 * data)
>> +static void lbs_ethtool_get_stats(struct net_device *dev,
>> +				  struct ethtool_stats *stats, u64 *data)
>>  {
>>  	struct lbs_private *priv = dev->priv;
>>  	struct cmd_ds_mesh_access mesh_access;
>> @@ -116,9 +116,7 @@ static void lbs_ethtool_get_stats(struct net_device * dev,
>>  	lbs_deb_enter(LBS_DEB_ETHTOOL);
>>  
>>  	/* Get Mesh Statistics */
>> -	ret = lbs_prepare_and_send_command(priv,
>> -			CMD_MESH_ACCESS, CMD_ACT_MESH_GET_STATS,
>> -			CMD_OPTION_WAITFORRSP, 0, &mesh_access);
>> +	ret = lbs_mesh_access(priv, CMD_ACT_MESH_GET_STATS, &mesh_access);
>>  
>>  	if (ret)
>>  		return;
>> 

Sebastian