From: <Tristram.Ha@microchip.com>
To: Andrew Lunn <andrew@lunn.ch>,
Florian Fainelli <f.fainelli@gmail.com>,
Pavel Machek <pavel@ucw.cz>,
Ruediger Schmitt <ruediger.schmitt@philips.com>
Cc: Tristram Ha <Tristram.Ha@microchip.com>,
Arkadi Sharshevsky <arkadis@mellanox.com>,
<UNGLinuxDriver@microchip.com>, <netdev@vger.kernel.org>
Subject: [PATCH v2 net-next 0/8] net: dsa: microchip: Modify KSZ9477 DSA driver in preparation to add other KSZ switch drivers
Date: Tue, 5 Dec 2017 17:46:30 -0800 [thread overview]
Message-ID: <1512524798-16210-1-git-send-email-Tristram.Ha@microchip.com> (raw)
From: Tristram Ha <Tristram.Ha@microchip.com>
This series of patches is to modify the original KSZ9477 DSA driver so
that other KSZ switch drivers can be added and use the common code.
There are several steps to accomplish this achievement. First is to
rename some function names with a prefix to indicate chip specific
function. Second is to move common code into header that can be shared.
Last is to modify tag_ksz.c so that it can handle many tail tag formats
used by different KSZ switch drivers.
ksz_common.c will contain the common code used by all KSZ switch drivers.
ksz9477.c will contain KSZ9477 code from the original ksz_common.c.
ksz9477_spi.c is renamed from ksz_spi.c.
ksz9477_reg.h is renamed from ksz_9477_reg.h.
ksz_common.h is added to provide common code access to KSZ switch
drivers.
ksz_spi.h is added to provide common SPI access functions to KSZ SPI
drivers.
v2
- Initialize reg_mutex before use
- The alu_mutex is only used inside chip specific functions
v1
- Each patch in the set is self-contained
- Use ksz9477 prefix to indicate KSZ9477 specific code
Tristram Ha (8):
Replace license with GPL.
Clean up code according to patch check suggestions.
Initialize mutex before use.
Rename some functions with ksz9477 prefix to separate chip specific
code from common code.
Rename ksz_spi.c to ksz9477_spi.c and update Kconfig in preparation to
add more KSZ switch drivers.
Break KSZ9477 DSA driver into two files in preparation to add more KSZ
switch drivers. Add common functions in ksz_common.h so that other
KSZ switch drivers can access code in ksz_common.c. Add ksz_spi.h
for common functions used by KSZ switch SPI drivers.
Prepare PHY for proper advertisement and get link status for the port.
Rename ksz_9477_reg.h to ksz9477_reg.h for consistency as the product
name is always KSZ####.
drivers/net/dsa/microchip/Kconfig | 12 +-
drivers/net/dsa/microchip/Makefile | 4 +-
drivers/net/dsa/microchip/ksz9477.c | 1331 ++++++++++++++++++++
.../microchip/{ksz_9477_reg.h => ksz9477_reg.h} | 23 +-
drivers/net/dsa/microchip/ksz9477_spi.c | 188 +++
drivers/net/dsa/microchip/ksz_common.c | 1176 +++--------------
drivers/net/dsa/microchip/ksz_common.h | 229 ++++
drivers/net/dsa/microchip/ksz_priv.h | 256 ++--
drivers/net/dsa/microchip/ksz_spi.c | 216 ----
drivers/net/dsa/microchip/ksz_spi.h | 82 ++
10 files changed, 2122 insertions(+), 1395 deletions(-)
create mode 100644 drivers/net/dsa/microchip/ksz9477.c
rename drivers/net/dsa/microchip/{ksz_9477_reg.h => ksz9477_reg.h} (98%)
create mode 100644 drivers/net/dsa/microchip/ksz9477_spi.c
create mode 100644 drivers/net/dsa/microchip/ksz_common.h
delete mode 100644 drivers/net/dsa/microchip/ksz_spi.c
create mode 100644 drivers/net/dsa/microchip/ksz_spi.h
--
1.9.1
next reply other threads:[~2017-12-06 1:46 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-12-06 1:46 Tristram.Ha [this message]
2017-12-06 1:46 ` [PATCH v2 net-next 1/8] net: dsa: microchip: Replace license with GPL Tristram.Ha
2017-12-06 1:46 ` [PATCH v2 net-next 2/8] net: dsa: microchip: Clean up code according to patch check suggestions Tristram.Ha
2017-12-06 1:46 ` [PATCH v2 net-next 3/8] net: dsa: microchip: Initialize mutex before use Tristram.Ha
2017-12-06 18:44 ` Andrew Lunn
2017-12-06 1:46 ` [PATCH v2 net-next 4/8] net: dsa: microchip: Rename some functions with ksz9477 prefix Tristram.Ha
2017-12-06 1:46 ` [PATCH v2 net-next 5/8] net: dsa: microchip: Rename ksz_spi.c to ksz9477_spi.c Tristram.Ha
2017-12-06 1:46 ` [PATCH v2 net-next 6/8] net: dsa: microchip: Break KSZ9477 DSA driver into two files Tristram.Ha
2017-12-06 1:46 ` [PATCH v2 net-next 7/8] net: dsa: microchip: Prepare PHY for proper advertisement Tristram.Ha
2017-12-06 18:56 ` Andrew Lunn
2017-12-06 21:55 ` Tristram.Ha
2017-12-06 22:56 ` Florian Fainelli
2017-12-06 1:46 ` [PATCH v2 net-next 8/8] net: dsa: microchip: Rename ksz_9477_reg.h to ksz9477_reg.h Tristram.Ha
2017-12-15 9:14 ` Pavel Machek
2018-08-16 0:28 ` [PATCH v2 net-next 0/8] net: dsa: microchip: Modify KSZ9477 DSA driver in preparation to add other KSZ switch drivers Florian Fainelli
2018-08-16 21:34 ` Tristram.Ha
2018-10-19 17:56 ` Florian Fainelli
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=1512524798-16210-1-git-send-email-Tristram.Ha@microchip.com \
--to=tristram.ha@microchip.com \
--cc=UNGLinuxDriver@microchip.com \
--cc=andrew@lunn.ch \
--cc=arkadis@mellanox.com \
--cc=f.fainelli@gmail.com \
--cc=netdev@vger.kernel.org \
--cc=pavel@ucw.cz \
--cc=ruediger.schmitt@philips.com \
/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;
as well as URLs for NNTP newsgroup(s).