On 12 April 2017 at 19:22, Kalle Valo <kvalo@xxxxxxxxxxxxxx> wrote: > + arend > > Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> writes: > >> 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 have no idea. At least me nor Arend didn't tag it for stable. I picked it up along with few other patches from LEDE 4.9 stable source and sent them @ stable. I'll review the series again. My apologies for the confusion. Regards, Amit Pundir > > -- > Kalle Valo