Re: [PATCH] yaboot 1.3.10 G5 support

[Benjamin Herrenschmidt <benh@kernel.crashing.org>]

On Wed, 2003-11-05 at 20:06, Ethan Benson wrote:
> 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?

No, it looks like chineese to me :) I was only answering about the
specifics of the disk@0 node in there

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

SATA2 ? It's in many ways better than SCSI. On the HW level, though,
it's definitely one port = one disk, you want more disk ? put many
ports. Makes a lot of sense in fact. It's only a few cables, allow
better length, etc...

Ben.