| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Sun, 08 Feb 2009 00:35:36 +0900 (JST) From: Atsushi Nemoto <anemo@....ocn.ne.jp> To: sshtylyov@...mvista.com Cc: bzolnier@...il.com, geert@...ux-m68k.org, linux-ide@...r.kernel.org, linux-kernel@...r.kernel.org, apw@...onical.com Subject: Re: [PATCH 01/15] ide: include <asm/ide.h> only when needed On Sat, 07 Feb 2009 01:03:59 +0300, Sergei Shtylyov <sshtylyov@...mvista.com> wrote: > > config CONFIG_IDE_BE_IO > > bool > > If it was that simple... Normally the BE case gets handled automagically > (moreover, there is MIPS option that additionally controls I/O and memory > space byte swapping). The case we have to address for TX493x is actually where > the usual magic fails (or actually the code just doesn't want to use that > option). So this doesn't look like a good name to me... Well, for TX493x (MIPS), we have CONFIG_SWAP_IO_SPACE for big endian and it works fine for PCI-IDE host controllers. For SoC internal controllers, no swapping is needed for both endian, thus custom tp_ops is needed only for big endian. So ... IDE_BE_IO looks actually not best name for this case. It is IDE_RAW_IO or something. But IDE_RAW_IO might not fit for other cases. IDE_SWAPPED_IO? Any other good name? --- Atsushi Nemoto -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists