From mboxrd@z Thu Jan 1 00:00:00 1970 From: Randy Dunlap Subject: Re: [PATCH -next] driver core: fix build for CONFIG_BLOCK=n Date: Thu, 23 Sep 2010 12:47:51 -0700 Message-ID: <20100923124751.cc59a349.rdunlap@xenotime.net> References: <20100920161239.f204a507.sfr@canb.auug.org.au> <20100920092034.0b3f5898.randy.dunlap@oracle.com> <20100921223115.GB21202@kroah.com> <20100923111541.97abcd05.rdunlap@xenotime.net> <20100923191629.GB18736@kroah.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: Received: from xenotime.net ([72.52.115.56]:41340 "HELO xenotime.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1754017Ab0IWTsA (ORCPT ); Thu, 23 Sep 2010 15:48:00 -0400 Received: from chimera.site ([173.50.240.230]) by xenotime.net for ; Thu, 23 Sep 2010 12:47:52 -0700 In-Reply-To: <20100923191629.GB18736@kroah.com> Sender: linux-next-owner@vger.kernel.org List-ID: To: Greg KH Cc: Stephen Rothwell , Kay Sievers , gregkh@suse.de, linux-next@vger.kernel.org, LKML On Thu, 23 Sep 2010 12:16:29 -0700 Greg KH wrote: > On Thu, Sep 23, 2010 at 11:15:41AM -0700, Randy Dunlap wrote: > > On Tue, 21 Sep 2010 15:31:15 -0700 Greg KH wrote: > > > > > On Mon, Sep 20, 2010 at 09:20:34AM -0700, Randy Dunlap wrote: > > > > From: Randy Dunlap > > > > > > > > Fix build when CONFIG_BLOCK is not enabled (since SEP-06). > > > > Fixes these build errors: > > > > > > > > drivers/base/core.c: In function 'get_device_parent': > > > > drivers/base/core.c:622: error: 'block_class' undeclared (first use in this function) > > > > drivers/base/core.c: In function 'device_add_class_symlinks': > > > > drivers/base/core.c:712: error: 'block_class' undeclared (first use in this function) > > > > drivers/base/core.c: In function 'device_remove_class_symlinks': > > > > drivers/base/core.c:742: error: 'block_class' undeclared (first use in this function) > > > > > > > > Signed-off-by: Randy Dunlap > > > > > > This should no longer be needed with the next linux-next tree as the > > > logic in this area has changed thanks to Andi's patch. ~~~~~~~~~~~~~~~~~~~~~~~ > > > > {sorry about resend, email problems} > > > > > > Still seeing this build error. (linux-next 2010-0923) > > Will you be merging some patch soon? > > Not unless someone provides it. I know of no valid user of the kernel > that does not enable CONFIG_BLOCK, so I really doubt this will ever hit > anyone in "real life" so it's _way_ down the list of things I have to do > at the moment, sorry. so what was your comment about Andi's patch (above)? Do you have it? Have you merged it? --- ~Randy *** Remember to use Documentation/SubmitChecklist when testing your code ***