| Message ID | 1471569995-10028-3-git-send-email-greearb@candelatech.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 3040420158c139f64776935587bfad2584152f4c |
| Delegated to: | Kalle Valo |
| Headers | show |
Hi Ben, On Thu, Aug 18, 2016 at 06:26:35PM -0700, greearb@candelatech.com wrote: > From: Ben Greear <greearb@candelatech.com> > > Helps to know the sta pointer. > > Signed-off-by: Ben Greear <greearb@candelatech.com> > --- > 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 > ath10k@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/ath10k
Mohammed Shafi Shajakhan <mohammed@codeaurora.org> writes: > Hi Ben, > > On Thu, Aug 18, 2016 at 06:26:35PM -0700, greearb@candelatech.com wrote: >> From: Ben Greear <greearb@candelatech.com> >> >> Helps to know the sta pointer. >> >> Signed-off-by: Ben Greear <greearb@candelatech.com> >> --- >> 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.
Ben Greear <greearb@candelatech.com> wrote: > From: Ben Greear <greearb@candelatech.com> > > Helps to know the sta pointer. > > Signed-off-by: Ben Greear <greearb@candelatech.com> Thanks, 1 patch applied to ath-next branch of ath.git: 3040420158c1 ath10k: improve logging message
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); ret = ath10k_peer_delete(ar, arvif->vdev_id, sta->addr); if (ret)