Hi Nipun, > -----Original Message----- > From: Nipun Gupta <[email protected]> > Sent: Thursday, May 13, 2021 4:02 AM > To: Chautru, Nicolas <[email protected]>; Hemant Agrawal > <[email protected]>; [email protected]; [email protected] > Cc: [email protected] > 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
