DPDK patches and discussions
 help / color / mirror / Atom feed
* Re: [dpdk-dev] [PATCH] maintainers: added liquidio driver maintainer
@ 2021-01-05  5:02 Shijith Thotton
  2021-02-22  5:48 ` Ugendreshwar Kudupudi
  0 siblings, 1 reply; 4+ messages in thread
From: Shijith Thotton @ 2021-01-05  5:02 UTC (permalink / raw)
  To: Ferruh Yigit, Ugendreshwar Kudupudi, dev

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [dpdk-dev] [PATCH] maintainers: added liquidio driver maintainer
  2021-01-05  5:02 [dpdk-dev] [PATCH] maintainers: added liquidio driver maintainer Shijith Thotton
@ 2021-02-22  5:48 ` Ugendreshwar Kudupudi
  0 siblings, 0 replies; 4+ messages in thread
From: Ugendreshwar Kudupudi @ 2021-02-22  5:48 UTC (permalink / raw)
  To: dev, Ferruh Yigit, Shijith Thotton
  Cc: ranjan.raj, huzefa.kankroliwala, parth.bera

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [dpdk-dev] [PATCH] maintainers: added liquidio driver maintainer
  2020-12-23 12:05 Ugendreshwar Kudupudi
@ 2021-01-04 13:55 ` Ferruh Yigit
  0 siblings, 0 replies; 4+ messages in thread
From: Ferruh Yigit @ 2021-01-04 13:55 UTC (permalink / raw)
  To: Ugendreshwar Kudupudi, dev

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
> 


^ permalink raw reply	[flat|nested] 4+ messages in thread

* [dpdk-dev] [PATCH] maintainers: added liquidio driver maintainer
@ 2020-12-23 12:05 Ugendreshwar Kudupudi
  2021-01-04 13:55 ` Ferruh Yigit
  0 siblings, 1 reply; 4+ messages in thread
From: Ugendreshwar Kudupudi @ 2020-12-23 12:05 UTC (permalink / raw)
  To: dev; +Cc: 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


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2021-02-22  5:48 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-05  5:02 [dpdk-dev] [PATCH] maintainers: added liquidio driver maintainer Shijith Thotton
2021-02-22  5:48 ` Ugendreshwar Kudupudi
  -- strict thread matches above, loose matches on Subject: below --
2020-12-23 12:05 Ugendreshwar Kudupudi
2021-01-04 13:55 ` Ferruh Yigit

DPDK patches and discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://inbox.dpdk.org/dev/0 dev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dev dev/ http://inbox.dpdk.org/dev \
		dev@dpdk.org
	public-inbox-index dev

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dev


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git