Differences
This shows you the differences between two versions of the page.
| Next revision | Previous revision | ||
|
soc:2008:mdeck:journal:weekb [2008/08/06 13:57] mdeck created |
soc:2008:mdeck:journal:weekb [2008/08/10 10:05] (current) mdeck |
||
|---|---|---|---|
| Line 13: | Line 13: | ||
| * [[http://git.etherboot.org/?p=people/mdeck/gpxe.git;a=commit;h=3605cbd396bc173a9639899788130617b85bf6a5|[Drivers-3c59x] MAC address handling, compaq_net_device support]] | * [[http://git.etherboot.org/?p=people/mdeck/gpxe.git;a=commit;h=3605cbd396bc173a9639899788130617b85bf6a5|[Drivers-3c59x] MAC address handling, compaq_net_device support]] | ||
| * [[http://git.etherboot.org/?p=people/mdeck/gpxe.git;a=commit;h=418a4b9067d518e06702f13baa12e99a58dbce7f|[Drivers-3c59x] Removed #ifdefs]] | * [[http://git.etherboot.org/?p=people/mdeck/gpxe.git;a=commit;h=418a4b9067d518e06702f13baa12e99a58dbce7f|[Drivers-3c59x] Removed #ifdefs]] | ||
| + | |||
| + | === 09 August === | ||
| + | |||
| + | Additional changes to 3c59x. It now successfully compiles. I have not yet tested it, and I anticipate additional changes in doing so. | ||
| + | |||
| + | * [[http://git.etherboot.org/?p=people/mdeck/gpxe.git;a=commit;h=a0f736c4f2c5e417a83adace3126fbede0f62dd3|[Drivers-3c59x] Removed EISA support]] | ||
| + | |||
| + | EISA support code was removed as per mcb30's recommendation during our Friday meeting. This will reduce the image size and simplify conversion. | ||
| + | |||
| + | * [[http://git.etherboot.org/?p=people/mdeck/gpxe.git;a=commit;h=f495d70763d49bd311be75d4cbd9ddf3e446b3de|[Drivers-3c59x] Removed linux timer code]] | ||
| + | |||
| + | Timer-dependent code was replaced with code occasionally triggered during a poll() call. | ||
| + | |||
| + | * [[http://git.etherboot.org/?p=people/mdeck/gpxe.git;a=commit;h=216284aa47374397ebbc059b5c5e322df7fbc31c|[Drivers-3c59x] Removed work-limiting variables]] | ||
| + | |||
| + | Interrupt functions, set to call during poll()s, had checks to limit the amount of work performed during a single call. I pulled out the code for now, as I consider it unnecessary. | ||
| + | |||
| + | * [[http://git.etherboot.org/?p=people/mdeck/gpxe.git;a=commit;h=e68d86b229c78aa270de3480e93a011b2e189159|[Drivers-3c59x] Compiles. Added mii.c, modified pcnet32 & sundance]] | ||
| + | |||
| + | Select MII structures, code, and definitions were pulled into the tree. The pcnet32 and sundance drivers had their own local mii_if_info definitions causing collisions, so these were commented out. | ||
| + | |||
| + | Significant amounts of additional small changes were made to adjust the code to gPXE, including datatypes, prototypes, gPXE structs and calls, casts, trimming out unnecessary debug prefixes, removing IRQ-related code, etc. Many changes due to migrating ''if_port'' from the net_device to the private struct, and fixing debug print characters because of changes in data types. | ||
| + | |||
| + | Next on my agenda is to test the code and perform debugging. I also will go through the debug statements and adjust their statement levels as necessary. | ||
| + | |||
| + | === 10 August === | ||
| + | |||
| + | After testing, I found the ''ioaddr'' PCI acquisition code needed adjustment. I also removed a stats interrupt disable notification, since stats will always fill up with the driver not clearing them. | ||
| + | |||
| + | * [[http://git.etherboot.org/?p=people/mdeck/gpxe.git;a=commit;h=6aea3d39cf5e9cbac9dabcff64c790ba52942bd8|[Drivers-3c59x] Fixed ioaddr acquisition code, removed stats notification]] | ||
| + | |||
| + | I ran ''scandisk'' a few times in DOS via AoE without any noticable network errors. The surface scan progress did appear ever-so-slightly jittery, so I wonder how things will turn out under more careful testing conditions. | ||
| + | |||
| + | Until more thorough testing is performed, I will leave the original driver code commented out, and will refrain from cleaning up the code. This should enhance the debuggability (I don't think that's a word) and simplify touching up the driver. | ||
| + | |||
| + | I currently don't see any problems with the driver. | ||
