| To: | Benjamin LaHaise <bcrl@xxxxxxxxx> |
|---|---|
| Subject: | Re: [2.6 patch] kill include/linux/eeprom.h |
| From: | Adrian Bunk <bunk@xxxxxxxxx> |
| Date: | Wed, 20 Apr 2005 01:56:46 +0200 |
| Cc: | jgarzik@xxxxxxxxx, linux-net@xxxxxxxxxxxxxxx, netdev@xxxxxxxxxxx |
| In-reply-to: | <20050419135648.GC7315@xxxxxxxxx> |
| References: | <20050419012935.GQ5489@xxxxxxxxx> <20050419135648.GC7315@xxxxxxxxx> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
| User-agent: | Mutt/1.5.6+20040907i |
On Tue, Apr 19, 2005 at 09:56:48AM -0400, Benjamin LaHaise wrote:
> At the very least your patch doesn't do a thorough enough job of
> removing the dead code -- there is no good reason to move the unused
> code into ns83820.c.
Where does my patch do this?
Only the one actually used function setup_ee_mem_bitbanger is moved to
ns83820.c .
> Also, someone needs to go around refactoring eeprom code out of the
> network drivers at some point.
I have no problem with this, but it has to be done right.
> -ben
cu
Adrian
--
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: NAPI and CPU utilization [was: NAPI, e100, and system performance problem], David S. Miller |
|---|---|
| Next by Date: | Re: [2.6 patch] kill include/linux/eeprom.h, Benjamin LaHaise |
| Previous by Thread: | Re: [2.6 patch] kill include/linux/eeprom.h, Nick Winlund |
| Next by Thread: | Re: [2.6 patch] kill include/linux/eeprom.h, Benjamin LaHaise |
| Indexes: | [Date] [Thread] [Top] [All Lists] |