Re: [PATCH 3/3] ath10k: Improve logging message.

2016-09-09 Thread Valo, Kalle 
Mohammed Shafi Shajakhan  writes:

> Hi Ben,
>
> On Thu, Aug 18, 2016 at 06:26:35PM -0700, gree...@candelatech.com wrote:
>> From: Ben Greear 
>> 
>> Helps to know the sta pointer.
>> 
>> Signed-off-by: Ben Greear 
>> ---
>>  drivers/net/wireless/ath/ath10k/mac.c | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>> 
>> diff --git a/drivers/net/wireless/ath/ath10k/mac.c 
>> b/drivers/net/wireless/ath/ath10k/mac.c
>> index d96c06e..82bc0da 100644
>> --- a/drivers/net/wireless/ath/ath10k/mac.c
>> +++ b/drivers/net/wireless/ath/ath10k/mac.c
>> @@ -6491,8 +6491,8 @@ static int ath10k_sta_state(struct ieee80211_hw *hw,
>>   * Existing station deletion.
>>   */
>>  ath10k_dbg(ar, ATH10K_DBG_MAC,
>> -   "mac vdev %d peer delete %pM (sta gone)\n",
>> -   arvif->vdev_id, sta->addr);
>> +   "mac vdev %d peer delete %pM (sta gone) sta: %p\n",
>> +   arvif->vdev_id, sta->addr, sta);
>
> good to rebase over Maharaja's change (%p to %pK) 
> https://patchwork.kernel.org/patch/9263691/

I added that to the patch in the pending branch.

-- 
Kalle Valo

Re: [PATCH 3/3] ath10k: Improve logging message.

2016-08-18 Thread Mohammed Shafi Shajakhan 
Hi Ben,

On Thu, Aug 18, 2016 at 06:26:35PM -0700, gree...@candelatech.com wrote:
> From: Ben Greear 
> 
> Helps to know the sta pointer.
> 
> Signed-off-by: Ben Greear 
> ---
>  drivers/net/wireless/ath/ath10k/mac.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/net/wireless/ath/ath10k/mac.c 
> b/drivers/net/wireless/ath/ath10k/mac.c
> index d96c06e..82bc0da 100644
> --- a/drivers/net/wireless/ath/ath10k/mac.c
> +++ b/drivers/net/wireless/ath/ath10k/mac.c
> @@ -6491,8 +6491,8 @@ static int ath10k_sta_state(struct ieee80211_hw *hw,
>* Existing station deletion.
>*/
>   ath10k_dbg(ar, ATH10K_DBG_MAC,
> -"mac vdev %d peer delete %pM (sta gone)\n",
> -arvif->vdev_id, sta->addr);
> +"mac vdev %d peer delete %pM (sta gone) sta: %p\n",
> +arvif->vdev_id, sta->addr, sta);

good to rebase over Maharaja's change (%p to %pK) 
https://patchwork.kernel.org/patch/9263691/

>  
>   ret = ath10k_peer_delete(ar, arvif->vdev_id, sta->addr);
>   if (ret)
> -- 
> 2.4.11
> 
> 
> ___
> ath10k mailing list
> ath...@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/ath10k