On 11/17/2010 2:53 AM, Dong, Chuanxiao wrote:

this code has a history of breaking all other systems.
I'm not even going to think about taking this until it's accepted by the
upstream maintainers.
The reason this code breaking all other systems is the wrong usage of Dekker 
algorithm in MMC block layer. But actually this code is only for MFLD and was 
wrapped by MFLD controller quirks. I think the wrong use of any platform 
specific code maybe can also break other systems. Agree? And if you are still 
not going to review these patches, let me know.

let me know how the upstream review goes.
really.

this stuff is not even "young" anymore; we're making exceptions for code that isn't upstream yet for your team, but the hard assumption is that the code will go upstream within a few (3) months... ... these patches are older than a few months already..... they really should have been sent and accepted upstream by now. And given their troublesome history and touching of generic code, I'm going to stick to that.

_______________________________________________
MeeGo-kernel mailing list
[email protected]
http://lists.meego.com/listinfo/meego-kernel

Reply via email to