From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [PATCH 07/28] nios2: I/O Mapping Date: Fri, 02 May 2014 14:22:30 +0200 Message-ID: <5393488.r5N1ijHzS4@wuerfel> References: <1397824031-4892-1-git-send-email-lftan@altera.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Return-path: Received: from mout.kundenserver.de ([212.227.17.10]:52710 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751678AbaEBMWg (ORCPT ); Fri, 2 May 2014 08:22:36 -0400 In-Reply-To: Sender: linux-arch-owner@vger.kernel.org List-ID: To: Ley Foon Tan Cc: Linux-Arch , "linux-kernel@vger.kernel.org" , "linux-doc@vger.kernel.org" , cltang@codesourcery.com On Friday 02 May 2014 18:37:55 Ley Foon Tan wrote: > On Thu, Apr 24, 2014 at 2:02 PM, Ley Foon Tan wrote: > > On Tue, Apr 22, 2014 at 9:59 PM, Arnd Bergmann wrote: > >> On Friday 18 April 2014, Ley Foon Tan wrote: > >> > >>> + > >>> +#include > >>> + > >>> +#define IO_SPACE_LIMIT 0xffffffff > >> > >> Please use 0xffff here, this should work for almost any PCI bus. > > Ah, CONFIG_PCI is not enable in nios2. So, I think IO_SPACE_LIMIT > > should set to 0. > > > > > >>> + > >>> +/* Use "Duff's Device" to unroll the loops. */ > >>> +#define __IO_OUT_LOOP(a, b, l) \ > >>> + do { \ > >>> + if (l > 0) { \ > >>> + int _n = (l + 7) / 8; \ > >>> + switch (l % 8) { \ > >>> + case 0: \ > >>> + do { \ > >>> + *a = *b++; \ > >> > >> I would recommend just doing all of this in out-of-line implementations > >> rather than macros and inline functions. > > Okay. Will move the macros to out-of-line function. > > I'm start working on this and found that it is better to keep it as > macros. This is because the input arguments "a" and "b" can be > byte/short/int pointers. Otherwise, we need to have separate function > calls for byte/short/int. But with the inline implementation, you have a separate instance for each caller, which seems more wasteful. You can put the function into a "lib" file to make sure it doesn't get linked in if there are no callers though. And of course you can keep the macro in the file implementing it so you don't have to write the same code three times. Arnd