From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760806AbZELLYo (ORCPT ); Tue, 12 May 2009 07:24:44 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1758245AbZELLYf (ORCPT ); Tue, 12 May 2009 07:24:35 -0400 Received: from fg-out-1718.google.com ([72.14.220.159]:37751 "EHLO fg-out-1718.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753363AbZELLYe (ORCPT ); Tue, 12 May 2009 07:24:34 -0400 Message-ID: <4A095C6B.9030303@monstr.eu> Date: Tue, 12 May 2009 13:24:27 +0200 From: Michal Simek Reply-To: monstr@monstr.eu User-Agent: Thunderbird 2.0.0.18 (X11/20081120) MIME-Version: 1.0 To: Arnd Bergmann CC: linux-kernel@vger.kernel.org, microblaze-uclinux@itee.uq.edu.au Subject: Re: [PATCH 11/12] microblaze: dont include asm/mmu.h in hw_exception_handler References: <20090511232019.927138561@arndb.de> <20090511232345.746701938@arndb.de>> <4A093D6E.7060309@monstr.eu> <200905121313.35501.arnd@arndb.de> In-Reply-To: <200905121313.35501.arnd@arndb.de> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Arnd Bergmann wrote: > On Tuesday 12 May 2009, Michal Simek wrote: >> 1. Please do not attach your patches - use in-line patches - >> it is easier for me and for others to review it. > > Something went wrong (again...) with my patches this time, so they > got rejected by vger. They are not attached however AFAICT, but > are inline with the "Content-Disposition: inline;" header, which > is generated by "quilt mail". Which tool do you use that does > not work with this correctly? I use sendmail and git-send-email and I use Thunderbird for reviewing. > >> - but on the other hand I need mmu.h for mmu patches - because of MICROBLAZE_TLB_SIZE >> - if you think that mmu.h shouldn't be included we have to >> move this definition to another file >> - you can look at my next branch to see mmu context > > Almost all architectures disallow using mmu.h in assembly, only > powerpc and sh have an '#ifndef __ASSEMBLY__' in there. I did the > change to allow using my asm-generic/mmu.h for microblaze-nommu. > > When your MMU code gets merged, this is no longer required though, > because you cannot simply fall back on the asm-generic version anyway. I have no problem to remove mmu.h from it but I have to move that one TLB macro to different sensible include file. It is related with mmu that's why I think someone in past add it to this file. The main point is which file. Any tip? Or the second solution is to move directly to .S file but any different include file will be better. Michal > > Arnd <>< -- Michal Simek, Ing. (M.Eng) w: www.monstr.eu p: +42-0-721842854