Re: [dpdk-dev] [PATCH] ethdev: group constant definitions in Doxygen

[Ferruh Yigit <ferruh.yigit@intel.com>]

On 9/23/2021 5:35 PM, Kevin Traynor wrote:
> On 30/08/2021 11:42, Thomas Monjalon wrote:
>> A lot of flags are parts of a group but are documented alone.
>> The Doxygen syntax @{ and @} for grouping is used
>> to make flags appear together and have a common description.
>>
>> Some Rx/Tx offload flags and RSS definitions are not grouped
>> because they need to be all properly documented first.
>>
>> Signed-off-by: Thomas Monjalon<thomas@monjalon.net>
>> ---
>>   lib/ethdev/rte_ethdev.h | 71 ++++++++++++++++++++++++++++-------------
>>   1 file changed, 48 insertions(+), 23 deletions(-)
> 
> lgtm, this is a nice enhancement when reading the api guide.
> 
> Acked-by: Kevin Traynor <ktraynor@redhat.com>
> 

Converting to explicit acks:
Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
Acked-by: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>


Applied to dpdk-next-net/main, thanks.