On 09/18/2017 02:41 PM, Florian Fainelli wrote: > Hi all, > > This patch series is a first pass set of clean-ups to reduce the number of > LOCs > between b53 and bcm_sf2 and sharing as many functions as possible. > > There is a number of additional cleanups queued up locally that require more > thorough testing.
David, I just spotted a missing EXPORT_SYMBOL() in patch 8 that was not flagged since I had temporarily disabled modular build, I will resubmit this shortly after checking the other patches too. Thanks! > > Thanks! > > Florian Fainelli (12): > net: dsa: b53: Remove is_cpu_port() > net: dsa: b53: Make b53_enable_cpu_port() take a port argument > net: dsa: b53: Defer port enabling to calling port_enable > net: dsa: bcm_sf2: Defer port enabling to calling port_enable > net: dsa: b53: Use a macro to define I/O operations > net: dsa: b53: Move Broadcom header setup to b53 > net: dsa: b53: Define EEE register page > net: dsa: b53: Move EEE functions to b53 > net: dsa: b53: Wire-up EEE > net: dsa: b53: Export b53_imp_vlan_setup() > net: dsa: bcm_sf2: Use SF2_NUM_EGRESS_QUEUES for CFP > net: dsa: bcm_sf2: Utilize b53_{enable,disable}_port > > drivers/net/dsa/b53/b53_common.c | 150 ++++++++++++++++++++++++++++++++---- > drivers/net/dsa/b53/b53_priv.h | 145 ++++++++--------------------------- > drivers/net/dsa/b53/b53_regs.h | 48 ++++++++++++ > drivers/net/dsa/bcm_sf2.c | 161 > +++------------------------------------ > drivers/net/dsa/bcm_sf2.h | 2 - > drivers/net/dsa/bcm_sf2_cfp.c | 6 +- > drivers/net/dsa/bcm_sf2_regs.h | 11 --- > 7 files changed, 227 insertions(+), 296 deletions(-) > -- Florian