diff mbox series

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

Message ID 20230911153246.137148-1-aubin.constans@microchip.com (mailing list archive)
State New, archived
Headers show
Series MAINTAINERS: mmc: take over as maintainer of MCI & SDHCI MICROCHIP DRIVERS | expand

Commit Message

Aubin Constans Sept. 11, 2023, 3:32 p.m. UTC
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(-)

Comments

Krzysztof Kozlowski Sept. 11, 2023, 3:52 p.m. UTC | #1
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
Ludovic Desroches Sept. 12, 2023, 5:21 a.m. UTC | #2
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
Ulf Hansson Sept. 14, 2023, 2:21 p.m. UTC | #3
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
>
Nicolas Ferre Sept. 14, 2023, 5:01 p.m. UTC | #4
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
>>
Eugen Hristev Sept. 18, 2023, 11:03 a.m. UTC | #5
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
>>>
>
Nicolas Ferre Oct. 23, 2023, 9:53 a.m. UTC | #6
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
>>>>
>>
>
Ulf Hansson Oct. 23, 2023, 11:50 a.m. UTC | #7
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
>
diff mbox series

Patch

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