[U-Boot] [PATCH] Canyonlands SATA harddisk driver

Wolfgang Denk
2009-03-27T16:31:44Z

Dear Kazuaki Ichinohe,

In message <49CB5F77.1060304@fsi.co.jp> you wrote:
...
> >Why do you add this 460EX SATA support to cmd_scsi.c? Wouldn't cmd_sata.c be a
> >better place? Or did I miss something here?
>
> The cmd_scsi.c which define CONFIG_CMD_SCSI are modified
> because I want to use scsiboot command of U-BOOT.

This makes no sense. It is a S-ATA driver, not a SCSI driver, and as
such it has to be supported in the context of the S-ATA support.

I explicitly NAK the changes to the SCSI related files.

Best regards,

Wolfgang Denk

--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
A quarrel is quickly settled when deserted by one party; there is no
battle unless there be two. - Seneca