RE: [EXTERNAL] [PATCH v2 0/9] DPAA2 crypto driver changes

[Gagandeep Singh <G.Singh@nxp.com>]

Hi,

> -----Original Message-----
> From: Akhil Goyal <gakhil@marvell.com>
> Sent: Wednesday, May 21, 2025 5:30 PM
> To: Gagandeep Singh <G.Singh@nxp.com>; dev@dpdk.org
> Cc: Hemant Agrawal <hemant.agrawal@nxp.com>
> Subject: RE: [EXTERNAL] [PATCH v2 0/9] DPAA2 crypto driver changes
> 
> > V2 changes:
> >  - fix checkpatch warning
> >  - fix 32 bit compilation error
> >  - fix a commit message
> >  - update document
> >
> > Gagandeep Singh (5):
> >   common/dpaax: fix invalid key command error
> >   common/dpaax: fix for PDCP AES only 12bit SN case
> >   common/dpaax: support 12bit SN in pdcp uplane
> >   crypto/dpaa2_sec: change custom device API to standard
> >   crypto/dpaa2_sec: add null algo capability
> >
> > Jun Yang (3):
> >   net/dpaa2: configure buffer layout
> >   mempool/dpaa2: mempool operation index
> >   crypto/dpaa2_sec: add support for simple IPsec FD
> >
> > Vanshika Shukla (1):
> >   crypto/dpaa2_sec: fix coverity Issues
> 
> Split the patchset for appropriate trees. There are patches for net and mempool
> which will not go in crypto tree. And I don't see them as dependent patches for
> crypto.

The patch "crypto/dpaa2_sec: add support for simple IPsec FD" is dependent on
both " net/dpaa2: configure buffer layout " and " mempool/dpaa2: mempool operation index ".
using API " rte_dpaa2_mpool_get_ops_idx " and Macro " DPAA2_DYN_TX_MIN_FD_OFFSET "
defined in these two patches.
Without these patches build will break.

> 
> >
> >  doc/guides/cryptodevs/dpaa2_sec.rst          |   2 +
> >  doc/guides/cryptodevs/features/dpaa2_sec.ini |   2 +
> >  drivers/bus/fslmc/portal/dpaa2_hw_pvt.h      |  10 +
> >  drivers/common/dpaax/caamflib/desc/pdcp.h    |  31 +++-
> >  drivers/crypto/dpaa2_sec/dpaa2_sec_dpseci.c  | 182 ++++++++++++-------
> >  drivers/crypto/dpaa2_sec/dpaa2_sec_priv.h    |  45 ++++-
> >  drivers/crypto/dpaa2_sec/meson.build         |   3 +-
> >  drivers/crypto/dpaa_sec/dpaa_sec.c           |  42 ++---
> >  drivers/mempool/dpaa2/dpaa2_hw_mempool.c     |  20 +-
> >  drivers/mempool/dpaa2/dpaa2_hw_mempool.h     |   5 +-
> >  drivers/net/dpaa2/base/dpaa2_hw_dpni.c       |  18 +-
> >  drivers/net/dpaa2/base/dpaa2_hw_dpni_annot.h |   6 +
> >  drivers/net/dpaa2/dpaa2_ethdev.h             |   6 +-
> >  13 files changed, 263 insertions(+), 109 deletions(-)
> >
> > --
> > 2.25.1