From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from 1wt.eu ([62.212.114.60]) by merlin.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1WD9IC-0003Vw-Mx for linux-mtd@lists.infradead.org; Tue, 11 Feb 2014 09:06:13 +0000 Date: Tue, 11 Feb 2014 10:05:33 +0100 From: Willy Tarreau To: Geert Uytterhoeven Subject: Re: [PATCH 1/1] ubi: Introduce block devices for UBI volumes Message-ID: <20140211090533.GA5414@1wt.eu> References: <20140210024827.GB9643@localhost> <1392017750.31031.8.camel@sauron.fi.intel.com> <20140210082714.GB10872@localhost> <20140210084616.GP22376@1wt.eu> <20140210142026.GB15607@localhost> <1392043814.32363.49.camel@sauron.fi.intel.com> <20140210184858.GA6484@localhost> <20140210214333.GH28464@1wt.eu> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Cc: Thomas Petazzoni , Mike Frysinger , Artem Bityutskiy , Richard Weinberger , "linux-mtd@lists.infradead.org" , Ezequiel Garcia , Michael Opdenacker , Piergiorgio Beruto , Brian Norris , David Woodhouse List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Tue, Feb 11, 2014 at 09:37:25AM +0100, Geert Uytterhoeven wrote: > On Mon, Feb 10, 2014 at 10:43 PM, Willy Tarreau wrote: > > On Mon, Feb 10, 2014 at 10:39:18PM +0100, Piergiorgio Beruto wrote: > >> Hi all, > >> Please, the possibility to disable at compile time unneeded code to save > >> memory and/or cpu speed would make the life of embedded developers much more > >> easier. > >> > >> Ubiblk was exactly what we were looking for, in my personal opinion the > >> possibility to disable caching and rw feature is of very high value. > > > > Reminds me the happy days of CONFIG_TINY :-) > > So in practice it's probably just a matter of turning the options to : > > > > Disable Caching (default N, say Y to save memory) > > Disable Write support (default N, say Y to reduce code size) > > But then allmodconfig builds will not exercise those code paths. > > Don't you want: > > bool "Enable Caching" if !EXPERT > default y > > bool "Enable Write support" if !EXPERT > default y > > ? Sounds like an excellent idea! Willy