RE: [PATCH v1 0/3] bbdev: remove offload cost

2023-01-03 Thread Chautru, Nicolas
Hi Maxime, Happy new year. What does the state " Awaiting Upstream" mean in patchwork for that serie? Can this be applied? Thanks Nic > -Original Message- > From: Chautru, Nicolas > Sent: Wednesday, December 14, 2022 3:34 PM > To: dev@dpdk.org; maxime.co

RE: [PATCH v1 0/3] bbdev: remove offload cost

2023-01-04 Thread Chautru, Nicolas
lag 51d49ccd23 test/bbdev: fix build issue with optional build flag I don’t see the last 3 commits from patchwork (ie. the serie from that email) [v1,3/3] app/bbdev-test: remove offload cost optional build flag bbdev: remove offload cost - - - 3-- 2022-12-14 Chautru, Nico

RE: [PATCH v1 6/6] baseband/acc: acc100 fix queue mapping to 64 bits

2023-01-04 Thread Chautru, Nicolas
> From: Vargas, Hernan > > Fix potential overflow for Q mapping extension to 64 bits. > > Fixes: 32e8b7ea35d ("baseband/acc100: refactor to segregate common > code") > > Signed-off-by: Hernan Vargas > --- (...) For the series, thanks Reviewed-by: Nicolas Chautru

RE: [PATCH v1 1/3] test/bbdev: explicit check for allocation failure

2023-01-06 Thread Chautru, Nicolas
Hi Maxime, Yes this new commit fixes the last commit on the tree. You might want to merge these 2 commits into one or not. I mentioned this in the cover letter I believe. Thanks Nic > -Original Message- > From: Maxime Coquelin > Sent: Friday, January 6, 2023 6:51 AM >

RE: no printf in drivers code

2023-02-03 Thread Chautru, Nicolas
> From: Thomas Monjalon > > Hello, > > We have too many drivers using printf() instead of rte_log(). > Please propose a plan to replace or remove the calls to printf(). > If no plan, I could propose one but you may not like it :) > > Affected drivers are: > baseband/acc Will push a pat

RE: [EXT] [PATCH v1 5/5] baseband/fpga_5gnr_fec: remove filler from HARQ

2022-05-19 Thread Chautru, Nicolas
optimization (no functional impact), no change required on stable branches. Thanks Nic > -Original Message- > From: Akhil Goyal > Sent: Monday, May 16, 2022 11:07 AM > To: Vargas, Hernan ; dev@dpdk.org; > t...@redhat.com > Cc: Chautru, Nicolas ; Zhang, Qi Z > > S

RE: [EXT] [PATCH v1 2/5] baseband/fpga_5gnr_fec: add FPGA Mutex

2022-05-19 Thread Chautru, Nicolas
ented through MMIO exposed per VF and common to all queues unlike the default enqueue operation where each queues are isolated by design. > -Original Message- > From: Akhil Goyal > Sent: Monday, May 16, 2022 11:09 AM > To: Vargas, Hernan ; dev@dpdk.org; > t...@redhat.com &g

RE: [PATCH v3 3/4] baseband/acc100: configuration of ACC101 from PF

2022-05-23 Thread Chautru, Nicolas
Hi Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Thursday, May 19, 2022 1:14 PM > To: Chautru, Nicolas ; dev@dpdk.org; > gak...@marvell.com; t...@redhat.com > Cc: tho...@monjalon.net; Kinsella, Ray ; > Richardson, Bruce ; > hemant.agra...

RE: [PATCH v2 1/5] baseband/acc100: introduce PMD for ACC101

2022-05-23 Thread Chautru, Nicolas
Hi Tom, I realize I had not replied to that one > -Original Message- > From: Tom Rix > Sent: Tuesday, May 10, 2022 4:56 AM > To: Chautru, Nicolas ; dev@dpdk.org; > gak...@marvell.com > Cc: tho...@monjalon.net; Kinsella, Ray ; > Richardson, Bruce ; > heman

RE: [PATCH v2 5/5] baseband/fpga_5gnr_fec: remove filler from HARQ

2022-05-23 Thread Chautru, Nicolas
Thanks Hernan > -Original Message- > From: Vargas, Hernan > Sent: Thursday, May 19, 2022 8:06 PM > To: dev@dpdk.org; gak...@marvell.com; t...@redhat.com > Cc: Chautru, Nicolas ; Zhang, Qi Z > ; Vargas, Hernan > Subject: [PATCH v2 5/5] baseband/fpga_5gnr_fec: re

RE: [PATCH v1] bbdev: add new operation for FFT processing

2022-05-25 Thread Chautru, Nicolas
/feedback. This can be used for vanilla FFT and/or chained FFT/iFFT + point-wise multiplication making it applicable for SRS processing. Thanks, Nic > -Original Message- > From: Chautru, Nicolas > Sent: Thursday, March 10, 2022 3:49 PM > To: dev@dpdk.org; gak...@ma

RE: [PATCH v5 5/5] baseband/acc100: configuration of ACC101 from PF

2022-05-25 Thread Chautru, Nicolas
Hi Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Wednesday, May 25, 2022 6:25 AM > To: Chautru, Nicolas ; dev@dpdk.org; > gak...@marvell.com; t...@redhat.com > Cc: tho...@monjalon.net; Kinsella, Ray ; > Richardson, Bruce ; > hemant.agra...

RE: [PATCH v5 4/5] baseband/acc100: modify validation code for ACC101

2022-05-25 Thread Chautru, Nicolas
> -Original Message- > From: Maxime Coquelin > Sent: Wednesday, May 25, 2022 7:33 AM > To: Chautru, Nicolas ; dev@dpdk.org; > gak...@marvell.com; t...@redhat.com > Cc: tho...@monjalon.net; Kinsella, Ray ; > Richardson, Bruce ; > hemant.agra...@nxp.com; Vargas

RE: [EXT] [PATCH v2 0/5] baseband/fpga_5gnr: maintenance changes to fpga_5gnr PMD

2022-05-31 Thread Chautru, Nicolas
pdk.org; > t...@redhat.com; Chautru, Nicolas > Cc: Zhang, Qi Z > Subject: RE: [EXT] [PATCH v2 0/5] baseband/fpga_5gnr: maintenance changes > to fpga_5gnr PMD > > Hi Nicolas, > > Can you review the series and ack? > > Regards, > Akhil > > > Few PMD c

RE: [PATCH v6 4/5] baseband/acc100: modify validation code for ACC101

2022-05-31 Thread Chautru, Nicolas
Hi Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Tuesday, May 31, 2022 1:03 AM > To: Chautru, Nicolas ; dev@dpdk.org; > gak...@marvell.com; t...@redhat.com > Cc: tho...@monjalon.net; Kinsella, Ray ; > Richardson, Bruce ; > hemant.agra...

RE: [PATCH v5 4/5] baseband/acc100: modify validation code for ACC101

2022-05-31 Thread Chautru, Nicolas
Hi Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Tuesday, May 31, 2022 1:00 AM > To: Chautru, Nicolas ; dev@dpdk.org; > gak...@marvell.com; t...@redhat.com > Cc: tho...@monjalon.net; Kinsella, Ray ; > Richardson, Bruce ; > hemant.agra...

RE: [PATCH v6 5/5] baseband/acc100: configuration of ACC101 from PF

2022-05-31 Thread Chautru, Nicolas
Hi Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Tuesday, May 31, 2022 12:36 AM > To: Chautru, Nicolas ; dev@dpdk.org; > gak...@marvell.com; t...@redhat.com > Cc: tho...@monjalon.net; Kinsella, Ray ; > Richardson, Bruce ; > hemant.agra...

RE: [EXT] [PATCH v6 3/5] baseband/acc100: introduce PMD for ACC101

2022-05-31 Thread Chautru, Nicolas
Hi Akhil, > -Original Message- > From: Akhil Goyal > Sent: Monday, May 30, 2022 12:40 AM > To: Chautru, Nicolas ; dev@dpdk.org; > t...@redhat.com; maxime.coque...@redhat.com > Cc: tho...@monjalon.net; Kinsella, Ray ; > Richardson, Bruce ; > hemant.agra...

RE: [PATCH v7 1/6] baseband/acc100: update companion PF configure function

2022-06-02 Thread Chautru, Nicolas
Hi Kevin, > -Original Message- > From: Kevin Traynor > Sent: Thursday, June 2, 2022 2:50 AM > To: Chautru, Nicolas ; dev@dpdk.org; > gak...@marvell.com; t...@redhat.com; maxime.coque...@redhat.com > Cc: tho...@monjalon.net; Kinsella, Ray ; > Richardson, Bruce ; >

RE: [PATCH v7 0/6] drivers/baseband: PMD to support ACC100/ACC101 devices

2022-06-06 Thread Chautru, Nicolas
Hi Akhil, Checking that this series on track to be applied. Let me know if anything required from our end. > -Original Message- > From: Chautru, Nicolas > Sent: Tuesday, May 31, 2022 3:32 PM > To: dev@dpdk.org; gak...@marvell.com; t...@redhat.com; > maxime.coque...@

RE: [PATCH v7 0/6] drivers/baseband: PMD to support ACC100/ACC101 devices

2022-06-06 Thread Chautru, Nicolas
Thanks Akhil for your reply. It should be okay to apply it after rc1 once you feel better. Thanks Nic From: Akhil Goyal Sent: Monday, June 6, 2022 8:04 AM To: Chautru, Nicolas ; dev@dpdk.org; t...@redhat.com; maxime.coque...@redhat.com Cc: tho...@monjalon.net; Kinsella, Ray ; Richardson

RE: [PATCH v1 0/2] bbdev: add device info on queue topology

2022-06-06 Thread Chautru, Nicolas
Hi Hemant, Could you review this serie, this is targeting 22.11 release. Let me know if unclear. Thanks Nic > -Original Message- > From: Chautru, Nicolas > Sent: Tuesday, March 8, 2022 4:23 PM > To: dev@dpdk.org; gak...@marvell.com > Cc: t...@redhat.com; hemant.agra...

RE: [PATCH v1] bbdev: add device status info

2022-06-07 Thread Chautru, Nicolas
Hi Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Tuesday, June 7, 2022 5:43 AM > To: Stephen Hemminger ; Chautru, Nicolas > > Cc: dev@dpdk.org; gak...@marvell.com; t...@redhat.com; > hemant.agra...@nxp.com; Vargas, Hernan > Subject: Re: [PAT

RE: [PATCH v1] bbdev: add device status info

2022-06-08 Thread Chautru, Nicolas
Hi Stephen, > -Original Message- > From: Stephen Hemminger > Sent: Tuesday, June 7, 2022 9:43 AM > To: Maxime Coquelin > Cc: Chautru, Nicolas ; dev@dpdk.org; > gak...@marvell.com; t...@redhat.com; hemant.agra...@nxp.com; Vargas, > Hernan > Subject: Re: [PAT

RE: [PATCH v1] bbdev: allow operation type enum for growth

2022-06-13 Thread Chautru, Nicolas
Hi Stephen, > -Original Message- > From: Stephen Hemminger > Sent: Monday, June 13, 2022 12:48 PM > To: Chautru, Nicolas > Cc: dev@dpdk.org; tho...@monjalon.net; maxime.coque...@redhat.com; > t...@redhat.com; m...@ashroe.eu; Richardson, Bruce > ; hemant.agra...@n

RE: [PATCH v4] doc: announce changes in bbdev related to enum extension

2022-06-17 Thread Chautru, Nicolas
Hi Thomas, Can this one be applied based on your review? Thanks Nic > -Original Message- > From: Maxime Coquelin > Sent: Thursday, June 9, 2022 12:54 AM > To: Chautru, Nicolas ; dev@dpdk.org; > gak...@marvell.com; tho...@monjalon.net > Cc: t...@redhat.com; Kinsella

RE: [PATCH v1] bbdev: allow operation type enum for growth

2022-06-17 Thread Chautru, Nicolas
> From: Thomas Monjalon > > This solution is what I proposed to the techboard some years ago, but the > preference was to completely remove the MAX values. > Thanks, good to see you had similar thought! I don't believe there is an actual recommendation captured in term of how to remove compl

RE: [EXT] [PATCH v12 0/7] bbdev changes for 22.11

2022-10-10 Thread Chautru, Nicolas
> Sent: Monday, October 10, 2022 12:36 AM > To: Chautru, Nicolas > Cc: dev@dpdk.org; maxime.coque...@redhat.com; t...@redhat.com; > m...@ashroe.eu; Richardson, Bruce ; > david.march...@redhat.com; step...@networkplumber.org; Zhang, > Mingshan ; hemant.agra...@nxp.com; Akhil >

RE: [EXT] Re: [PATCH v9 13/14] baseband/acc: add PF configure companion function

2022-10-10 Thread Chautru, Nicolas
Hi Akhil, Maxime, > From: Akhil Goyal > Sent: Monday, October 10, 2022 3:09 AM > Subject: RE: [EXT] Re: [PATCH v9 13/14] baseband/acc: add PF configure > companion function > > > > diff --git a/drivers/baseband/acc/version.map > > b/drivers/baseband/acc/version.map > > > index b4ff13e38f..27fbb

RE: [PATCH v9 09/14] baseband/acc: add LTE processing functions

2022-10-10 Thread Chautru, Nicolas
Hi Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Monday, October 10, 2022 2:48 AM > To: Chautru, Nicolas ; dev@dpdk.org; > gak...@marvell.com > Cc: t...@redhat.com; m...@ashroe.eu; Richardson, Bruce > ; hemant.agra...@nxp.com; > david.m

RE: [EXT] Re: [PATCH v9 13/14] baseband/acc: add PF configure companion function

2022-10-11 Thread Chautru, Nicolas
Hi Akhil, Maxime, > -Original Message- > From: Akhil Goyal > Sent: Monday, October 10, 2022 11:12 AM > To: Chautru, Nicolas ; Maxime Coquelin > ; dev@dpdk.org > Cc: t...@redhat.com; m...@ashroe.eu; Richardson, Bruce > ; hemant.agra...@nxp.com; > david.m

RE: [PATCH v9 09/14] baseband/acc: add LTE processing functions

2022-10-11 Thread Chautru, Nicolas
Hi Akhil, Maxime, > -Original Message- > From: Akhil Goyal > Subject: RE: [PATCH v9 09/14] baseband/acc: add LTE processing functions > > > > > +/* Enqueue one encode operations for ACC200 device in TB mode. */ > > > > +static inline int enqueue_enc_one_op_tb(struct acc_queue *q, > > >

RE: [EXT] [PATCH v11 06/16] baseband/acc: add info get function for ACC200

2022-10-12 Thread Chautru, Nicolas
Hi Akhil, OK. Fixing this this morning + test_vector extra line. Thanks Nic > -Original Message- > From: Akhil Goyal > Sent: Wednesday, October 12, 2022 5:31 AM > To: Chautru, Nicolas ; dev@dpdk.org; > maxime.coque...@redhat.com > Cc: t...@redhat.com; m...@ashroe.eu;

RE: [EXT] [PATCH v12 06/16] baseband/acc: add info get function for ACC200

2022-10-12 Thread Chautru, Nicolas
Hi Akhil, > -Original Message- > From: Akhil Goyal > Sent: Wednesday, October 12, 2022 1:13 PM > To: Chautru, Nicolas ; dev@dpdk.org; > maxime.coque...@redhat.com > Cc: t...@redhat.com; m...@ashroe.eu; Richardson, Bruce > ; hemant.agra...@nxp.com; > david.m

RE: [EXT] Re: [PATCH v9 13/14] baseband/acc: add PF configure companion function

2022-10-12 Thread Chautru, Nicolas
Hi Maxime, Akhil, <...> > > This is same comment as for ACC100 vs. ACC101. > > Having a single API would be the way to go, given the prototype of > > the functions are identical. > > > > Keep acc200 function, but internal only, rte_acc_configure() would > > call the acc100

RE: [EXT] Re: [PATCH v9 13/14] baseband/acc: add PF configure companion function

2022-10-12 Thread Chautru, Nicolas
Hi Maxime, Akhil, (replying to myself) > > Hi Maxime, Akhil, > > <...> > > > This is same comment as for ACC100 vs. ACC101. > > > Having a single API would be the way to go, given the prototype > > > of the functions are identical. > > > > > > Keep acc200 function, but inte

RE: baseband PMD APIs

2022-10-13 Thread Chautru, Nicolas
- > From: Thomas Monjalon > Sent: Thursday, October 13, 2022 2:25 AM > To: Chautru, Nicolas ; Gagandeep Singh > ; Hemant Agrawal > Cc: Akhil Goyal ; david.march...@redhat.com; > maxime.coque...@redhat.com; Tom Rix ; dev@dpdk.org > Subject: Re: baseband PMD APIs > > To bbdev ma

RE: [EXT] [PATCH v3 00/30] baseband/acc100: changes for 22.11

2022-10-13 Thread Chautru, Nicolas
Hi Akhil, > -Original Message- > From: Akhil Goyal > Sent: Thursday, October 13, 2022 6:02 AM > To: Vargas, Hernan ; dev@dpdk.org; > t...@redhat.com; maxime.coque...@redhat.com > Cc: Chautru, Nicolas ; Zhang, Qi Z > > Subject: RE: [EXT] [PATCH v3 00/30] base

RE: [PATCH v3 18/30] baseband/acc100: enable input validation by default

2022-10-19 Thread Chautru, Nicolas
Hi Maxime, > From: Maxime Coquelin > On 10/12/22 04:53, Hernan Vargas wrote: > > Enable validation functions by default and provide a new flag > > RTE_LIBRTE_SKIP_VALIDATE if the user wants to run without validating > > input to save cycles. > > I would prefer a devarg, so that it can be enable

RE: [PATCH v5 28/29] baseband/acc100: add workaround for deRM corner cases

2022-10-21 Thread Chautru, Nicolas
Hi Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Friday, October 21, 2022 2:33 AM > To: Vargas, Hernan ; dev@dpdk.org; > gak...@marvell.com; t...@redhat.com > Cc: Chautru, Nicolas ; Zhang, Qi Z > > Subject: Re: [PATCH v5 28/29] baseband/acc100:

RE: [PATCH v4 01/14] baseband/acc100: remove unused registers

2022-09-21 Thread Chautru, Nicolas
iginal Message- > From: Chautru, Nicolas > Sent: Wednesday, September 21, 2022 5:27 PM > To: dev@dpdk.org; tho...@monjalon.net > Cc: maxime.coque...@redhat.com; t...@redhat.com; m...@ashroe.eu; > Richardson, Bruce ; > hemant.agra...@nxp.com; david.march...@redhat.com; >

RE: [EXT] [PATCH v7 5/7] bbdev: add new operation for FFT processing

2022-09-22 Thread Chautru, Nicolas
Hi Akhil, > -Original Message- > From: Akhil Goyal > Sent: Thursday, September 22, 2022 7:20 AM > To: Chautru, Nicolas ; dev@dpdk.org; > tho...@monjalon.net; hemant.agra...@nxp.com > Cc: maxime.coque...@redhat.com; t...@redhat.com; m...@ashroe.eu; > Richardson,

RE: [EXT] [PATCH v7 5/7] bbdev: add new operation for FFT processing

2022-09-22 Thread Chautru, Nicolas
Hi Akhil, > -Original Message- > From: Akhil Goyal > Sent: Thursday, September 22, 2022 9:49 AM > To: Chautru, Nicolas ; dev@dpdk.org; > tho...@monjalon.net; hemant.agra...@nxp.com > Cc: maxime.coque...@redhat.com; t...@redhat.com; m...@ashroe.eu; > Richardson,

RE: [PATCH v4 03/14] baseband/acc: rename directory from acc100 to acc

2022-09-22 Thread Chautru, Nicolas
Hi Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Thursday, September 22, 2022 6:18 AM > To: Chautru, Nicolas ; dev@dpdk.org; > tho...@monjalon.net > Cc: t...@redhat.com; m...@ashroe.eu; Richardson, Bruce > ; hemant.agra...@nxp.com; > david.m

RE: [PATCH v4 06/14] baseband/acc: add info get function for ACC200

2022-09-22 Thread Chautru, Nicolas
Hi Maxime, OK, chanding this in v5. > -Original Message- > From: Maxime Coquelin > Sent: Thursday, September 22, 2022 7:12 AM > To: Chautru, Nicolas ; dev@dpdk.org; > tho...@monjalon.net > Cc: t...@redhat.com; m...@ashroe.eu; Richardson, Bruce > ; hemant.agra...@n

RE: [PATCH v4 07/14] baseband/acc: add queue configuration for ACC200

2022-09-22 Thread Chautru, Nicolas
Hi Maxime, Will update this in the new version. Thanks. > -Original Message- > From: Maxime Coquelin > Sent: Thursday, September 22, 2022 7:30 AM > To: Chautru, Nicolas ; dev@dpdk.org; > tho...@monjalon.net > Cc: t...@redhat.com; m...@ashroe.eu; Richardson, Bru

RE: [EXT] [PATCH v9 0/7] bbdev changes for 22.11

2022-09-22 Thread Chautru, Nicolas
Thanks Akhil. I put the change to the release note here: https://patches.dpdk.org/project/dpdk/patch/20220922205833.41496-2-nicolas.chau...@intel.com/ > -Original Message- > From: Akhil Goyal > Sent: Thursday, September 22, 2022 11:18 AM > To: Chautru, Nicolas ; dev@dpd

RE: [PATCH v4 08/14] baseband/acc: add LDPC processing functions

2022-09-23 Thread Chautru, Nicolas
Hi Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Friday, September 23, 2022 1:30 AM > To: Chautru, Nicolas ; dev@dpdk.org; > tho...@monjalon.net > Cc: t...@redhat.com; m...@ashroe.eu; Richardson, Bruce > ; hemant.agra...@nxp.com; > david.m

RE: [PATCH v4 09/14] baseband/acc: add LTE processing functions

2022-09-23 Thread Chautru, Nicolas
Hi Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Friday, September 23, 2022 2:00 AM > To: Chautru, Nicolas ; dev@dpdk.org; > tho...@monjalon.net > Cc: t...@redhat.com; m...@ashroe.eu; Richardson, Bruce > ; hemant.agra...@nxp.com; > david.m

RE: [PATCH v4 10/14] baseband/acc: add support for FFT operations

2022-09-23 Thread Chautru, Nicolas
Hi Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Friday, September 23, 2022 2:06 AM > To: Chautru, Nicolas ; dev@dpdk.org; > tho...@monjalon.net > Cc: t...@redhat.com; m...@ashroe.eu; Richardson, Bruce > ; hemant.agra...@nxp.com; > david.m

RE: [PATCH v4 11/14] baseband/acc: support interrupt

2022-09-23 Thread Chautru, Nicolas
Hi Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Friday, September 23, 2022 2:17 AM > To: Chautru, Nicolas ; dev@dpdk.org; > tho...@monjalon.net > Cc: t...@redhat.com; m...@ashroe.eu; Richardson, Bruce > ; hemant.agra...@nxp.com; > david.m

RE: [PATCH v4 12/14] baseband/acc: add device status and vf2pf comms

2022-09-23 Thread Chautru, Nicolas
Hi Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Friday, September 23, 2022 2:23 AM > To: Chautru, Nicolas ; dev@dpdk.org; > tho...@monjalon.net > Cc: t...@redhat.com; m...@ashroe.eu; Richardson, Bruce > ; hemant.agra...@nxp.com; > david.m

RE: [PATCH v4 13/14] baseband/acc: add PF configure companion function

2022-09-23 Thread Chautru, Nicolas
Hi Maxime, Thanks will send an updated serie now. Much appreciated. > -Original Message- > From: Maxime Coquelin > Sent: Friday, September 23, 2022 2:26 AM > To: Chautru, Nicolas ; dev@dpdk.org; > tho...@monjalon.net > Cc: t...@redhat.com; m...@ashroe.eu;

RE: [EXT] [PATCH v7 6/7] bbdev: add queue related warning and status information

2022-09-24 Thread Chautru, Nicolas
Hi Ferruh, Ray, Akhil, > > -Original Message- > > From: Ferruh Yigit > > Sent: Friday, September 23, 2022 4:28 PM > > To: Akhil Goyal ; Nicolas Chautru > > ; dev@dpdk.org; tho...@monjalon.net; > > hemant.agra...@nxp.com; Ray Kinsella > > Cc: maxime.coque...@redhat.com; t...@redhat.com;

RE: [EXT] [PATCH v7 6/7] bbdev: add queue related warning and status information

2022-09-27 Thread Chautru, Nicolas
th_fdir_conf -> rte_eth_conf (in the middle)``. Need to identify this kind of usages and fix in 20.11, otherwise this blocks us extending existing enum/define. One solution can be using a fixed size array instead of ``.*MAX.*`` value. > -Original Message- > Fr

RE: [EXT] [PATCH v7 6/7] bbdev: add queue related warning and status information

2022-09-29 Thread Chautru, Nicolas
we are able to move forward. There is no full consensus but not strong opinion either from anyone. Thanks, Nic > -Original Message- > From: Akhil Goyal > Sent: Thursday, September 29, 2022 11:33 AM > To: Ferruh Yigit ; Chautru, Nicolas > ; dev@dpdk.org; Maxime Coqueli

RE: [PATCH v6 10/14] baseband/acc: add support for FFT operations

2022-09-29 Thread Chautru, Nicolas
Hi Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Tuesday, September 27, 2022 6:45 AM > To: Chautru, Nicolas ; dev@dpdk.org; > tho...@monjalon.net > Cc: t...@redhat.com; m...@ashroe.eu; Richardson, Bruce > ; hemant.agra...@nxp.com; > david.m

RE: [PATCH v6 11/14] baseband/acc: support interrupt

2022-09-29 Thread Chautru, Nicolas
Hi Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Tuesday, September 27, 2022 6:50 AM > To: Chautru, Nicolas ; dev@dpdk.org; > tho...@monjalon.net > Cc: t...@redhat.com; m...@ashroe.eu; Richardson, Bruce > ; hemant.agra...@nxp.com; > david.m

RE: [PATCH v6 09/14] baseband/acc: add LTE processing functions

2022-09-29 Thread Chautru, Nicolas
Thanks Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Tuesday, September 27, 2022 6:43 AM > To: Chautru, Nicolas ; dev@dpdk.org; > tho...@monjalon.net > Cc: t...@redhat.com; m...@ashroe.eu; Richardson, Bruce > ; hemant.agra...@nxp.com; > david.m

RE: [PATCH v10 6/7] bbdev: add queue related warning and status information

2022-10-03 Thread Chautru, Nicolas
his is not a MAX enum. Does that make sense and do you agree this is best consensus so far to move forward? Thanks Thomas, Nic > -Original Message- > From: Thomas Monjalon > Sent: Monday, October 3, 2022 1:29 AM > To: Chautru, Nicolas > Cc: dev@dpdk.org; gak...@mar

RE: [EXT] [PATCH v12 0/7] bbdev changes for 22.11

2022-10-06 Thread Chautru, Nicolas
ges. Nic > -Original Message- > From: Akhil Goyal > Sent: Thursday, October 6, 2022 10:32 AM > To: Chautru, Nicolas ; dev@dpdk.org; > tho...@monjalon.net > Cc: maxime.coque...@redhat.com; t...@redhat.com; m...@ashroe.eu; > Richardson, Bruce ; > david.march...@redhat.com;

RE: [EXT] [PATCH v7 04/14] baseband/acc: introduce PMD for ACC200

2022-10-07 Thread Chautru, Nicolas
Hi Akhil, > -Original Message- > From: Akhil Goyal > Sent: Friday, October 7, 2022 11:16 AM > To: Chautru, Nicolas ; dev@dpdk.org; > tho...@monjalon.net > Cc: maxime.coque...@redhat.com; t...@redhat.com; m...@ashroe.eu; > Richardson, Bruce ; > hemant.agra

RE: [PATCH v6 1/1] baseband/acc100: add workaround for deRM corner cases

2022-10-25 Thread Chautru, Nicolas
Hi Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Tuesday, October 25, 2022 1:26 AM > To: Vargas, Hernan ; dev@dpdk.org; > gak...@marvell.com; t...@redhat.com > Cc: Chautru, Nicolas ; Zhang, Qi Z > ; David Marchand ; > Thomas Monjalon > Subject:

RE: [PATCH 10/14] baseband/ark: introduce ark baseband driver

2022-10-26 Thread Chautru, Nicolas
n the implementation doesn't fully match your cover letter (I will add some more comments on this for 11), but unclear to me whether this is intentional or not. Thanks Nic > -Original Message- > From: John Miller > Sent: Wednesday, October 26, 2022 12:46 PM > To: Chaut

RE: [PATCH 11/14] baseband/ark: introduce ark baseband driver custom functions

2022-10-26 Thread Chautru, Nicolas
Hi John, > -Original Message- > From: John Miller > Sent: Wednesday, October 26, 2022 12:46 PM > To: Chautru, Nicolas > Cc: dev@dpdk.org; ed.cz...@atomicrules.com; Shepard Siegel > ; John Miller > > Subject: [PATCH 11/14] baseband/ark: introduce ark baseband d

RE: [PATCH v12 04/16] baseband/acc: introduce PMD for ACC200

2022-10-31 Thread Chautru, Nicolas
Hi Thomas, > -Original Message- > From: Thomas Monjalon > Sent: Sunday, October 30, 2022 9:03 AM > To: Chautru, Nicolas > Cc: dev@dpdk.org; gak...@marvell.com; maxime.coque...@redhat.com; > t...@redhat.com; Richardson, Bruce ; > hemant.agra...@nxp.com; david.march.

RE: [PATCH 10/14] baseband/ark: introduce ark baseband driver

2022-10-31 Thread Chautru, Nicolas
Hi John, > From: John Miller > Sent: Monday, October 31, 2022 10:34 AM > To: Chautru, Nicolas > Cc: dev@dpdk.org; ed.cz...@atomicrules.com; Shepard Siegel > ; Maxime Coquelin > Subject: Re: [PATCH 10/14] baseband/ark: introduce ark baseband driver > > Hi Nicolas, &g

RE: [PATCH v12 04/16] baseband/acc: introduce PMD for ACC200

2022-10-31 Thread Chautru, Nicolas
Hi Thomas, > -Original Message- > From: Thomas Monjalon > 31/10/2022 16:43, Chautru, Nicolas: > > From: Thomas Monjalon > > > 12/10/2022 19:59, Nicolas Chautru: > > > > +Bind PF UIO driver(s) > > > > +~ > > &

RE: [PATCH v1 6/6] baseband/acc: fix redundant function definition

2022-11-02 Thread Chautru, Nicolas
Thanks Maxime for the quick reviews. > -Original Message- > From: Maxime Coquelin > Sent: Wednesday, November 2, 2022 12:19 AM > To: Chautru, Nicolas ; dev@dpdk.org; > gak...@marvell.com; Vargas, Hernan > Subject: Re: [PATCH v1 6/6] baseband/acc: fix redundant f

RE: [PATCH v7 1/1] baseband/acc100: add detection for deRM corner cases

2022-11-03 Thread Chautru, Nicolas
Hi Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Thursday, November 3, 2022 7:10 AM > To: Vargas, Hernan ; dev@dpdk.org; > gak...@marvell.com; t...@redhat.com > Cc: Chautru, Nicolas ; Zhang, Qi Z > > Subject: Re: [PATCH v7 1/1] baseband/acc100

RE: [PATCH v7 1/1] baseband/acc100: add detection for deRM corner cases

2022-11-03 Thread Chautru, Nicolas
Hi Maxime, Akhil, > -Original Message- > From: Maxime Coquelin > Sent: Thursday, November 3, 2022 8:08 AM > To: Chautru, Nicolas ; Vargas, Hernan > ; dev@dpdk.org; gak...@marvell.com; > t...@redhat.com > Cc: Zhang, Qi Z > Subject: Re: [PATCH v7 1/1] baseband/a

RE: [PATCH v7 1/1] baseband/acc100: add detection for deRM corner cases

2022-11-03 Thread Chautru, Nicolas
Given how close we are to 22.11, let's not argue over this. Hernan sending a new patch based on Maxime recommendation. > -Original Message- > From: Chautru, Nicolas > Sent: Thursday, November 3, 2022 9:21 AM > To: Maxime Coquelin ; Vargas, Hernan > ; dev@dpdk.org

RE: [PATCH v1 1/1] baseband/acc: fix check after deref and dead code

2022-11-04 Thread Chautru, Nicolas
Hi Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Friday, November 4, 2022 1:52 AM > To: Vargas, Hernan ; dev@dpdk.org; > gak...@marvell.com; t...@redhat.com > Cc: Chautru, Nicolas ; Zhang, Qi Z > > Subject: Re: [PATCH v1 1/1] baseband/acc: fix ch

RE: [PATCH v12 04/16] baseband/acc: introduce PMD for ACC200

2022-11-07 Thread Chautru, Nicolas
Hi Thomas, Reminder : do you mind kindly clarifying/confirming below. Then we can update the docs accordingly. Thanks. > -Original Message- > From: Chautru, Nicolas > Sent: Monday, October 31, 2022 2:41 PM > To: Thomas Monjalon > Cc: dev@dpdk.org; gak...@marvell.co

RE: [PATCH v1 1/1] baseband/acc: fix check after deref and dead code

2022-11-08 Thread Chautru, Nicolas
; From: Chautru, Nicolas > Sent: Friday, November 4, 2022 10:39 AM > To: Maxime Coquelin ; Vargas, Hernan > ; dev@dpdk.org; gak...@marvell.com; > t...@redhat.com > Cc: Zhang, Qi Z > Subject: RE: [PATCH v1 1/1] baseband/acc: fix check after deref and dead code > > Hi Maxime

RE: [PATCH v12 04/16] baseband/acc: introduce PMD for ACC200

2022-11-08 Thread Chautru, Nicolas
Hi Thomas, > -Original Message- > From: Thomas Monjalon > 08/11/2022 00:52, Chautru, Nicolas: > > Hi Thomas, > > Reminder : do you mind kindly clarifying/confirming below. Then we can > update the docs accordingly. Thanks. > > > > From: Chautru,

RE: [PATCH v1 1/1] baseband/acc: fix check after deref and dead code

2022-11-10 Thread Chautru, Nicolas
Hi David, > -Original Message- > From: David Marchand > Sent: Thursday, November 10, 2022 1:49 AM > To: Vargas, Hernan > Cc: dev@dpdk.org; gak...@marvell.com; t...@redhat.com; > maxime.coque...@redhat.com; Chautru, Nicolas ; > Zhang, Qi Z > Subject: Re: [PA

RE: [PATCH v1 1/1] baseband/acc: fix check after deref and dead code

2022-11-15 Thread Chautru, Nicolas
David, Reminder on this one. Ok to apply as is? If not consensus we cannot also mark the Coverity warning as not an issue but please check notes below. Thanks Nic > -Original Message- > From: Chautru, Nicolas > Sent: Thursday, November 10, 2022 1:15 PM > To: David March

RE: [PATCH v1 1/1] baseband/acc: fix check after deref and dead code

2022-11-15 Thread Chautru, Nicolas
> Sent: Tuesday, November 15, 2022 8:00 AM > To: Chautru, Nicolas ; Vargas, Hernan > > Cc: dev@dpdk.org; gak...@marvell.com; t...@redhat.com; > maxime.coque...@redhat.com; Zhang, Qi Z > Subject: Re: [PATCH v1 1/1] baseband/acc: fix check after deref and dead > code > > Hello

RE: [PATCH v1 1/1] baseband/acc: fix check after deref and dead code

2022-11-15 Thread Chautru, Nicolas
r and accepted. As mentioned we can clean up some of these checks in 23.03, hence marked as deferred. Thanks Nic > -Original Message- > From: Chautru, Nicolas > Sent: Tuesday, November 15, 2022 10:04 AM > To: David Marchand ; Vargas, Hernan > > Cc: dev@dpdk.org

RE: [PATCH v2 2/2] doc: simplify the binding steps

2022-11-22 Thread Chautru, Nicolas
Hi Thomas, > -Original Message- > From: Thomas Monjalon > Sent: Tuesday, November 22, 2022 7:01 AM > To: Chautru, Nicolas > Cc: dev@dpdk.org; gak...@marvell.com; maxime.coque...@redhat.com; > Vargas, Hernan > Subject: Re: [PATCH v2 2/2] doc: simplify the binding

RE: [PATCH v2 2/2] doc: simplify the binding steps

2022-11-23 Thread Chautru, Nicolas
Hi Thomas, > -Original Message- > From: Thomas Monjalon > Sent: Wednesday, November 23, 2022 12:35 AM > To: Chautru, Nicolas > Cc: dev@dpdk.org; gak...@marvell.com; maxime.coque...@redhat.com; > Vargas, Hernan > Subject: Re: [PATCH v2 2/2] doc: simplify the bin

RE: [PATCH] maintainers: split baseband from crypto tree

2022-12-05 Thread Chautru, Nicolas
> > Baseband patches are now split from dpdk-next-crypto, All baseband > patches need to be submitted to dpdk-next-baseband which will be > maintained by Maxime. > > Cc: maxime.coque...@redhat.com > > Signed-off-by: Akhil Goyal > --- > MAINTAINERS | 12 > 1 file changed, 8 inserti

RE: [PATCH v1 2/2] test/bbdev: fix build issue with optional build flag

2022-12-05 Thread Chautru, Nicolas
Hi Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Thursday, November 24, 2022 12:09 PM > To: Chautru, Nicolas ; dev@dpdk.org; > tho...@monjalon.net; gak...@marvell.com > Cc: Vargas, Hernan ; vipin.vargh...@amd.com; > Mcnamara, John ; ferruh.yi...@a

RE: [PATCH v2 3/3] test/bbdev: explicit check for allocation failure

2022-12-07 Thread Chautru, Nicolas
ecember 7, 2022 12:40 AM > To: Chautru, Nicolas ; dev@dpdk.org > Cc: Vargas, Hernan ; vipin.vargh...@amd.com > Subject: Re: [PATCH v2 3/3] test/bbdev: explicit check for allocation failure > > > > On 12/6/22 20:14, Nicolas Chautru wrote: > > Adding check for error on ret

RE: [PATCH v4 4/7] drivers/baseband: update PMDs to expose queue per operation

2022-08-15 Thread Chautru, Nicolas
Hi Tom, Back from time off, replying to that previous email. > -Original Message- > From: Tom Rix > Sent: Monday, July 18, 2022 6:21 AM > To: Chautru, Nicolas ; dev@dpdk.org; > tho...@monjalon.net; gak...@marvell.com; hemant.agra...@nxp.com > Cc: maxime.coqu

RE: [PATCH v1 00/10] baseband/acc200

2022-08-15 Thread Chautru, Nicolas
Hi Tom, I had answered all of your comments from v4 before I went on time off. Let me know if any concern acking that v5, thanks Nic > -Original Message- > From: Tom Rix > Sent: Friday, July 22, 2022 1:20 PM > To: Vargas, Hernan ; Maxime Coquelin > ; Chautru, Nicolas &

RE: [PATCH v5 0/7] bbdev changes for 22.11

2022-08-15 Thread Chautru, Nicolas
; From: Chautru, Nicolas > Sent: Wednesday, July 6, 2022 4:28 PM > To: dev@dpdk.org; tho...@monjalon.net; gak...@marvell.com; > hemant.agra...@nxp.com > Cc: maxime.coque...@redhat.com; t...@redhat.com; m...@ashroe.eu; > Richardson, Bruce ; > david.march...@redhat.com; step...@net

RE: [EXT] [PATCH v2 00/37] baseband/acc100: changes for 22.11

2022-08-24 Thread Chautru, Nicolas
nal Message- > From: Akhil Goyal > Sent: Tuesday, August 23, 2022 9:00 AM > To: Vargas, Hernan ; dev@dpdk.org; > t...@redhat.com > Cc: Chautru, Nicolas ; Zhang, Qi Z > > Subject: RE: [EXT] [PATCH v2 00/37] baseband/acc100: changes for 22.11 > > Hi Hernan, > >

RE: [PATCH v5 2/7] bbdev: add device status info

2022-08-25 Thread Chautru, Nicolas
Thanks Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Thursday, August 25, 2022 7:19 AM > To: Chautru, Nicolas ; dev@dpdk.org; > tho...@monjalon.net; gak...@marvell.com; hemant.agra...@nxp.com > Cc: t...@redhat.com; m...@ashroe.eu; Richardson, Bru

RE: [PATCH v5 2/7] bbdev: add device status info

2022-08-29 Thread Chautru, Nicolas
Hi Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Friday, August 26, 2022 3:13 AM > To: Chautru, Nicolas ; dev@dpdk.org; > tho...@monjalon.net; gak...@marvell.com; hemant.agra...@nxp.com > Cc: t...@redhat.com; m...@ashroe.eu; Richardson, Bruce > ; davi

RE: [PATCH v6 5/7] bbdev: add new operation for FFT processing

2022-08-29 Thread Chautru, Nicolas
Hi Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Friday, August 26, 2022 5:08 AM > To: Chautru, Nicolas ; dev@dpdk.org; > tho...@monjalon.net; gak...@marvell.com; hemant.agra...@nxp.com > Cc: t...@redhat.com; m...@ashroe.eu; Richardson, Bruce > ; davi

RE: [PATCH v5 2/7] bbdev: add device status info

2022-08-30 Thread Chautru, Nicolas
Hi Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Tuesday, August 30, 2022 12:09 AM > To: Chautru, Nicolas ; dev@dpdk.org; > tho...@monjalon.net; gak...@marvell.com; hemant.agra...@nxp.com > Cc: t...@redhat.com; m...@ashroe.eu; Richardson, Bru

RE: [PATCH v1 00/10] baseband/acc200

2022-08-30 Thread Chautru, Nicolas
Hi Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Tuesday, August 30, 2022 12:45 AM > To: Chautru, Nicolas ; dev@dpdk.org; > tho...@monjalon.net; gak...@marvell.com; hemant.agra...@nxp.com; > t...@redhat.com; Vargas, Hernan > Cc: m...@ashroe.

RE: [PATCH v1 00/10] baseband/acc200

2022-08-31 Thread Chautru, Nicolas
Hi Thomas, Tom, > -Original Message- > From: Tom Rix > Sent: Wednesday, August 31, 2022 12:26 PM > To: Chautru, Nicolas ; Maxime Coquelin > ; dev@dpdk.org; tho...@monjalon.net; > gak...@marvell.com; hemant.agra...@nxp.com; Vargas, Hernan > > Cc: m...@ashr

RE: [PATCH v1 00/10] baseband/acc200

2022-08-31 Thread Chautru, Nicolas
Hi Tom, > -Original Message- > From: Tom Rix > Sent: Wednesday, August 31, 2022 5:28 PM > To: Chautru, Nicolas ; Maxime Coquelin > ; dev@dpdk.org; tho...@monjalon.net; > gak...@marvell.com; hemant.agra...@nxp.com; Vargas, Hernan > > Cc: m...@ashroe.eu; Richardso

RE: [PATCH v1 00/10] baseband/acc200

2022-09-01 Thread Chautru, Nicolas
Hi Tom, > -Original Message- > From: Tom Rix > Sent: Thursday, September 1, 2022 6:49 AM > To: Chautru, Nicolas ; Maxime Coquelin > ; dev@dpdk.org; tho...@monjalon.net; > gak...@marvell.com; hemant.agra...@nxp.com; Vargas, Hernan > > Cc: m...@ashroe.eu; Richardso

RE: [PATCH v7 0/7] bbdev changes for 22.11

2022-09-06 Thread Chautru, Nicolas
Hi Akhil, Can this very serie be applied now? I would notably like to rebase the other PMDs series based on main branch once this one is merged. Thanks! Nic > -Original Message- > From: Chautru, Nicolas > Sent: Monday, August 29, 2022 11:07 AM > To: dev@d

RE: [PATCH v2 00/37] baseband/acc100: changes for 22.11

2022-09-06 Thread Chautru, Nicolas
Hi Tom, Maxime, Hermant, Can we get a few reviews/or and acks for the patches in that serie please? Much appreciated, Nic > -Original Message- > From: Vargas, Hernan > Sent: Friday, August 19, 2022 7:31 PM > To: dev@dpdk.org; gak...@marvell.com; t...@redhat.com > Cc: C

RE: [EXT] Re: [PATCH v1 00/10] baseband/acc200

2022-09-14 Thread Chautru, Nicolas
Hi Thomas, Akhil, Bruce, Maxime, > -Original Message- > From: Thomas Monjalon > Sent: Wednesday, September 14, 2022 7:23 AM > To: Richardson, Bruce ; Maxime Coquelin > ; Akhil Goyal ; > Chautru, Nicolas > Cc: dev@dpdk.org; hemant.agra...@nxp.com; Vargas, H

  1   2   3   4   >