From: Tobias Waldekranz <email@example.com> To: Mickey Rachamim <firstname.lastname@example.org>, Jakub Kicinski <email@example.com>, Vadym Kochan <firstname.lastname@example.org> Cc: Andrew Lunn <email@example.com>, "David S. Miller" <firstname.lastname@example.org>, "netdev\@vger.kernel.org" <email@example.com>, "linux-kernel\@vger.kernel.org" <firstname.lastname@example.org>, Vladimir Oltean <email@example.com> Subject: RE: [EXT] Re: [PATCH net-next 5/7] net: marvell: prestera: add LAG support Date: Tue, 09 Feb 2021 22:34:07 +0100 [thread overview] Message-ID: <firstname.lastname@example.org> (raw) In-Reply-To: <BN6PR18MB158781B17E633670912AEED6BA8E9@BN6PR18MB1587.namprd18.prod.outlook.com> On Tue, Feb 09, 2021 at 20:31, Mickey Rachamim <email@example.com> wrote: > Hi Andrew, Jakub, Tobias, > > On Tuesday, February 9, 2021 7:35 PM Jakub Kicinski wrote: >> Sounds like we have 3 people who don't like FW-heavy designs dominating the kernel - this conversation can only go one way. >> Marvell, Plvision anything to share? AFAIU the values of Linux kernel are open source, healthy community, empowering users. With the SDK on the embedded CPU your driver does not seem to tick any of these boxes. > > I'll try to share Marvell's insight and plans regarding our Prestera drivers; > > We do understand the importance and the vision behind the open-source community - while being committed to quality, functionality and the developers/end-users. > > We started working on the Prestera driver in Q2 2019. it took us more than a year to get the first approved driver into 5.10, and we just started. > Right at the beginning - we implemented PP function into the Kernel driver like the SDMA operation (This is the RX/TX DMA engine). > Yet, the FW itself - is an SW package that supports many Marvell Prestera Switching families of devices - this is a significant SW package that will take many working years to adapt to the Kernel environment. > We do plan to port more and more PP functions as Kernel drivers along the way. This is very encouraging to hear. I understand that it is a massive undertaking. > We also are working with the community to extend Kernel functionality with a new feature beneficial to all Kernel users (e.g. Devlink changes) and we will continue to do it. > By extending the Prestera driver to in-kernel implementation with more PP features - we will simplify the FW logic and enables cost-effective solutions to the market/developers. Until that day arrives, are there any chances of Marvell opening up CPSS in the same way DSDT was re-licensed some years back? Being able to clone github.com/Marvell-switching/prestera-firmware (or whatever) and build the firmware from source would go a long way to alleviate my fears at least. In such a world, I at least have a chance of debugging any issue all the way to the bottom of the stack. It would also make it possible for the community to help out with the porting effort. > Regards, > Mickey.
next prev parent reply other threads:[~2021-02-10 1:16 UTC|newest] Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-02-03 16:54 [PATCH net-next 0/7] Marvell Prestera Switchdev misc updates Vadym Kochan 2021-02-03 16:54 ` [PATCH net-next 1/7] net: marvell: prestera: bump supported firmware version to 2.5 Vadym Kochan 2021-02-03 16:54 ` [PATCH net-next 2/7] net: marvell: prestera: disable events interrupt while handling Vadym Kochan 2021-02-05 5:10 ` Jakub Kicinski 2021-02-05 11:28 ` Vadym Kochan 2021-02-03 16:54 ` [PATCH net-next 3/7] net: marvell: prestera: add support for AC3X 98DX3265 device Vadym Kochan 2021-02-03 16:54 ` [PATCH net-next 4/7] net: marvell: prestera: move netdev topology validation to prestera_main Vadym Kochan 2021-02-05 14:09 ` Vladimir Oltean 2021-02-03 16:54 ` [PATCH net-next 5/7] net: marvell: prestera: add LAG support Vadym Kochan 2021-02-05 5:16 ` Jakub Kicinski 2021-02-08 19:54 ` Tobias Waldekranz 2021-02-08 21:05 ` Jakub Kicinski 2021-02-08 22:30 ` Andrew Lunn 2021-02-09 12:37 ` Tobias Waldekranz 2021-02-09 11:56 ` Tobias Waldekranz 2021-02-09 17:48 ` Jakub Kicinski 2021-02-09 13:58 ` Andrew Lunn 2021-02-09 17:35 ` Jakub Kicinski 2021-02-09 20:31 ` [EXT] " Mickey Rachamim 2021-02-09 21:34 ` Tobias Waldekranz [this message] 2021-02-10 10:41 ` Mickey Rachamim 2021-02-10 21:44 ` Tobias Waldekranz 2021-02-10 0:28 ` Andrew Lunn 2021-02-10 10:42 ` Mickey Rachamim 2021-02-10 19:25 ` Jakub Kicinski 2021-02-10 20:52 ` Taras Chornyi 2021-02-05 15:24 ` Vladimir Oltean 2021-02-03 16:54 ` [PATCH net-next 6/7] net: marvell: prestera: align flood setting according to latest firmware version Vadym Kochan 2021-02-03 16:54 ` [PATCH net-next 7/7] net: marvell: prestera: fix port event handling on init Vadym Kochan 2021-02-05 5:19 ` Jakub Kicinski 2021-02-05 12:31 ` Vadym Kochan
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 \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --subject='RE: [EXT] Re: [PATCH net-next 5/7] net: marvell: prestera: add LAG support' \ /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
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).