mbox series

[net,v2,0/5] net: dsa: qca8k: multiple fix on mdio read/write

Message ID 20221229163336.2487-1-ansuelsmth@gmail.com
Headers
Series net: dsa: qca8k: multiple fix on mdio read/write |

Message

Christian Marangi Dec. 29, 2022, 4:33 p.m. UTC 
  Due to some problems in reading the Documentation and elaborating it
some wrong assumption were done. The error was reported and notice only
now due to how things are setup in the code flow.

First 2 patch fix mgmt eth where the lenght calculation is very
confusing and in step of word size. (the related commit description have
an extensive description about how this mess works)

Last 3 patch revert the broken mdio cache and apply a correct version
that should still save some extra mdio in phy poll secnario.

These 5 patch fix each related problem and apply what the Documentation
actually say.

Changes v2:
- Add cover letter
- Fix typo in revert patch

Christian Marangi (5):
  net: dsa: qca8k: fix wrong length value for mgmt eth packet
  net: dsa: tag_qca: fix wrong MGMT_DATA2 size
  Revert "net: dsa: qca8k: cache lo and hi for mdio write"
  net: dsa: qca8k: introduce single mii read/write lo/hi
  net: dsa: qca8k: improve mdio master read/write by using single lo/hi

 drivers/net/dsa/qca/qca8k-8xxx.c | 164 ++++++++++++++++++++-----------
 drivers/net/dsa/qca/qca8k.h      |   5 -
 include/linux/dsa/tag_qca.h      |   4 +-
 3 files changed, 109 insertions(+), 64 deletions(-)

Comments

patchwork-bot+netdevbpf@kernel.org Jan. 1, 2023, noon UTC | #1 
Hello:

This series was applied to netdev/net.git (master)
by David S. Miller <davem@davemloft.net>:

On Thu, 29 Dec 2022 17:33:31 +0100 you wrote:
> Due to some problems in reading the Documentation and elaborating it
> some wrong assumption were done. The error was reported and notice only
> now due to how things are setup in the code flow.
> 
> First 2 patch fix mgmt eth where the lenght calculation is very
> confusing and in step of word size. (the related commit description have
> an extensive description about how this mess works)
> 
> [...]

Here is the summary with links:
  - [net,v2,1/5] net: dsa: qca8k: fix wrong length value for mgmt eth packet
    https://git.kernel.org/netdev/net/c/9807ae697461
  - [net,v2,2/5] net: dsa: tag_qca: fix wrong MGMT_DATA2 size
    https://git.kernel.org/netdev/net/c/d9dba91be71f
  - [net,v2,3/5] Revert "net: dsa: qca8k: cache lo and hi for mdio write"
    https://git.kernel.org/netdev/net/c/03cb9e6d0b32
  - [net,v2,4/5] net: dsa: qca8k: introduce single mii read/write lo/hi
    https://git.kernel.org/netdev/net/c/cfbd6de588ef
  - [net,v2,5/5] net: dsa: qca8k: improve mdio master read/write by using single lo/hi
    https://git.kernel.org/netdev/net/c/a4165830ca23

You are awesome, thank you!