All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] fix IDE_BUS(dev) macro
@ 2012-04-17 19:06 Luka Perkov
  2012-04-18 21:37 ` David Purdy
                   ` (4 more replies)
  0 siblings, 5 replies; 24+ messages in thread
From: Luka Perkov @ 2012-04-17 19:06 UTC (permalink / raw)
  To: u-boot

IDE_BUS assumes that each bus has two devices and thus returns the first
bus even when the second one should be probed.

Signed-off-by: Simon Baatz <gmbnomis@gmail.com>
Tested-by: Luka Perkov <uboot@lukaperkov.net>
---

Simon discovered this while adding support for new board IB NAS6210.

More info can be found here:

http://lists.denx.de/pipermail/u-boot/2012-April/122525.html

When this is commited I will do a coding style cleanup. There are tabs
after few "#define" parts in include/ide.h.

 include/ide.h |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/ide.h b/include/ide.h
index 8ecc9dd..385e909 100644
--- a/include/ide.h
+++ b/include/ide.h
@@ -24,7 +24,7 @@
 #ifndef	_IDE_H
 #define _IDE_H
 
-#define	IDE_BUS(dev)	(dev >> 1)
+#define IDE_BUS(dev)	(dev / (CONFIG_SYS_IDE_MAXDEVICE / CONFIG_SYS_IDE_MAXBUS))
 
 #define	ATA_CURR_BASE(dev)	(CONFIG_SYS_ATA_BASE_ADDR+ide_bus_offset[IDE_BUS(dev)])
 

^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH] fix IDE_BUS(dev) macro
  2012-04-17 19:06 [U-Boot] [PATCH] fix IDE_BUS(dev) macro Luka Perkov
@ 2012-04-18 21:37 ` David Purdy
  2012-04-19  6:38   ` Albert ARIBAUD
  2012-04-20  5:17 ` Prafulla Wadaskar
                   ` (3 subsequent siblings)
  4 siblings, 1 reply; 24+ messages in thread
From: David Purdy @ 2012-04-18 21:37 UTC (permalink / raw)
  To: u-boot

On Tue, Apr 17, 2012 at 2:06 PM, Luka Perkov <uboot@lukaperkov.net> wrote:
> IDE_BUS assumes that each bus has two devices and thus returns the first
> bus even when the second one should be probed.
>
> Signed-off-by: Simon Baatz <gmbnomis@gmail.com>
> Tested-by: Luka Perkov <uboot@lukaperkov.net>
> ---
>
> Simon discovered this while adding support for new board IB NAS6210.
>
> More info can be found here:
>
> http://lists.denx.de/pipermail/u-boot/2012-April/122525.html
>
> When this is commited I will do a coding style cleanup. There are tabs
> after few "#define" parts in include/ide.h.
>
> ?include/ide.h | ? ?2 +-
> ?1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/ide.h b/include/ide.h
> index 8ecc9dd..385e909 100644
> --- a/include/ide.h
> +++ b/include/ide.h
> @@ -24,7 +24,7 @@
> ?#ifndef ? ? ? ?_IDE_H

Simon, Luka, Prafulla, etal,

I've tested this out on the Seagate GoFlex Net (Kirkwood) device,
which also has dual SATA ports, and it resolved the long-standing (and
irritating) bug/unwanted-feature that required us to boot from _only_
the right side port.

I would guess that other Kirkwood dual-SATA-port boxes would be
happier because of this, as well.

Nice catch, Simon.

regards,

Dave

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH] fix IDE_BUS(dev) macro
  2012-04-18 21:37 ` David Purdy
@ 2012-04-19  6:38   ` Albert ARIBAUD
  2012-04-21 19:38     ` David Purdy
                       ` (2 more replies)
  0 siblings, 3 replies; 24+ messages in thread
From: Albert ARIBAUD @ 2012-04-19  6:38 UTC (permalink / raw)
  To: u-boot

Hi Dave,

Le 18/04/2012 23:37, David Purdy a ?crit :
> On Tue, Apr 17, 2012 at 2:06 PM, Luka Perkov<uboot@lukaperkov.net>  wrote:
>> IDE_BUS assumes that each bus has two devices and thus returns the first
>> bus even when the second one should be probed.
>>
>> Signed-off-by: Simon Baatz<gmbnomis@gmail.com>
>> Tested-by: Luka Perkov<uboot@lukaperkov.net>
>> ---
>>
>> Simon discovered this while adding support for new board IB NAS6210.
>>
>> More info can be found here:
>>
>> http://lists.denx.de/pipermail/u-boot/2012-April/122525.html
>>
>> When this is commited I will do a coding style cleanup. There are tabs
>> after few "#define" parts in include/ide.h.
>>
>>   include/ide.h |    2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/include/ide.h b/include/ide.h
>> index 8ecc9dd..385e909 100644
>> --- a/include/ide.h
>> +++ b/include/ide.h
>> @@ -24,7 +24,7 @@
>>   #ifndef        _IDE_H
>
> Simon, Luka, Prafulla, etal,
>
> I've tested this out on the Seagate GoFlex Net (Kirkwood) device,
> which also has dual SATA ports, and it resolved the long-standing (and
> irritating) bug/unwanted-feature that required us to boot from _only_
> the right side port.
>
> I would guess that other Kirkwood dual-SATA-port boxes would be
> happier because of this, as well.
>
> Nice catch, Simon.

Not my main area of expertise here, but I am not sure how this plays on 
Marvell non-kirkwood platforms (e.g., orion5x).

ISTR it is not the first time we deal with the whole IDE number of bus / 
number of ports [ / useable ports ] issue, and we may be running in 
circles here, fixing one platform and breaking another.

I'll try this on EDMiniV2 in the coming days, and let people know the 
results in this thread.

> regards,
>
> Dave

Amicalement,
-- 
Albert.

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH] fix IDE_BUS(dev) macro
  2012-04-17 19:06 [U-Boot] [PATCH] fix IDE_BUS(dev) macro Luka Perkov
  2012-04-18 21:37 ` David Purdy
@ 2012-04-20  5:17 ` Prafulla Wadaskar
  2012-06-03 15:03 ` Vágó Tibor
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 24+ messages in thread
From: Prafulla Wadaskar @ 2012-04-20  5:17 UTC (permalink / raw)
  To: u-boot



> -----Original Message-----
> From: u-boot-bounces at lists.denx.de [mailto:u-boot-
> bounces at lists.denx.de] On Behalf Of Luka Perkov
> Sent: 18 April 2012 00:37
> To: u-boot at lists.denx.de
> Subject: [U-Boot] [PATCH] fix IDE_BUS(dev) macro
> 
> IDE_BUS assumes that each bus has two devices and thus returns the
> first
> bus even when the second one should be probed.
> 
> Signed-off-by: Simon Baatz <gmbnomis@gmail.com>
> Tested-by: Luka Perkov <uboot@lukaperkov.net>
> ---
> 
> Simon discovered this while adding support for new board IB NAS6210.
> 
> More info can be found here:
> 
> http://lists.denx.de/pipermail/u-boot/2012-April/122525.html
> 
> When this is commited I will do a coding style cleanup. There are tabs
> after few "#define" parts in include/ide.h.
> 
>  include/ide.h |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/ide.h b/include/ide.h
> index 8ecc9dd..385e909 100644
> --- a/include/ide.h
> +++ b/include/ide.h
> @@ -24,7 +24,7 @@
>  #ifndef	_IDE_H
>  #define _IDE_H
> 
> -#define	IDE_BUS(dev)	(dev >> 1)
> +#define IDE_BUS(dev)	(dev / (CONFIG_SYS_IDE_MAXDEVICE /
> CONFIG_SYS_IDE_MAXBUS))
> 
>  #define	ATA_CURR_BASE(dev)
> 	(CONFIG_SYS_ATA_BASE_ADDR+ide_bus_offset[IDE_BUS(dev)])
>

Dear Wolfgang

This is generic patch to IDE framework, this patch is dependency for (being pulled) Kirkwood based boards.

Will you please kindly pull it? OR I should do the needful?

Regards..
Prafulla . . .

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH] fix IDE_BUS(dev) macro
  2012-04-19  6:38   ` Albert ARIBAUD
@ 2012-04-21 19:38     ` David Purdy
  2012-04-29 20:19     ` Luka Perkov
  2012-06-03 20:23     ` [U-Boot] [PATCH] " Rogan Dawes
  2 siblings, 0 replies; 24+ messages in thread
From: David Purdy @ 2012-04-21 19:38 UTC (permalink / raw)
  To: u-boot

On Thu, Apr 19, 2012 at 1:38 AM, Albert ARIBAUD
<albert.u.boot@aribaud.net> wrote:
> Hi Dave,
>
>
>
>
> Not my main area of expertise here, but I am not sure how this plays on
> Marvell non-kirkwood platforms (e.g., orion5x).
>
> ISTR it is not the first time we deal with the whole IDE number of bus /
> number of ports [ / useable ports ] issue, and we may be running in circles
> here, fixing one platform and breaking another.

I, too, remember remember reading some discussions about changes to
that very line.


> I'll try this on EDMiniV2 in the coming days, and let people know the
> results in this thread.
>
>
>
> Amicalement,
> --
> Albert.

After further testing, I've found a possible (minor?) anomaly, but not
on a supported (official) Kirkwood device.  The device I tested it on
is a Seagate GoFlex Net (dual SATA).

The behavior I see is this:

With  Toshiba 160GB and Seagate 320GB hdd's in ports 0 and 1 (right &
left) respectively, I get perfect results.  I can access both drives
from U-Boot, and boot from both.

If I switch the drives around  (ie.  swap them) so the Toshiba is is
port 1 (left), I can no longer probe it.   It could very well be a
timing problem, or it could be simply a quirk that is simply
attributable to this hdd (which seems to check out completely
healthy).

If I place _just_ the Seagate drive in port 1 (left), it is still
correctly recognized.  This is true whether it is the lone hdd, or if
another hdd is in port 0.

Just thought I'd let you know.  Maybe this is a _specific_ behavior
subset that you all can test on your respective boxes.

My feeling is that the patch is nevertheless still valid  -  for what
I've seen on Kirkwood, it is an improvement over the previous
situation.  Just my 2 cents.

Dave

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH] fix IDE_BUS(dev) macro
  2012-04-19  6:38   ` Albert ARIBAUD
  2012-04-21 19:38     ` David Purdy
@ 2012-04-29 20:19     ` Luka Perkov
  2012-05-14 19:29       ` Luka Perkov
  2012-06-03 20:23     ` [U-Boot] [PATCH] " Rogan Dawes
  2 siblings, 1 reply; 24+ messages in thread
From: Luka Perkov @ 2012-04-29 20:19 UTC (permalink / raw)
  To: u-boot

Hi Albert,

On Thu, Apr 19, 2012 at 08:38:19AM +0200, Albert ARIBAUD wrote:
> Not my main area of expertise here, but I am not sure how this plays
> on Marvell non-kirkwood platforms (e.g., orion5x).
> 
> ISTR it is not the first time we deal with the whole IDE number of
> bus / number of ports [ / useable ports ] issue, and we may be
> running in circles here, fixing one platform and breaking another.
> 
> I'll try this on EDMiniV2 in the coming days, and let people know
> the results in this thread.

I was just wondering did you have the time to test this patch on your
board?

Regards,
Luka

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH] fix IDE_BUS(dev) macro
  2012-04-29 20:19     ` Luka Perkov
@ 2012-05-14 19:29       ` Luka Perkov
  2012-05-15 19:42         ` Albert ARIBAUD
  2012-10-08 18:46         ` [U-Boot] " Tom Rini
  0 siblings, 2 replies; 24+ messages in thread
From: Luka Perkov @ 2012-05-14 19:29 UTC (permalink / raw)
  To: u-boot

Hi Albert,

On Sun, Apr 29, 2012 at 10:19:41PM +0200, Luka Perkov wrote:
> On Thu, Apr 19, 2012 at 08:38:19AM +0200, Albert ARIBAUD wrote:
> > Not my main area of expertise here, but I am not sure how this plays
> > on Marvell non-kirkwood platforms (e.g., orion5x).
> > 
> > ISTR it is not the first time we deal with the whole IDE number of
> > bus / number of ports [ / useable ports ] issue, and we may be
> > running in circles here, fixing one platform and breaking another.
> > 
> > I'll try this on EDMiniV2 in the coming days, and let people know
> > the results in this thread.
> 
> I was just wondering did you have the time to test this patch on your
> board?

Albert, did you maybe have time to test this patch. I have included it
bellow so you dont have to search for it...

Signed-off-by: Simon Baatz <gmbnomis@gmail.com>
Tested-by: Luka Perkov <uboot@lukaperkov.net>
---

Simon discovered this while adding support for new board IB NAS6210.

More info can be found here:

http://lists.denx.de/pipermail/u-boot/2012-April/122525.html

 include/ide.h |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/ide.h b/include/ide.h
index 8ecc9dd..385e909 100644
--- a/include/ide.h
+++ b/include/ide.h
@@ -24,7 +24,7 @@
 #ifndef	_IDE_H
 #define _IDE_H
 
-#define	IDE_BUS(dev)	(dev >> 1)
+#define IDE_BUS(dev)	(dev / (CONFIG_SYS_IDE_MAXDEVICE / CONFIG_SYS_IDE_MAXBUS))
 
 #define	ATA_CURR_BASE(dev)	(CONFIG_SYS_ATA_BASE_ADDR+ide_bus_offset[IDE_BUS(dev)])
 

^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH] fix IDE_BUS(dev) macro
  2012-05-14 19:29       ` Luka Perkov
@ 2012-05-15 19:42         ` Albert ARIBAUD
  2012-05-15 20:44           ` Luka Perkov
  2012-06-07 13:30           ` Luka Perkov
  2012-10-08 18:46         ` [U-Boot] " Tom Rini
  1 sibling, 2 replies; 24+ messages in thread
From: Albert ARIBAUD @ 2012-05-15 19:42 UTC (permalink / raw)
  To: u-boot

Hi Luka,

Le 14/05/2012 21:29, Luka Perkov a ?crit :
> Hi Albert,
>
> On Sun, Apr 29, 2012 at 10:19:41PM +0200, Luka Perkov wrote:
>> On Thu, Apr 19, 2012 at 08:38:19AM +0200, Albert ARIBAUD wrote:
>>> Not my main area of expertise here, but I am not sure how this plays
>>> on Marvell non-kirkwood platforms (e.g., orion5x).
>>>
>>> ISTR it is not the first time we deal with the whole IDE number of
>>> bus / number of ports [ / useable ports ] issue, and we may be
>>> running in circles here, fixing one platform and breaking another.
>>>
>>> I'll try this on EDMiniV2 in the coming days, and let people know
>>> the results in this thread.
>>
>> I was just wondering did you have the time to test this patch on your
>> board?
>
> Albert, did you maybe have time to test this patch. I have included it
> bellow so you dont have to search for it...
>
> Signed-off-by: Simon Baatz<gmbnomis@gmail.com>
> Tested-by: Luka Perkov<uboot@lukaperkov.net>
> ---
>
> Simon discovered this while adding support for new board IB NAS6210.
>
> More info can be found here:
>
> http://lists.denx.de/pipermail/u-boot/2012-April/122525.html
>
>   include/ide.h |    2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/ide.h b/include/ide.h
> index 8ecc9dd..385e909 100644
> --- a/include/ide.h
> +++ b/include/ide.h
> @@ -24,7 +24,7 @@
>   #ifndef	_IDE_H
>   #define _IDE_H
>
> -#define	IDE_BUS(dev)	(dev>>  1)
> +#define IDE_BUS(dev)	(dev / (CONFIG_SYS_IDE_MAXDEVICE / CONFIG_SYS_IDE_MAXBUS))
>
>   #define	ATA_CURR_BASE(dev)	(CONFIG_SYS_ATA_BASE_ADDR+ide_bus_offset[IDE_BUS(dev)])

Sorry, been busier than usual. I'll try and test this tomorrow night; if 
not, that'll be next week I'm afraid.

Amicalement,
-- 
Albert.

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH] fix IDE_BUS(dev) macro
  2012-05-15 19:42         ` Albert ARIBAUD
@ 2012-05-15 20:44           ` Luka Perkov
  2012-06-07 13:30           ` Luka Perkov
  1 sibling, 0 replies; 24+ messages in thread
From: Luka Perkov @ 2012-05-15 20:44 UTC (permalink / raw)
  To: u-boot

Hi Albert,

On Tue, May 15, 2012 at 09:42:59PM +0200, Albert ARIBAUD wrote:
> >diff --git a/include/ide.h b/include/ide.h
> >index 8ecc9dd..385e909 100644
> >--- a/include/ide.h
> >+++ b/include/ide.h
> >@@ -24,7 +24,7 @@
> >  #ifndef	_IDE_H
> >  #define _IDE_H
> >
> >-#define	IDE_BUS(dev)	(dev>>  1)
> >+#define IDE_BUS(dev)	(dev / (CONFIG_SYS_IDE_MAXDEVICE / CONFIG_SYS_IDE_MAXBUS))
> >
> >  #define	ATA_CURR_BASE(dev)	(CONFIG_SYS_ATA_BASE_ADDR+ide_bus_offset[IDE_BUS(dev)])
> 
> Sorry, been busier than usual. I'll try and test this tomorrow
> night; if not, that'll be next week I'm afraid.

It's no hurry... I was pinging you so this does not end up forgotten.

If this does not work for your boards we can always use #ifdef's.

Regards,
Luka

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH] fix IDE_BUS(dev) macro
  2012-04-17 19:06 [U-Boot] [PATCH] fix IDE_BUS(dev) macro Luka Perkov
  2012-04-18 21:37 ` David Purdy
  2012-04-20  5:17 ` Prafulla Wadaskar
@ 2012-06-03 15:03 ` Vágó Tibor
  2012-06-03 15:03 ` Vágó Tibor
  2012-10-07  7:15 ` uboot at lukaperkov.net
  4 siblings, 0 replies; 24+ messages in thread
From: Vágó Tibor @ 2012-06-03 15:03 UTC (permalink / raw)
  To: u-boot

Hi,

I've tested this IDE patch on my NAS6220 and it works.

U-boot detect both HDD correctly and boot from them.

Cheers,
Tibor


-------- Original message --------
IDE_BUS assumes that each bus has two devices and thus returns the first
bus even when the second one should be probed.

Signed-off-by: Simon Baatz <gmbnomis@gmail.com>
Tested-by: Luka Perkov <uboot@lukaperkov.net>
---

Simon discovered this while adding support for new board IB NAS6210.

More info can be found here:

http://lists.denx.de/pipermail/u-boot/2012-April/122525.html

When this is commited I will do a coding style cleanup. There are tabs
after few "#define" parts in include/ide.h.

  include/ide.h |    2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/ide.h b/include/ide.h
index 8ecc9dd..385e909 100644
--- a/include/ide.h
+++ b/include/ide.h
@@ -24,7 +24,7 @@
  #ifndef	_IDE_H
  #define _IDE_H

-#define	IDE_BUS(dev)	(dev >> 1)
+#define IDE_BUS(dev)	(dev / (CONFIG_SYS_IDE_MAXDEVICE / 
CONFIG_SYS_IDE_MAXBUS))

  #define	ATA_CURR_BASE(dev) 
(CONFIG_SYS_ATA_BASE_ADDR+ide_bus_offset[IDE_BUS(dev)])

^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH] fix IDE_BUS(dev) macro
  2012-04-17 19:06 [U-Boot] [PATCH] fix IDE_BUS(dev) macro Luka Perkov
                   ` (2 preceding siblings ...)
  2012-06-03 15:03 ` Vágó Tibor
@ 2012-06-03 15:03 ` Vágó Tibor
  2012-06-03 21:24   ` Luka Perkov
  2012-10-07  7:15 ` uboot at lukaperkov.net
  4 siblings, 1 reply; 24+ messages in thread
From: Vágó Tibor @ 2012-06-03 15:03 UTC (permalink / raw)
  To: u-boot

Hi,

I've tested this IDE patch on my NAS6220 and it works.

U-boot detect both HDD correctly and boot from them.

Cheers,
Tibor


-------- Original message --------
IDE_BUS assumes that each bus has two devices and thus returns the first
bus even when the second one should be probed.

Signed-off-by: Simon Baatz <gmbnomis@gmail.com>
Tested-by: Luka Perkov <uboot@lukaperkov.net>
---

Simon discovered this while adding support for new board IB NAS6210.

More info can be found here:

http://lists.denx.de/pipermail/u-boot/2012-April/122525.html

When this is commited I will do a coding style cleanup. There are tabs
after few "#define" parts in include/ide.h.

  include/ide.h |    2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/ide.h b/include/ide.h
index 8ecc9dd..385e909 100644
--- a/include/ide.h
+++ b/include/ide.h
@@ -24,7 +24,7 @@
  #ifndef	_IDE_H
  #define _IDE_H

-#define	IDE_BUS(dev)	(dev >> 1)
+#define IDE_BUS(dev)	(dev / (CONFIG_SYS_IDE_MAXDEVICE / 
CONFIG_SYS_IDE_MAXBUS))

  #define	ATA_CURR_BASE(dev) 
(CONFIG_SYS_ATA_BASE_ADDR+ide_bus_offset[IDE_BUS(dev)])

^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH] fix IDE_BUS(dev) macro
  2012-04-19  6:38   ` Albert ARIBAUD
  2012-04-21 19:38     ` David Purdy
  2012-04-29 20:19     ` Luka Perkov
@ 2012-06-03 20:23     ` Rogan Dawes
  2012-06-03 21:25       ` Luka Perkov
  2 siblings, 1 reply; 24+ messages in thread
From: Rogan Dawes @ 2012-06-03 20:23 UTC (permalink / raw)
  To: u-boot

On 19/04/2012 08:38, Albert ARIBAUD wrote:
> Hi Dave,
>
> Le 18/04/2012 23:37, David Purdy a ?crit :
>> On Tue, Apr 17, 2012 at 2:06 PM, Luka Perkov<uboot@lukaperkov.net> wrote:
>>> IDE_BUS assumes that each bus has two devices and thus returns the first
>>> bus even when the second one should be probed.
>>>
>>> Signed-off-by: Simon Baatz<gmbnomis@gmail.com>
>>> Tested-by: Luka Perkov<uboot@lukaperkov.net>
>>> ---
>>>
>>> Simon discovered this while adding support for new board IB NAS6210.
>>>
>>> More info can be found here:
>>>
>>> http://lists.denx.de/pipermail/u-boot/2012-April/122525.html
>>>
>>> When this is commited I will do a coding style cleanup. There are tabs
>>> after few "#define" parts in include/ide.h.
>>>
>>> include/ide.h | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/include/ide.h b/include/ide.h
>>> index 8ecc9dd..385e909 100644
>>> --- a/include/ide.h
>>> +++ b/include/ide.h
>>> @@ -24,7 +24,7 @@
>>> #ifndef _IDE_H
>>
>> Simon, Luka, Prafulla, etal,
>>
>> I've tested this out on the Seagate GoFlex Net (Kirkwood) device,
>> which also has dual SATA ports, and it resolved the long-standing (and
>> irritating) bug/unwanted-feature that required us to boot from _only_
>> the right side port.
>>
>> I would guess that other Kirkwood dual-SATA-port boxes would be
>> happier because of this, as well.
>>
>> Nice catch, Simon.
>
> Not my main area of expertise here, but I am not sure how this plays on
> Marvell non-kirkwood platforms (e.g., orion5x).
>
> ISTR it is not the first time we deal with the whole IDE number of bus /
> number of ports [ / useable ports ] issue, and we may be running in
> circles here, fixing one platform and breaking another.
>
> I'll try this on EDMiniV2 in the coming days, and let people know the
> results in this thread.
>
>> regards,
>>
>> Dave
>
> Amicalement,

I had something similar for my DNS323 port (orion5x-based), which other 
people may be remembering.

http://lists.denx.de/pipermail/u-boot/2010-August/075589.html

I seem to recall Wolfgang was not terribly enthused about it for some 
reason, though.

Rogan

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH] fix IDE_BUS(dev) macro
  2012-06-03 15:03 ` Vágó Tibor
@ 2012-06-03 21:24   ` Luka Perkov
  0 siblings, 0 replies; 24+ messages in thread
From: Luka Perkov @ 2012-06-03 21:24 UTC (permalink / raw)
  To: u-boot

Hi Tibor,

On Sun, Jun 03, 2012 at 05:03:13PM +0200, V?g? Tibor wrote:
> Hi,
> 
> I've tested this IDE patch on my NAS6220 and it works.
> 
> U-boot detect both HDD correctly and boot from them.

Thank you for testing.

Regards,
Luka

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH] fix IDE_BUS(dev) macro
  2012-06-03 20:23     ` [U-Boot] [PATCH] " Rogan Dawes
@ 2012-06-03 21:25       ` Luka Perkov
  0 siblings, 0 replies; 24+ messages in thread
From: Luka Perkov @ 2012-06-03 21:25 UTC (permalink / raw)
  To: u-boot

Hi Rogan,

On Sun, Jun 03, 2012 at 10:23:15PM +0200, Rogan Dawes wrote:
> I had something similar for my DNS323 port (orion5x-based), which
> other people may be remembering.
> 
> http://lists.denx.de/pipermail/u-boot/2010-August/075589.html
> 
> I seem to recall Wolfgang was not terribly enthused about it for
> some reason, though.

I'm waiting for Albert to do testing on his boards. I was going to ping
him again about this one next week ;)

Regards,
Luka

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH] fix IDE_BUS(dev) macro
  2012-05-15 19:42         ` Albert ARIBAUD
  2012-05-15 20:44           ` Luka Perkov
@ 2012-06-07 13:30           ` Luka Perkov
  2012-06-20  8:05             ` Albert ARIBAUD
  1 sibling, 1 reply; 24+ messages in thread
From: Luka Perkov @ 2012-06-07 13:30 UTC (permalink / raw)
  To: u-boot

Hi Albert,

On Tue, May 15, 2012 at 09:42:59PM +0200, Albert ARIBAUD wrote:
> >On Sun, Apr 29, 2012 at 10:19:41PM +0200, Luka Perkov wrote:
> >>On Thu, Apr 19, 2012 at 08:38:19AM +0200, Albert ARIBAUD wrote:
> >>>Not my main area of expertise here, but I am not sure how this plays
> >>>on Marvell non-kirkwood platforms (e.g., orion5x).
> >>>
> >>>ISTR it is not the first time we deal with the whole IDE number of
> >>>bus / number of ports [ / useable ports ] issue, and we may be
> >>>running in circles here, fixing one platform and breaking another.
> >>>
> >>>I'll try this on EDMiniV2 in the coming days, and let people know
> >>>the results in this thread.
> >>
> >>I was just wondering did you have the time to test this patch on your
> >>board?
> >
> >Albert, did you maybe have time to test this patch. I have included it
> >bellow so you dont have to search for it...
> >
> >Signed-off-by: Simon Baatz<gmbnomis@gmail.com>
> >Tested-by: Luka Perkov<uboot@lukaperkov.net>
> >---
> >
> >Simon discovered this while adding support for new board IB NAS6210.
> >
> >More info can be found here:
> >
> >http://lists.denx.de/pipermail/u-boot/2012-April/122525.html
> >
> >  include/ide.h |    2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> >diff --git a/include/ide.h b/include/ide.h
> >index 8ecc9dd..385e909 100644
> >--- a/include/ide.h
> >+++ b/include/ide.h
> >@@ -24,7 +24,7 @@
> >  #ifndef	_IDE_H
> >  #define _IDE_H
> >
> >-#define	IDE_BUS(dev)	(dev>>  1)
> >+#define IDE_BUS(dev)	(dev / (CONFIG_SYS_IDE_MAXDEVICE / CONFIG_SYS_IDE_MAXBUS))
> >
> >  #define	ATA_CURR_BASE(dev)	(CONFIG_SYS_ATA_BASE_ADDR+ide_bus_offset[IDE_BUS(dev)])
> 
> Sorry, been busier than usual. I'll try and test this tomorrow
> night; if not, that'll be next week I'm afraid.

It's me beeing boring again, sorry... Can we get your feedback on this one?

Regards,
Luka

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH] fix IDE_BUS(dev) macro
  2012-06-07 13:30           ` Luka Perkov
@ 2012-06-20  8:05             ` Albert ARIBAUD
  2012-08-09 20:36               ` Wolfgang Denk
  0 siblings, 1 reply; 24+ messages in thread
From: Albert ARIBAUD @ 2012-06-20  8:05 UTC (permalink / raw)
  To: u-boot

Sorry Luka (and all), been tied up pretty heavily recently, had to
adjust to find some free time again.

I will test your patch today and post results tonight.

2012/6/7 Luka Perkov <uboot@lukaperkov.net>
>
> Hi Albert,
>
> On Tue, May 15, 2012 at 09:42:59PM +0200, Albert ARIBAUD wrote:
> > >On Sun, Apr 29, 2012 at 10:19:41PM +0200, Luka Perkov wrote:
> > >>On Thu, Apr 19, 2012 at 08:38:19AM +0200, Albert ARIBAUD wrote:
> > >>>Not my main area of expertise here, but I am not sure how this plays
> > >>>on Marvell non-kirkwood platforms (e.g., orion5x).
> > >>>
> > >>>ISTR it is not the first time we deal with the whole IDE number of
> > >>>bus / number of ports [ / useable ports ] issue, and we may be
> > >>>running in circles here, fixing one platform and breaking another.
> > >>>
> > >>>I'll try this on EDMiniV2 in the coming days, and let people know
> > >>>the results in this thread.
> > >>
> > >>I was just wondering did you have the time to test this patch on your
> > >>board?
> > >
> > >Albert, did you maybe have time to test this patch. I have included it
> > >bellow so you dont have to search for it...
> > >
> > >Signed-off-by: Simon Baatz<gmbnomis@gmail.com>
> > >Tested-by: Luka Perkov<uboot@lukaperkov.net>
> > >---
> > >
> > >Simon discovered this while adding support for new board IB NAS6210.
> > >
> > >More info can be found here:
> > >
> > >http://lists.denx.de/pipermail/u-boot/2012-April/122525.html
> > >
> > > ?include/ide.h | ? ?2 +-
> > > ?1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > >diff --git a/include/ide.h b/include/ide.h
> > >index 8ecc9dd..385e909 100644
> > >--- a/include/ide.h
> > >+++ b/include/ide.h
> > >@@ -24,7 +24,7 @@
> > > ?#ifndef ? ?_IDE_H
> > > ?#define _IDE_H
> > >
> > >-#define ? ? IDE_BUS(dev) ? ?(dev>> ?1)
> > >+#define IDE_BUS(dev) ? ? ? ?(dev / (CONFIG_SYS_IDE_MAXDEVICE / CONFIG_SYS_IDE_MAXBUS))
> > >
> > > ?#define ? ?ATA_CURR_BASE(dev) ? ? ?(CONFIG_SYS_ATA_BASE_ADDR+ide_bus_offset[IDE_BUS(dev)])
> >
> > Sorry, been busier than usual. I'll try and test this tomorrow
> > night; if not, that'll be next week I'm afraid.
>
> It's me beeing boring again, sorry... Can we get your feedback on this one?
>
> Regards,
> Luka
>

Amicalement,
Albert.

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH] fix IDE_BUS(dev) macro
  2012-06-20  8:05             ` Albert ARIBAUD
@ 2012-08-09 20:36               ` Wolfgang Denk
  2012-08-13 12:27                 ` Albert ARIBAUD
  0 siblings, 1 reply; 24+ messages in thread
From: Wolfgang Denk @ 2012-08-09 20:36 UTC (permalink / raw)
  To: u-boot

Dear Albert ARIBAUD,

In message <CAJHhwataYrT3GBzhrDcqhphRP5HbX3vd2XyuyYMSLeD8XWE8WA@mail.gmail.com> you wrote:
> Sorry Luka (and all), been tied up pretty heavily recently, had to
> adjust to find some free time again.
> 
> I will test your patch today and post results tonight.

What was the outcome of this?  I cannot find any further messages to
this thread...

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
One possible reason that things aren't going  according  to  plan  is
that there never was a plan in the first place.

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH] fix IDE_BUS(dev) macro
  2012-08-09 20:36               ` Wolfgang Denk
@ 2012-08-13 12:27                 ` Albert ARIBAUD
  2012-08-14  8:30                   ` Albert ARIBAUD
  2012-08-14 12:33                   ` DrEagle
  0 siblings, 2 replies; 24+ messages in thread
From: Albert ARIBAUD @ 2012-08-13 12:27 UTC (permalink / raw)
  To: u-boot

Hi Wolfgang,

On Thu, 09 Aug 2012 22:36:33 +0200, Wolfgang Denk <wd@denx.de> wrote:
> Dear Albert ARIBAUD,
> 
> In message
> <CAJHhwataYrT3GBzhrDcqhphRP5HbX3vd2XyuyYMSLeD8XWE8WA@mail.gmail.com>
> you wrote:
> > Sorry Luka (and all), been tied up pretty heavily recently, had to
> > adjust to find some free time again.
> > 
> > I will test your patch today and post results tonight.
> 
> What was the outcome of this?  I cannot find any further messages to
> this thread...

Sorry, I was away from home for a few days with no or little Internet
access.

I wanted to test this on orion5x/edminiv2; I should be able to do this
today in the course of regression-testing preparing an orion5x patch
I want to submit in this window. 

> Best regards,
> 
> Wolfgang Denk

Amicalement,
-- 
Albert.

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH] fix IDE_BUS(dev) macro
  2012-08-13 12:27                 ` Albert ARIBAUD
@ 2012-08-14  8:30                   ` Albert ARIBAUD
  2012-08-14 12:33                   ` DrEagle
  1 sibling, 0 replies; 24+ messages in thread
From: Albert ARIBAUD @ 2012-08-14  8:30 UTC (permalink / raw)
  To: u-boot

On Mon, 13 Aug 2012 14:27:16 +0200, Albert ARIBAUD
<albert.u.boot@aribaud.net> wrote:
> Hi Wolfgang,
> 
> On Thu, 09 Aug 2012 22:36:33 +0200, Wolfgang Denk <wd@denx.de> wrote:
> > Dear Albert ARIBAUD,
> > 
> > In message
> > <CAJHhwataYrT3GBzhrDcqhphRP5HbX3vd2XyuyYMSLeD8XWE8WA@mail.gmail.com>
> > you wrote:
> > > Sorry Luka (and all), been tied up pretty heavily recently, had to
> > > adjust to find some free time again.
> > > 
> > > I will test your patch today and post results tonight.
> > 
> > What was the outcome of this?  I cannot find any further messages to
> > this thread...
> 
> Sorry, I was away from home for a few days with no or little Internet
> access.
> 
> I wanted to test this on orion5x/edminiv2; I should be able to do this
> today in the course of regression-testing preparing an orion5x patch
> I want to submit in this window. 

... and of course I spent all my allotted time yesterday on catching up
about pull requests, sorry. I should be able to test this today, above
the current tip of u-boot-arm/master.

Amicalement,
-- 
Albert.

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH] fix IDE_BUS(dev) macro
  2012-08-13 12:27                 ` Albert ARIBAUD
  2012-08-14  8:30                   ` Albert ARIBAUD
@ 2012-08-14 12:33                   ` DrEagle
  1 sibling, 0 replies; 24+ messages in thread
From: DrEagle @ 2012-08-14 12:33 UTC (permalink / raw)
  To: u-boot

Hi,

It works great for my IB-6220 (Dual Sata).

Without the patch, the same disk is seen twice, I have only checked
disks serial numbers :

ib62x0 => version

U-Boot 2012.07-00123-g4d3c95f-dirty (Aug 13 2012 - 11:30:58) RaidSonic
ICY BOX IB-NAS62x0
arm-linux-gnueabi-gcc (Debian 4.4.5-8) 4.4.5
GNU ld (GNU Binutils for Debian) 2.20.1.20100303
ib62x0 => ide reset

Reset IDE: Bus 0: OK Bus 1: OK
  Device 0: Model: WDC WD20EARX-00MMMB0 Firm: 80.00A80 Ser#:
WD-WCAWZ2075392
            Type: Hard Disk
            Supports 48-bit addressing
            Capacity: 1907729.0 MB = 1863.0 GB (-387938128 x 512)
  Device 1: Model: WDC WD20EARX-00MMMB0 Firm: 80.00A80 Ser#:
WD-WCAWZ2075392
            Type: Hard Disk
            Supports 48-bit addressing
            Capacity: 1907729.0 MB = 1863.0 GB (-387938128 x 512)
ib62x0 =>

---
With the patch applied, the two disks are seen :
---
patch -p0 -i ../fix_ide.diff

File to patch: include/ide.h

ib62x0 => version

U-Boot 2012.07-00123-g4d3c95f-dirty (Aug 14 2012 - 14:22:49) RaidSonic
ICY BOX IB-NAS62x0
arm-linux-gnueabi-gcc (Debian 4.4.5-8) 4.4.5
GNU ld (GNU Binutils for Debian) 2.20.1.20100303
ib62x0 => ide reset

Reset IDE: Bus 0: OK Bus 1: OK
  Device 0: Model: WDC WD20EARX-00MMMB0 Firm: 80.00A80 Ser#:
WD-WCAWZ2075392
            Type: Hard Disk
            Supports 48-bit addressing
            Capacity: 1907729.0 MB = 1863.0 GB (-387938128 x 512)
  Device 1: Model: WDC WD20EARX-00MMMB0 Firm: 80.00A80 Ser#:
WD-WCAWZ2083815
            Type: Hard Disk
            Supports 48-bit addressing
            Capacity: 1907729.0 MB = 1863.0 GB (-387938128 x 512)
ib62x0 =>
ib62x0 => ide device 0

IDE device 0: Model: WDC WD20EARX-00MMMB0 Firm: 80.00A80 Ser#:
WD-WCAWZ2075392
            Type: Hard Disk
            Supports 48-bit addressing
            Capacity: 1907729.0 MB = 1863.0 GB (-387938128 x 512)
... is now current device
ib62x0 => ide device 1

IDE device 1: Model: WDC WD20EARX-00MMMB0 Firm: 80.00A80 Ser#:
WD-WCAWZ2083815
            Type: Hard Disk
            Supports 48-bit addressing
            Capacity: 1907729.0 MB = 1863.0 GB (-387938128 x 512)
... is now current device

---
For me this patch has fixed the detection bug of ide disks.

Hope this tests may help.

Le 13/08/2012 14:27, Albert ARIBAUD a ?crit :
> Hi Wolfgang,
> 
> On Thu, 09 Aug 2012 22:36:33 +0200, Wolfgang Denk <wd@denx.de> wrote:
>> Dear Albert ARIBAUD,
>>
>> In message
>> <CAJHhwataYrT3GBzhrDcqhphRP5HbX3vd2XyuyYMSLeD8XWE8WA@mail.gmail.com>
>> you wrote:
>>> Sorry Luka (and all), been tied up pretty heavily recently, had to
>>> adjust to find some free time again.
>>>
>>> I will test your patch today and post results tonight.
>>
>> What was the outcome of this?  I cannot find any further messages to
>> this thread...
> 
> Sorry, I was away from home for a few days with no or little Internet
> access.
> 
> I wanted to test this on orion5x/edminiv2; I should be able to do this
> today in the course of regression-testing preparing an orion5x patch
> I want to submit in this window. 
> 
>> Best regards,
>>
>> Wolfgang Denk
> 
> Amicalement,
> 

drEagle

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH] fix IDE_BUS(dev) macro
  2012-04-17 19:06 [U-Boot] [PATCH] fix IDE_BUS(dev) macro Luka Perkov
                   ` (3 preceding siblings ...)
  2012-06-03 15:03 ` Vágó Tibor
@ 2012-10-07  7:15 ` uboot at lukaperkov.net
  2012-10-07  9:28   ` Albert ARIBAUD
  4 siblings, 1 reply; 24+ messages in thread
From: uboot at lukaperkov.net @ 2012-10-07  7:15 UTC (permalink / raw)
  To: u-boot

Hi Tom, Albert, Prafulla and others,

On Tue, Apr 17, 2012 at 09:06:49PM +0200, Luka Perkov wrote:
> IDE_BUS assumes that each bus has two devices and thus returns the first
> bus even when the second one should be probed.
> 
> Signed-off-by: Simon Baatz <gmbnomis@gmail.com>
> Tested-by: Luka Perkov <uboot@lukaperkov.net>
> ---
> 
> Simon discovered this while adding support for new board IB NAS6210.
> 
> More info can be found here:
> 
> http://lists.denx.de/pipermail/u-boot/2012-April/122525.html
> 
> When this is commited I will do a coding style cleanup. There are tabs
> after few "#define" parts in include/ide.h.
> 
>  include/ide.h |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/ide.h b/include/ide.h
> index 8ecc9dd..385e909 100644
> --- a/include/ide.h
> +++ b/include/ide.h
> @@ -24,7 +24,7 @@
>  #ifndef	_IDE_H
>  #define _IDE_H
>  
> -#define	IDE_BUS(dev)	(dev >> 1)
> +#define IDE_BUS(dev)	(dev / (CONFIG_SYS_IDE_MAXDEVICE / CONFIG_SYS_IDE_MAXBUS))
>  
>  #define	ATA_CURR_BASE(dev)	(CONFIG_SYS_ATA_BASE_ADDR+ide_bus_offset[IDE_BUS(dev)])
>  

I have assigned this patch to Tom in patchwork here:

http://patchwork.ozlabs.org/patch/159129/

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH] fix IDE_BUS(dev) macro
  2012-10-07  7:15 ` uboot at lukaperkov.net
@ 2012-10-07  9:28   ` Albert ARIBAUD
  2012-10-07 14:40     ` Tom Rini
  0 siblings, 1 reply; 24+ messages in thread
From: Albert ARIBAUD @ 2012-10-07  9:28 UTC (permalink / raw)
  To: u-boot

Hi uboot at lukaperkov.net,

On Sun, 7 Oct 2012 09:15:52 +0200, uboot at lukaperkov.net wrote:

> Hi Tom, Albert, Prafulla and others,
> 
> On Tue, Apr 17, 2012 at 09:06:49PM +0200, Luka Perkov wrote:
> > IDE_BUS assumes that each bus has two devices and thus returns the first
> > bus even when the second one should be probed.
> > 
> > Signed-off-by: Simon Baatz <gmbnomis@gmail.com>
> > Tested-by: Luka Perkov <uboot@lukaperkov.net>
> > ---
> > 
> > Simon discovered this while adding support for new board IB NAS6210.
> > 
> > More info can be found here:
> > 
> > http://lists.denx.de/pipermail/u-boot/2012-April/122525.html
> > 
> > When this is commited I will do a coding style cleanup. There are tabs
> > after few "#define" parts in include/ide.h.
> > 
> >  include/ide.h |    2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/include/ide.h b/include/ide.h
> > index 8ecc9dd..385e909 100644
> > --- a/include/ide.h
> > +++ b/include/ide.h
> > @@ -24,7 +24,7 @@
> >  #ifndef	_IDE_H
> >  #define _IDE_H
> >  
> > -#define	IDE_BUS(dev)	(dev >> 1)
> > +#define IDE_BUS(dev)	(dev / (CONFIG_SYS_IDE_MAXDEVICE / CONFIG_SYS_IDE_MAXBUS))
> >  
> >  #define	ATA_CURR_BASE(dev)	(CONFIG_SYS_ATA_BASE_ADDR+ide_bus_offset[IDE_BUS(dev)])
> >  
> 
> I have assigned this patch to Tom in patchwork here:
> 
> http://patchwork.ozlabs.org/patch/159129/
> 
> From when it was submitted we have got confirmations that this patch
> does the job for kirkwood boards on the u-boot mailing list:
> 
> http://lists.denx.de/pipermail/u-boot/2012-April/122684.html
> http://lists.denx.de/pipermail/u-boot/2012-June/125658.html
> 
> It would be nice if this patch would enter v2012.10 release. At least
> someone could pull it into his repo so it can find it's way into master
> eventually.

As discussed on IRC, there are tests that actually cover the plaforms I
should have tested myself, so I personally think this can go in 2012.10.

> Regards,
> Luka

Amicalement,
-- 
Albert.

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH] fix IDE_BUS(dev) macro
  2012-10-07  9:28   ` Albert ARIBAUD
@ 2012-10-07 14:40     ` Tom Rini
  0 siblings, 0 replies; 24+ messages in thread
From: Tom Rini @ 2012-10-07 14:40 UTC (permalink / raw)
  To: u-boot

On Sun, Oct 7, 2012 at 2:28 AM, Albert ARIBAUD
<albert.u.boot@aribaud.net> wrote:
> Hi uboot at lukaperkov.net,
>
> On Sun, 7 Oct 2012 09:15:52 +0200, uboot at lukaperkov.net wrote:
>
>> Hi Tom, Albert, Prafulla and others,
>>
>> On Tue, Apr 17, 2012 at 09:06:49PM +0200, Luka Perkov wrote:
>> > IDE_BUS assumes that each bus has two devices and thus returns the first
>> > bus even when the second one should be probed.
>> >
>> > Signed-off-by: Simon Baatz <gmbnomis@gmail.com>
>> > Tested-by: Luka Perkov <uboot@lukaperkov.net>
>> > ---
>> >
>> > Simon discovered this while adding support for new board IB NAS6210.
>> >
>> > More info can be found here:
>> >
>> > http://lists.denx.de/pipermail/u-boot/2012-April/122525.html
>> >
>> > When this is commited I will do a coding style cleanup. There are tabs
>> > after few "#define" parts in include/ide.h.
>> >
>> >  include/ide.h |    2 +-
>> >  1 file changed, 1 insertion(+), 1 deletion(-)
>> >
>> > diff --git a/include/ide.h b/include/ide.h
>> > index 8ecc9dd..385e909 100644
>> > --- a/include/ide.h
>> > +++ b/include/ide.h
>> > @@ -24,7 +24,7 @@
>> >  #ifndef    _IDE_H
>> >  #define _IDE_H
>> >
>> > -#define    IDE_BUS(dev)    (dev >> 1)
>> > +#define IDE_BUS(dev)       (dev / (CONFIG_SYS_IDE_MAXDEVICE / CONFIG_SYS_IDE_MAXBUS))
>> >
>> >  #define    ATA_CURR_BASE(dev)      (CONFIG_SYS_ATA_BASE_ADDR+ide_bus_offset[IDE_BUS(dev)])
>> >
>>
>> I have assigned this patch to Tom in patchwork here:
>>
>> http://patchwork.ozlabs.org/patch/159129/
>>
>> From when it was submitted we have got confirmations that this patch
>> does the job for kirkwood boards on the u-boot mailing list:
>>
>> http://lists.denx.de/pipermail/u-boot/2012-April/122684.html
>> http://lists.denx.de/pipermail/u-boot/2012-June/125658.html
>>
>> It would be nice if this patch would enter v2012.10 release. At least
>> someone could pull it into his repo so it can find it's way into master
>> eventually.
>
> As discussed on IRC, there are tests that actually cover the plaforms I
> should have tested myself, so I personally think this can go in 2012.10.

OK, I'll pick this up Monday, thanks.

-- 
Tom

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] fix IDE_BUS(dev) macro
  2012-05-14 19:29       ` Luka Perkov
  2012-05-15 19:42         ` Albert ARIBAUD
@ 2012-10-08 18:46         ` Tom Rini
  1 sibling, 0 replies; 24+ messages in thread
From: Tom Rini @ 2012-10-08 18:46 UTC (permalink / raw)
  To: u-boot

On Mon, May 14, 2012 at 09:29:10AM -0000, Luka Perkov wrote:

> Hi Albert,
> 
> On Sun, Apr 29, 2012 at 10:19:41PM +0200, Luka Perkov wrote:
> > On Thu, Apr 19, 2012 at 08:38:19AM +0200, Albert ARIBAUD wrote:
> > > Not my main area of expertise here, but I am not sure how this plays
> > > on Marvell non-kirkwood platforms (e.g., orion5x).
> > > 
> > > ISTR it is not the first time we deal with the whole IDE number of
> > > bus / number of ports [ / useable ports ] issue, and we may be
> > > running in circles here, fixing one platform and breaking another.
> > > 
> > > I'll try this on EDMiniV2 in the coming days, and let people know
> > > the results in this thread.
> > 
> > I was just wondering did you have the time to test this patch on your
> > board?
> 
> Albert, did you maybe have time to test this patch. I have included it
> bellow so you dont have to search for it...
> 
> Signed-off-by: Simon Baatz <gmbnomis@gmail.com>
> Tested-by: Luka Perkov <uboot@lukaperkov.net>

With a reworded commit message, applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20121008/5de3d0d2/attachment.pgp>

^ permalink raw reply	[flat|nested] 24+ messages in thread

end of thread, other threads:[~2012-10-08 18:46 UTC | newest]

Thread overview: 24+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-04-17 19:06 [U-Boot] [PATCH] fix IDE_BUS(dev) macro Luka Perkov
2012-04-18 21:37 ` David Purdy
2012-04-19  6:38   ` Albert ARIBAUD
2012-04-21 19:38     ` David Purdy
2012-04-29 20:19     ` Luka Perkov
2012-05-14 19:29       ` Luka Perkov
2012-05-15 19:42         ` Albert ARIBAUD
2012-05-15 20:44           ` Luka Perkov
2012-06-07 13:30           ` Luka Perkov
2012-06-20  8:05             ` Albert ARIBAUD
2012-08-09 20:36               ` Wolfgang Denk
2012-08-13 12:27                 ` Albert ARIBAUD
2012-08-14  8:30                   ` Albert ARIBAUD
2012-08-14 12:33                   ` DrEagle
2012-10-08 18:46         ` [U-Boot] " Tom Rini
2012-06-03 20:23     ` [U-Boot] [PATCH] " Rogan Dawes
2012-06-03 21:25       ` Luka Perkov
2012-04-20  5:17 ` Prafulla Wadaskar
2012-06-03 15:03 ` Vágó Tibor
2012-06-03 15:03 ` Vágó Tibor
2012-06-03 21:24   ` Luka Perkov
2012-10-07  7:15 ` uboot at lukaperkov.net
2012-10-07  9:28   ` Albert ARIBAUD
2012-10-07 14:40     ` Tom Rini

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.