Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1728428 > unrolled thread

[PATCH RFC 0/6] Modify KSZ9477 DSA driver in preparation to add other KSZ switch drivers.

Started by<Tristram.Ha@microchip.com>
First post2017-09-07 23:10 +0200
Last post2017-09-08 15:40 +0200
Articles 4 — 3 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  [PATCH RFC 0/6] Modify KSZ9477 DSA driver in preparation to add  other KSZ switch drivers. <Tristram.Ha@microchip.com> - 2017-09-07 23:10 +0200
    Re: [PATCH RFC 0/6] Modify KSZ9477 DSA driver in preparation to add  other KSZ switch drivers. Pavel Machek <pavel@ucw.cz> - 2017-09-08 11:00 +0200
      RE: [PATCH RFC 0/6] Modify KSZ9477 DSA driver in preparation to add  other KSZ switch drivers. <Tristram.Ha@microchip.com> - 2017-09-09 03:20 +0200
    Re: [PATCH RFC 0/6] Modify KSZ9477 DSA driver in preparation to add other KSZ switch drivers. Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2017-09-08 15:40 +0200

#1728428 — [PATCH RFC 0/6] Modify KSZ9477 DSA driver in preparation to add other KSZ switch drivers.

From<Tristram.Ha@microchip.com>
Date2017-09-07 23:10 +0200
Subject[PATCH RFC 0/6] Modify KSZ9477 DSA driver in preparation to add other KSZ switch drivers.
Message-ID<uncdH-5xq-1@gated-at.bofh.it>
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.

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

[toc] | [next] | [standalone]


#1728704

FromPavel Machek <pavel@ucw.cz>
Date2017-09-08 11:00 +0200
Message-ID<unniO-4qa-5@gated-at.bofh.it>
In reply to#1728428

[Multipart message — attachments visible in raw view] — view raw

Hi!

> 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.
> 

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

[toc] | [prev] | [next] | [standalone]


#1729363

From<Tristram.Ha@microchip.com>
Date2017-09-09 03:20 +0200
Message-ID<unCBb-6Ng-1@gated-at.bofh.it>
In reply to#1728704
> -----Original Message-----
> From: Pavel Machek [mailto:pavel@ucw.cz]
> Sent: Friday, September 08, 2017 1:54 AM
> To: Tristram Ha - C24268
> Cc: andrew@lunn.ch; muvarov@gmail.com; nathan.leigh.conrad@gmail.com;
> vivien.didelot@savoirfairelinux.com; f.fainelli@gmail.com;
> netdev@vger.kernel.org; linux-kernel@vger.kernel.org; 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 <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.
> >
> 
> 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.

[toc] | [prev] | [next] | [standalone]


#1728896 — Re: [PATCH RFC 0/6] Modify KSZ9477 DSA driver in preparation to add other KSZ switch drivers.

FromVivien Didelot <vivien.didelot@savoirfairelinux.com>
Date2017-09-08 15:40 +0200
SubjectRe: [PATCH RFC 0/6] Modify KSZ9477 DSA driver in preparation to add other KSZ switch drivers.
Message-ID<unrFN-7D6-49@gated-at.bofh.it>
In reply to#1728428
Hi Tristram,

Tristram.Ha@microchip.com writes:

> 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.

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

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web