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

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

[Ugendreshwar Kudupudi]

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
-M: Shijith Thotton <sthotton@marvell.com>
-M: Srisivasubramanian Srinivasan <srinivasan@marvell.com>
+Cavium LiquidIO
+M: Ugendreshwar Kudupudi <ukudupudi@marvell.com>
 T: git://dpdk.org/next/dpdk-next-net-mrvl
 F: drivers/net/liquidio/
 F: doc/guides/nics/liquidio.rst
-- 
2.17.1

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

[Ferruh Yigit]

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
> 

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

[Ugendreshwar Kudupudi]

++ Ranjan and team

Hello Ferruh Yigit

Based on my conversations and review with my team, looks like these changes have been already up-streamed.

Ranjan (in CC) will give further details and take the conversation forward.

Regards
Ugen

Ugendreshwar Kudupudi | TME
Cavium Networks (I) Pvt Ltd | 1st & 2nd Floor, 6-3-567 North East Building, Erra Manzil, Somajiguda, Hyderabad, Telangana 500082 India
Phone: +91 40 6733 4000 |  Ext: 4101 | Mobile: +91 9008511933 |
M A R V E L L | www.marvell.com<http://www.marvell.com/> | ukudupudi@marvell.com<mailto:youremail@marvell.com>
________________________________
From: Shijith Thotton <sthotton@marvell.com>
Sent: Tuesday, January 5, 2021 10:32 AM
To: Ferruh Yigit <ferruh.yigit@intel.com>; Ugendreshwar Kudupudi <ukudupudi@marvell.com>; dev@dpdk.org <dev@dpdk.org>
Subject: Re: [dpdk-dev] [PATCH] maintainers: added liquidio driver maintainer

>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
>>

Srisivasubramanian is not part of Marvell now.
Acked-by: Shijith Thotton <sthotton@marvell.com>

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

[Shijith Thotton]

>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
>>

Srisivasubramanian is not part of Marvell now.
Acked-by: Shijith Thotton <sthotton@marvell.com>