MAINTAINERS: mmc: take over as maintainer of MCI & SDHCI MICROCHIP DRIVERS

[PATCH] MAINTAINERS: mmc: take over as maintainer of MCI & SDHCI MICROCHIP DRIVERS

Posted by Aubin Constans 2 years, 3 months ago

On the one hand Eugen has taken responsibilities outside Microchip,
on the other hand I have some experience with the Microchip SDMMC
SDHCI controller.
Change Eugen as reviewer and take over maintainership of the SDHCI
MICROCHIP DRIVER.
Also, take over maintainership of its predecessor, that is the MCI
MICROCHIP DRIVER.

Cc: Eugen Hristev <eugen.hristev@collabora.com>
Cc: Ludovic Desroches <ludovic.desroches@microchip.com>
Signed-off-by: Aubin Constans <aubin.constans@microchip.com>
---
 MAINTAINERS | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 2833e2da63e0..52beaf4f7fbb 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -14022,7 +14022,7 @@ F:	Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml
 F:	drivers/iio/adc/mcp3911.c
 
 MICROCHIP MMC/SD/SDIO MCI DRIVER
-M:	Ludovic Desroches <ludovic.desroches@microchip.com>
+M:	Aubin Constans <aubin.constans@microchip.com>
 S:	Maintained
 F:	drivers/mmc/host/atmel-mci.c
 
@@ -19235,7 +19235,8 @@ F:	Documentation/devicetree/bindings/mmc/sdhci-common.yaml
 F:	drivers/mmc/host/sdhci*
 
 SECURE DIGITAL HOST CONTROLLER INTERFACE (SDHCI) MICROCHIP DRIVER
-M:	Eugen Hristev <eugen.hristev@microchip.com>
+M:	Aubin Constans <aubin.constans@microchip.com>
+R:	Eugen Hristev <eugen.hristev@collabora.com>
 L:	linux-mmc@vger.kernel.org
 S:	Supported
 F:	drivers/mmc/host/sdhci-of-at91.c
-- 
2.39.2

Re: [PATCH] MAINTAINERS: mmc: take over as maintainer of MCI & SDHCI MICROCHIP DRIVERS

Posted by Ulf Hansson 2 years, 1 month ago

On Mon, 11 Sept 2023 at 17:33, Aubin Constans
<aubin.constans@microchip.com> wrote:
>
> On the one hand Eugen has taken responsibilities outside Microchip,
> on the other hand I have some experience with the Microchip SDMMC
> SDHCI controller.
> Change Eugen as reviewer and take over maintainership of the SDHCI
> MICROCHIP DRIVER.
> Also, take over maintainership of its predecessor, that is the MCI
> MICROCHIP DRIVER.
>
> Cc: Eugen Hristev <eugen.hristev@collabora.com>
> Cc: Ludovic Desroches <ludovic.desroches@microchip.com>
> Signed-off-by: Aubin Constans <aubin.constans@microchip.com>

Thanks for volunteering to help out with the maintenance!

Applied for next.

Kind regards
Uffe



> ---
>  MAINTAINERS | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 2833e2da63e0..52beaf4f7fbb 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -14022,7 +14022,7 @@ F:      Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml
>  F:     drivers/iio/adc/mcp3911.c
>
>  MICROCHIP MMC/SD/SDIO MCI DRIVER
> -M:     Ludovic Desroches <ludovic.desroches@microchip.com>
> +M:     Aubin Constans <aubin.constans@microchip.com>
>  S:     Maintained
>  F:     drivers/mmc/host/atmel-mci.c
>
> @@ -19235,7 +19235,8 @@ F:      Documentation/devicetree/bindings/mmc/sdhci-common.yaml
>  F:     drivers/mmc/host/sdhci*
>
>  SECURE DIGITAL HOST CONTROLLER INTERFACE (SDHCI) MICROCHIP DRIVER
> -M:     Eugen Hristev <eugen.hristev@microchip.com>
> +M:     Aubin Constans <aubin.constans@microchip.com>
> +R:     Eugen Hristev <eugen.hristev@collabora.com>
>  L:     linux-mmc@vger.kernel.org
>  S:     Supported
>  F:     drivers/mmc/host/sdhci-of-at91.c
> --
> 2.39.2
>

Re: [PATCH] MAINTAINERS: mmc: take over as maintainer of MCI & SDHCI MICROCHIP DRIVERS

Posted by Ludovic.Desroches@microchip.com 2 years, 3 months ago

On 9/11/23 17:32, Aubin Constans wrote:
> On the one hand Eugen has taken responsibilities outside Microchip,
> on the other hand I have some experience with the Microchip SDMMC
> SDHCI controller.
> Change Eugen as reviewer and take over maintainership of the SDHCI
> MICROCHIP DRIVER.
> Also, take over maintainership of its predecessor, that is the MCI
> MICROCHIP DRIVER.
> 
> Cc: Eugen Hristev <eugen.hristev@collabora.com>
> Cc: Ludovic Desroches <ludovic.desroches@microchip.com>
> Signed-off-by: Aubin Constans <aubin.constans@microchip.com>

For atmel-mci:
Acked-by: Ludovic Desroches <ludovic.desroches@microchip.com>

> ---
>   MAINTAINERS | 5 +++--
>   1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 2833e2da63e0..52beaf4f7fbb 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -14022,7 +14022,7 @@ F:	Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml
>   F:	drivers/iio/adc/mcp3911.c
>   
>   MICROCHIP MMC/SD/SDIO MCI DRIVER
> -M:	Ludovic Desroches <ludovic.desroches@microchip.com>
> +M:	Aubin Constans <aubin.constans@microchip.com>
>   S:	Maintained
>   F:	drivers/mmc/host/atmel-mci.c
>   
> @@ -19235,7 +19235,8 @@ F:	Documentation/devicetree/bindings/mmc/sdhci-common.yaml
>   F:	drivers/mmc/host/sdhci*
>   
>   SECURE DIGITAL HOST CONTROLLER INTERFACE (SDHCI) MICROCHIP DRIVER
> -M:	Eugen Hristev <eugen.hristev@microchip.com>
> +M:	Aubin Constans <aubin.constans@microchip.com>
> +R:	Eugen Hristev <eugen.hristev@collabora.com>
>   L:	linux-mmc@vger.kernel.org
>   S:	Supported
>   F:	drivers/mmc/host/sdhci-of-at91.c

Re: [PATCH] MAINTAINERS: mmc: take over as maintainer of MCI & SDHCI MICROCHIP DRIVERS

Posted by Ulf Hansson 2 years, 3 months ago

On Tue, 12 Sept 2023 at 07:21, <Ludovic.Desroches@microchip.com> wrote:
>
> On 9/11/23 17:32, Aubin Constans wrote:
> > On the one hand Eugen has taken responsibilities outside Microchip,
> > on the other hand I have some experience with the Microchip SDMMC
> > SDHCI controller.
> > Change Eugen as reviewer and take over maintainership of the SDHCI
> > MICROCHIP DRIVER.
> > Also, take over maintainership of its predecessor, that is the MCI
> > MICROCHIP DRIVER.
> >
> > Cc: Eugen Hristev <eugen.hristev@collabora.com>
> > Cc: Ludovic Desroches <ludovic.desroches@microchip.com>
> > Signed-off-by: Aubin Constans <aubin.constans@microchip.com>
>
> For atmel-mci:
> Acked-by: Ludovic Desroches <ludovic.desroches@microchip.com>

Sounds like the patch could be split up, as there is an agreement on
the atmel-mci part.

Aubin, can you make a separate patch for the atmel-mci part and add
Ludovic's ack to it?

In regards to the sdhci driver, I suggest you just add yourself as a
maintainer too, along with Eugen.

Kind regards
Uffe

>
> > ---
> >   MAINTAINERS | 5 +++--
> >   1 file changed, 3 insertions(+), 2 deletions(-)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 2833e2da63e0..52beaf4f7fbb 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -14022,7 +14022,7 @@ F:    Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml
> >   F:  drivers/iio/adc/mcp3911.c
> >
> >   MICROCHIP MMC/SD/SDIO MCI DRIVER
> > -M:   Ludovic Desroches <ludovic.desroches@microchip.com>
> > +M:   Aubin Constans <aubin.constans@microchip.com>
> >   S:  Maintained
> >   F:  drivers/mmc/host/atmel-mci.c
> >
> > @@ -19235,7 +19235,8 @@ F:    Documentation/devicetree/bindings/mmc/sdhci-common.yaml
> >   F:  drivers/mmc/host/sdhci*
> >
> >   SECURE DIGITAL HOST CONTROLLER INTERFACE (SDHCI) MICROCHIP DRIVER
> > -M:   Eugen Hristev <eugen.hristev@microchip.com>
> > +M:   Aubin Constans <aubin.constans@microchip.com>
> > +R:   Eugen Hristev <eugen.hristev@collabora.com>
> >   L:  linux-mmc@vger.kernel.org
> >   S:  Supported
> >   F:  drivers/mmc/host/sdhci-of-at91.c
>

Re: [PATCH] MAINTAINERS: mmc: take over as maintainer of MCI & SDHCI MICROCHIP DRIVERS

Posted by Nicolas Ferre 2 years, 3 months ago

On 14/09/2023 at 16:21, Ulf Hansson wrote:
> On Tue, 12 Sept 2023 at 07:21, <Ludovic.Desroches@microchip.com> wrote:
>>
>> On 9/11/23 17:32, Aubin Constans wrote:
>>> On the one hand Eugen has taken responsibilities outside Microchip,
>>> on the other hand I have some experience with the Microchip SDMMC
>>> SDHCI controller.
>>> Change Eugen as reviewer and take over maintainership of the SDHCI
>>> MICROCHIP DRIVER.
>>> Also, take over maintainership of its predecessor, that is the MCI
>>> MICROCHIP DRIVER.
>>>
>>> Cc: Eugen Hristev <eugen.hristev@collabora.com>
>>> Cc: Ludovic Desroches <ludovic.desroches@microchip.com>
>>> Signed-off-by: Aubin Constans <aubin.constans@microchip.com>
>>
>> For atmel-mci:
>> Acked-by: Ludovic Desroches <ludovic.desroches@microchip.com>
> 
> Sounds like the patch could be split up, as there is an agreement on
> the atmel-mci part.
> 
> Aubin, can you make a separate patch for the atmel-mci part and add
> Ludovic's ack to it?
> 
> In regards to the sdhci driver, I suggest you just add yourself as a
> maintainer too, along with Eugen.

For adding more background to this patch, in fact it was created in 
coordination with Eugen. We can wait a bit for him to catch-up with 
emails, no hurry there as Aubin won't have access to emails for a few 
days anyway.

If it has an interest you can add my:
Acked-by: Nicolas Ferre <nicolas.ferre@microchip.com>
But definitively, no problem to delay this change.

Thanks for your understanding. Best regards,
   Nicolas

>>> ---
>>>    MAINTAINERS | 5 +++--
>>>    1 file changed, 3 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>> index 2833e2da63e0..52beaf4f7fbb 100644
>>> --- a/MAINTAINERS
>>> +++ b/MAINTAINERS
>>> @@ -14022,7 +14022,7 @@ F:    Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml
>>>    F:  drivers/iio/adc/mcp3911.c
>>>
>>>    MICROCHIP MMC/SD/SDIO MCI DRIVER
>>> -M:   Ludovic Desroches <ludovic.desroches@microchip.com>
>>> +M:   Aubin Constans <aubin.constans@microchip.com>
>>>    S:  Maintained
>>>    F:  drivers/mmc/host/atmel-mci.c
>>>
>>> @@ -19235,7 +19235,8 @@ F:    Documentation/devicetree/bindings/mmc/sdhci-common.yaml
>>>    F:  drivers/mmc/host/sdhci*
>>>
>>>    SECURE DIGITAL HOST CONTROLLER INTERFACE (SDHCI) MICROCHIP DRIVER
>>> -M:   Eugen Hristev <eugen.hristev@microchip.com>
>>> +M:   Aubin Constans <aubin.constans@microchip.com>
>>> +R:   Eugen Hristev <eugen.hristev@collabora.com>
>>>    L:  linux-mmc@vger.kernel.org
>>>    S:  Supported
>>>    F:  drivers/mmc/host/sdhci-of-at91.c
>>

Re: [PATCH] MAINTAINERS: mmc: take over as maintainer of MCI & SDHCI MICROCHIP DRIVERS

Posted by Eugen Hristev 2 years, 3 months ago

On 9/14/23 20:01, Nicolas Ferre wrote:
> On 14/09/2023 at 16:21, Ulf Hansson wrote:
>> On Tue, 12 Sept 2023 at 07:21, <Ludovic.Desroches@microchip.com> wrote:
>>>
>>> On 9/11/23 17:32, Aubin Constans wrote:
>>>> On the one hand Eugen has taken responsibilities outside Microchip,
>>>> on the other hand I have some experience with the Microchip SDMMC
>>>> SDHCI controller.
>>>> Change Eugen as reviewer and take over maintainership of the SDHCI
>>>> MICROCHIP DRIVER.
>>>> Also, take over maintainership of its predecessor, that is the MCI
>>>> MICROCHIP DRIVER.
>>>>
>>>> Cc: Eugen Hristev <eugen.hristev@collabora.com>
>>>> Cc: Ludovic Desroches <ludovic.desroches@microchip.com>
>>>> Signed-off-by: Aubin Constans <aubin.constans@microchip.com>
>>>
>>> For atmel-mci:
>>> Acked-by: Ludovic Desroches <ludovic.desroches@microchip.com>
>>
>> Sounds like the patch could be split up, as there is an agreement on
>> the atmel-mci part.
>>
>> Aubin, can you make a separate patch for the atmel-mci part and add
>> Ludovic's ack to it?
>>
>> In regards to the sdhci driver, I suggest you just add yourself as a
>> maintainer too, along with Eugen.
> 
> For adding more background to this patch, in fact it was created in 
> coordination with Eugen. We can wait a bit for him to catch-up with 
> emails, no hurry there as Aubin won't have access to emails for a few 
> days anyway.
> 
> If it has an interest you can add my:
> Acked-by: Nicolas Ferre <nicolas.ferre@microchip.com>
> But definitively, no problem to delay this change.

Hi,

I am fine with this change, as Nicolas has already discussed this with me.

Acked-by: Eugen Hristev <eugen.hristev@collabora.com>

P.S. I have a mailmap entry for the new e-mail so the change is not 
compulsory, you can change as you see fit.

Eugen

> 
> Thanks for your understanding. Best regards,
>    Nicolas
> 
>>>> ---
>>>>    MAINTAINERS | 5 +++--
>>>>    1 file changed, 3 insertions(+), 2 deletions(-)
>>>>
>>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>>> index 2833e2da63e0..52beaf4f7fbb 100644
>>>> --- a/MAINTAINERS
>>>> +++ b/MAINTAINERS
>>>> @@ -14022,7 +14022,7 @@ F:    
>>>> Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml
>>>>    F:  drivers/iio/adc/mcp3911.c
>>>>
>>>>    MICROCHIP MMC/SD/SDIO MCI DRIVER
>>>> -M:   Ludovic Desroches <ludovic.desroches@microchip.com>
>>>> +M:   Aubin Constans <aubin.constans@microchip.com>
>>>>    S:  Maintained
>>>>    F:  drivers/mmc/host/atmel-mci.c
>>>>
>>>> @@ -19235,7 +19235,8 @@ F:    
>>>> Documentation/devicetree/bindings/mmc/sdhci-common.yaml
>>>>    F:  drivers/mmc/host/sdhci*
>>>>
>>>>    SECURE DIGITAL HOST CONTROLLER INTERFACE (SDHCI) MICROCHIP DRIVER
>>>> -M:   Eugen Hristev <eugen.hristev@microchip.com>
>>>> +M:   Aubin Constans <aubin.constans@microchip.com>
>>>> +R:   Eugen Hristev <eugen.hristev@collabora.com>
>>>>    L:  linux-mmc@vger.kernel.org
>>>>    S:  Supported
>>>>    F:  drivers/mmc/host/sdhci-of-at91.c
>>>
>

Re: [PATCH] MAINTAINERS: mmc: take over as maintainer of MCI & SDHCI MICROCHIP DRIVERS

Posted by Nicolas Ferre 2 years, 1 month ago

Ulf,

On 18/09/2023 at 13:03, Eugen Hristev wrote:
> On 9/14/23 20:01, Nicolas Ferre wrote:
>> On 14/09/2023 at 16:21, Ulf Hansson wrote:
>>> On Tue, 12 Sept 2023 at 07:21, <Ludovic.Desroches@microchip.com> wrote:
>>>>
>>>> On 9/11/23 17:32, Aubin Constans wrote:
>>>>> On the one hand Eugen has taken responsibilities outside Microchip,
>>>>> on the other hand I have some experience with the Microchip SDMMC
>>>>> SDHCI controller.
>>>>> Change Eugen as reviewer and take over maintainership of the SDHCI
>>>>> MICROCHIP DRIVER.
>>>>> Also, take over maintainership of its predecessor, that is the MCI
>>>>> MICROCHIP DRIVER.
>>>>>
>>>>> Cc: Eugen Hristev <eugen.hristev@collabora.com>
>>>>> Cc: Ludovic Desroches <ludovic.desroches@microchip.com>
>>>>> Signed-off-by: Aubin Constans <aubin.constans@microchip.com>
>>>>
>>>> For atmel-mci:
>>>> Acked-by: Ludovic Desroches <ludovic.desroches@microchip.com>
>>>
>>> Sounds like the patch could be split up, as there is an agreement on
>>> the atmel-mci part.
>>>
>>> Aubin, can you make a separate patch for the atmel-mci part and add
>>> Ludovic's ack to it?
>>>
>>> In regards to the sdhci driver, I suggest you just add yourself as a
>>> maintainer too, along with Eugen.
>>
>> For adding more background to this patch, in fact it was created in
>> coordination with Eugen. We can wait a bit for him to catch-up with
>> emails, no hurry there as Aubin won't have access to emails for a few
>> days anyway.
>>
>> If it has an interest you can add my:
>> Acked-by: Nicolas Ferre <nicolas.ferre@microchip.com>
>> But definitively, no problem to delay this change.
> 
> Hi,
> 
> I am fine with this change, as Nicolas has already discussed this with me.
> 
> Acked-by: Eugen Hristev <eugen.hristev@collabora.com>

I think we have all the required tags since Eugen email: aka ping...

Tell me if you prefer that we use another route to Mainline (as you may 
not have remaining changes to queue). Best regards,
   Nicolas

> P.S. I have a mailmap entry for the new e-mail so the change is not
> compulsory, you can change as you see fit.
> 
> Eugen
> 
>>
>> Thanks for your understanding. Best regards,
>>     Nicolas
>>
>>>>> ---
>>>>>     MAINTAINERS | 5 +++--
>>>>>     1 file changed, 3 insertions(+), 2 deletions(-)
>>>>>
>>>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>>>> index 2833e2da63e0..52beaf4f7fbb 100644
>>>>> --- a/MAINTAINERS
>>>>> +++ b/MAINTAINERS
>>>>> @@ -14022,7 +14022,7 @@ F:
>>>>> Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml
>>>>>     F:  drivers/iio/adc/mcp3911.c
>>>>>
>>>>>     MICROCHIP MMC/SD/SDIO MCI DRIVER
>>>>> -M:   Ludovic Desroches <ludovic.desroches@microchip.com>
>>>>> +M:   Aubin Constans <aubin.constans@microchip.com>
>>>>>     S:  Maintained
>>>>>     F:  drivers/mmc/host/atmel-mci.c
>>>>>
>>>>> @@ -19235,7 +19235,8 @@ F:
>>>>> Documentation/devicetree/bindings/mmc/sdhci-common.yaml
>>>>>     F:  drivers/mmc/host/sdhci*
>>>>>
>>>>>     SECURE DIGITAL HOST CONTROLLER INTERFACE (SDHCI) MICROCHIP DRIVER
>>>>> -M:   Eugen Hristev <eugen.hristev@microchip.com>
>>>>> +M:   Aubin Constans <aubin.constans@microchip.com>
>>>>> +R:   Eugen Hristev <eugen.hristev@collabora.com>
>>>>>     L:  linux-mmc@vger.kernel.org
>>>>>     S:  Supported
>>>>>     F:  drivers/mmc/host/sdhci-of-at91.c
>>>>
>>
>

Re: [PATCH] MAINTAINERS: mmc: take over as maintainer of MCI & SDHCI MICROCHIP DRIVERS

Posted by Krzysztof Kozlowski 2 years, 3 months ago

On 11/09/2023 17:32, Aubin Constans wrote:
> On the one hand Eugen has taken responsibilities outside Microchip,

That's not a reason to drop Eugen or change his status to a reviewer...
Unless of course this is what Eugen wants?
Normally you should just update the email address, although if it is
still not updated, then indeed change might be needed here.

> on the other hand I have some experience with the Microchip SDMMC
> SDHCI controller.
> Change Eugen as reviewer and take over maintainership of the SDHCI
> MICROCHIP DRIVER.

Best regards,
Krzysztof