From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ksp.mff.cuni.cz ([195.113.26.206]:35355 "EHLO atrey.karlin.mff.cuni.cz" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753377Ab0DTFTw (ORCPT ); Tue, 20 Apr 2010 01:19:52 -0400 Date: Tue, 20 Apr 2010 07:18:54 +0200 From: Pavel Machek Subject: Re: [PATCH 2/5] arm: msm: smd: use either package v3 or v4 not both Message-ID: <20100420051854.GE2075@ucw.cz> References: <> <1271700189-8376-2-git-send-email-dwalker@codeaurora.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1271700189-8376-2-git-send-email-dwalker@codeaurora.org> Sender: linux-arm-msm-owner@vger.kernel.org List-ID: To: Daniel Walker Cc: linux-arm-msm@vger.kernel.org, dima@android.com, Daniel Walker On Mon 2010-04-19 11:03:06, Daniel Walker wrote: > From: Daniel Walker > > This modifies SMD to use either the package v3 or package v4, > but not both. The current code tries to allocate as v4 on all > system which can produce a scary looking error message on boot up, > > smem_find(16, 40): wrong size 16424 > smd_alloc_channel() cid=02 size=08192 'SMD_RPCCALL' > > With this error the code then falls back on the package v3 allocation > method. This method is inefficient because it causes a slow down > on some systems even when the allocation method can be determined > at compile time. It also causes a kernel size increase that effects > all system and is not needed. Well... it adds about 100 bytes to kernel text and produces unmeasurable slowdown. > --- a/arch/arm/mach-msm/Kconfig > +++ b/arch/arm/mach-msm/Kconfig > @@ -7,6 +7,7 @@ choice > config ARCH_MSM7X00A > bool "MSM7x00A / MSM7x01A" > select ARCH_MSM_ARM11 > + select MSM_SMD_PKG3 > select CPU_V6 > > config ARCH_QSD8X50 > @@ -325,6 +326,9 @@ config MSM_SERIAL_DEBUGGER_CONSOLE > Enables a console so that printk messages are displayed on > the debugger serial port as the occur. > > +config MSM_SMD_PKG3 > + bool > + ...but I guess that as long as kconfig system gets it right automatically... its probably ok. ....well, will that prevent us from producing 'generic' kernel in future? Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html