Hi Nicolas,
Please submit a patch for renaming rte_acc100_pmd.h to acc100_pmd.h as it is an internal file. > > > v7: updates suggested by Akhil and Maxime. Removed RTE prefix for internal > > #define, removed > > build-time single FEC engine configuration mode, documentation fix, coding > > guideline fix, renamed new file without rte prefix. > > > > v6: use formally an unified driver for AC100 and AC101. Exposing single > > unified > > function for the PF device configuration used by bbdev-test. > > > > v5: minor fix to doc and intermediate commit caught in CI > > > > v4: incremental updates based on reviews Changed order of patchset to > include > > two ACC100 specific commits first: updating formally the companion configure > > function from PF and the protection adjustment. > > Then next 3 commits for the actual changes specific to ACC101: the device > > specific function for FCW 5GUL generation used as a local function, combined > > documentations into a single .rst common to both devices, updated > comments, > > removed unused #defines. > > > > v3: Feedback from Tom Rix: missing copyright, refactor bbdev-test section > > calling the configure companion function for ACC100/101, taking the Pmon > > commit out which is not directly required. > > > > v2: Based on good feedback from Thomas and David, now implementing the > > ACC101 PMD as a close derivative from existing ACC100 PMD with hooks to > > have different behaviour based on variant. > > This prevents code duplication and only rely on different functions and > > behaviour when hardware requires or support differences. > > Note that these are pending changes for ACC100 which would be specific to > that > > device and not ACC101 but these can be managed based on the new > > implementation, ie. is_acc100() etc... (such incremental changes for ACC100 > > trending 22.11 but confirming this is future proof). > > The serie also includes commits which were meant for ACC101 but are also > > valuable for ACC100. > > > > v1: This serie introduces the PMD for the new bbdev device ACC101 (aka > Mount > > Cirrus). > > This is a derivative from previous Mount Bryce ACC100 which includes silicon > > improvement, bug fixes, capacity improvement for 5GNR and feature > > improvement. > > > > > > Nicolas Chautru (6): > > baseband/acc100: update companion PF configure function > > baseband/acc100: add protection for some negative scenario > > baseband/acc100: remove RTE prefix for internal macro > > baseband/acc100: introduce PMD for ACC101 > > baseband/acc100: modify validation code for ACC101 > > baseband/acc100: configuration of ACC101 from PF > > > > MAINTAINERS | 1 + > > app/test-bbdev/test_bbdev_perf.c | 6 +- > > doc/guides/bbdevs/acc100.rst | 37 +- > > doc/guides/bbdevs/features/acc101.ini | 13 + > > doc/guides/rel_notes/release_22_07.rst | 3 + > > drivers/baseband/acc100/acc100_pf_enum.h | 18 + > > drivers/baseband/acc100/acc101_pmd.h | 50 +++ > > drivers/baseband/acc100/rte_acc100_cfg.h | 4 +- > > drivers/baseband/acc100/rte_acc100_pmd.c | 687 > > ++++++++++++++++++++++++++++--- > > drivers/baseband/acc100/rte_acc100_pmd.h | 37 +- > > drivers/baseband/acc100/version.map | 3 +- > > 11 files changed, 768 insertions(+), 91 deletions(-) > > create mode 100644 doc/guides/bbdevs/features/acc101.ini > > create mode 100644 drivers/baseband/acc100/acc101_pmd.h > > Series applied to dpdk-next-crypto > > Thanks.