Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932134Ab3FAA0p (ORCPT ); Fri, 31 May 2013 20:26:45 -0400 Received: from mail-pd0-f177.google.com ([209.85.192.177]:59749 "EHLO mail-pd0-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755009Ab3FAA0h (ORCPT ); Fri, 31 May 2013 20:26:37 -0400 Message-ID: <51A93FA0.1050203@biophys.uni-duesseldorf.de> Date: Sat, 01 Jun 2013 12:26:08 +1200 From: schmitz User-Agent: Mozilla/5.0 (X11; U; Linux ppc; en-US; rv:1.8.1.4) Gecko/20070509 Iceape/1.1.2 (Debian-1.1.2-1) MIME-Version: 1.0 To: Geert Uytterhoeven CC: Chen Gang , Greg Ungerer , schmitz@debian.org, Sam Ravnborg , Greg KH , linux-m68k , "linux-kernel@vger.kernel.org" , Linux-Arch Subject: Re: [PATCH] arch: m68k: include: asm: the 3rd parameter of 'insl' and 'outsl' need '<< 2' References: <51A71E49.1000707@asianux.com> In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3640 Lines: 89 Geert , >> >> The related git number: >> for parport.h: "4914802 m68k,m68knommu: merge header files" in 2009 >> for io_mm.h: "84b16b7 m68k/atari: ROM port ISA adapter support" in Apr 6 2013 >> >> The related warning (make EXTRA_CFLAG=-W ARCH=m68k allmodconfig): >> arch/m68k/include/asm/parport.h:14:0: warning: "insl" redefined [enabled by default] >> arch/m68k/include/asm/io_mm.h:403:0: note: this is the location of the previous definition >> arch/m68k/include/asm/parport.h:15:0: warning: "outsl" redefined [enabled by default] >> arch/m68k/include/asm/io_mm.h:406:0: note: this is the location of the previous definition >> Is that the same problem Thorsten reported recently? parport.h should either use what the arch io.h include defined, or (in the case of Q40 on m68k) undef and redefine as needed. >> >> Signed-off-by: Chen Gang >> --- >> arch/m68k/include/asm/io_mm.h | 5 +++-- >> arch/m68k/include/asm/parport.h | 9 +++++++-- >> 2 files changed, 10 insertions(+), 4 deletions(-) >> >> diff --git a/arch/m68k/include/asm/io_mm.h b/arch/m68k/include/asm/io_mm.h >> index ffdf54f4..66be3b2 100644 >> --- a/arch/m68k/include/asm/io_mm.h >> +++ b/arch/m68k/include/asm/io_mm.h >> @@ -400,10 +400,11 @@ static inline void isa_delay(void) >> >> #define insb(port, buf, nr) ((port) < 1024 ? isa_rom_insb((port), (buf), (nr)) : isa_insb((port), (buf), (nr))) >> #define insw(port, buf, nr) ((port) < 1024 ? isa_rom_insw((port), (buf), (nr)) : isa_insw((port), (buf), (nr))) >> -#define insl isa_insl >> +#define insl(port, buf, len) isa_insb((port), (buf), (len) << 2) >> > > Oops, changes from 32-bit accesses to byte accesses? > That's in the Atari specific branch - please explain why you think this needs to be done. Has this patch been tested by running on ARAnyM, at least? Unless this has been properly tested on Atari (hardware), please leave as-is. > >> #define outsb(port, buf, nr) ((port) < 1024 ? isa_rom_outsb((port), (buf), (nr)) : isa_outsb((port), (buf), (nr))) >> #define outsw(port, buf, nr) ((port) < 1024 ? isa_rom_outsw((port), (buf), (nr)) : isa_outsw((port), (buf), (nr))) >> -#define outsl isa_outsl >> +#define outsl(port, buf, len) isa_outsb((port), (buf), (len) << 2) >> >> #define readb(addr) in_8(addr) >> #define writeb(val, addr) out_8((addr), (val)) >> diff --git a/arch/m68k/include/asm/parport.h b/arch/m68k/include/asm/parport.h >> index 5ea75e6..e8e4a2a 100644 >> --- a/arch/m68k/include/asm/parport.h >> +++ b/arch/m68k/include/asm/parport.h >> @@ -11,8 +11,13 @@ >> #ifndef _ASM_M68K_PARPORT_H >> #define _ASM_M68K_PARPORT_H 1 >> >> -#define insl(port,buf,len) isa_insb(port,buf,(len)<<2) >> -#define outsl(port,buf,len) isa_outsb(port,buf,(len)<<2) >> +#ifndef insl >> +#define insl(port, buf, len) isa_insb((port), (buf), (len) << 2) >> +#endif >> + >> +#ifndef outsl >> +#define outsl(port, buf, len) isa_outsb((port), (buf), (len) << 2) >> +#endif >> I think that should read #undef insl #define insl(port,buf,len) isa_insb(port,buf,(len)<<2) instead. I distinctly remember this brought up a few weeks ago. > Now the (re)definitions are identical to the originals, so they can just > be removed. But the ones in are not correct anymore, IMHO. > Seconded. Cheers, Michael -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/