AW: [U-Boot-Users] [PATCH] fix comment in net.c

Jonas Mark (ST-FIR/ENG1) * Mark.Jonas at de.bosch.com
Thu Nov 10 09:19:29 CET 2005


Good morning Pantelis,

> Ron Alder wrote:
>> This patch fixes a comment in net/net.c
>> 
>> CHANGELOG:
>> * Fix comment in net/net.c
>>  Patch by Ron Alder, 09 Nov 2005
>> 
>> 
>>
------------------------------------------------------------------------
>> 
>> diff --git a/net/net.c b/net/net.c
>> --- a/net/net.c
>> +++ b/net/net.c
>> @@ -829,7 +829,6 @@ static ushort CDP_compute_csum(const uch
}
>>  		if (len) {
>>  			leftover = (signed short)(*(const signed char
*)buff);
>> -			/* * XXX CISCO SUCKS big time! (and blows too)
*/
>>  			result = (result & 0xffff0000) | ((result +
leftover) &
>>  		0x0000ffff);  		} while (result >> 16)
> 
> WTF is this? Politically correct day?

I've been with Motorola and Freescale and indeed political correctness
is
a high value there.

> Since I wrote the comment in question let me elaborate.
> 
> CDP uses the ip checksum algorithm with a twist; for the last
> byte it *sign* extends and sums. Obviously this is is completely
> braindead, and warants the comment and much more :)

I agree with you that removing your comment does not yield any real
benefit -
except maybe for the people working at Cisco. But I guess you agree that
instead of insulting people a comment that elaborates why you were
forced to
do what you did would have been much better.

> Regards
> 
> Pantelis

Regards
Mark





More information about the U-Boot mailing list