[net-next,v3,00/17] net: dsa: microchip: add error handling and register access validation

Message

Oleksij Rempel Aug. 26, 2022, 10:56 a.m. UTC

changes v4:
- add Reviewed-by: Vladimir Oltean <[e-mail]> to all patches
- fix checkpatch warnings.

changes v3:
- fix build error in the middle of the patch stack.

changes v2:
- add regmap_ranges for KSZ9477
- drop output clock devicetree in driver validation patches. DTs need
  some more refactoring and can be done in a separate patch set.
- remove some unused variables.

This patch series adds error handling for the PHY read/write path and optional
register access validation.
After adding regmap_ranges for KSZ8563 some bugs was detected, so
critical bug fixes are sorted before ragmap_range patch.

Potentially this bug fixes can be ported to stable kernels, but need to be
reworked.

Oleksij Rempel (17):
  net: dsa: microchip: add separate struct ksz_chip_data for KSZ8563
    chip
  net: dsa: microchip: do per-port Gbit detection instead of per-chip
  net: dsa: microchip: don't announce extended register support on non
    Gbit chips
  net: dsa: microchip: allow to pass return values for PHY read/write
    accesses
  net: dsa: microchip: forward error value on all ksz_pread/ksz_pwrite
    functions
  net: dsa: microchip: ksz9477: add error handling to ksz9477_r/w_phy
  net: dsa: microchip: ksz8795: add error handling to ksz8_r/w_phy
  net: dsa: microchip: KSZ9893: do not write to not supported Output
    Clock Control Register
  net: dsa: microchip: add support for regmap_access_tables
  net: dsa: microchip: add regmap_range for KSZ8563 chip
  net: dsa: microchip: ksz9477: remove MII_CTRL1000 check from
    ksz9477_w_phy()
  net: dsa: microchip: add regmap_range for KSZ9477 chip
  net: dsa: microchip: ksz9477: use internal_phy instead of phy_port_cnt
  net: dsa: microchip: remove unused port phy variable
  net: dsa: microchip: ksz9477: remove unused "on" variable
  net: dsa: microchip: remove unused sgmii variable
  net: dsa: microchip: remove IS_9893 flag

 drivers/net/dsa/microchip/ksz8.h         |   4 +-
 drivers/net/dsa/microchip/ksz8795.c      | 111 ++++--
 drivers/net/dsa/microchip/ksz9477.c      |  85 ++---
 drivers/net/dsa/microchip/ksz9477.h      |   4 +-
 drivers/net/dsa/microchip/ksz_common.c   | 450 ++++++++++++++++++++++-
 drivers/net/dsa/microchip/ksz_common.h   | 102 +++--
 drivers/net/dsa/microchip/ksz_spi.c      |   5 +-
 drivers/net/dsa/microchip/lan937x.h      |   4 +-
 drivers/net/dsa/microchip/lan937x_main.c |   8 +-
 9 files changed, 651 insertions(+), 122 deletions(-)

Comments

[e-mail] Aug. 31, 2022, 8:50 a.m. UTC | #1

Hello:

This series was applied to netdev/net-next.git (master)
by David S. Miller <[e-mail]>:

On Fri, 26 Aug 2022 12:56:17 +0200 you wrote:
> changes v4:
> - add Reviewed-by: Vladimir Oltean <[e-mail]> to all patches
> - fix checkpatch warnings.
> 
> changes v3:
> - fix build error in the middle of the patch stack.
> 
> [...]

Here is the summary with links:
  - [net-next,v4,01/17] net: dsa: microchip: add separate struct ksz_chip_data for KSZ8563 chip
    https://git.kernel.org/netdev/net-next/c/b44908095612
  - [net-next,v4,02/17] net: dsa: microchip: do per-port Gbit detection instead of per-chip
    https://git.kernel.org/netdev/net-next/c/505bf3205aaa
  - [net-next,v4,03/17] net: dsa: microchip: don't announce extended register support on non Gbit chips
    https://git.kernel.org/netdev/net-next/c/d7539fc2b41a
  - [net-next,v4,04/17] net: dsa: microchip: allow to pass return values for PHY read/write accesses
    https://git.kernel.org/netdev/net-next/c/8f4204567923
  - [net-next,v4,05/17] net: dsa: microchip: forward error value on all ksz_pread/ksz_pwrite functions
    https://git.kernel.org/netdev/net-next/c/d38bc3b4b8a6
  - [net-next,v4,06/17] net: dsa: microchip: ksz9477: add error handling to ksz9477_r/w_phy
    https://git.kernel.org/netdev/net-next/c/9da975e1bbef
  - [net-next,v4,07/17] net: dsa: microchip: ksz8795: add error handling to ksz8_r/w_phy
    https://git.kernel.org/netdev/net-next/c/9590fc4a2af5
  - [net-next,v4,08/17] net: dsa: microchip: KSZ9893: do not write to not supported Output Clock Control Register
    https://git.kernel.org/netdev/net-next/c/b5708dc6539d
  - [net-next,v4,09/17] net: dsa: microchip: add support for regmap_access_tables
    https://git.kernel.org/netdev/net-next/c/ec6ba50c65c1
  - [net-next,v4,10/17] net: dsa: microchip: add regmap_range for KSZ8563 chip
    https://git.kernel.org/netdev/net-next/c/41131bac9a9a
  - [net-next,v4,11/17] net: dsa: microchip: ksz9477: remove MII_CTRL1000 check from ksz9477_w_phy()
    https://git.kernel.org/netdev/net-next/c/5bd3ecd121e3
  - [net-next,v4,12/17] net: dsa: microchip: add regmap_range for KSZ9477 chip
    https://git.kernel.org/netdev/net-next/c/74e792b5f2dd
  - [net-next,v4,13/17] net: dsa: microchip: ksz9477: use internal_phy instead of phy_port_cnt
    https://git.kernel.org/netdev/net-next/c/0a7fbd514edf
  - [net-next,v4,14/17] net: dsa: microchip: remove unused port phy variable
    https://git.kernel.org/netdev/net-next/c/6aaa8e7d2002
  - [net-next,v4,15/17] net: dsa: microchip: ksz9477: remove unused "on" variable
    https://git.kernel.org/netdev/net-next/c/7d39143449ea
  - [net-next,v4,16/17] net: dsa: microchip: remove unused sgmii variable
    https://git.kernel.org/netdev/net-next/c/e7f695210140
  - [net-next,v4,17/17] net: dsa: microchip: remove IS_9893 flag
    https://git.kernel.org/netdev/net-next/c/32cbac21b9f4

You are awesome, thank you!