Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758338AbZAIVtu (ORCPT ); Fri, 9 Jan 2009 16:49:50 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1755948AbZAIVtk (ORCPT ); Fri, 9 Jan 2009 16:49:40 -0500 Received: from rcsinet11.oracle.com ([148.87.113.123]:16799 "EHLO rgminet11.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753759AbZAIVtj (ORCPT ); Fri, 9 Jan 2009 16:49:39 -0500 Message-ID: <4967C61E.9090807@oracle.com> Date: Fri, 09 Jan 2009 13:48:14 -0800 From: Randy Dunlap Organization: Oracle Linux Engineering User-Agent: Thunderbird 2.0.0.6 (X11/20070801) MIME-Version: 1.0 To: Alexey Korolev CC: Stephen Rothwell , linux-next@vger.kernel.org, LKML , David Woodhouse , Vasiliy Leonenko , Linus Torvalds Subject: Re: linux-next: Tree for January 6 (MTD) References: <20090106182311.525e4b14.sfr@canb.auug.org.au> <496386DB.8090606@oracle.com> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit X-Source-IP: acsmt706.oracle.com [141.146.40.84] X-Auth-Type: Internal IP X-CT-RefId: str=0001.0A090202.4967C622.023A:SCFSTAT928724,ss=1,fgs=0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2300 Lines: 67 This commit was just merged: Gitweb: http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=60f26520e7bd4479dbebf77317c45fc2255ba2e8 Commit: 60f26520e7bd4479dbebf77317c45fc2255ba2e8 Parent: d81408304b06a71c28417445202af9cd6673168d Author: Alexey Korolev AuthorDate: Tue Dec 16 18:24:14 2008 +0000 Committer: David Woodhouse CommitDate: Mon Jan 5 13:58:47 2009 +0100 [MTD] LPDDR Makefile and KConfig but the patch below is missing/still needed. Alexey Korolev wrote: > Hi Randy, > >> # CONFIG_MTD_LPDDR is not set >> CONFIG_MTD_QINFO_PROBE=y >> >> There appears to be too little control/coordination between those >> two kconfig symbols. Surely one of them (MTD_QINFO_PROBE) should >> depend on the other (MTD_LPDDR). And why even build 2 modules >> when MTD_QINFO_PROBE is enabled? Why not combine the code into >> one module? > Thanks a lot for good finding. Here is a fix for the problem. > > About combining: it is assumed to have one QINFO probing module for several > kinds of LPDDR command set drivers. If we combine two modules into > one it would make further extension rather hard. So it is better to have > two modules. > To fix a problem I just added dependency in Kconfig file. > This temporary fix which will work until we have just one cmdset module. > In near future we will make a fix which parse command set types > and select proper driver according to device type. > > Thanks, > Alexey > > Signed-off-by: Alexey Korolev > --- > diff --git a/drivers/mtd/lpddr/Kconfig b/drivers/mtd/lpddr/Kconfig > index acd4ea9..5a401d8 100644 > --- a/drivers/mtd/lpddr/Kconfig > +++ b/drivers/mtd/lpddr/Kconfig > @@ -12,6 +12,7 @@ config MTD_LPDDR > DDR memories, intended for battery-operated systems. > > config MTD_QINFO_PROBE > + depends on MTD_LPDDR > tristate "Detect flash chips by QINFO probe" > help > Device Information for LPDDR chips is offered through the Overlay > > -- ~Randy -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/