From: Albert ARIBAUD <albert.aribaud@free.fr>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] orion5x: edminiv2: add libata support
Date: Thu, 01 Jul 2010 10:52:48 +0200 [thread overview]
Message-ID: <4C2C5760.5020708@free.fr> (raw)
In-Reply-To: <1277969793.2221.126.camel@tor-desktop>
Le 01/07/2010 09:36, Tor Krill a ?crit :
> On Wed, 2010-06-30 at 23:30 +0200, Albert Aribaud wrote:
>> Signed-off-by: Albert Aribaud<albert.aribaud@free.fr>
>> ---
>> This patch:
>> - adds support in libata for the orion5x MVSATAHC controller;
>> - enables orion5x MVSTAHC port 1 on the edmini board;
>> - adds IDE and EXT2 commands to the edminiv2 command set.
>
> Just a heads up. I posted a patch with a sata-driver for kirkwood a
> while back. (We use this with our board and it seems to work fine)
>
> http://lists.denx.de/pipermail/u-boot/2010-June/073147.html
>
> Which might have some similarities to the sata controller on orion.
>
> /Tor
Thanks Tor -- Now we're both informed of each other's dev. :)
First off : the MVSATAHC controller is indeed quite similar between both
SoCs, at least from a u-boot perspective, where disk usage is limited to
loading a kernel and initrd. IIRC its code is already shared on the
Linux side between orion5x and kirkwood.
I see your sata_mv driver originates in the Linux kernel driver and
makes use of DMA. This might be a real gain when loading big files often
; in my case, I considered that u-boot will only kick in once in a while
and load a less-than-2MB kernel and possibly a less-than-3MB initrd, and
while relying on the IDE PIO code will certainly make it slower than
with sata_mv, the occasional couple of 10th of ms is a fair price for
the very small footprint -- ED Mini V2 is short on NOR Flash space.
Amicalement,
--
Albert.
prev parent reply other threads:[~2010-07-01 8:52 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-06-30 21:30 [U-Boot] [PATCH] orion5x: edminiv2: add libata support Albert Aribaud
2010-06-30 21:39 ` Wolfgang Denk
2010-06-30 22:35 ` Albert ARIBAUD
2010-06-30 22:48 ` Wolfgang Denk
2010-07-01 0:35 ` Albert ARIBAUD
2010-07-01 0:37 ` [U-Boot] [PATCH 1/4] ide: add configuration CONFIG_IDE_SWAP_IO Albert Aribaud
2010-07-01 0:38 ` [U-Boot] [PATCH 2/4] orion5x: edminiv2: add CMD_IDE support Albert Aribaud
2010-07-01 0:38 ` [U-Boot] [PATCH 3/4] orion5x: fix typo in comment Albert Aribaud
2010-07-01 0:38 ` [U-Boot] [PATCH 4/4] edminiv2: inttroduce CONFIG_SKIP_LOWLEVEL_INIT Albert Aribaud
2010-06-30 21:42 ` [U-Boot] [PATCH] orion5x: edminiv2: add libata support Albert ARIBAUD
2010-07-01 7:36 ` Tor Krill
2010-07-01 8:41 ` Prafulla Wadaskar
2010-07-01 8:51 ` Tor Krill
2010-07-01 12:00 ` Albert ARIBAUD
2010-07-01 8:51 ` Albert ARIBAUD
2010-07-01 8:52 ` Albert ARIBAUD [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4C2C5760.5020708@free.fr \
--to=albert.aribaud@free.fr \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox