Re: [dpdk-dev] [PATCH] maintainers: added liquidio driver maintainer

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

On 12/23/2020 12:05 PM, Ugendreshwar Kudupudi wrote:
> Added Marvell LiquidIO Driver Maintainer
> 
> Signed-off-by: Ugendreshwar Kudupudi <ukudupudi@marvell.com>
> ---
>   MAINTAINERS | 5 ++---
>   1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index eafe9f8c4..a8d68f7ed 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -616,9 +616,8 @@ F: drivers/net/thunderx/
>   F: doc/guides/nics/thunderx.rst
>   F: doc/guides/nics/features/thunderx.ini
>   
> -Cavium LiquidIO - UNMAINTAINED

The driver is marked as unmaintained because of the missing close/remove 
implementation at first place.
Can you please update that part first before removing the unmaintained tag.

Both remove() and close() should free the driver allocated resources, and both 
should ensure device is stopped first. Also remove() can be called after close() 
called, please check other samples too.

> -M: Shijith Thotton <sthotton@marvell.com>
> -M: Srisivasubramanian Srinivasan <srinivasan@marvell.com>
> +Cavium LiquidIO
> +M: Ugendreshwar Kudupudi <ukudupudi@marvell.com>

Welcome Ugendreshwar, can Shijith and Srisivasubramanian also ack this patch?

>   T: git://dpdk.org/next/dpdk-next-net-mrvl
>   F: drivers/net/liquidio/
>   F: doc/guides/nics/liquidio.rst
>