public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Albert ARIBAUD <albert.aribaud@free.fr>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH V5 4/4] edminiv2: add mvsata_ide and cmd_ide support
Date: Sun, 01 Aug 2010 19:34:59 +0200	[thread overview]
Message-ID: <4C55B043.3020604@free.fr> (raw)
In-Reply-To: <F766E4F80769BD478052FB6533FA745D19A6576506@SC-VEXCH4.marvell.com>

Hi Prafulla et al.,

Le 01/08/2010 18:48, Prafulla Wadaskar a ?crit :
>
>
>> -----Original Message-----
>> From: Albert ARIBAUD [mailto:albert.aribaud at free.fr]
>> Sent: Friday, July 23, 2010 4:21 PM
>> To: Prafulla Wadaskar
>> Cc: u-boot at lists.denx.de
>> Subject: Re: [PATCH V5 4/4] edminiv2: add mvsata_ide and
>> cmd_ide support
> ...snip...
>>
>>> Secondly I tried to generate similar patches for Kirkwood
>> oneprd_base (attached) on the top of this patch series
>>> But somehow "ide reset" is unable to detect device on port.
>>> If you have openrd_base board, can you pls give a try?
>>   >
>>   >  May be you may need to look at mpp config.
>>
>> I have an OpenRD Client, not base; client has a SATA
>> connector to which
>> I have a disk connected, so I could give it a try but we'll
>> need to make
>> sur how many SATA ports kirkwood has and which ones are used
>> on either
>> openrd client and base (orion5x has two sata ports and edmini
>> uses port
>> 1, not 0, for its sole SATA conector). I am away from home right now,
>> but I will be able to give it a try as soon as August 1st.
>
> Hi Albert
> Ping ...if you are back

I am, and I have done some work on your problem, but for the moment I am 
still at the same stage as you -- a continuous sequence of dots when 
doing an ide reset, right?

I have #define'd DEBUG in cmd_ide.c and compared the debug messages for 
my ED Mini V2 (working) and my OpenRD-Client (not working). I have also 
performed manual SATA and IDE reset sequences. Short story:

1) the board code correctly initializes the SATA link, which correctly 
detects a disk if there is one:

  Marvell>> md.l f1082300 4
f1082300: 00000113 14010000 00000300 010300b0    ................
Marvell>> md.l f1084300 4
f1084300: 00000000 00000000 00000300 010300b0    ................
Marvell>>

(this is consistent with my system which has a 1.5 GBps disk on port 0).

2) The problem is that the drive appears continuously non-RDY (SStatus 
register remains at 0x80 while on the ED Mini V2 it ends up at 0x50, 
which basically means it is ready).

I will now compare the MPP / GPIO settings performed by the 
FLASH-resident U-Boot (its 'ide reset' command works ok) to those in 
effect when the patched u-boot fails. Maybe one GPIO controls power to 
the internal SATA port, although I doubt it -- if the drive was off, the 
controller would not see it at all, and SStatus at F1082300 would not 
report it present.

> Pls include my email address in your white list, the mails to you bounces back to me.

It's not an issue of (not) whitelisting your e-mail address; it's your 
mail server being blacklisted by my email provider. If you have a recent 
non-delivery report please send it to me in private to my gmail account 
(albert.aribaud at gmail.com), I'll try to sort things out and contact your 
mail server's admin with the relevant details.

> Regards..
> Prafulla . .

Amicalement,
-- 
Albert.

  reply	other threads:[~2010-08-01 17:34 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-07-13 12:02 [U-Boot] [PATCH V5 1/4] ide: add configuration CONFIG_IDE_SWAP_IO Albert Aribaud
2010-07-13 12:02 ` [U-Boot] [PATCH V5 2/4] ide: add mvsata_ide driver Albert Aribaud
2010-07-13 12:02   ` [U-Boot] [PATCH V5 3/4] cmd_ide: add support for orion5x Albert Aribaud
2010-07-13 12:02     ` [U-Boot] [PATCH V5 4/4] edminiv2: add mvsata_ide and cmd_ide support Albert Aribaud
2010-07-19  7:31       ` Prafulla Wadaskar
2010-07-21 10:26       ` Prafulla Wadaskar
2010-07-23 10:50         ` Albert ARIBAUD
2010-08-01 16:48           ` Prafulla Wadaskar
2010-08-01 17:34             ` Albert ARIBAUD [this message]
2010-08-02  3:35               ` Prafulla Wadaskar
2010-08-02  6:13                 ` Albert ARIBAUD
2010-08-03 12:01                   ` Albert ARIBAUD
2010-08-03 12:10                     ` Prafulla Wadaskar
2010-08-04  6:08                     ` Prafulla Wadaskar
2010-08-04  7:23                       ` Albert ARIBAUD
2010-08-04 20:03                         ` Albert ARIBAUD
2010-08-05  5:23                           ` Prafulla Wadaskar
2010-08-04 20:35                       ` Albert ARIBAUD
2010-08-05  5:27                         ` Prafulla Wadaskar
2010-08-05 12:35                           ` Albert ARIBAUD
2010-08-07 21:15                         ` Wolfgang Denk
2010-07-21 10:17     ` [U-Boot] [PATCH V5 3/4] cmd_ide: add support for orion5x Prafulla Wadaskar
2010-07-21 10:16   ` [U-Boot] [PATCH V5 2/4] ide: add mvsata_ide driver Prafulla Wadaskar
2010-07-21 17:04     ` Albert ARIBAUD
2010-07-14 15:03 ` [U-Boot] [PATCH V5 1/4] ide: add configuration CONFIG_IDE_SWAP_IO Albert ARIBAUD
2010-07-20 13:43   ` Wolfgang Denk

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=4C55B043.3020604@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