All of lore.kernel.org
 help / color / mirror / Atom feed
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.

      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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.