Subject: Re: new EtherIP driver for 4.0
To: None <>
From: Christos Zoulas <>
List: tech-net
Date: 11/22/2006 20:02:59
In article <>,
Rui Paulo  <> wrote:
>On Nov 21, 2006, at 8:50 PM, Hans Rosenfeld wrote:
>> On Tue, Oct 31, 2006 at 03:09:01PM +0000, Rui Paulo wrote:
>>> I took a very skim look and I couldn't find anything obviously wrong.
>>> There are some KNF nits but nothing important.
>> After some brainstorming with Rui I made some (mostly cosmetic)  
>> changes to
>> the code to make it conform to KNF rules. I put the latest version in
>> for others to  
>> take
>> another look at it.
>Anyone has anything to say ? I plan to commit this soon.

1. The whole thing needs to be passed through unexpand and possibly
   the indentation in ethersubr.c is wrong.
2. s/bzero/memset/
3. etherip_ether_aton() -> ether_snprintf(), same as the tap interface.

Go for it.