On 2017-04-12 at 15:18:06 +0200, Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> wrote: > On Wed, Apr 05, 2017 at 04:02:22PM +0530, Amit Pundir wrote: > > From: Tobias Klauser <tklauser@xxxxxxxxxx> > > > > Instead of using a private copy of struct net_device_stats in struct > > brcm_if, use stats from struct net_device. Also remove the now > > unnecessary .ndo_get_stats function. > > > > Signed-off-by: Tobias Klauser <tklauser@xxxxxxxxxx> > > Acked-by: Arend van Spriel <arend.vanspriel@xxxxxxxxxxxx> > > Signed-off-by: Kalle Valo <kvalo@xxxxxxxxxxxxxx> > > (cherry picked from commit 91b632803ee4e47c5a5c4dc3d8bf5abf9c16107a) > > Signed-off-by: Amit Pundir <amit.pundir@xxxxxxxxxx> > > --- > > .../wireless/broadcom/brcm80211/brcmfmac/core.c | 26 +++++++--------------- > > .../wireless/broadcom/brcm80211/brcmfmac/core.h | 2 -- > > 2 files changed, 8 insertions(+), 20 deletions(-) > > Again, how is this a proper stable patch? I don't know how this ended up being submitted for inclusion in stable, but from by perspective this patch clearly doesn't qualify for inclusion.