From: Florian Fainelli <f.fainelli@gmail.com>
To: Tristram.Ha@microchip.com, marek.vasut@gmail.com
Cc: arkadis@mellanox.com, UNGLinuxDriver@microchip.com,
netdev@vger.kernel.org, andrew@lunn.ch, pavel@ucw.cz,
ruediger.schmitt@philips.com
Subject: Re: [PATCH v2 net-next 0/8] net: dsa: microchip: Modify KSZ9477 DSA driver in preparation to add other KSZ switch drivers
Date: Fri, 19 Oct 2018 10:56:02 -0700 [thread overview]
Message-ID: <b4393977-7b5a-89a3-d7a2-8936457166dd@gmail.com> (raw)
In-Reply-To: <93AF473E2DA327428DE3D46B72B1E9FD411AC863@CHN-SV-EXMX02.mchp-main.com>
On 08/16/2018 02:34 PM, Tristram.Ha@microchip.com wrote:
>> -----Original Message-----
>> From: Florian Fainelli <f.fainelli@gmail.com>
>> Sent: Wednesday, August 15, 2018 5:29 PM
>> To: Tristram Ha - C24268 <Tristram.Ha@microchip.com>; Andrew Lunn
>> <andrew@lunn.ch>; Pavel Machek <pavel@ucw.cz>; Ruediger Schmitt
>> <ruediger.schmitt@philips.com>
>> Cc: Arkadi Sharshevsky <arkadis@mellanox.com>; UNGLinuxDriver
>> <UNGLinuxDriver@microchip.com>; netdev@vger.kernel.org
>> Subject: Re: [PATCH v2 net-next 0/8] net: dsa: microchip: Modify KSZ9477
>> DSA driver in preparation to add other KSZ switch drivers
>>
>> On 12/05/2017 05:46 PM, Tristram.Ha@microchip.com wrote:
>>> 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.
>>
>> Is something gating this series from getting included? It's been nearly
>> 8 months now and this has not been include nor resubmitted, any plans to
>> rebase that patch series and work towards inclusion in net-next when it
>> opens back again?
>>
>> Thank you!
>
> Sorry for the long delay. I will restart my kernel submission effort next month
> after finishing the work on current development project.
>
Tristram, any chance of resubmitting this or should someone with access
to those switches take up your series and submit it?
--
Florian
prev parent reply other threads:[~2018-10-20 2:03 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-12-06 1:46 [PATCH v2 net-next 0/8] net: dsa: microchip: Modify KSZ9477 DSA driver in preparation to add other KSZ switch drivers Tristram.Ha
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 [this message]
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=b4393977-7b5a-89a3-d7a2-8936457166dd@gmail.com \
--to=f.fainelli@gmail.com \
--cc=Tristram.Ha@microchip.com \
--cc=UNGLinuxDriver@microchip.com \
--cc=andrew@lunn.ch \
--cc=arkadis@mellanox.com \
--cc=marek.vasut@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).