2023-09-14 22:28:03

by Nicolas Ferre

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

On 14/09/2023 at 16:21, Ulf Hansson wrote:
> On Tue, 12 Sept 2023 at 07:21, <[email protected]> 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 <[email protected]>
>>> Cc: Ludovic Desroches <[email protected]>
>>> Signed-off-by: Aubin Constans <[email protected]>
>>
>> For atmel-mci:
>> Acked-by: Ludovic Desroches <[email protected]>
>
> 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 <[email protected]>
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 <[email protected]>
>>> +M: Aubin Constans <[email protected]>
>>> 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 <[email protected]>
>>> +M: Aubin Constans <[email protected]>
>>> +R: Eugen Hristev <[email protected]>
>>> L: [email protected]
>>> S: Supported
>>> F: drivers/mmc/host/sdhci-of-at91.c
>>


2023-09-19 00:22:27

by Eugen Hristev

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

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, <[email protected]> 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 <[email protected]>
>>>> Cc: Ludovic Desroches <[email protected]>
>>>> Signed-off-by: Aubin Constans <[email protected]>
>>>
>>> For atmel-mci:
>>> Acked-by: Ludovic Desroches <[email protected]>
>>
>> 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 <[email protected]>
> 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 <[email protected]>

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 <[email protected]>
>>>> +M:   Aubin Constans <[email protected]>
>>>>    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 <[email protected]>
>>>> +M:   Aubin Constans <[email protected]>
>>>> +R:   Eugen Hristev <[email protected]>
>>>>    L:  [email protected]
>>>>    S:  Supported
>>>>    F:  drivers/mmc/host/sdhci-of-at91.c
>>>
>

2023-10-23 09:54:10

by Nicolas Ferre

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

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, <[email protected]> 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 <[email protected]>
>>>>> Cc: Ludovic Desroches <[email protected]>
>>>>> Signed-off-by: Aubin Constans <[email protected]>
>>>>
>>>> For atmel-mci:
>>>> Acked-by: Ludovic Desroches <[email protected]>
>>>
>>> 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 <[email protected]>
>> 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 <[email protected]>

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 <[email protected]>
>>>>> +M: Aubin Constans <[email protected]>
>>>>> 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 <[email protected]>
>>>>> +M: Aubin Constans <[email protected]>
>>>>> +R: Eugen Hristev <[email protected]>
>>>>> L: [email protected]
>>>>> S: Supported
>>>>> F: drivers/mmc/host/sdhci-of-at91.c
>>>>
>>
>