[U-Boot-Users] [PATCH v3 0/10] SATA framework and Freescale SATA controller support patch set

[Wolfgang Denk]

In message <1206524999.3662.4.camel at localhost.localdomain> you wrote:
> 
> Dave Liu (10):
>       ata: merge the ata_piix driver
>       ata: add the support for SATA framework
>       ata: make the ata_piix driver using new SATA framework
>       ata: add the libata support
>       ata: add the fis struct for SATA
>       ata: enable the sata init on bootup
>       ata: add the readme for SATA command line
>       mpc83xx: initialize serdes for MPC837xEMDS boards
>       drivers: Add the support for Freescale SATA controller
>       mpc83xx: enable the SATA interface on mpc837xemds board

You don't mention anythiong about the sc520 board here, yet...

>  board/freescale/mpc837xemds/mpc837xemds.c |   33 +
>  common/Makefile                           |    2 +-
>  common/cmd_sata.c                         |  768 +++
> +--------------------
>  disk/part.c                               |   14 +
>  disk/part_dos.c                           |    2 +
>  disk/part_iso.c                           |    2 +
>  disk/part_mac.c                           |    1 +
>  doc/README.sata                           |   69 +++
>  drivers/block/Makefile                    |    2 +
>  drivers/block/ata_piix.c                  |  585 ++++++++++++++++++-
>  drivers/block/fsl_sata.c                  |  921 ++++++++++++++++++++++
> +++++++
>  drivers/block/fsl_sata.h                  |  374 ++++++++++++
>  drivers/block/libata.c                    |  158 +++++
>  include/configs/MPC837XEMDS.h             |   28 +
>  include/configs/sc520_cdp.h               |   27 +-

...you're trying to sneak in configuration changes here.

Have these been tested?

If yes, then please split these off into a separate patch.

If not, then just dump it.

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
The first thing we do is kill all the lawyers.
(Shakespeare. II Henry VI, Act IV, scene ii)