Messages in this thread | | | From | Razvan Stefanescu <> | Subject | RE: [RESEND PATCH 2/6] staging: fsl-dpaa2/ethsw: Add Freescale DPAA2 Ethernet Switch driver | Date | Mon, 2 Oct 2017 07:18:52 +0000 |
| |
> -----Original Message----- > From: Florian Fainelli [mailto:f.fainelli@gmail.com] > Sent: Friday, September 29, 2017 19:11 > To: Razvan Stefanescu <razvan.stefanescu@nxp.com>; Bogdan Purcareata > <bogdan.purcareata@nxp.com>; gregkh@linuxfoundation.org > Cc: devel@driverdev.osuosl.org; linux-kernel@vger.kernel.org; > netdev@vger.kernel.org; agraf@suse.de; arnd@arndb.de; Alexandru Marginean > <alexandru.marginean@nxp.com>; Ruxandra Ioana Radulescu > <ruxandra.radulescu@nxp.com>; Laurentiu Tudor <laurentiu.tudor@nxp.com>; > stuyoder@gmail.com > Subject: RE: [RESEND PATCH 2/6] staging: fsl-dpaa2/ethsw: Add Freescale DPAA2 > Ethernet Switch driver > > On September 29, 2017 6:59:18 AM PDT, Razvan Stefanescu > <razvan.stefanescu@nxp.com> wrote: > > > > > >> -----Original Message----- > >> From: Bogdan Purcareata > >> Sent: Friday, September 29, 2017 16:36 > >> To: Razvan Stefanescu <razvan.stefanescu@nxp.com>; > >> gregkh@linuxfoundation.org > >> Cc: devel@driverdev.osuosl.org; linux-kernel@vger.kernel.org; > >> netdev@vger.kernel.org; agraf@suse.de; arnd@arndb.de; Alexandru > >Marginean > >> <alexandru.marginean@nxp.com>; Ruxandra Ioana Radulescu > >> <ruxandra.radulescu@nxp.com>; Laurentiu Tudor > ><laurentiu.tudor@nxp.com>; > >> stuyoder@gmail.com > >> Subject: RE: [RESEND PATCH 2/6] staging: fsl-dpaa2/ethsw: Add > >Freescale DPAA2 > >> Ethernet Switch driver > >> > >> > Introduce the DPAA2 Ethernet Switch driver, which manages Datapath > >Switch > >> > (DPSW) objects discovered on the MC bus. > >> > > >> > Suggested-by: Alexandru Marginean <alexandru.marginean@nxp.com> > >> > Signed-off-by: Razvan Stefanescu <razvan.stefanescu@nxp.com> > > This looks pretty good for a new switchdev driver, is there a reason you can't > target drivers/net/ethernet instead of staging? Is it because the MC bus code is > still in staging (AFAICT)? > Yes, driver depends on MC bus, which is still in staging. Also, control traffic code will require access to DPIO functions, that are also in staging.
Best regards, Razvan S. > -- > Florian
| |