Dear Stefan Roese, In message <[email protected]> you wrote: > > I don't like this "l+=91" statement. How about making it a bit more flexible. > Something like this: > > #define MIN_PACKET_LENGTH 64 > #define MAX_PACKET_LENGTH 1518 > #ifndef CONFIG_SYS_POST_ETH_LOOPS > #define CONFIG_SYS_POST_ETH_LOOPS 10 > #endif > #define PACKET_INCR ((MAX_PACKET_LENGTH - MIN_PACKET_LENGTH) / \ > CONFIG_SYS_POST_ETH_LOOPS)
Agreed. For the final version, please check how MAX_PACKET_LENGTH is used (I didn't do this) and verify that 1518 is the correct value to use here. Thanks. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [email protected] Prof: So the American government went to IBM to come up with a data encryption standard and they came up with ... Student: EBCDIC! _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
