[v1] MAINTAINERS: net: Update reviewers for TI's Ethernet drivers

[PATCH] MAINTAINERS: net: Update reviewers for TI's Ethernet drivers

Posted by Ravi Gunasekaran 2 years, 1 month ago

Grygorii is no longer associated with TI and messages addressed to
him bounce.

Add Siddharth and myself as reviewers.

Signed-off-by: Ravi Gunasekaran <r-gunasekaran@ti.com>
---
 MAINTAINERS | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 7b151710e8c5..bd52c33bca02 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -21775,7 +21775,8 @@ F:	Documentation/devicetree/bindings/counter/ti-eqep.yaml
 F:	drivers/counter/ti-eqep.c
 
 TI ETHERNET SWITCH DRIVER (CPSW)
-R:	Grygorii Strashko <grygorii.strashko@ti.com>
+R:	Siddharth Vadapalli <s-vadapalli@ti.com>
+R:	Ravi Gunasekaran <r-gunasekaran@ti.com>
 L:	linux-omap@vger.kernel.org
 L:	netdev@vger.kernel.org
 S:	Maintained
-- 
2.17.1

Re: [PATCH] MAINTAINERS: net: Update reviewers for TI's Ethernet drivers

Posted by Roger Quadros 2 years, 1 month ago

Hi Ravi,

On 10/11/2023 10:42, Ravi Gunasekaran wrote:
> Grygorii is no longer associated with TI and messages addressed to
> him bounce.
> 
> Add Siddharth and myself as reviewers.
> 
> Signed-off-by: Ravi Gunasekaran <r-gunasekaran@ti.com>
> ---
>  MAINTAINERS | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 7b151710e8c5..bd52c33bca02 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -21775,7 +21775,8 @@ F:	Documentation/devicetree/bindings/counter/ti-eqep.yaml
>  F:	drivers/counter/ti-eqep.c
>  
>  TI ETHERNET SWITCH DRIVER (CPSW)
> -R:	Grygorii Strashko <grygorii.strashko@ti.com>
> +R:	Siddharth Vadapalli <s-vadapalli@ti.com>
> +R:	Ravi Gunasekaran <r-gunasekaran@ti.com>

Could you please add me as Reviewer as well. Thanks!

>  L:	linux-omap@vger.kernel.org
>  L:	netdev@vger.kernel.org
>  S:	Maintained

> F:      drivers/net/ethernet/ti/cpsw*
> F:      drivers/net/ethernet/ti/davinci*

What about am65-cpsw*?

And drivers/net/ethernet/ti/icssg/*

I also see 

OMAP GPIO DRIVER
M:      Grygorii Strashko <grygorii.strashko@ti.com>

Maybe a separate patch to remove the invalid email-id?

-- 
cheers,
-roger

Re: [PATCH] MAINTAINERS: net: Update reviewers for TI's Ethernet drivers

Posted by Ravi Gunasekaran 2 years, 1 month ago

Roger,

On 11/10/23 2:21 PM, Roger Quadros wrote:
> Hi Ravi,
> 
> On 10/11/2023 10:42, Ravi Gunasekaran wrote:
>> Grygorii is no longer associated with TI and messages addressed to
>> him bounce.
>>
>> Add Siddharth and myself as reviewers.
>>
>> Signed-off-by: Ravi Gunasekaran <r-gunasekaran@ti.com>
>> ---
>>  MAINTAINERS | 3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 7b151710e8c5..bd52c33bca02 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -21775,7 +21775,8 @@ F:	Documentation/devicetree/bindings/counter/ti-eqep.yaml
>>  F:	drivers/counter/ti-eqep.c
>>  
>>  TI ETHERNET SWITCH DRIVER (CPSW)
>> -R:	Grygorii Strashko <grygorii.strashko@ti.com>
>> +R:	Siddharth Vadapalli <s-vadapalli@ti.com>
>> +R:	Ravi Gunasekaran <r-gunasekaran@ti.com>
> 
> Could you please add me as Reviewer as well. Thanks!

Thanks for volunteering to be a reviewer.

I posted a v2 adding you as a reviewer.
https://lore.kernel.org/all/20231110092749.3618-1-r-gunasekaran@ti.com/

> 
>>  L:	linux-omap@vger.kernel.org
>>  L:	netdev@vger.kernel.org
>>  S:	Maintained
> 
>> F:      drivers/net/ethernet/ti/cpsw*
>> F:      drivers/net/ethernet/ti/davinci*
> 
> What about am65-cpsw*?
> 
> And drivers/net/ethernet/ti/icssg/*

I would prefer a separate entry for ICSSG. Will let Danish comment on this.

> 
> I also see 
> 
> OMAP GPIO DRIVER
> M:      Grygorii Strashko <grygorii.strashko@ti.com>
> 
> Maybe a separate patch to remove the invalid email-id?
> 
Yes, that's the plan. One of us from TI would be posting shortly.


-- 
Regards,
Ravi

Re: [PATCH] MAINTAINERS: net: Update reviewers for TI's Ethernet drivers

Posted by Anwar, Md Danish 2 years, 1 month ago

On 11/10/2023 3:08 PM, Ravi Gunasekaran wrote:
> Roger,
> 
> On 11/10/23 2:21 PM, Roger Quadros wrote:
>> Hi Ravi,
>>
>> On 10/11/2023 10:42, Ravi Gunasekaran wrote:
>>> Grygorii is no longer associated with TI and messages addressed to
>>> him bounce.
>>>
>>> Add Siddharth and myself as reviewers.
>>>
>>> Signed-off-by: Ravi Gunasekaran <r-gunasekaran@ti.com>
>>> ---
>>>  MAINTAINERS | 3 ++-
>>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>> index 7b151710e8c5..bd52c33bca02 100644
>>> --- a/MAINTAINERS
>>> +++ b/MAINTAINERS
>>> @@ -21775,7 +21775,8 @@ F:	Documentation/devicetree/bindings/counter/ti-eqep.yaml
>>>  F:	drivers/counter/ti-eqep.c
>>>  
>>>  TI ETHERNET SWITCH DRIVER (CPSW)
>>> -R:	Grygorii Strashko <grygorii.strashko@ti.com>
>>> +R:	Siddharth Vadapalli <s-vadapalli@ti.com>
>>> +R:	Ravi Gunasekaran <r-gunasekaran@ti.com>
>>
>> Could you please add me as Reviewer as well. Thanks!
> 
> Thanks for volunteering to be a reviewer.
> 
> I posted a v2 adding you as a reviewer.
> https://lore.kernel.org/all/20231110092749.3618-1-r-gunasekaran@ti.com/
> 
>>
>>>  L:	linux-omap@vger.kernel.org
>>>  L:	netdev@vger.kernel.org
>>>  S:	Maintained
>>
>>> F:      drivers/net/ethernet/ti/cpsw*
>>> F:      drivers/net/ethernet/ti/davinci*
>>
>> What about am65-cpsw*?
>>
>> And drivers/net/ethernet/ti/icssg/*
> 
> I would prefer a separate entry for ICSSG. Will let Danish comment on this.
> 

Sure, I will share a separate patch for this. Roger, do you want me to
add you as a reviewer in for ICSSG driver as well?

>>
>> I also see 
>>
>> OMAP GPIO DRIVER
>> M:      Grygorii Strashko <grygorii.strashko@ti.com>
>>
>> Maybe a separate patch to remove the invalid email-id?
>>
> Yes, that's the plan. One of us from TI would be posting shortly.
> 
> 

-- 
Thanks and Regards,
Md Danish Anwar

Re: [PATCH] MAINTAINERS: net: Update reviewers for TI's Ethernet drivers

Posted by Roger Quadros 2 years, 1 month ago


On 10/11/2023 18:21, Anwar, Md Danish wrote:
> On 11/10/2023 3:08 PM, Ravi Gunasekaran wrote:
>> Roger,
>>
>> On 11/10/23 2:21 PM, Roger Quadros wrote:
>>> Hi Ravi,
>>>
>>> On 10/11/2023 10:42, Ravi Gunasekaran wrote:
>>>> Grygorii is no longer associated with TI and messages addressed to
>>>> him bounce.
>>>>
>>>> Add Siddharth and myself as reviewers.
>>>>
>>>> Signed-off-by: Ravi Gunasekaran <r-gunasekaran@ti.com>
>>>> ---
>>>>  MAINTAINERS | 3 ++-
>>>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>>>
>>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>>> index 7b151710e8c5..bd52c33bca02 100644
>>>> --- a/MAINTAINERS
>>>> +++ b/MAINTAINERS
>>>> @@ -21775,7 +21775,8 @@ F:	Documentation/devicetree/bindings/counter/ti-eqep.yaml
>>>>  F:	drivers/counter/ti-eqep.c
>>>>  
>>>>  TI ETHERNET SWITCH DRIVER (CPSW)
>>>> -R:	Grygorii Strashko <grygorii.strashko@ti.com>
>>>> +R:	Siddharth Vadapalli <s-vadapalli@ti.com>
>>>> +R:	Ravi Gunasekaran <r-gunasekaran@ti.com>
>>>
>>> Could you please add me as Reviewer as well. Thanks!
>>
>> Thanks for volunteering to be a reviewer.
>>
>> I posted a v2 adding you as a reviewer.
>> https://lore.kernel.org/all/20231110092749.3618-1-r-gunasekaran@ti.com/
>>
>>>
>>>>  L:	linux-omap@vger.kernel.org
>>>>  L:	netdev@vger.kernel.org
>>>>  S:	Maintained
>>>
>>>> F:      drivers/net/ethernet/ti/cpsw*
>>>> F:      drivers/net/ethernet/ti/davinci*
>>>
>>> What about am65-cpsw*?
>>>
>>> And drivers/net/ethernet/ti/icssg/*
>>
>> I would prefer a separate entry for ICSSG. Will let Danish comment on this.
>>
> 
> Sure, I will share a separate patch for this. Roger, do you want me to
> add you as a reviewer in for ICSSG driver as well?
> 

Yes please. Thanks!

>>>
>>> I also see 
>>>
>>> OMAP GPIO DRIVER
>>> M:      Grygorii Strashko <grygorii.strashko@ti.com>
>>>
>>> Maybe a separate patch to remove the invalid email-id?
>>>
>> Yes, that's the plan. One of us from TI would be posting shortly.
>>
>>
> 

-- 
cheers,
-roger