Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758650AbZFMGzz (ORCPT ); Sat, 13 Jun 2009 02:55:55 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752124AbZFMGzr (ORCPT ); Sat, 13 Jun 2009 02:55:47 -0400 Received: from 219-87-157-169.static.tfn.net.tw ([219.87.157.169]:33273 "EHLO mswedge2.sunplus.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752018AbZFMGzq (ORCPT ); Sat, 13 Jun 2009 02:55:46 -0400 In-Reply-To: <200906092019.06812.arnd@arndb.de> To: Arnd Bergmann Cc: Andrew Morton , linux-arch@vger.kernel.org, linux-kernel@vger.kernel.org, torvalds@linux-foundation.org MIME-Version: 1.0 Subject: Re: [PATCH 00/27] score arch files for linux X-Mailer: Lotus Notes Release 6.5 September 26, 2003 Message-ID: From: liqin.chen@sunplusct.com Date: Sat, 13 Jun 2009 14:50:47 +0800 X-MIMETrack: Serialize by Router on ctmail01/SunplusCT(Release 7.0.3FP1|February 24, 2008) at 2009/06/13 ?? 02:50:48, Serialize complete at 2009/06/13 ?? 02:50:48 Content-Type: text/plain; charset="GB2312" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by alpha.home.local id n5D6uU54011807 Content-Length: 1800 Lines: 47 Hi Arnd, Arnd Bergmann д?? 2009-06-10 02:19:06: > On Tuesday 09 June 2009, liqin.chen@sunplusct.com wrote: > > Hi Arnd, Andrew Morton and linus, > > > > According to your comment on score arch code, > > we update score code base on asm-generic#next repository. > > use arch microblaze and blackfin as reference. > > Ok, just finished looking over the patches and I am generally > very happy about their quality, it usually takes much longer > for a submission of this size to get into a mergeable shape. > > The only thing that I think needs to be fixed before a > 2.6.31 release is my comment about unistd.h still containing > the __ARCH_WANT_SYSCALL_* #defines. I consider that my fault > because I did not make it clear enough what those are meant > for in asm-generic/unistd.h. > > One more general thought: I noticed that you split the header > files by alphabetical ordering. While there is no good way to > split a set of interdependent files, a logical grouping > (e.g. ABI, memory management, device, cpu) would be nice. > > Everything else I found are only polite suggestions of what > can still be improved, not issues that I think need to > be addressed immediately. I especially love how the work > that Remis and I put into the generic header files starts > paying off, because it makes reviewing an architecture so > much simpler. > > Please add my 'Reviewed-by: Arnd Bergmann ' > to your patches. > Based on your comments, I use asm-generic/current.h, hardirq.h, unaligned.h etc instead of files under arch/include/asm folder. It could work in score code. Best Regards Liqin -- ????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m???? ????????I?