Hi Nipun,
> -----Original Message-----
> From: Nipun Gupta <nipun.gu...@nxp.com>
> Sent: Thursday, May 13, 2021 4:02 AM
> To: Chautru, Nicolas <nicolas.chau...@intel.com>; Hemant Agrawal
> <hemant.agra...@nxp.com>; dev@dpdk.org; gak...@marvell.com
> Cc: david.march...@redhat.com
> Subject: RE: [PATCH v4 5/8] baseband/la12xx: add queue and modem config
> support
>
>
>
> > > + .type = RTE_BBDEV_OP_LDPC_DEC,
> > > + .cap.ldpc_dec = {
> > > + .capability_flags =
> > > + RTE_BBDEV_LDPC_CRC_TYPE_24A_CHECK |
> > > + RTE_BBDEV_LDPC_LLR_COMPRESSION |
> >
> > Are you sure you support the 6bits LLR compression option? I don't see
> > it being used in the PMD below. Nor in doc.
> > Let me know if you are unclear what that flag really mean.
>
> We support LLR compression in the Hardware by default.
> We do not need to call ' ldpc_input_llr_scaling' API in testbbdev.
>
So you always assume to pack up the input data as 6 bits back-to-back? You
understand what that feature mean?
My concern is that you would not support current API and many unit test would
fail then for you PMD.
Are you able to run the bbdev-test and see failures (all tests should be either
skipped : missing capability) or passing
> Regards,
> Nipun
