Re: [PATCH] TXx9: Add TX4938 ATA support

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

 



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


[Index of Archives]     [Linux MIPS Home]     [LKML Archive]     [Linux ARM Kernel]     [Linux ARM]     [Linux]     [Git]     [Yosemite News]     [Linux SCSI]     [Linux Hams]

  Powered by Linux