Hello. Atsushi Nemoto wrote:
+ struct tx4938ide_platform_info pdata = { + .ioport_shift = shift, + .gbus_clock = tune ? txx9_gbus_clock : 0,
Any reason not to supply the GBUS clock?
The EBUS channel might be used for both ATA and ISA or other local bus devices. In that case, the board setup code should initialize best timings for all devices and the IDE driver should not overrite it.
Ah, I forgot about the ATA address space being appeandage to ISA address space on TX4939...
I'm afraid you can't just early return from the set_pio_mode() method...
Do you mean I should use IDE_HFLAG_NO_SET_MODE instead of just returning from set_pio_mode?
No, that's for the smart RAID controllers that program the transfer modes themselves. In your case, hwif->port_ops->set_pio_mode() must be NULL -- if you're not going to allow the mode programming, that is.
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