* [PATCH v2 0/1] Device queue initialization @ 2024-08-23 14:56 Hernan Vargas 2024-08-23 14:56 ` [PATCH v2 1/1] drivers/baseband: device queues initialization Hernan Vargas 0 siblings, 1 reply; 5+ messages in thread From: Hernan Vargas @ 2024-08-23 14:56 UTC (permalink / raw) To: dev, gakhil, trix, maxime.coquelin Cc: nicolas.chautru, qi.z.zhang, Hernan Vargas Reworded commit message for cosmetic queue initialization change. Hernan Vargas (1): drivers/baseband: device queues initialization drivers/baseband/acc/rte_acc100_pmd.c | 1 + drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c | 2 ++ drivers/baseband/fpga_lte_fec/fpga_lte_fec.c | 2 ++ drivers/baseband/la12xx/bbdev_la12xx.c | 2 ++ 4 files changed, 7 insertions(+) -- 2.37.1 ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH v2 1/1] drivers/baseband: device queues initialization 2024-08-23 14:56 [PATCH v2 0/1] Device queue initialization Hernan Vargas @ 2024-08-23 14:56 ` Hernan Vargas 2024-08-26 5:14 ` Hemant Agrawal ` (2 more replies) 0 siblings, 3 replies; 5+ messages in thread From: Hernan Vargas @ 2024-08-23 14:56 UTC (permalink / raw) To: dev, gakhil, trix, maxime.coquelin Cc: nicolas.chautru, qi.z.zhang, Hernan Vargas Cosmetic change to keep code consistency in PMD for queue initialization set to zero. No functional impact. Signed-off-by: Hernan Vargas <hernan.vargas@intel.com> --- drivers/baseband/acc/rte_acc100_pmd.c | 1 + drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c | 2 ++ drivers/baseband/fpga_lte_fec/fpga_lte_fec.c | 2 ++ drivers/baseband/la12xx/bbdev_la12xx.c | 2 ++ 4 files changed, 7 insertions(+) diff --git a/drivers/baseband/acc/rte_acc100_pmd.c b/drivers/baseband/acc/rte_acc100_pmd.c index ab69350080f0..9b12fa7048c2 100644 --- a/drivers/baseband/acc/rte_acc100_pmd.c +++ b/drivers/baseband/acc/rte_acc100_pmd.c @@ -1030,6 +1030,7 @@ acc100_dev_info_get(struct rte_bbdev *dev, dev_info->num_queues[RTE_BBDEV_OP_LDPC_ENC] = d->acc_conf.q_dl_5g.num_aqs_per_groups * d->acc_conf.q_dl_5g.num_qgroups; dev_info->num_queues[RTE_BBDEV_OP_FFT] = 0; + dev_info->num_queues[RTE_BBDEV_OP_MLDTS] = 0; dev_info->queue_priority[RTE_BBDEV_OP_TURBO_DEC] = d->acc_conf.q_ul_4g.num_qgroups; dev_info->queue_priority[RTE_BBDEV_OP_TURBO_ENC] = d->acc_conf.q_dl_4g.num_qgroups; dev_info->queue_priority[RTE_BBDEV_OP_LDPC_DEC] = d->acc_conf.q_ul_5g.num_qgroups; diff --git a/drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c b/drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c index 9b253cde280d..dad16ad422b5 100644 --- a/drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c +++ b/drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c @@ -625,6 +625,8 @@ fpga_5gnr_dev_info_get(struct rte_bbdev *dev, struct rte_bbdev_driver_info *dev_ dev_info->num_queues[RTE_BBDEV_OP_TURBO_ENC] = 0; dev_info->num_queues[RTE_BBDEV_OP_LDPC_DEC] = dev_info->max_num_queues / 2; dev_info->num_queues[RTE_BBDEV_OP_LDPC_ENC] = dev_info->max_num_queues / 2; + dev_info->num_queues[RTE_BBDEV_OP_FFT] = 0; + dev_info->num_queues[RTE_BBDEV_OP_MLDTS] = 0; dev_info->queue_priority[RTE_BBDEV_OP_LDPC_DEC] = 1; dev_info->queue_priority[RTE_BBDEV_OP_LDPC_ENC] = 1; } diff --git a/drivers/baseband/fpga_lte_fec/fpga_lte_fec.c b/drivers/baseband/fpga_lte_fec/fpga_lte_fec.c index a4a963f74dbf..c6a3c3805a04 100644 --- a/drivers/baseband/fpga_lte_fec/fpga_lte_fec.c +++ b/drivers/baseband/fpga_lte_fec/fpga_lte_fec.c @@ -659,6 +659,8 @@ fpga_dev_info_get(struct rte_bbdev *dev, dev_info->num_queues[RTE_BBDEV_OP_TURBO_ENC] = dev_info->max_num_queues / 2; dev_info->num_queues[RTE_BBDEV_OP_LDPC_DEC] = 0; dev_info->num_queues[RTE_BBDEV_OP_LDPC_ENC] = 0; + dev_info->num_queues[RTE_BBDEV_OP_FFT] = 0; + dev_info->num_queues[RTE_BBDEV_OP_MLDTS] = 0; dev_info->queue_priority[RTE_BBDEV_OP_TURBO_DEC] = 1; dev_info->queue_priority[RTE_BBDEV_OP_TURBO_ENC] = 1; } diff --git a/drivers/baseband/la12xx/bbdev_la12xx.c b/drivers/baseband/la12xx/bbdev_la12xx.c index 1a56e73abdab..c13eabbf4b6e 100644 --- a/drivers/baseband/la12xx/bbdev_la12xx.c +++ b/drivers/baseband/la12xx/bbdev_la12xx.c @@ -108,6 +108,8 @@ la12xx_info_get(struct rte_bbdev *dev __rte_unused, dev_info->num_queues[RTE_BBDEV_OP_TURBO_ENC] = 0; dev_info->num_queues[RTE_BBDEV_OP_LDPC_DEC] = LA12XX_MAX_QUEUES / 2; dev_info->num_queues[RTE_BBDEV_OP_LDPC_ENC] = LA12XX_MAX_QUEUES / 2; + dev_info->num_queues[RTE_BBDEV_OP_FFT] = 0; + dev_info->num_queues[RTE_BBDEV_OP_MLDTS] = 0; dev_info->queue_priority[RTE_BBDEV_OP_LDPC_DEC] = 1; dev_info->queue_priority[RTE_BBDEV_OP_LDPC_ENC] = 1; rte_bbdev_log_debug("got device info from %u", dev->data->dev_id); -- 2.37.1 ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2 1/1] drivers/baseband: device queues initialization 2024-08-23 14:56 ` [PATCH v2 1/1] drivers/baseband: device queues initialization Hernan Vargas @ 2024-08-26 5:14 ` Hemant Agrawal 2024-09-10 15:34 ` Maxime Coquelin 2024-09-13 13:50 ` Maxime Coquelin 2 siblings, 0 replies; 5+ messages in thread From: Hemant Agrawal @ 2024-08-26 5:14 UTC (permalink / raw) To: Hernan Vargas, dev, gakhil, trix, maxime.coquelin Cc: nicolas.chautru, qi.z.zhang On 23-08-2024 20:26, Hernan Vargas wrote: > Cosmetic change to keep code consistency in PMD for queue > initialization set to zero. > No functional impact. > > Signed-off-by: Hernan Vargas <hernan.vargas@intel.com> > --- > drivers/baseband/acc/rte_acc100_pmd.c | 1 + > drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c | 2 ++ > drivers/baseband/fpga_lte_fec/fpga_lte_fec.c | 2 ++ > drivers/baseband/la12xx/bbdev_la12xx.c | 2 ++ > 4 files changed, 7 insertions(+) > > diff --git a/drivers/baseband/acc/rte_acc100_pmd.c b/drivers/baseband/acc/rte_acc100_pmd.c > index ab69350080f0..9b12fa7048c2 100644 > --- a/drivers/baseband/acc/rte_acc100_pmd.c > +++ b/drivers/baseband/acc/rte_acc100_pmd.c > @@ -1030,6 +1030,7 @@ acc100_dev_info_get(struct rte_bbdev *dev, > dev_info->num_queues[RTE_BBDEV_OP_LDPC_ENC] = d->acc_conf.q_dl_5g.num_aqs_per_groups * > d->acc_conf.q_dl_5g.num_qgroups; > dev_info->num_queues[RTE_BBDEV_OP_FFT] = 0; > + dev_info->num_queues[RTE_BBDEV_OP_MLDTS] = 0; > dev_info->queue_priority[RTE_BBDEV_OP_TURBO_DEC] = d->acc_conf.q_ul_4g.num_qgroups; > dev_info->queue_priority[RTE_BBDEV_OP_TURBO_ENC] = d->acc_conf.q_dl_4g.num_qgroups; > dev_info->queue_priority[RTE_BBDEV_OP_LDPC_DEC] = d->acc_conf.q_ul_5g.num_qgroups; > diff --git a/drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c b/drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c > index 9b253cde280d..dad16ad422b5 100644 > --- a/drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c > +++ b/drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c > @@ -625,6 +625,8 @@ fpga_5gnr_dev_info_get(struct rte_bbdev *dev, struct rte_bbdev_driver_info *dev_ > dev_info->num_queues[RTE_BBDEV_OP_TURBO_ENC] = 0; > dev_info->num_queues[RTE_BBDEV_OP_LDPC_DEC] = dev_info->max_num_queues / 2; > dev_info->num_queues[RTE_BBDEV_OP_LDPC_ENC] = dev_info->max_num_queues / 2; > + dev_info->num_queues[RTE_BBDEV_OP_FFT] = 0; > + dev_info->num_queues[RTE_BBDEV_OP_MLDTS] = 0; > dev_info->queue_priority[RTE_BBDEV_OP_LDPC_DEC] = 1; > dev_info->queue_priority[RTE_BBDEV_OP_LDPC_ENC] = 1; > } > diff --git a/drivers/baseband/fpga_lte_fec/fpga_lte_fec.c b/drivers/baseband/fpga_lte_fec/fpga_lte_fec.c > index a4a963f74dbf..c6a3c3805a04 100644 > --- a/drivers/baseband/fpga_lte_fec/fpga_lte_fec.c > +++ b/drivers/baseband/fpga_lte_fec/fpga_lte_fec.c > @@ -659,6 +659,8 @@ fpga_dev_info_get(struct rte_bbdev *dev, > dev_info->num_queues[RTE_BBDEV_OP_TURBO_ENC] = dev_info->max_num_queues / 2; > dev_info->num_queues[RTE_BBDEV_OP_LDPC_DEC] = 0; > dev_info->num_queues[RTE_BBDEV_OP_LDPC_ENC] = 0; > + dev_info->num_queues[RTE_BBDEV_OP_FFT] = 0; > + dev_info->num_queues[RTE_BBDEV_OP_MLDTS] = 0; > dev_info->queue_priority[RTE_BBDEV_OP_TURBO_DEC] = 1; > dev_info->queue_priority[RTE_BBDEV_OP_TURBO_ENC] = 1; > } > diff --git a/drivers/baseband/la12xx/bbdev_la12xx.c b/drivers/baseband/la12xx/bbdev_la12xx.c > index 1a56e73abdab..c13eabbf4b6e 100644 > --- a/drivers/baseband/la12xx/bbdev_la12xx.c > +++ b/drivers/baseband/la12xx/bbdev_la12xx.c > @@ -108,6 +108,8 @@ la12xx_info_get(struct rte_bbdev *dev __rte_unused, > dev_info->num_queues[RTE_BBDEV_OP_TURBO_ENC] = 0; > dev_info->num_queues[RTE_BBDEV_OP_LDPC_DEC] = LA12XX_MAX_QUEUES / 2; > dev_info->num_queues[RTE_BBDEV_OP_LDPC_ENC] = LA12XX_MAX_QUEUES / 2; > + dev_info->num_queues[RTE_BBDEV_OP_FFT] = 0; > + dev_info->num_queues[RTE_BBDEV_OP_MLDTS] = 0; > dev_info->queue_priority[RTE_BBDEV_OP_LDPC_DEC] = 1; > dev_info->queue_priority[RTE_BBDEV_OP_LDPC_ENC] = 1; > rte_bbdev_log_debug("got device info from %u", dev->data->dev_id); Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com> ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2 1/1] drivers/baseband: device queues initialization 2024-08-23 14:56 ` [PATCH v2 1/1] drivers/baseband: device queues initialization Hernan Vargas 2024-08-26 5:14 ` Hemant Agrawal @ 2024-09-10 15:34 ` Maxime Coquelin 2024-09-13 13:50 ` Maxime Coquelin 2 siblings, 0 replies; 5+ messages in thread From: Maxime Coquelin @ 2024-09-10 15:34 UTC (permalink / raw) To: Hernan Vargas, dev, gakhil, trix; +Cc: nicolas.chautru, qi.z.zhang On 8/23/24 16:56, Hernan Vargas wrote: > Cosmetic change to keep code consistency in PMD for queue > initialization set to zero. > No functional impact. > > Signed-off-by: Hernan Vargas <hernan.vargas@intel.com> > --- > drivers/baseband/acc/rte_acc100_pmd.c | 1 + > drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c | 2 ++ > drivers/baseband/fpga_lte_fec/fpga_lte_fec.c | 2 ++ > drivers/baseband/la12xx/bbdev_la12xx.c | 2 ++ > 4 files changed, 7 insertions(+) > Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com> Thanks, Maxime ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2 1/1] drivers/baseband: device queues initialization 2024-08-23 14:56 ` [PATCH v2 1/1] drivers/baseband: device queues initialization Hernan Vargas 2024-08-26 5:14 ` Hemant Agrawal 2024-09-10 15:34 ` Maxime Coquelin @ 2024-09-13 13:50 ` Maxime Coquelin 2 siblings, 0 replies; 5+ messages in thread From: Maxime Coquelin @ 2024-09-13 13:50 UTC (permalink / raw) To: Hernan Vargas, dev, gakhil, trix; +Cc: nicolas.chautru, qi.z.zhang On 8/23/24 16:56, Hernan Vargas wrote: > Cosmetic change to keep code consistency in PMD for queue > initialization set to zero. > No functional impact. > > Signed-off-by: Hernan Vargas <hernan.vargas@intel.com> > --- > drivers/baseband/acc/rte_acc100_pmd.c | 1 + > drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c | 2 ++ > drivers/baseband/fpga_lte_fec/fpga_lte_fec.c | 2 ++ > drivers/baseband/la12xx/bbdev_la12xx.c | 2 ++ > 4 files changed, 7 insertions(+) > Applied to dpdk-next-baseband/for-main Thanks, Maxime ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2024-09-13 13:50 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2024-08-23 14:56 [PATCH v2 0/1] Device queue initialization Hernan Vargas 2024-08-23 14:56 ` [PATCH v2 1/1] drivers/baseband: device queues initialization Hernan Vargas 2024-08-26 5:14 ` Hemant Agrawal 2024-09-10 15:34 ` Maxime Coquelin 2024-09-13 13:50 ` Maxime Coquelin
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).