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

Re: [PATCH] yaboot 1.3.10 G5 support



On Wed, Nov 05, 2003 at 07:16:35PM +1100, Benjamin Herrenschmidt wrote:
> 
> > > +	    echo "${DEVICE_PATH##*device-tree}/k2-sata@$DEVICE_ID/disk@0:$PARTITION"
> > 
> > this doesn't look quite right to me why do you have that disk@0 ?  I
> > don't think this patch will work for anything but the first disk.
> > 
> > please confirm this work in all configurations with all disks before i
> > will apply.
> 
> My underdtanding of things is that SATA 1 doesn't allow for "slaves"
> so there can be only one disk per port, though there are 2 ports
> (k2-sata0 and k2-sata1).

ok, so the above patch looks sane to you then?

> SATA 2 supports some "extender" mecanism where you can hook more
> disks, but I don't think we have to care about this yet and it's
> not a SATA 2 chipset anyway :)

/me continues to be amazed how all these so called `new' technologies
continue to be kludgy crippled hacks compared to SCSI.

-- 
Ethan Benson
http://www.alaska.net/~erbenson/

Attachment: pgp00003.pgp
Description: PGP signature