From mboxrd@z Thu Jan  1 00:00:00 1970
From: Timo Ketola <timo@exertus.fi>
Date: Wed, 18 Apr 2012 12:15:22 +0300
Subject: [U-Boot] [PATCH 6/9] imx: usb: There is no such register
In-Reply-To: <4F8E83BD.8050609@denx.de>
References: <1334223234-23383-1-git-send-email-timo@exertus.fi>
	<1334735852-23415-1-git-send-email-timo@exertus.fi>
	<1334735852-23415-7-git-send-email-timo@exertus.fi>
	<4F8E83BD.8050609@denx.de>
Message-ID: <4F8E862A.50109@exertus.fi>
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

On 18.04.2012 12:05, Stefano Babic wrote:
> As far as I can see, only MX31 and MX25 boards are using this file.
> Other i.MX have its own initialization file. So #if defined(CONFIG_MX31)
> || defined(CONFIG_MX25) is always true.

So, would it be OK to remove this check altogether?

> However, where is this code ? In current u-boot I see only #if
> defined(CONFIG_MX31) at this line. Is it your patch correct ?

My previous patch 5 touched that one.

>> -	setbits_le32(&ehci->control, USB_EN);
>> -
>
> As far as I can see, it tries to overwrite a capability register, that
> is for our luck read-only. Good catch !

Thanks.

--

Timo