Re: [0/2] net: dsa: vsc73xx: Adjustments for vsc73xx_platform_probe()

2019-09-21 Thread Markus Elfring
> Indeed, looks like we have a mix of clean ups of varying clarity here. > I will just drop all, including the devm_platform_ioremap_resource() > conversion patches from patchwork for now. I would prefer an other patch management in this case. > Markus, please repost them all once net-next

Re: [0/2] net: dsa: vsc73xx: Adjustments for vsc73xx_platform_probe()

2019-09-20 Thread Jakub Kicinski
On Fri, 20 Sep 2019 09:36:57 -0700, Florian Fainelli wrote: > On 9/20/19 8:30 AM, Markus Elfring wrote: > >> netdev is closed at the moment for patch. > > > > I wonder about this information. > > This is covered here: > >

Re: [0/2] net: dsa: vsc73xx: Adjustments for vsc73xx_platform_probe()

2019-09-20 Thread Florian Fainelli
On 9/20/19 8:30 AM, Markus Elfring wrote: >> netdev is closed at the moment for patch. > > I wonder about this information. This is covered here: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/networking/netdev-FAQ.rst#n40 and you can skip the reading and

Re: [0/2] net: dsa: vsc73xx: Adjustments for vsc73xx_platform_probe()

2019-09-20 Thread Markus Elfring
> netdev is closed at the moment for patch. I wonder about this information. > Please repost once it reopens, in about 2 weeks time. I hope that the presented change possibilities can be integrated in the near future also without a repetition of this small patch series.

Re: [PATCH 0/2] net: dsa: vsc73xx: Adjustments for vsc73xx_platform_probe()

2019-09-20 Thread Andrew Lunn
On Fri, Sep 20, 2019 at 04:25:53PM +0200, Markus Elfring wrote: > From: Markus Elfring > Date: Fri, 20 Sep 2019 16:17:18 +0200 > > Two update suggestions were taken into account > from static source code analysis. Hi Markus netdev is closed at the moment for patch. Please repost once it

[PATCH 0/2] net: dsa: vsc73xx: Adjustments for vsc73xx_platform_probe()

2019-09-20 Thread Markus Elfring
From: Markus Elfring Date: Fri, 20 Sep 2019 16:17:18 +0200 Two update suggestions were taken into account from static source code analysis. Markus Elfring (2): Use devm_platform_ioremap_resource() Return an error code as a constant drivers/net/dsa/vitesse-vsc73xx-platform.c | 16