[PATCH 00/15] soc: octeontx2: Add RVU admin function driver
From: andrew@lunn.ch (Andrew Lunn)
Date: 2018-08-30 13:27:12
Also in:
lkml, netdev
From: andrew@lunn.ch (Andrew Lunn)
Date: 2018-08-30 13:27:12
Also in:
lkml, netdev
quoted
quoted
My feeling overall is that we need a review from the network driver folks more than the arm-soc team etc, and that maybe the driver as a whole should go into drivers/net/ethernet.This driver doesn't handle any network IO and moreever this driver has to handle configuration requests from crypto driver as well. There will be separate network and crypto drivers which will be upstreamed into drivers/net/ethernet and drivers/crypto. And in future silicons there will be different types of functional blocks which will be added into this resource virtualization unit (RVU). Hence i thought this driver is not a right fit in drivers/net/ethernet.
Hi Sunil
Do you have a git branch for everything? I would like to look at the
actual Ethernet driver, and the full API this driver exports to other
drivers.
I think there real question here is, do you have split between this
driver and the actual device drivers in the right place? For me, link
up/down detection should be in the Ethernet driver, since it is not
shared with the crypto driver.
Thanks
Andrew