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&amp;data=04%7C01%7Cnipun.gupta%40nxp.com
> %7Cd99e521071fc4162dfa508d9890c1b8a%7C686ea1d3bc2b4c6fa92cd99c5c30
> 1635%7C0%7C0%7C637691507373998592%7CUnknown%7CTWFpbGZsb3d8eyJ
> WIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C
> 1000&amp;sdata=ogEvbxTKLnCAPs1cuIN76bXb1Mjty0crd3wotOmw%2Fcg%3D
> &amp;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

Reply via email to