On Tue, Aug 28, 2018 at 5:40 PM Arnd Bergmann <[email protected]> wrote: > > On Tue, Aug 28, 2018 at 12:58 PM <[email protected]> wrote: > > > > From: Sunil Goutham <[email protected]> > > > > This patch adds basic template for Marvell OcteonTX2's > > CGX ethernet interface driver. Just the probe. > > RVU AF driver will use APIs exported by this driver > > for various things like PF to physical interface mapping, > > loopback mode, interface stats etc. > > > > Signed-off-by: Sunil Goutham <[email protected]> > > --- > > drivers/soc/marvell/Kconfig | 10 +++ > > drivers/soc/marvell/octeontx2/Makefile | 2 + > > drivers/soc/marvell/octeontx2/cgx.c | 117 > > +++++++++++++++++++++++++++++++++ > > drivers/soc/marvell/octeontx2/cgx.h | 20 ++++++ > > If this is a regular PCI ethernet driver, why do you put it into driver/soc > rather than drivers/net/ethernet/ ?
No, this is not a ethernet driver, as mentioned in the cover letter this driver and AF driver doesn't handle any IO. There will be a separate ethernet driver (will submit that as well in future) which will communicate with these drivers for configuring hardware. The driver in question here is for a serdes controller which handles physical ethernet interfaces. Admin function driver gathers info w.r.t current state of physical ethernet interfaces from this driver and notifies actual ethernet driver about changes, if any. Thanks, Sunil. > > Arnd
