From mboxrd@z Thu Jan 1 00:00:00 1970 From: Geert Uytterhoeven Subject: Re: [PATCH] ide: remove M68K_IDE_SWAPW define from Date: Sun, 10 Aug 2008 20:16:10 +0200 (CEST) Message-ID: References: <200808101746.12697.bzolnier@gmail.com> Mime-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Return-path: In-Reply-To: <200808101746.12697.bzolnier@gmail.com> Sender: linux-kernel-owner@vger.kernel.org To: Bartlomiej Zolnierkiewicz Cc: linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org, Roman Zippel List-Id: linux-ide@vger.kernel.org On Sun, 10 Aug 2008, Bartlomiej Zolnierkiewicz wrote: > Since we solved this by overriding default ->{in,out}put_data > methods in {q40,falcon}ide M68K_IDE_SWAP define can go away. > > Cc: Geert Uytterhoeven > Cc: Roman Zippel > Signed-off-by: Bartlomiej Zolnierkiewicz Acked-by: Geert Uytterhoeven > --- > include/asm-m68k/ide.h | 9 --------- > 1 file changed, 9 deletions(-) > > Index: b/include/asm-m68k/ide.h > =================================================================== > --- a/include/asm-m68k/ide.h > +++ b/include/asm-m68k/ide.h > @@ -92,15 +92,6 @@ > #define outsw_swapw(port, addr, n) raw_outsw_swapw((u16 *)port, addr, n) > #endif > > - > -/* Q40 and Atari have byteswapped IDE busses and since many interesting > - * values in the identification string are text, chars and words they > - * happened to be almost correct without swapping.. However *_capacity > - * is needed for drives over 8 GB. RZ */ > -#if defined(CONFIG_Q40) || defined(CONFIG_ATARI) > -#define M68K_IDE_SWAPW (MACH_IS_Q40 || MACH_IS_ATARI) > -#endif > - > #ifdef CONFIG_BLK_DEV_FALCON_IDE > #define IDE_ARCH_LOCK Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds