[PATCH v2 28/37] baseband/acc100: make desc optimization optional

[Maxime Coquelin]

On 8/20/22 04:31, Hernan Vargas wrote:
> Add ACC100_DESC_OPTIMIZATION flag to enable muxing of encode operations
> with common FCW.
> 
> Signed-off-by: Hernan Vargas <hernan.vargas at intel.com>
> ---
>   drivers/baseband/acc100/rte_acc100_pmd.c | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/baseband/acc100/rte_acc100_pmd.c b/drivers/baseband/acc100/rte_acc100_pmd.c
> index 69de204293..7b23529ec3 100644
> --- a/drivers/baseband/acc100/rte_acc100_pmd.c
> +++ b/drivers/baseband/acc100/rte_acc100_pmd.c
> @@ -4569,9 +4569,10 @@ acc100_enqueue_ldpc_dec_cb(struct rte_bbdev_queue_data *q_data,
>   			break;
>   		}
>   		avail -= 1;
> -
> +#ifdef ACC100_DESC_OPTIMIZATION
>   		if (i > 0)
>   			same_op = cmp_ldpc_dec_op(&ops[i-1]);
> +#endif

Nack, don't put optimizations under #ifdefs as I explained on an earlier
patch.

>   		rte_bbdev_log(INFO, "Op %d %d %d %d %d %d %d %d %d %d %d %d\n",
>   			i, ops[i]->ldpc_dec.op_flags, ops[i]->ldpc_dec.rv_index,
>   			ops[i]->ldpc_dec.iter_max, ops[i]->ldpc_dec.iter_count,