On 1/6/23 06:44, Hernan Vargas wrote:
Use define constant ACC_HARQ_ALIGN_64B instead of hardcoded number.
No functional impact.

Signed-off-by: Hernan Vargas <hernan.var...@intel.com>
---
  drivers/baseband/acc/rte_acc100_pmd.c | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/baseband/acc/rte_acc100_pmd.c 
b/drivers/baseband/acc/rte_acc100_pmd.c
index 1ca95f4440..4ac8061ebd 100644
--- a/drivers/baseband/acc/rte_acc100_pmd.c
+++ b/drivers/baseband/acc/rte_acc100_pmd.c
@@ -1392,7 +1392,7 @@ acc101_fcw_ld_fill(struct rte_bbdev_dec_op *op, struct 
acc_fcw_ld *fcw,
                harq_in_length = RTE_MIN(harq_in_length, op->ldpc_dec.n_cb
                                - op->ldpc_dec.n_filler);
                /* Alignment on next 64B - Already enforced from HC output */
-               harq_in_length = RTE_ALIGN_FLOOR(harq_in_length, 64);
+               harq_in_length = RTE_ALIGN_FLOOR(harq_in_length, 
ACC_HARQ_ALIGN_64B);
                fcw->hcin_size0 = harq_in_length;
                fcw->hcin_offset = 0;
                fcw->hcin_size1 = 0;
@@ -1435,7 +1435,7 @@ acc101_fcw_ld_fill(struct rte_bbdev_dec_op *op, struct 
acc_fcw_ld *fcw,
                /* Cannot exceed the pruned Ncb circular buffer */
                harq_out_length = RTE_MIN(harq_out_length, ncb_p);
                /* Alignment on next 64B */
-               harq_out_length = RTE_ALIGN_CEIL(harq_out_length, 64);
+               harq_out_length = RTE_ALIGN_CEIL(harq_out_length, 
ACC_HARQ_ALIGN_64B);
                fcw->hcout_size0 = harq_out_length;
                fcw->hcout_size1 = 0;
                fcw->hcout_offset = 0;

Reviewed-by: Maxime Coquelin <maxime.coque...@redhat.com>

Thanks,
Maxime

Reply via email to