From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stefan Roese <sr@denx.de>
Date: Wed, 15 Jul 2015 13:16:10 +0200
Subject: [U-Boot] [PATCH] arm: mvebu: add Armada XP SATA support
In-Reply-To: <1436950918-8400-1-git-send-email-anton.schubert@gmx.de>
References: <1436950918-8400-1-git-send-email-anton.schubert@gmx.de>
Message-ID: <55A640FA.1010906@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Anton,

On 15.07.2015 11:01, Anton Schubert wrote:
> This patch initializes the SATA address windows on Armada XP and
> allows it to work with the existing mvsata_ide driver.
> It also adds the necessary configuration for the db-mv784mp-gp board.

But it doesn't enable the IDE interface and commands. I would
prefer to enable CONFIG_CMD_IDE by default for this board. Please
add this to v2 of this patch.

I tested this on the db-mv784mp-gp board and get this output:

Marvell>> ide info

IDE device 0: Model: TS8GHSD310 Firm: 20120518 Ser#: 20121206A51815115288
            Type: Hard Disk
            Capacity: 7627.9 MB = 7.4 GB (15621984 x 512)
IDE device 1: Model: TS8GHSD310 Firm: 20120518 Ser#: 20121206A51815115288
            Type: Hard Disk
            Capacity: 7627.9 MB = 7.4 GB (15621984 x 512)

So this 8GiB device is listed twice (I'm not even sure where its
connected - I've never used IDE on it and no external disk is
connected).

Could you please comment on this?

Thanks,
Stefan