| Message ID | 20220517094333.27225-1-arun.ramadoss@microchip.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | net: dsa: microchip: refactor the ksz switch init function | expand |
Hello: This series was applied to netdev/net-next.git (master) by David S. Miller <davem@davemloft.net>: On Tue, 17 May 2022 15:13:24 +0530 you wrote: > During the ksz_switch_register function, it calls the individual switches init > functions (ksz8795.c and ksz9477.c). Both these functions have few things in > common like, copying the chip specific data to struct ksz_dev, allocating > ksz_port memory and mib_names memory & cnt. And to add the new LAN937x series > switch, these allocations has to be replicated. > Based on the review feedback of LAN937x part support patch, refactored the > switch init function to move allocations to switch register. > > [...] Here is the summary with links: - [net-next,1/9] net: dsa: microchip: ksz8795: update the port_cnt value in ksz_chip_data https://git.kernel.org/netdev/net-next/c/a30bf805592e - [net-next,2/9] net: dsa: microchip: move ksz_chip_data to ksz_common https://git.kernel.org/netdev/net-next/c/462d525018f0 - [net-next,3/9] net: dsa: microchip: perform the compatibility check for dev probed https://git.kernel.org/netdev/net-next/c/eee16b147121 - [net-next,4/9] net: dsa: microchip: move struct mib_names to ksz_chip_data https://git.kernel.org/netdev/net-next/c/a530e6f2204a - [net-next,5/9] net: dsa: microchip: move port memory allocation to ksz_common https://git.kernel.org/netdev/net-next/c/198b34783ab1 - [net-next,6/9] net: dsa: microchip: move get_strings to ksz_common https://git.kernel.org/netdev/net-next/c/997d2126ac61 - [net-next,7/9] net: dsa: move mib->cnt_ptr reset code to ksz_common.c https://git.kernel.org/netdev/net-next/c/b094c679662c - [net-next,8/9] net: dsa: microchip: add the phylink get_caps https://git.kernel.org/netdev/net-next/c/65ac79e18120 - [net-next,9/9] net: dsa: microchip: remove unused members in ksz_device https://git.kernel.org/netdev/net-next/c/008db08b64f4 You are awesome, thank you!
During the ksz_switch_register function, it calls the individual switches init functions (ksz8795.c and ksz9477.c). Both these functions have few things in common like, copying the chip specific data to struct ksz_dev, allocating ksz_port memory and mib_names memory & cnt. And to add the new LAN937x series switch, these allocations has to be replicated. Based on the review feedback of LAN937x part support patch, refactored the switch init function to move allocations to switch register. Link:https://patchwork.kernel.org/project/netdevbpf/patch/20220504151755.11737-8-arun.ramadoss@microchip.com/ Changes in Patch v1 - Added the macros for the chip id - Updated the ksz8863_smi of_device_id data - Moved the patch 4 port allocation after the patch on mib_names to ksz_common Changes in RFC v2 - Fixed the compilation issue Arun Ramadoss (8): net: dsa: microchip: ksz8795: update the port_cnt value in ksz_chip_data net: dsa: microchip: move ksz_chip_data to ksz_common net: dsa: microchip: perform the compatibility check for dev probed net: dsa: microchip: move struct mib_names to ksz_chip_data net: dsa: microchip: move port memory allocation to ksz_common net: dsa: microchip: move get_strings to ksz_common net: dsa: microchip: add the phylink get_caps net: dsa: microchip: remove unused members in ksz_device Prasanna Vengateshan (1): net: dsa: move mib->cnt_ptr reset code to ksz_common.c drivers/net/dsa/microchip/ksz8795.c | 252 +----------- drivers/net/dsa/microchip/ksz8795_spi.c | 35 +- drivers/net/dsa/microchip/ksz8863_smi.c | 10 +- drivers/net/dsa/microchip/ksz9477.c | 200 ++-------- drivers/net/dsa/microchip/ksz9477_i2c.c | 30 +- drivers/net/dsa/microchip/ksz9477_spi.c | 30 +- drivers/net/dsa/microchip/ksz_common.c | 485 +++++++++++++++++++++++- drivers/net/dsa/microchip/ksz_common.h | 79 +++- 8 files changed, 664 insertions(+), 457 deletions(-) base-commit: 6251264fedde83ade6f0f1f7049037469dd4de0b