On Mon, Dec 14, 2009 at 10:25:34AM +0100, Stefan Weil wrote: > Michael S. Tsirkin suggested more documentation for > two temporary status values. Well, here it is. > > Signed-off-by: Stefan Weil <w...@mail.berlios.de>
Looks good to me. > --- > hw/eepro100.c | 7 ++++--- > 1 files changed, 4 insertions(+), 3 deletions(-) > > diff --git a/hw/eepro100.c b/hw/eepro100.c > index 22e5bed..5a77409 100644 > --- a/hw/eepro100.c > +++ b/hw/eepro100.c > @@ -213,9 +213,10 @@ typedef struct { > uint32_t ru_offset; /* RU address offset */ > uint32_t statsaddr; /* pointer to eepro100_stats_t */ > > - /* Temporary data. */ > - eepro100_tx_t tx; > - uint32_t cb_address; > + /* Temporary status information (no need to save these values), > + * used while processing CU commands. */ > + eepro100_tx_t tx; /* transmit buffer descriptor */ > + uint32_t cb_address; /* = cu_base + cu_offset */ > > /* Statistical counters. Also used for wake-up packet (i82559). */ > eepro100_stats_t statistics; > -- > 1.6.5