[eepro100-bug] options documentation seems bad

[Donald Becker]

On Thu, 28 Mar 2002 aaron@aaroncummings.com wrote:

> This is in regards to the documentation for using the options attribute
> to force speed and half vs full duplex.
>
> http://www.scyld.com/network/eepro100.html says:

[[ Old table deleted. ]]

> But the code in eepro100.c seems to indicate:
>  Hex   Decimal  Meaning
>  0x200 512      Force 100mbps, full duplex
>  0x100 256      Force 100mbps, half duplex
>  0x020  32      Force 10mbps, full duplex
>  0x000   0      Force 10mbps, half duplex

Good point.  I changed this 10 months ago for most of my drivers, and I
didn't update the eepro100 web page to match the driver.

Thanks for pointing this out.  Now fixed:
  http://www.scyld.com/network/eepro100.html


BTW, that last table entry should be 0x10, meaning 10Mbps shared between
Rx and Tx.  "0x20" means 10+10Mbps Tx+Rx.
Would it have been more intuitive if the settings were in decimal rather
than hex?

The CVS log entry for that change was:
________________
revision 1.15
date: 2001/05/17 21:12:15;  author: becker;  state: Exp;  lines: +84 -67
eepro100.c:v1.14 5/17/2001
...
Changed the module values for forcing the speed and duplex.
Use option values 0x20 and 0x200 for forcing full duplex operation.
Use option values 0x10 and 0x100 for forcing half duplex fixed speed.


-- 
Donald Becker				becker@scyld.com
Scyld Computing Corporation		http://www.scyld.com
410 Severn Ave. Suite 210		Second Generation Beowulf Clusters
Annapolis MD 21403			410-990-9993