Re: [PATCH] mmc: mmci: assume maintainership

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Mar 15, 2012 at 03:30:36PM +0100, Linus Walleij wrote:
> From: Linus Walleij <linus.walleij@xxxxxxxxxx>
> 
> So since this driver is crucial for us (as in ST-Ericsson)
> to have actively maintained, and since it is marked orphaned
> I will assume maintainership for it starting with the 3.5
> integration cycle. The idea is to collect patches and send
> either patch sets of pull requests to Chris.

Actually, I still want to be involved because I'm far from convinced with
some of the patches coming from Ulf are correct for ARMs implementation.
That's why I'm purposely not applying the MMCI patches in the patch system
in a particularly fast manner - I want there to be a decent amount of
testing between each set of patches.

For example, the next patch in the series to be applied is:

"mmc: mmci: Decrease current consumption in suspend"

and as I've explained before, it does this by saving the MMCI power and
clock registers and clearing them.  This has the effect of stopping the
clock, and in the case of ARMs hardware, removing power from the card.
Therefore, the card will require full bring up, including reassigning
the card address for MMC cards on resume, and simply starting the next
command will not be possible.

As I've already tried to explain, on the normal power management suspend
callback, these registers should _already_ have been placed into their
"card not present" state, so saving and zeroing them has no useful effect.

Moreover, saving and restoring them across suspend is wrong because on
ARM hardware there's a sequence of power-off, power-up, power-on which
needs to be gone through to restore power.

So, I've queued up the 10 patches which I'm happy to take for v3.4, and
that's all I'm taking for v3.4 until I can find the strength to have
another discussion with Ulf over this issue.

As there's an interdependence between the various patches, I can't apply
any of the later patches even if I wanted to without these "power saving"
patches.

So, it's not really unmaintained or orphaned...

-- 
Russell King
 Linux kernel    2.6 ARM Linux   - http://www.arm.linux.org.uk/
 maintainer of:
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux