From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751614AbdBXTx5 (ORCPT ); Fri, 24 Feb 2017 14:53:57 -0500 Received: from mail-pg0-f66.google.com ([74.125.83.66]:34332 "EHLO mail-pg0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751471AbdBXTxt (ORCPT ); Fri, 24 Feb 2017 14:53:49 -0500 Date: Sat, 25 Feb 2017 04:44:34 +0900 From: Stafford Horne To: Jonas Bonn Cc: Stefan Kristiansson , linux@roeck-us.net, openrisc@lists.librecores.org, linux-kernel@vger.kernel.org, Valentin Rothberg Subject: Re: [PATCH v4 22/24] arch/openrisc/lib/memcpy.c: use correct OR1200 option Message-ID: <20170224194434.GR2449@lianli.shorne-pla.net> References: <7ec3096414795682d3540c4b8020ff52b12b34a6.1487910170.git.shorne@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.7.1 (2016-10-04) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Feb 24, 2017 at 10:24:31AM +0100, Jonas Bonn wrote: > Hi Stafford, > > Squash this patch into the memcpy patch in order to prevent surprises for > anyone doing a git bisection through here. Hi Jonas, Thanks for reviewing. Good spot. I intentionally didn't squash this one just to give Valentin credit on the fix. Also, since it wouldnt really be broken (build or boot issues) in the initial patch I didnt think it would be a bisect issue. Also, I noticed a lot of Valintin's patches pushed by people are separated out not squashed. -Stafford > /Jonas > > On 02/24/2017 05:32 AM, Stafford Horne wrote: > > From: Valentin Rothberg > > > > The Kconfig option for OR12000 is OR1K_1200. > > > > Signed-off-by: Valentin Rothberg > > Signed-off-by: Stafford Horne > > --- > > arch/openrisc/lib/memcpy.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/arch/openrisc/lib/memcpy.c b/arch/openrisc/lib/memcpy.c > > index 4706f01..669887a 100644 > > --- a/arch/openrisc/lib/memcpy.c > > +++ b/arch/openrisc/lib/memcpy.c > > @@ -17,7 +17,7 @@ > > #include > > -#ifdef CONFIG_OR1200 > > +#ifdef CONFIG_OR1K_1200 > > /* > > * Do memcpy with word copies and loop unrolling. This gives the > > * best performance on the OR1200 and MOR1KX archirectures > >