Skip to contentSkip to conversation

Orphaning MMC host drivers

RFC In review Pierre Ossman 5 participants 10 messages May 14, 2007 – Jul 15, 2007

lore
Share & tools
lore

Pierre Ossman

author
#0 lore raw

RFC 1/1 Orphaning MMC host drivers

I've reached the point where I've grown tired of trying to figure out who has hardware for what. I intend to commit the following in a few days so if you care about the quality of the drivers it's time to step up to the plate.

There's a rather long list of cc here, but I've included everyone that has been involved in the drivers in one way or another in case one or two maintainers can actually be found.

Even if you don't feel like maintaining this, feel free to suggest mailing lists that should be added to the entries.

Rgds

Signature · 5 lines
     -- Pierre Ossman

  Linux kernel, MMC maintainer        http://www.kernel.org
  PulseAudio, core developer          http://pulseaudio.org
  rdesktop, core developer          http://www.rdesktop.org

commit 11c3d5e4b871fc69d72f67811d4eb6bfd7084c43

Author: Pierre Ossman <[email protected]>
Date:   Mon May 14 21:25:26 2007 +0200

mmc: mark unmaintained drivers

Most of the host controller drivers in the MMC layer lacks an official maintainer. Make sure this is mentioned in MAINTAINERS in case someone wants to pick up the ball.

Signed-off-by Pierre Ossman <[email protected]>
MAINTAINERS +18 -0
index 68a56ad..8f1b60c 100644
@@ -346,6 +346,9 @@ P:	Ivan Kokshaysky
 M:	[email protected]
 S:	Maintained for 2.4; PCI support for 2.6.
 
+AMD AU1XX0 MMC DRIVER
+S:	Orphan
+
 AMD GEODE PROCESSOR/CHIPSET SUPPORT
 P:	Jordan Crouse
 M:	[email protected]

@@ -418,6 +421,9 @@ P:	Ian Molton
 M:	[email protected]
 S:	Maintained
 
+ARM PRIMECELL MMCI PL180/1 DRIVER
+S:	Orphan
+
 ARM/ADI ROADRUNNER MACHINE SUPPORT
 P:	Lennert Buytenhek
 M:	[email protected]

@@ -649,6 +655,9 @@ L:	[email protected] (subscribers-only)
 W:	http://linux-atm.sourceforge.net
 S:	Maintained
 
+ATMEL AT91 MCI DRIVER
+S:	Orphan
+
 ATMEL MACB ETHERNET DRIVER
 P:	Haavard Skinnemoen
 M:	[email protected]

@@ -2382,6 +2391,9 @@ M:	[email protected]
 W:	http://popies.net/meye/
 S:	Maintained
 
+MOTOROLA I.MX MMCI DRIVER
+S:	Orphan
+
 MOUSE AND MISC DEVICES [GENERAL]
 P:	Alessandro Rubini
 M:	[email protected]

@@ -2902,6 +2914,9 @@ M:	[email protected]
 L:	[email protected]	(subscribers-only)
 S:	Maintained
 
+PXA MMCI DRIVER
+S:	Orphan
+
 QLOGIC QLA2XXX FC-SCSI DRIVER
 P:	Andrew Vasquez
 M:	[email protected]

@@ -3417,6 +3432,9 @@ P:      Alex Dubov
 M:      [email protected]
 S:      Maintained
 
+TI OMAP MMC INTERFACE DRIVER
+S:	Orphan
+
 TI OMAP RANDOM NUMBER GENERATOR SUPPORT
 P:	Deepak Saxena
 M:	[email protected]
#1 lore raw

Review M:        [email protected]

Quoted 11 lines · Pierre Ossman
(quoted diff) +0 -0
@@ -2382,6 +2391,9 @@ M:        [email protected]

 W:     http://popies.net/meye/
 S:     Maintained

(quoted diff) +3 -0
+MOTOROLA I.MX MMCI DRIVER
+S:     Orphan
+

 MOUSE AND MISC DEVICES [GENERAL]
 P:     Alessandro Rubini
 M:     [email protected]

(quoted diff) +0 -0
@@ -2902,6 +2914,9 @@ M:        [email protected]

Hello Pierre,

as I have already stated, I would take responsibility for I.MX SDHC driver. It is not perfect, but I am trying to keep and fix it and I would be happy to receive feedback or help from others.

The actual state is such, that read works for both SD/MMC, but write is failing. The write path is timing sensitive and some other kernel changes above 2.6.18 exposes these problems. I have some ideas, how to dig for this problem in mind, but I did not find time to try them yet.

I have already proposed MAINTAINERS change, but you did not liked to point L: field to "sdhci-devel" list. I propose ALKML mailing list this time as you suggested.

Do you include change or should I send my own patch for review? Would you suggest change in driver name to MOTOROLA I.MX MMCI DRIVER ?

Best wishes

Pavel

My original proposal with L: changed to ALKML

MAINTAINERS +7 -0
Index: linux-2.6.21/MAINTAINERS
===================================================================
@@ -1717,6 +1717,13 @@ M:       [email protected]
 L:     [email protected] (subscribers-only)
 S:     Maintained
 
+IMX MMC/SD HOST CONTROLLER INTERFACE DRIVER
+P:     Pavel Pisa
+M:     [email protected]
+L:     [email protected] (subscribers-only)
+W:     http://mmc.drzeus.cx/wiki/Controllers/Freescale/SDHC
+S:     Maintained
+
 INFINIBAND SUBSYSTEM
 P:     Roland Dreier
 M:     [email protected]
author
#2 lore raw
Quoted 6 lines · Pavel Pisa

as I have already stated, I would take responsibility for I.MX SDHC driver. It is not perfect, but I am trying to keep and fix it and I would be happy to receive feedback or help from others.

Great.

Quoted 5 lines

Do you include change or should I send my own patch for review? Would you suggest change in driver name to MOTOROLA I.MX MMCI DRIVER ?

I'll included it in my end.

As for the name, I think it's easier for people to find stuff if the vendor name is included. I'll take your name (which is more detailed) and stick a MOTOROLA in the front.

Rgds

Signature · 5 lines
     -- Pierre Ossman

  Linux kernel, MMC maintainer        http://www.kernel.org
  PulseAudio, core developer          http://pulseaudio.org
  rdesktop, core developer          http://www.rdesktop.org
author
#4 lore raw
Quoted 4 lines · Khasim

Hi Pierre,

Carlos Aguiar and Anderson Briglia are interested in making sure the driver works for existing boards as they have access to them, and I can make it work for new omaps (2430, 3430).

Great. I'll update my info.

Signature · 5 lines
     -- Pierre Ossman

  Linux kernel, MMC maintainer        http://www.kernel.org
  PulseAudio, core developer          http://pulseaudio.org
  rdesktop, core developer          http://www.rdesktop.org
#5 lore raw
In reply to Pierre Ossman
Quoted 4 lines · Pierre Ossman

I've reached the point where I've grown tired of trying to figure out who has hardware for what. I intend to commit the following in a few days so if you care about the quality of the drivers it's time to step up to the plate.

You can add me for mmci.c

Signature · 3 lines
Russell King
 Linux kernel    2.6 ARM Linux   - http://www.arm.linux.org.uk/
 maintainer of:
author
#6 lore raw
Quoted 8 lines · Russell King
Quoted 4 lines · Pierre Ossman

I've reached the point where I've grown tired of trying to figure out who has hardware for what. I intend to commit the following in a few days so if you care about the quality of the drivers it's time to step up to the plate.

You can add me for mmci.c

Great. Do you want the LAKML there aswell?

Signature · 5 lines
     -- Pierre Ossman

  Linux kernel, MMC maintainer        http://www.kernel.org
  PulseAudio, core developer          http://pulseaudio.org
  rdesktop, core developer          http://www.rdesktop.org
#7 lore raw
Quoted 11 lines · Pierre Ossman
Quoted 8 lines · Russell King
Quoted 4 lines · Pierre Ossman

I've reached the point where I've grown tired of trying to figure out who has hardware for what. I intend to commit the following in a few days so if you care about the quality of the drivers it's time to step up to the plate.

You can add me for mmci.c

Great. Do you want the LAKML there aswell?

Probably be a good idea.

Signature · 3 lines
Russell King
 Linux kernel    2.6 ARM Linux   - http://www.arm.linux.org.uk/
 maintainer of:
#8 lore raw
In reply to Pierre Ossman

Pierre Ossman :

MAINTAINERS L: [email protected] (subscribers-only) · line 660

Quoted 3 lines
MAINTAINERS +3 -0
+ATMEL AT91 MCI DRIVER
+S:	Orphan
+

Hi Pierre,

I come back on this request... better late than never ;-)

As I have patched the AT91 MCI driver these days, with the kind help of a few people, I think I can now add myself as maintainer for this driver. Here is the entry below.

Kind regards,

Signature · 18 lines
Nicolas Ferre

--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -662,7 +662,12 @@
 S:	Maintained

 ATMEL AT91 MCI DRIVER
-S:	Orphan
+P:	Nicolas Ferre
+M:	[email protected]
+L:	[email protected] (subscribers-only)
+W:	http://www.atmel.com/products/AT91/
+W:	http://www.at91.com/
+S:	Maintained

 ATMEL MACB ETHERNET DRIVER
 P:	Haavard Skinnemoen
author
#9 lore raw
Quoted 9 lines · On Fri, 13 Jul 2007 13:35:35 +0200 Nicolas Ferre <[email protected]>…

Hi Pierre,

I come back on this request... better late than never ;-)

As I have patched the AT91 MCI driver these days, with the kind help of a few people, I think I can now add myself as maintainer for this driver. Here is the entry below.

Wonderful. And good luck ;)

Rgds

Signature · 5 lines
     -- Pierre Ossman

  Linux kernel, MMC maintainer        http://www.kernel.org
  PulseAudio, core developer          http://pulseaudio.org
  rdesktop, core developer          http://www.rdesktop.org