> -----Original Message-----
> From: Stephen Hemminger <step...@networkplumber.org>
> Sent: 27 December 2021 23:21
> To: Nipun Gupta <nipun.gu...@nxp.com>
> Cc: dev@dpdk.org; tho...@monjalon.net; ferruh.yi...@intel.com; Hemant
> Agrawal <hemant.agra...@nxp.com>
> Subject: Re: [PATCH v2 00/16] features and fixes on NXP eth devices
>
> On Mon, 27 Dec 2021 21:46:29 +0530
> nipun.gu...@nxp.com wrote:
>
> > From: Nipun Gupta <nipun.gu...@nxp.com>
> >
> > This series adds few features and important fixes on DPAA,
> > PFE and ENETC devices.
> >
> > Features added:
> > - level 2 support for shaping on DPAA2
> > - loopback configuration for DPNI devices on DPAA2
> > - Multiple TXQ's enqueue for ordered queues for performance
> > - VFs support on ENETC
> >
> > Fixes:
> > - fix unregistering interrupt handler on DPAA2
> > - fix timestamping for IEEE1588 on DPAA1
> >
> > Changes in v2:
> > - fix checkpatch errors
> >
> > Apeksha Gupta (1):
> > net/pfe: remove setting unused value
> >
> > Gagandeep Singh (4):
> > net/dpaa2: add support for level 2 in traffic management
> > net/enetc: add support for VFs
> > net/pfe: disable HW CRC stripping
> > net/pfe: reduce driver initialization time
> >
> > Jun Yang (4):
> > net/dpaa2: support multiple txqs en-queue for ordered
> > net/dpaa2: secondary process handling for dpni
> > bus/fslmc: add and scan dprc devices
> > net/dpaa2: support recycle loopback port
> >
> > Nipun Gupta (4):
> > bus/fslmc: update MC to 10.29
> > bus/fslmc: use dmb oshst for synchronization before I/O
> > net/dpaa: check status before configuring shared MAC
> > net/dpaa: enable checksum for shared MAC interface
> >
> > Rohit Raj (1):
> > net/dpaa2: warn user in case of high nb desc
> >
> > Vanshika Shukla (2):
> > net/dpaa2: fix unregistering interrupt handler
> > net/dpaa2: fix timestamping for IEEE1588
> >
> > doc/guides/nics/dpaa2.rst | 2 +-
> > drivers/bus/dpaa/base/fman/fman_hw.c | 11 +
> > drivers/bus/dpaa/include/fsl_fman.h | 2 +
> > drivers/bus/dpaa/version.map | 1 +
> > drivers/bus/fslmc/fslmc_bus.c | 15 +-
> > drivers/bus/fslmc/fslmc_vfio.c | 18 +-
> > drivers/bus/fslmc/mc/dprc.c | 129 ++++
> > drivers/bus/fslmc/mc/fsl_dpmng.h | 2 +-
> > drivers/bus/fslmc/mc/fsl_dprc.h | 46 ++
> > drivers/bus/fslmc/mc/fsl_dprc_cmd.h | 48 ++
> > drivers/bus/fslmc/meson.build | 4 +-
> > drivers/bus/fslmc/portal/dpaa2_hw_dprc.c | 100 +++
> > drivers/bus/fslmc/portal/dpaa2_hw_pvt.h | 15 +-
> > drivers/bus/fslmc/qbman/include/compat.h | 4 +-
> > drivers/bus/fslmc/rte_fslmc.h | 10 +-
> > drivers/event/dpaa2/dpaa2_eventdev.c | 12 +-
> > drivers/mempool/dpaa2/dpaa2_hw_mempool.c | 23 +
> > drivers/mempool/dpaa2/rte_dpaa2_mempool.h | 15 +
> > drivers/mempool/dpaa2/version.map | 1 +
> > drivers/net/dpaa/dpaa_ethdev.c | 17 +-
> > drivers/net/dpaa2/base/dpaa2_hw_dpni.c | 5 +-
> > drivers/net/dpaa2/dpaa2_ethdev.c | 117 +++-
> > drivers/net/dpaa2/dpaa2_ethdev.h | 38 +-
> > drivers/net/dpaa2/dpaa2_ptp.c | 8 +-
> > drivers/net/dpaa2/dpaa2_recycle.c | 780 ++++++++++++++++++++++
> > drivers/net/dpaa2/dpaa2_rxtx.c | 181 ++++-
> > drivers/net/dpaa2/dpaa2_tm.c | 563 +++++++++++++---
> > drivers/net/dpaa2/dpaa2_tm.h | 17 +-
> > drivers/net/dpaa2/mc/dpdmux.c | 8 +
> > drivers/net/dpaa2/mc/dpkg.c | 7 +-
> > drivers/net/dpaa2/mc/dpni.c | 417 ++++++++----
> > drivers/net/dpaa2/mc/fsl_dpdmux.h | 3 +
> > drivers/net/dpaa2/mc/fsl_dpdmux_cmd.h | 5 +-
> > drivers/net/dpaa2/mc/fsl_dpni.h | 173 +++--
> > drivers/net/dpaa2/mc/fsl_dpni_cmd.h | 137 ++--
> > drivers/net/dpaa2/meson.build | 1 +
> > drivers/net/dpaa2/version.map | 1 +
> > drivers/net/enetc/enetc_ethdev.c | 25 +-
> > drivers/net/pfe/pfe_ethdev.c | 10 +-
> > drivers/net/pfe/pfe_hal.c | 4 +-
> > drivers/net/pfe/pfe_hif.c | 6 +-
> > 41 files changed, 2528 insertions(+), 453 deletions(-)
> > create mode 100644 drivers/bus/fslmc/mc/dprc.c
> > create mode 100644 drivers/bus/fslmc/mc/fsl_dprc.h
> > create mode 100644 drivers/bus/fslmc/mc/fsl_dprc_cmd.h
> > create mode 100644 drivers/bus/fslmc/portal/dpaa2_hw_dprc.c
> > create mode 100644 drivers/net/dpaa2/dpaa2_recycle.c
> >
>
> In general best to split the patch series by device not by vendor.
> That way it is easier to review and changes to one device can go in
> if there are still issues with other devices
Agree !! Will send it separately from next time. This time though I
would send like to them in this only as otherwise it would break the
versioning.
Thanks,
Nipun