Sure Nicolas, this seems fine. I'll use this change instead of existing patch 1/10 and send a re-spin.
Thanks, Nipun > -----Original Message----- > From: Nicolas Chautru <nicolas.chau...@intel.com> > Sent: Thursday, October 7, 2021 2:29 AM > To: dev@dpdk.org; gak...@marvell.com; Nipun Gupta <nipun.gu...@nxp.com>; > t...@redhat.com > Cc: tho...@monjalon.net; mingshan.zh...@intel.com; arun.jo...@intel.com; > Hemant Agrawal <hemant.agra...@nxp.com>; david.march...@redhat.com; > Nicolas Chautru <nicolas.chau...@intel.com> > Subject: [PATCH v9] bbdev: add device info related to data endianness > assumption > > Suggesting this bbdev api change attached to address the requirement for the > new > bbdev PMD from this serie : > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatches. > dpdk.org%2Fproject%2Fdpdk%2Fpatch%2F20211006113112.11163-2- > nipun.gupta%40nxp.com%2F&data=04%7C01%7Cnipun.gupta%40nxp.com > %7Cd99e521071fc4162dfa508d9890c1b8a%7C686ea1d3bc2b4c6fa92cd99c5c30 > 1635%7C0%7C0%7C637691507373998592%7CUnknown%7CTWFpbGZsb3d8eyJ > WIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C > 1000&sdata=ogEvbxTKLnCAPs1cuIN76bXb1Mjty0crd3wotOmw%2Fcg%3D > &reserved=0 > Nipin, please review and you can use this one in case that is okay with you. > This is defined as an enum and default values set in existing PMDs. > > Nicolas Chautru (1): > bbdev: add device info related to data endianness assumption > > doc/guides/rel_notes/release_21_11.rst | 1 + > drivers/baseband/acc100/rte_acc100_pmd.c | 1 + > drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c | 1 + > drivers/baseband/fpga_lte_fec/fpga_lte_fec.c | 1 + > drivers/baseband/turbo_sw/bbdev_turbo_software.c | 1 + > lib/bbdev/rte_bbdev.h | 8 ++++++++ > 6 files changed, 13 insertions(+) > > -- > 1.8.3.1