From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from sunset.davemloft.net (unknown [74.93.104.97]) by ozlabs.org (Postfix) with ESMTP id 629FFDDF19 for ; Wed, 13 Feb 2008 20:05:19 +1100 (EST) Date: Wed, 13 Feb 2008 01:05:50 -0800 (PST) Message-Id: <20080213.010550.211671868.davem@davemloft.net> To: sam@ravnborg.org Subject: Re: [GIT]: Make LMB code sharable with sparc64. From: David Miller In-Reply-To: <20080213085735.GB8091@uranus.ravnborg.org> References: <20080213.004120.22092044.davem@davemloft.net> <20080213085735.GB8091@uranus.ravnborg.org> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Cc: sparclinux@vger.kernel.org, linuxppc-dev@ozlabs.org, linux-kernel@vger.kernel.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Sam Ravnborg Date: Wed, 13 Feb 2008 09:57:35 +0100 > Review had been easier if the patch was inlined. Sorry :) > Can we plase have this changed to use: > > config SPARC64 > + select HAVE_LMB > > And then in lib/Kconfig have > +config HAVE_LMB > + bool > > So we avoid creating a new variable each time we introdce LMB support. > You would need to update powerpc Kconfig too of course. Sure, I'll rework this in the next day or so to do things that way. Thanks for the review.