From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [PATCH 02/20] OMAP: McBSP: Wakeups utilized Date: Thu, 6 Aug 2009 15:05:25 +0300 Message-ID: <20090806120525.GJ2358@atomide.com> References: <1248958183-15015-1-git-send-email-eduardo.valentin@nokia.com> <1248958183-15015-2-git-send-email-eduardo.valentin@nokia.com> <1248958183-15015-3-git-send-email-eduardo.valentin@nokia.com> <20090730215602.8fdcbaf1.jhnikula@gmail.com> <20090731075622.GA27083@esdhcp037198.research.nokia.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mho-01-ewr.mailhop.org ([204.13.248.71]:54180 "EHLO mho-01-ewr.mailhop.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755257AbZHFMFc (ORCPT ); Thu, 6 Aug 2009 08:05:32 -0400 Content-Disposition: inline In-Reply-To: <20090731075622.GA27083@esdhcp037198.research.nokia.com> Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Eduardo Valentin Cc: ext Jarkko Nikula , "linux-omap@vger.kernel.org" , "alsa-devel@vger.kernel.org" , "Nurkkala Eero.An (EXT-Offcode/Oulu)" , "Ujfalusi Peter (Nokia-D/Tampere)" * Eduardo Valentin [090731 11:08]: > On Thu, Jul 30, 2009 at 08:56:02PM +0200, ext Jarkko Nikula wrote: > > On Thu, 30 Jul 2009 15:49:25 +0300 > > Eduardo Valentin wrote: > > > > > --- a/arch/arm/plat-omap/mcbsp.c > > > +++ b/arch/arm/plat-omap/mcbsp.c > > > @@ -257,6 +257,23 @@ int omap_mcbsp_request(unsigned int id) > > > clk_enable(mcbsp->iclk); > > > clk_enable(mcbsp->fclk); > > > > > > +#ifdef CONFIG_ARCH_OMAP34XX > > > + /* > > > > I'm sure there is a way to avoid all of these added ifdefs in this patch > > and in patches 7 and 8 as well. > > hmmm.. these ifdef's are required because I've put the extra field (for > 34xx only) inside an ifdef on their declaration in mcbsp.h. We can remove it > there, but will leave a few extra bytes on !omap3xx binaries. Hmm which field are you talking about? The define does not matter, and the u16 syscon also gets optimized out if 34xx is not selected. Tony