From mboxrd@z Thu Jan 1 00:00:00 1970 From: Adrian Hunter Subject: Re: New MMC maintainer needed Date: Mon, 03 Aug 2009 14:13:28 +0300 Message-ID: <4A76C658.6050002@nokia.com> References: <20090714153601.6dfe70ff@mjolnir.ossman.eu> <20090722151744.fffd7bf5.akpm@linux-foundation.org> <20090728222334.0c543c47@mjolnir.ossman.eu> <20090731122623.254fd0f1@mjolnir.ossman.eu> <20090731105407.GA31900@console-pimps.org> <20090803123429.390a636f@mjolnir.ossman.eu> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20090803123429.390a636f@mjolnir.ossman.eu> Sender: linux-kernel-owner@vger.kernel.org List-ID: Content-Type: text/plain; charset="us-ascii"; format="flowed" To: Pierre Ossman Cc: Matt Fleming , Andrew Morton , "linux-kernel@vger.kernel.org" , "linux-embedded@vger.kernel.org" , "nico@cam.org" , "nicolas.ferre@rfo.atmel.com" , "hskinnemoen@atmel.com" , "tony@atomide.com" , "david-b@pacbell.net" , "manuel.lauss@gmail.com" , "mirq-linux@rere.qmqm.pl" , "ppisa@pikron.com" , "Lavinen Jarkko (Nokia-D/Helsinki)" , "ben@fluff.org" , "saschasommer@freenet.de" , "avorontsov@ru.mvista.com" , "oakad@yahoo.com" , "ian@mnementh.co.uk" , HaraldWelte@ Pierre Ossman wrote: > On Fri, 31 Jul 2009 11:54:07 +0100 > Matt Fleming wrote: > >> On Fri, Jul 31, 2009 at 12:26:23PM +0200, Pierre Ossman wrote: >>> [PATCH 0/32] mmc and omap_hsmmc patches >>> http://marc.info/?t=124722953900010&r=1&w=2 >>> >>> I haven't looked through these at all. The ones affecting the core >>> probably need some thorough reviews. >>> >>> I did notice the patch to say which cards a controller supports though, >>> and I'm very sceptical about that one. The scanning process should work >>> anyway, and the performance impact should be negligible as it is only >>> on init. So that patch only adds complexity and confusion IMO. >>> >> How much complexity does it really add? Surely it's better to give the >> host controller driver writers the ability to not entertain supporting >> some cards if they cannot be used? If they want to avoid the scanning >> process for certain cards, why not let them? >> > > Let's look at the pros and cons of this: But the cons are all subjective. > Con: > > - The scanning code gets less clear as you increase the number of > possible paths through it. > > - Different systems will have different init sequences, possibly > provoking bugs in the cards. > > - Host driver writers now have more capability bits they have to > consider. And these might be less than obvious since SD/MMC/SDIO are > normally compatible so these bits seem useless. > > - With the current logic (which was better in the first version), > "normal" drivers will have to explicitly state that they work as > intended by setting all bits. And the pro is objective. > Pro: > > - A slightly reduced scanning time. That's great! Why do you disregard this so easily? > I simply don't see it as being worth it. Linux patches generally need > to provide the answer to "Why?", not just be able to avoid "Why not?". You have just supplied answers to both "Why?" and "Why not?". In my opinion "Why?" outweighs "Why not?" because the pro is objective and the cons and not.