DPDK patches and discussions
 help / color / mirror / Atom feed
From: Maxime Coquelin <maxime.coquelin@redhat.com>
To: Nicolas Chautru <nicolas.chautru@intel.com>, dev@dpdk.org
Cc: hemant.agrawal@nxp.com, david.marchand@redhat.com,
	hernan.vargas@intel.com
Subject: Re: [PATCH v6 00/12] VRB2 bbdev PMD introduction
Date: Thu, 12 Oct 2023 14:53:13 +0200	[thread overview]
Message-ID: <9f7890a8-33b0-ddf4-67f2-7c2d6268c091@redhat.com> (raw)
In-Reply-To: <20231010202049.771959-1-nicolas.chautru@intel.com>



On 10/10/23 22:20, Nicolas Chautru wrote:
> v6: removed one empty line typo.
>      Thanks
> 
> v5: missed a line in v4 patch 2/12.
>      change to 9/12 with suggested refactor.
> 
> v4: updates based on Maxime's suggestions:
> - FFT windowing exposure tweaked to show number of windows
> as well in capacity to be more generic and future-proof.
> - few code refactor notably to avoid code duplication
> - cosmetic and doc update
> 
> v3: updates based on v2 review:
> - split into smaller incremental commits
> - FFT windowing exposed through a more generic structure
> - refactor using wrapper functions to manage device variants
> - removed custom dump function
> - consider the request unsupport SO option as an error
> instead of fall-back.
> - cosmetic and doc update.
> Thanks
> 
> v2: doc, comments and commit-log updates.
> 
> This serie includes changes to the VRB BBDEV PMD for 23.11.
> 
> This allows the VRB unified driver to support the new VRB2
> implementation variant on GNR-D.
> 
> This also include minor change to the dev_info to expose FFT version
> flexibility to expose information to the application on what windows
> LUT is configured dynamically on the device.
> 
> Nicolas Chautru (12):
>    bbdev: add FFT window width member in driver info
>    baseband/acc: add FFT window width in the VRB PMD
>    baseband/acc: remove the 4G SO capability for VRB1
>    baseband/acc: allocate FCW memory separately
>    baseband/acc: add support for MLD operation
>    baseband/acc: refactor to allow unified driver extension
>    baseband/acc: adding VRB2 device variant
>    baseband/acc: add FEC capabilities for the VRB2 variant
>    baseband/acc: add FFT support to VRB2 variant
>    baseband/acc: add MLD support in VRB2 variant
>    baseband/acc: add support for VRB2 engine error detection
>    baseband/acc: add configure helper for VRB2
> 
>   doc/guides/bbdevs/features/vrb2.ini    |   14 +
>   doc/guides/bbdevs/index.rst            |    1 +
>   doc/guides/bbdevs/vrb1.rst             |    4 -
>   doc/guides/bbdevs/vrb2.rst             |  206 ++++
>   doc/guides/rel_notes/release_23_11.rst |    3 +
>   drivers/baseband/acc/acc100_pmd.h      |    2 +
>   drivers/baseband/acc/acc_common.h      |  185 ++-
>   drivers/baseband/acc/rte_acc100_pmd.c  |    6 +-
>   drivers/baseband/acc/rte_vrb_pmd.c     | 1571 +++++++++++++++++++++---
>   drivers/baseband/acc/vrb1_pf_enum.h    |   17 +-
>   drivers/baseband/acc/vrb2_pf_enum.h    |  124 ++
>   drivers/baseband/acc/vrb2_vf_enum.h    |  121 ++
>   drivers/baseband/acc/vrb_cfg.h         |   16 +
>   drivers/baseband/acc/vrb_pmd.h         |  173 ++-
>   lib/bbdev/rte_bbdev.h                  |    2 +
>   lib/bbdev/rte_bbdev_op.h               |    2 +
>   16 files changed, 2258 insertions(+), 189 deletions(-)
>   create mode 100644 doc/guides/bbdevs/features/vrb2.ini
>   create mode 100644 doc/guides/bbdevs/vrb2.rst
>   create mode 100644 drivers/baseband/acc/vrb2_pf_enum.h
>   create mode 100644 drivers/baseband/acc/vrb2_vf_enum.h
> 

Applied to next-baseband/for-main.

Thanks,
Maxime


      parent reply	other threads:[~2023-10-12 12:53 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-10-10 20:20 Nicolas Chautru
2023-10-10 20:20 ` [PATCH v6 01/12] bbdev: add FFT window width member in driver info Nicolas Chautru
2023-10-10 20:20 ` [PATCH v6 02/12] baseband/acc: add FFT window width in the VRB PMD Nicolas Chautru
2023-10-10 20:20 ` [PATCH v6 03/12] baseband/acc: remove the 4G SO capability for VRB1 Nicolas Chautru
2023-10-10 20:20 ` [PATCH v6 04/12] baseband/acc: allocate FCW memory separately Nicolas Chautru
2023-10-10 20:20 ` [PATCH v6 05/12] baseband/acc: add support for MLD operation Nicolas Chautru
2023-10-10 20:20 ` [PATCH v6 06/12] baseband/acc: refactor to allow unified driver extension Nicolas Chautru
2023-10-10 20:20 ` [PATCH v6 07/12] baseband/acc: adding VRB2 device variant Nicolas Chautru
2023-10-10 20:20 ` [PATCH v6 08/12] baseband/acc: add FEC capabilities for the VRB2 variant Nicolas Chautru
2023-10-10 20:20 ` [PATCH v6 09/12] baseband/acc: add FFT support to " Nicolas Chautru
2023-10-10 20:20 ` [PATCH v6 10/12] baseband/acc: add MLD support in " Nicolas Chautru
2023-10-10 20:20 ` [PATCH v6 11/12] baseband/acc: add support for VRB2 engine error detection Nicolas Chautru
2023-10-10 20:20 ` [PATCH v6 12/12] baseband/acc: add configure helper for VRB2 Nicolas Chautru
2023-10-12 12:53 ` Maxime Coquelin [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=9f7890a8-33b0-ddf4-67f2-7c2d6268c091@redhat.com \
    --to=maxime.coquelin@redhat.com \
    --cc=david.marchand@redhat.com \
    --cc=dev@dpdk.org \
    --cc=hemant.agrawal@nxp.com \
    --cc=hernan.vargas@intel.com \
    --cc=nicolas.chautru@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).