From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pavel Machek Subject: Re: [PATCH 1/3] libata: add missing PM callbacks Date: Tue, 6 Mar 2007 09:22:12 +0100 Message-ID: <20070306082212.GC2040@elf.ucw.cz> References: <20070302083001.GK20322@htj.dyndns.org> <45E8B3C8.2090009@garzik.org> <45E8E749.4030406@gmail.com> <20070306080509.GA2040@elf.ucw.cz> <45ED22AB.4090901@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from gprs189-60.eurotel.cz ([160.218.189.60]:36328 "EHLO amd.ucw.cz" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752929AbXCFIWX (ORCPT ); Tue, 6 Mar 2007 03:22:23 -0500 Content-Disposition: inline In-Reply-To: <45ED22AB.4090901@gmail.com> Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Tejun Heo Cc: Jeff Garzik , linux-ide@vger.kernel.org, Alan , Linux Kernel Mailing List Hi! > > Much complexity for little gain. Who is running _without_ CONFIG_PM > > these days? > > Embedded people, I guess. The problem here is that if we are gonna > support !CONFIG_PM configuration and try to reduce the kernel/module > images size for such case, we end up sprinkling #ifdef's all over huge > number of device drivers. Ok, in separate mail I see you already did the work... > If we determine to drop !CONFIG_PM configuration, I'm happy with that > too but we need to determine something here. Alternatives... > > 1. drop !CONFIG_PM configuration > 2. continue to sprinkle #ifdef's over device drivers > 3. find out prettier way to mark PM functions ...of 3., and code is nicely generic and fairly simple. I thought it would be too much work to implemente it, and it would get too messy. Given that it is so simple, I believe 3. is the best option. We can still do 1. in future if we feel like so... Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html