[U-Boot] [RFC PATCH 2/2] block: Move ceva driver to DM
Simon Glass
sjg at chromium.org
Sat Sep 24 19:26:33 CEST 2016
Hi Michal,
On 8 September 2016 at 07:57, Michal Simek <michal.simek at xilinx.com> wrote:
> This patch also includes ARM64 zynqmp changes:
> - Remove platform non DM initialization
> - Remove hardcoded sata base address
>
> Signed-off-by: Michal Simek <michal.simek at xilinx.com>
> ---
>
> There are probably more things to test and to check but
> on my platform I can connect only one HDD. But IP itself
> have two ports which are not handled properly.
> I have tried to reuse as much infrastructure as is available.
> There need to be cleanup for SATA/SCSI/AHCI names.
>
> There is also sata cmd and it is a question if make sense to keep it in
> the tree because it is subset of scsi commands.
>
> scsi scan needs to be called first and maybe make sense to call it
> automatically as was done before.
>
> Simon: Please check if I did it at least partially right.
>
> TODO:
> CONFIG_DM_SATA should be moved to Kconfig
>
> LOG:
>
> ZynqMP> scsi scan
> SATA link 0 timeout.
> Target spinup took 0 ms.
> AHCI 0001.0301 32 slots 2 ports 6 Gbps 0x3 impl SATA mode
> flags: 64bit ncq pm clo only pmp fbss pio slum part ccc apst
> scanning bus for devices...
> Device 0: (1:0) Vendor: ATA Prod.: KINGSTON SVP200S Rev: 501A
> Type: Hard Disk
> Capacity: 57241.8 MB = 55.9 GB (117231408 x 512)
> Found 1 device(s).
> ZynqMP> ls sata 0
> <DIR> 4096 .
> <DIR> 4096 ..
> <DIR> 4096 bin
> <DIR> 4096 boot
> <DIR> 4096 dev
> <DIR> 12288 etc
> <DIR> 4096 home
> <DIR> 4096 lib
> <DIR> 4096 lost+found
> <DIR> 4096 media
> <DIR> 4096 mnt
> <DIR> 4096 opt
> <DIR> 4096 proc
> <DIR> 4096 root
> <DIR> 4096 run
>
> ---
> arch/arm/include/asm/arch-zynqmp/hardware.h | 2 --
> board/xilinx/zynqmp/zynqmp.c | 11 -------
> drivers/block/sata_ceva.c | 49 +++++++++++++++++++++++++++--
> include/configs/xilinx_zynqmp.h | 7 +++--
> 4 files changed, 52 insertions(+), 17 deletions(-)
Looks good to me - this is how a driver should be organised.
Regards,
Simon
More information about the U-Boot
mailing list