Re: [PATCH 01/15] ide: include <asm/ide.h> only when needed

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello.

Atsushi Nemoto 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.

Yeah, SoCs *typically* can handle different endianness for the integrated devices in a transparent way. TX4939 didn't do that consistently still, and that's where the MIPS address swizzling macros could have helped but Atsushi chose to reserve their usage only to the external bus accesses. I however don't think that TX4938 case should have been handled the same way as TX4939 since in this case the controller is *not* SoC integrated device and the IDE registers are situated on the chip's external bus with their mapping is actually board specific, if I don't mistake (I don't have the datasheet at hand).

So ... IDE_BE_IO looks actually not best name for this case.  It is
IDE_RAW_IO or something.

  Yes, I was going to suggest exactly that.

But IDE_RAW_IO might not fit for other cases.  IDE_SWAPPED_IO?

I'd prefer IDE_RAW_IO because it can be swapped when using the standard accessors as well. Frankly speaking, I'm not sure why ide-h8300 needs its own accessors while this arch's io.h has an abundance of swapping and not swapping accessors already defined...

Any other good name?

  I'm still not convinced that it's really worth the trouble...

---
Atsushi Nemoto

MBR, Sergei


--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Filesystems]     [Linux SCSI]     [Linux RAID]     [Git]     [Kernel Newbies]     [Linux Newbie]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Samba]     [Device Mapper]

  Powered by Linux