From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from imap.sh.mvista.com (unknown [63.81.120.155]) by ozlabs.org (Postfix) with ESMTP id 1F9EDDDE0A for ; Thu, 1 Nov 2007 01:12:26 +1100 (EST) Message-ID: <47288D4F.2090504@ru.mvista.com> Date: Wed, 31 Oct 2007 17:12:31 +0300 From: Sergei Shtylyov MIME-Version: 1.0 To: Paul Mackerras Subject: Re: [POWERPC] Fix off-by-one error in setting decrementer on Book E References: <18213.19469.827909.663373@cargo.ozlabs.ibm.com> <47260FE5.5080107@ru.mvista.com> <18216.19826.611308.659235@cargo.ozlabs.ibm.com> In-Reply-To: <18216.19826.611308.659235@cargo.ozlabs.ibm.com> Content-Type: text/plain; charset=us-ascii; format=flowed Cc: linuxppc-dev@ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Hello. Paul Mackerras wrote: > If I take out the removed lines in the rest of your patch, I get: >>+ */ >> #if defined(CONFIG_40x) >> mtspr(SPRN_PIT, val); >>+#else >>+#if !defined(CONFIG_BOOKE) >>+ val = val ? val - 1 : 0; >>+#endif >>+#if defined(CONFIG_8xx_CPU6) >> set_dec_cpu6(val); >>+#if defined(CONFIG_PPC_ISERIES) > I think you're missing a #else here. You've cut off #endif's at the end. But yes, you are correct -- I've lost it while pasting this hunk into mail. :-/ > Paul. WBR, Sergei