| Message ID | 20230911153246.137148-1-aubin.constans@microchip.com |
|---|---|
| State | New |
| Headers | show |
| Series | MAINTAINERS: mmc: take over as maintainer of MCI & SDHCI MICROCHIP DRIVERS | expand |
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 >
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 >>
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 >>> >
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 >>>> >> >
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 --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
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(-)