public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Christian Marangi <ansuelsmth@gmail.com>
To: Andrew Lunn <andrew@lunn.ch>,
	Florian Fainelli <f.fainelli@gmail.com>,
	Vladimir Oltean <olteanv@gmail.com>,
	"David S. Miller" <davem@davemloft.net>,
	Eric Dumazet <edumazet@google.com>,
	Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
	Christian Marangi <ansuelsmth@gmail.com>,
	"Russell King (Oracle)" <rmk+kernel@armlinux.org.uk>,
	netdev@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: [net PATCH v2 0/5] net: dsa: qca8k: multiple fix on mdio read/write
Date: Thu, 29 Dec 2022 17:33:31 +0100	[thread overview]
Message-ID: <20221229163336.2487-1-ansuelsmth@gmail.com> (raw)

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(-)

-- 
2.37.2


             reply	other threads:[~2022-12-29 16:44 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-12-29 16:33 Christian Marangi [this message]
2022-12-29 16:33 ` [net PATCH v2 1/5] net: dsa: qca8k: fix wrong length value for mgmt eth packet Christian Marangi
2022-12-29 16:33 ` [net PATCH v2 2/5] net: dsa: tag_qca: fix wrong MGMT_DATA2 size Christian Marangi
2022-12-29 16:33 ` [net PATCH v2 3/5] Revert "net: dsa: qca8k: cache lo and hi for mdio write" Christian Marangi
2022-12-29 16:33 ` [net PATCH v2 4/5] net: dsa: qca8k: introduce single mii read/write lo/hi Christian Marangi
2022-12-29 16:33 ` [net PATCH v2 5/5] net: dsa: qca8k: improve mdio master read/write by using single lo/hi Christian Marangi
2023-01-01 12:00 ` [net PATCH v2 0/5] net: dsa: qca8k: multiple fix on mdio read/write patchwork-bot+netdevbpf

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20221229163336.2487-1-ansuelsmth@gmail.com \
    --to=ansuelsmth@gmail.com \
    --cc=andrew@lunn.ch \
    --cc=davem@davemloft.net \
    --cc=edumazet@google.com \
    --cc=f.fainelli@gmail.com \
    --cc=kuba@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=olteanv@gmail.com \
    --cc=pabeni@redhat.com \
    --cc=rmk+kernel@armlinux.org.uk \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox