On 9/15/2022 11:44 AM, Chaoyong He wrote:
* Changes since v8
- Update the nfp.rst
- Fix the 'app_hw' to 'app_fw'
- Remove the ovs compatible header file
- Remove the use of
rte_eth_dev_configure()/rte_eth_rx_burst()/rte_eth_dev_start() API
* Changes since v7
- Adjust the logics to make sure not break the pci probe process
- Change 'app' to 'app_fw' in all logics to avoid confuse
- Fix problem about log level
* Changes since v6
- Fix the compile error
* Changes since v5
- Compare integer with 0 explicitly
- Change helper macro to function
- Implement the dummy functions
- Remove some unnecessary logics
* Changes since v4
- Remove the unneeded '__rte_unused' attribute
- Fixup a potential memory leak problem
* Changes since v3
- Add the 'Depends-on' tag
* Changes since v2
- Remove the use of rte_panic()
* Changes since v1
- Fix the compile error
Depends-on: series-23707 ("Add support of NFP3800 chip and firmware with NFDk")
Chaoyong He (12):
net/nfp: move app specific attributes to own struct
net/nfp: simplify initialization and remove dead code
net/nfp: move app specific init logic to own function
net/nfp: add initial flower firmware support
net/nfp: add flower PF setup logic
net/nfp: add flower PF related routines
net/nfp: add flower ctrl VNIC related logics
net/nfp: move common rxtx function for flower use
net/nfp: add flower ctrl VNIC rxtx logic
net/nfp: add flower representor framework
net/nfp: move rxtx function to header file
net/nfp: add flower PF rxtx logic
Hi Chaoyong,
Patchset looks good, except from two issues we have discussed before,
those issues are:
* Creating a new ethdev just for driver-FW control communication
* Application (OvS) specific code in the driver
I commented them separately and cc'ed more folks, we can proceed when
above are resolved.
Thanks,
ferruh
