From mboxrd@z Thu Jan  1 00:00:00 1970
From: Amit Virdi <amit.virdi@st.com>
Date: Tue, 27 Mar 2012 14:32:08 +0530
Subject: [U-Boot] [PATCH 15/25] SPEAr: Enable autoneg for ethernet
In-Reply-To: <201203071518.53629.sr@denx.de>
References: <1331121854-20494-1-git-send-email-amit.virdi@st.com>
	<1331121854-20494-16-git-send-email-amit.virdi@st.com>
	<201203071518.53629.sr@denx.de>
Message-ID: <4F718210.1090009@st.com>
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

Stefan,

On 3/7/2012 7:48 PM, Stefan Roese wrote:
> On Wednesday 07 March 2012 13:04:04 Amit Virdi wrote:
>> From: Shiraz Hashim<shiraz.hashim@st.com>
>>
>> The linux-2.6.37 base port has few problems with nfs boot. The boot
>> fails while getting timeout on attempting to mount root file system.
>> This helps the ethernet to bring up faster in linux boot thus avoiding
>> the time out.
>> Besides, it was reported that few phys on SPEAr board are failing in
>> certain network conditions which is avoided by enabling autonegotiation.
>> See issue #115943 in global bug tracker.
>
> Hmmm. This sounds like a problem in the Linux network/PHY code then. Which
> needs to get fixed there instead of changing something here in U-Boot. Please

Probably, this is the problem with phys used in some boards.

> think about situations where the ethernet driver is not called at all in U-
> Boot (booting from NOR/NAND). With my changes to defer the ethernet probing to
> the first usage.
>

Keeping your changes, we can still move ahead with autoneg ON. I can 
think of two solutions for this -

Solution 1:
  - Use fix configuration for all SPEAr boards except the one on which 
the problem has been reported. In that case we can ON autoneg.

Solution 2:
  - Enable autoneg for all SPEAr and specify a larger delay parameter if 
rootfs has to be mounted from nfs. This would ensure that timeout won't 
happen.

I would prefer solution 2. What's your opinion?

> One more comment below.
>
>> Reported-by: Deepak Sikri<deepak.sikri@st.com>
>> Reported-by: Armando Visconti<armando.visconti@st.com>
>> Signed-off-by: Shiraz Hashim<shiraz.hashim@st.com>
>> Signed-off-by: Amit Virdi<amit.virdi@st.com>
>> ---
>>   include/configs/spear-common.h |    1 +
>>   1 files changed, 1 insertions(+), 0 deletions(-)
>>
>> diff --git a/include/configs/spear-common.h
>> b/include/configs/spear-common.h index 5b6ef72..26642f1 100644
>> --- a/include/configs/spear-common.h
>> +++ b/include/configs/spear-common.h
>> @@ -35,6 +35,7 @@
>>   #define CONFIG_DW0_PHY				1
>>   #define CONFIG_NET_MULTI
>>   #define CONFIG_PHY_RESET_DELAY			(10000)		/*
> in usec */
>> +#define CONFIG_DW_AUTONEG			1
>
> Please don't add the "1" here.
>

Ok.

Thanks
Amit Virdi