From: Arseny Solokha <asolo...@kb.kras.ru> Date: Wed, 4 Sep 2019 20:52:18 +0700
> This is a cleanup series for the gianfar Ethernet driver, following up a > discussion in [1]. It is intended to precede a conversion of gianfar from > PHYLIB to PHYLINK API, which will be submitted later in its version 2. > However, it won't make a conversion cleaner, except for the last patch in > this series. Obviously this series is not intended for -stable. > > The first patch looks super controversial to me, as it moves lots of code > around for the sole purpose of getting rid of static forward declarations > in two translation units. On the other hand, this change is purely > mechanical and cannot do any harm other than cluttering git blame output. > I can prepare an alternative patch for only swapping adjacent functions > around, if necessary. > > The second patch is a trivial follow-up to the first one, making functions > that are only called from the same translation unit static. > > The third patch removes some now unused macro and structure definitions > from gianfar.h, slipped away from various cleanups in the past. > > The fourth patch, also suggested in [1], makes the driver consistently use > PHY connection type value obtained from a Device Tree node, instead of > ignoring it and using the one auto-detected by MAC, when connecting to PHY. > Obviously a value has to be specified correctly in DT source, or omitted > altogether, in which case the driver will fall back to auto-detection. When > querying a DT node, the driver will also take both applicable properties > into account by making a proper API call instead of open-coding the lookup > half-way correctly. > > [1] > https://lore.kernel.org/netdev/ca+h21hruqt6ngg5ksdswrgh_w5gtgf4fjamcwjne7qjrjus...@mail.gmail.com/ Series applied, thanks.
