From: Tristram Ha <[email protected]>
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.
This patch set is against net-next.
drivers/net/dsa/microchip/Makefile | 2 +-
drivers/net/dsa/microchip/ksz9477.c | 1317 ++++++++++++++++++++++++++++++++
drivers/net/dsa/microchip/ksz_common.c | 1156 +---------------------------
drivers/net/dsa/microchip/ksz_priv.h | 105 ++-
drivers/net/dsa/microchip/ksz_spi.c | 13 +-
net/dsa/tag_ksz.c | 40 +-
6 files changed, 1458 insertions(+), 1175 deletions(-) create mode 100644 drivers/net/dsa/microchip/ksz9477.c
Hi!
> From: Tristram Ha <[email protected]>
>
> 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.
>
Please wrap the lines from time to time...
> This patch set is against net-next.
>
> drivers/net/dsa/microchip/Makefile | 2 +-
> drivers/net/dsa/microchip/ksz9477.c | 1317 ++++++++++++++++++++++++++++++++
We already have ksz_9477_reg.h. So should this be ksz_9477.c for consistency?
> drivers/net/dsa/microchip/ksz_common.c | 1156 +---------------------------
> drivers/net/dsa/microchip/ksz_priv.h | 105 ++-
> drivers/net/dsa/microchip/ksz_spi.c | 13 +-
> net/dsa/tag_ksz.c | 40 +-
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
Hi Tristram,
[email protected] writes:
> From: Tristram Ha <[email protected]>
>
> 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.
Please see Documentation/process/submitting-patches.rst. Use a correct
formatting for the messages, and send you patchset as a thread.
> This patch set is against net-next.
The net subsystem is somehow special. Please see
Documentation/networking/netdev-FAQ.txt. The tree must be indicated in
the subject prefix of the patch series.
Thanks,
Vivien
> -----Original Message-----
> From: Pavel Machek [mailto:[email protected]]
> Sent: Friday, September 08, 2017 1:54 AM
> To: Tristram Ha - C24268
> Cc: [email protected]; [email protected]; [email protected];
> [email protected]; [email protected];
> [email protected]; [email protected]; Woojung Huh -
> C21699
> Subject: Re: [PATCH RFC 0/6] Modify KSZ9477 DSA driver in preparation to
> add other KSZ switch drivers.
>
> Hi!
>
> > From: Tristram Ha <[email protected]>
> >
> > 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.
> >
>
> Please wrap the lines from time to time...
>
>
> > This patch set is against net-next.
> >
> > drivers/net/dsa/microchip/Makefile | 2 +-
> > drivers/net/dsa/microchip/ksz9477.c | 1317
> ++++++++++++++++++++++++++++++++
>
> We already have ksz_9477_reg.h. So should this be ksz_9477.c for
> consistency?
The product name is KSZ9477 and other switches are also like KSZ####,
so I would prefer to have no separation between KSZ and the product
number. I think the file ksz_9477_reg.h was named that way because
the other files were named ksz_common.c and ksz_spi.c. If need to
we can change the file name.
Hi!
> > > drivers/net/dsa/microchip/Makefile | 2 +-
> > > drivers/net/dsa/microchip/ksz9477.c | 1317
> > ++++++++++++++++++++++++++++++++
> >
> > We already have ksz_9477_reg.h. So should this be ksz_9477.c for
> > consistency?
>
> The product name is KSZ9477 and other switches are also like KSZ####,
> so I would prefer to have no separation between KSZ and the product
> number. I think the file ksz_9477_reg.h was named that way because
> the other files were named ksz_common.c and ksz_spi.c. If need to
> we can change the file name.
I don't care either way, but please make it consistent.
Thanks,
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html