On Sun, 19 Oct 2008 22:34:06 +0400, Sergei Shtylyov <sshtylyov@xxxxxxxxxxxxx> 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.
> 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?
> > + for (i = 0; i < 8; i++) {
> > + /* check EBCCRn.ISA, EBCCRn.BSZ, EBCCRn.ME */
> > + if ((__raw_readq(&tx4938_ebuscptr->cr[i]) & 0x00f00008)
> > + == 0x00e00008)
> > + break;
> > + }
> > + if (i == 8)
> > + return;
> > + pdata.ebus_ch = i;
>
> Why not grab the base address from this register as well and put it into
> the resource?
Well, ... no reason :-) I will do that way. Thank you.
---
Atsushi Nemoto
--
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
