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

[Dave Liu]

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

Currently, the ata_piix driver only is used by sc520 board.

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

I have not the sc520_cdp x86 board,
so I can not test it.

but building/compiling it is passed.

> If yes, then please split these off into a separate patch.
> 
> If not, then just dump it.

dump it, sorry, what do you mean?

Thanks,
Dave