MAINTAINERS: Add Aksh Garg as PCI CADENCE reviewer

[PATCH] MAINTAINERS: Add Aksh Garg as PCI CADENCE reviewer

Posted by Aksh Garg 3 weeks, 1 day ago

I wish to contribute to the review process for Cadence PCIe IP drivers,
hence add myself as a reviewer.

Signed-off-by: Aksh Garg <a-garg7@ti.com>
---
 MAINTAINERS | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 2fb1c75afd16..e2c188a358ed 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -20315,10 +20315,11 @@ F:	Documentation/devicetree/bindings/pci/marvell,armada8k-pcie.yaml
 F:	drivers/pci/controller/dwc/pcie-armada8k.c
 
 PCI DRIVER FOR CADENCE PCIE IP
+R:	Aksh Garg <a-garg7@ti.com>
 L:	linux-pci@vger.kernel.org
 S:	Orphan
 F:	Documentation/devicetree/bindings/pci/cdns,*
-F:	drivers/pci/controller/cadence/*cadence*
+F:	drivers/pci/controller/cadence/
 
 PCI DRIVER FOR CIX Sky1
 M:	Hans Zhang <hans.zhang@cixtech.com>
-- 
2.34.1

Re: [PATCH] MAINTAINERS: Add Aksh Garg as PCI CADENCE reviewer

Posted by Bjorn Helgaas 3 weeks ago

On Fri, May 08, 2026 at 11:39:51AM +0530, Aksh Garg wrote:
> I wish to contribute to the review process for Cadence PCIe IP drivers,
> hence add myself as a reviewer.
> 
> Signed-off-by: Aksh Garg <a-garg7@ti.com>

Applied to pci/for-linus for v7.1, thanks!

> ---
>  MAINTAINERS | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 2fb1c75afd16..e2c188a358ed 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -20315,10 +20315,11 @@ F:	Documentation/devicetree/bindings/pci/marvell,armada8k-pcie.yaml
>  F:	drivers/pci/controller/dwc/pcie-armada8k.c
>  
>  PCI DRIVER FOR CADENCE PCIE IP
> +R:	Aksh Garg <a-garg7@ti.com>
>  L:	linux-pci@vger.kernel.org
>  S:	Orphan
>  F:	Documentation/devicetree/bindings/pci/cdns,*
> -F:	drivers/pci/controller/cadence/*cadence*
> +F:	drivers/pci/controller/cadence/
>  
>  PCI DRIVER FOR CIX Sky1
>  M:	Hans Zhang <hans.zhang@cixtech.com>
> -- 
> 2.34.1
>

Re: [PATCH] MAINTAINERS: Add Aksh Garg as PCI CADENCE reviewer

Posted by Hans Zhang 3 weeks, 1 day ago

Hi Aksh,

Let's wait for one day, or two days, to see if Vignesh is willing to do it.

For drivers/pci/controller/cadence, currently Aksh and Siddharth hope to 
become Reviewers. Manikandan believes that he can become a Maintainer. I 
myself also hope to apply for the role of maintainer. Then let's review 
the common driver related to Cadence PCIe IP, as well as the driver of 
the product.

Siddharth:
Depending on the frequency with which we are able to Review patches 
consistently, we could eventually take up the role of being a Maintainer 
as well.

Best regards,
Hans

On 5/8/26 14:09, Aksh Garg wrote:
> I wish to contribute to the review process for Cadence PCIe IP drivers,
> hence add myself as a reviewer.
> 
> Signed-off-by: Aksh Garg <a-garg7@ti.com>
> ---
>   MAINTAINERS | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 2fb1c75afd16..e2c188a358ed 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -20315,10 +20315,11 @@ F:	Documentation/devicetree/bindings/pci/marvell,armada8k-pcie.yaml
>   F:	drivers/pci/controller/dwc/pcie-armada8k.c
>   
>   PCI DRIVER FOR CADENCE PCIE IP
> +R:	Aksh Garg <a-garg7@ti.com>
>   L:	linux-pci@vger.kernel.org
>   S:	Orphan
>   F:	Documentation/devicetree/bindings/pci/cdns,*
> -F:	drivers/pci/controller/cadence/*cadence*
> +F:	drivers/pci/controller/cadence/
>   
>   PCI DRIVER FOR CIX Sky1
>   M:	Hans Zhang <hans.zhang@cixtech.com>

Re: [PATCH] MAINTAINERS: Add Aksh Garg as PCI CADENCE reviewer

Posted by Krzysztof Wilczyński 2 weeks, 5 days ago

> For drivers/pci/controller/cadence, currently Aksh and Siddharth hope to
> become Reviewers. Manikandan believes that he can become a Maintainer.

Manikandan needs to also learn how to disable automatic responder, per:

  "I am ooo for Diwali till 23rd October."

This is a rather long break until October.  I am sure, a lot of the mailing
list users already growing tired of receiving this all the time.

Thank you!

	Krzysztof

Re: [PATCH] MAINTAINERS: Add Aksh Garg as PCI CADENCE reviewer

Posted by Hans Zhang 3 weeks, 1 day ago


On 5/8/26 15:08, Hans Zhang wrote:
> Hi Aksh,
> 
> Let's wait for one day, or two days, to see if Vignesh is willing to do it.
> 
> For drivers/pci/controller/cadence, currently Aksh and Siddharth hope to 
> become Reviewers. Manikandan believes that he can become a Maintainer. I 
> myself also hope to apply for the role of maintainer. Then let's review 
> the common driver related to Cadence PCIe IP, as well as the driver of 
> the product.
> 
> Siddharth:
> Depending on the frequency with which we are able to Review patches 
> consistently, we could eventually take up the role of being a Maintainer 
> as well.
> 


Hi Aksh,

Privately, Vignesh has replied as follows:

Vignesh:
Agreed, I would prefer if Manikandan can own the driver as he would have
better visibility wrt IP documentations as part of Cadence.

Sorry, I don't have the bandwidth to own the driver. Siddharth and Aksh
can help review and test on all TI platforms with Cadence PCIe IP.




Could you please add Siddharth as a Reviewer? I and Manikandan are the 
maintainers. Of course, we need to see if Bjorn and Mani agree. Then, 
send this patch in the next version.


Best regards,
Hans


> Best regards,
> Hans
> 
> On 5/8/26 14:09, Aksh Garg wrote:
>> I wish to contribute to the review process for Cadence PCIe IP drivers,
>> hence add myself as a reviewer.
>>
>> Signed-off-by: Aksh Garg <a-garg7@ti.com>
>> ---
>>   MAINTAINERS | 3 ++-
>>   1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 2fb1c75afd16..e2c188a358ed 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -20315,10 +20315,11 @@ F:    Documentation/devicetree/bindings/pci/ 
>> marvell,armada8k-pcie.yaml
>>   F:    drivers/pci/controller/dwc/pcie-armada8k.c
>>   PCI DRIVER FOR CADENCE PCIE IP
>> +R:    Aksh Garg <a-garg7@ti.com>
>>   L:    linux-pci@vger.kernel.org
>>   S:    Orphan
>>   F:    Documentation/devicetree/bindings/pci/cdns,*
>> -F:    drivers/pci/controller/cadence/*cadence*
>> +F:    drivers/pci/controller/cadence/
>>   PCI DRIVER FOR CIX Sky1
>>   M:    Hans Zhang <hans.zhang@cixtech.com>
>

Re: [PATCH] MAINTAINERS: Add Aksh Garg as PCI CADENCE reviewer

Posted by Manivannan Sadhasivam 2 weeks, 5 days ago

On Fri, May 08, 2026 at 07:34:01PM +0800, Hans Zhang wrote:
> 
> 
> On 5/8/26 15:08, Hans Zhang wrote:
> > Hi Aksh,
> > 
> > Let's wait for one day, or two days, to see if Vignesh is willing to do it.
> > 
> > For drivers/pci/controller/cadence, currently Aksh and Siddharth hope to
> > become Reviewers. Manikandan believes that he can become a Maintainer. I
> > myself also hope to apply for the role of maintainer. Then let's review
> > the common driver related to Cadence PCIe IP, as well as the driver of
> > the product.
> > 
> > Siddharth:
> > Depending on the frequency with which we are able to Review patches
> > consistently, we could eventually take up the role of being a Maintainer
> > as well.
> > 
> 
> 
> Hi Aksh,
> 
> Privately, Vignesh has replied as follows:
> 
> Vignesh:
> Agreed, I would prefer if Manikandan can own the driver as he would have
> better visibility wrt IP documentations as part of Cadence.
> 
> Sorry, I don't have the bandwidth to own the driver. Siddharth and Aksh
> can help review and test on all TI platforms with Cadence PCIe IP.
> 
> 
> 
> 
> Could you please add Siddharth as a Reviewer? I and Manikandan are the
> maintainers. Of course, we need to see if Bjorn and Mani agree. Then, send
> this patch in the next version.
> 

$ git log --oneline --author=Manikandan drivers/pci/|wc -l
4

I wouldn't vote a developer who has committed 4 patches for a driver to become
a maintainer. But considering the fact that the driver is currently in 'Orphan'
state, and the developer works for the company that owns the IP, I'm OK with it.

But atleast that patch has to come from him, stating that he wishes to maintain
the driver. He can add reviewers in the same patch though.

- Mani

-- 
மணிவண்ணன் சதாசிவம்