Differences
This shows you the differences between two versions of the page.
| Both sides previous revision Previous revision | |||
|
soc:2008:mdeck:journal:week8 [2008/07/19 17:39] mdeck |
soc:2008:mdeck:journal:week8 [2008/07/20 13:51] (current) mdeck |
||
|---|---|---|---|
| Line 67: | Line 67: | ||
| The driver is still currently cluttered throughout with commented-out original-driver code. Once I get feedback from mdc and others, I'll begin pruning out the excess. | The driver is still currently cluttered throughout with commented-out original-driver code. Once I get feedback from mdc and others, I'll begin pruning out the excess. | ||
| + | |||
| + | === 20 July === | ||
| + | |||
| + | I decided to jump the gun and prune away on **skge**. | ||
| + | |||
| + | First, a new branch and a merge of recent updates into the tree: | ||
| + | * [[http://git.etherboot.org/?p=people/mdeck/gpxe.git;a=commitdiff;h=39e778e1930ef5e18bbbd3dcd1d7deddf5597aef|Merge branch 'master' of git://git.etherboot.org/scm/gpxe into skge]] | ||
| + | |||
| + | Most of those cluttering commented lines of old code were removed: | ||
| + | * [[http://git.etherboot.org/?p=people/mdeck/gpxe.git;a=commitdiff;h=d259e62817c58d891834615903c1a5fb601a00ec|[Drivers-skge] Removed unnecessary linux-driver code]] | ||
| + | |||
| + | A few functions were revived to support the XM PHY fully: | ||
| + | * [[http://git.etherboot.org/?p=people/mdeck/gpxe.git;a=commitdiff;h=9ad495c1ada5c4e473f05fc51687abcfa4595750|[Drivers-skge] XM link-state support added]] | ||
| + | |||
| + | Additional functions were clipped, debug statements were placed throughout. All todo's were taken care of. Note I left the EEPROM functions in place, but commented out, in case ''nvs'' support for this is added later: | ||
| + | * [[http://git.etherboot.org/?p=people/mdeck/gpxe.git;a=commitdiff;h=e00cdf2b52ad76d67ba985dfdec6989983499dec|[Drivers-skge] Cleanup extraneous fns, added DBG statements]] | ||
| + | |||
| + | I currently don't foresee additional work to this driver until a code review with mdc. | ||
