From mboxrd@z Thu Jan 1 00:00:00 1970 From: Geert Uytterhoeven Subject: Re: [PATCH v2 net-next] net/core: ensure features get disabled on new lower devs Date: Tue, 17 Nov 2015 10:02:37 +0100 Message-ID: References: <1446583017-19021-1-git-send-email-jarod@redhat.com> <1446610172-21420-1-git-send-email-jarod@redhat.com> <20151104.215632.964832400121049446.davem@davemloft.net> <56452E2A.9050501@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Cc: David Miller , Jarod Wilson , "linux-kernel@vger.kernel.org" , Eric Dumazet , Jay Vosburgh , Veaceslav Falico , Andy Gospodarek , =?UTF-8?B?SmnFmcOtIFDDrXJrbw==?= , Nikolay Aleksandrov , Michal Kubecek , Alexander Duyck , "netdev@vger.kernel.org" , Andrew Lunn , Vivien Didelot , Linux-sh list To: Florian Fainelli Return-path: In-Reply-To: <56452E2A.9050501@gmail.com> Sender: linux-sh-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Fri, Nov 13, 2015 at 1:26 AM, Florian Fainelli wrote: > On 04/11/15 18:56, David Miller wrote: >>> Fixes: fd867d51f889 ("net/core: generic support for disabling netdev features down stack") >> ... >>> Reported-by: Nikolay Aleksandrov >>> Signed-off-by: Jarod Wilson >>> --- >>> v2: Based on suggestions from Alex, and with not changing err to ret, this >>> patch actually becomes quite minimal and doesn't ugly up the code much. >> >> Applied, thanks. > > This causes some warnings to be displayed for DSA stacked devices: And a few more: sh-eth ee700000.ethernet eth0: set_features() failed (-1); wanted 0x0000000000004000, left 0x0000000000004800 g_ether gadget usb0: set_features() failed (-1); wanted 0x0000000000004000, left 0x0000000000004800 Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds