On Thu, 20 Jan 2005 16:00:38 -0800
"Michael Chan" <mchan@xxxxxxxxxxxx> wrote:
> Detailed changes:
Thanks a lot Michael, patch applied.
Another thing that makes patches like this one easier to review
is to split them up as much as is reasonable possible. As a specific
suggestion, for this patch we might have first done a change which
added all of the NVRAM chip IDs and probed for them, the next change
might be the change from tw32_f() to tw32() for the chip register
accesses, the next change would make use of the NVRAM chip ID type
and finally we'd have the change that implemented tg3_set_eeprom().
That's how you likely, at least logically, implemented this patch
in a step by step fashion. If you really do split the change up into
pieces like that, the individual patches are much easier to verify
and review than the whole thing put together and submitted all at
once.
Thanks again.
|