From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoph Hellwig Subject: Re: [PATCH 5/6] mmc: debugfs: Move card status retrieveal into the block layer Date: Tue, 23 May 2017 11:52:22 +0200 Message-ID: <20170523095222.GA25307@lst.de> References: <20170519133732.27470-1-linus.walleij@linaro.org> <20170519133732.27470-6-linus.walleij@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-block-owner@vger.kernel.org To: Linus Walleij Cc: Ulf Hansson , "linux-mmc@vger.kernel.org" , Adrian Hunter , linux-block@vger.kernel.org, Jens Axboe , Christoph Hellwig , Arnd Bergmann , Bartlomiej Zolnierkiewicz , Paolo Valente , Avri Altman List-Id: linux-mmc@vger.kernel.org On Tue, May 23, 2017 at 11:49:48AM +0200, Linus Walleij wrote: > In that case CONFIG_MMC_BLOCK contains 'm' instead of 'y' > which confusingly does not evaluate to true in the preprocessor > (it assumes it is a misspelled 'n' I guess). > > And then the autobuilders wreak havoc. > > And that is why the IS_ENABLED() defines exist in the first > place IIUC. > > I'm all for making CONFIG_MMC_BLOCK into a bool... but > don't know how people (Intel laptops) feel about that extra > code in their kernel at all times. All the time? At least only if any mmc/sd host driver is loaded, right? I don't think those additional 20k fatten the cow (as we say in German).