netdev
[Top] [All Lists]

Re: [2.6 patch] kill include/linux/eeprom.h

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>