[PR 17] GRETH: Fixed autonego timeout overflow problem

[Thomas Doerfler]

Hi,

Am 19.03.2012 23:26, schrieb Ralf Corsepius:
> On 03/19/2012 08:59 PM, Joel Sherrill wrote:
>> This looks ok to me.
> 
> I am having strong doubts on this patch.
> 
> Daniel, could you elaborate the problem you are trying to solve?

I agree with Ralf, the solution seems a bit unclear related to the casts
involved...

What exactly is the reason for the casts? Why are they applied in an
asymetric way?

wkr,

Thomas.


> 
> Ralf
> 
> 
>> I would have thought we had some support routines to do this but alas.
>>
>> Please commit someone.
>> ________________________________________
>> From: rtems-devel-bounces at rtems.org [rtems-devel-bounces at rtems.org] On
>> Behalf Of Daniel Hellstrom [daniel at gaisler.com]
>> Sent: Monday, February 06, 2012 8:15 AM
>> To: rtems-devel at rtems.org
>> Subject: [PR 17] GRETH: Fixed autonego timeout overflow problem
>>
>> Signed-off-by: Daniel Hellstrom<daniel at gaisler.com>
>> ---
>>   c/src/libchip/network/greth.c |    2 +-
>>   1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/c/src/libchip/network/greth.c
>> b/c/src/libchip/network/greth.c
>> index aff4d0f..b0e70b4 100644
>> --- a/c/src/libchip/network/greth.c
>> +++ b/c/src/libchip/network/greth.c
>> @@ -339,7 +339,7 @@ greth_initialize_hardware (struct greth_softc *sc)
>>               while (!(((phystatus = read_mii(phyaddr, 1))>>  5)&  1)) {
>>                       if ( rtems_clock_get_tod_timeval(&tnow) !=
>> RTEMS_SUCCESSFUL )
>>                         printk("rtems_clock_get_tod_timeval failed\n\r");
>> -                    msecs =
>> (tnow.tv_sec-tstart.tv_sec)*1000+(tnow.tv_usec-tstart.tv_usec)/1000;
>> +                    msecs =
>> (int)(tnow.tv_sec-tstart.tv_sec)*1000+((int)tnow.tv_usec-(int)tstart.tv_usec)/1000;
>>
>>                       if ( msecs>  GRETH_AUTONEGO_TIMEOUT_MS ){
>>                               sc->auto_neg_time = msecs;
>>                               sc->auto_neg = -1; /* Failed */
>> -- 
>> 1.7.0.4
>>
>> _______________________________________________
>> rtems-devel mailing list
>> rtems-devel at rtems.org
>> http://www.rtems.org/mailman/listinfo/rtems-devel
>>
>> _______________________________________________
>> rtems-devel mailing list
>> rtems-devel at rtems.org
>> http://www.rtems.org/mailman/listinfo/rtems-devel
> 
> _______________________________________________
> rtems-devel mailing list
> rtems-devel at rtems.org
> http://www.rtems.org/mailman/listinfo/rtems-devel


-- 
--------------------------------------------
Embedded Brains GmbH
Thomas Doerfler           Obere Lagerstr. 30
D-82178 Puchheim          Germany
email: Thomas.Doerfler at embedded-brains.de
Phone: +49-89-18908079-2
Fax:   +49-89-18908079-9