* 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
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).