From mboxrd@z Thu Jan  1 00:00:00 1970
From: Tolunay Orkun <listmember@orkun.us>
Date: Tue, 28 Feb 2006 14:32:40 -0600
Subject: [U-Boot-Users] [PATCH] CFI support for Versatile & Integrator/CP
 boards
In-Reply-To: <4404AA53.6040803@lucent.com>
References: <CAM-OWA1XvpFVjCRGry00000004@cam-owa1.Emea.Arm.com> <200602281024.13958.sr@denx.de> <4404A2A0.5000009@orkun.us> <4404AA53.6040803@lucent.com>
Message-ID: <4404B368.30007@orkun.us>
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

Marcus Hall wrote:
>> I now think Marcus patch is preferable but it would also be better to 
>> inform on the console regarding timeout case as well.
> 
> 
> Well, flash_status_check() does print an error message if it does 
> timeout...

You are right. We don't need print in flash_full_status_check for timeout 
but we still reset the flash 2nd time in flash_full_status_check for 
timeouts. Probably harmless but unnecessary.

I guess, it is best to leave it as it is now after your patch is applied.

Tolunay