netdev
[Top] [All Lists]

Re: [PATCH] /driver/net/wan/sbs520

To: Adam Anthony <AAnthony@xxxxxxx>
Subject: Re: [PATCH] /driver/net/wan/sbs520
From: Francois Romieu <romieu@xxxxxxxxxxxxx>
Date: Mon, 10 Jan 2005 19:00:52 +0100
Cc: netdev@xxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
In-reply-to: <4F23E557A0317D45864097982DE907941A32B1@pilotmail.sbscorp.sbs.com>
References: <4F23E557A0317D45864097982DE907941A32B1@pilotmail.sbscorp.sbs.com>
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: Mutt/1.4.1i
Adam Anthony <AAnthony@xxxxxxx> :
[...]
>        It would be great to receive some feedback on our work, and we hope
> that this driver will eventually be added to the kernel.

It will probably require a few extra steps:
- read Documentation/CodingStyle (mixed case, typedef from hell, ugly #ifdef);
- grep ^static
  -> no static functions ? Uh ?
- use non-obsolete API (pci_find_device in 2005 ?);
- convert the os independant wrappers.

Btw it would probably make sense 1) to figure out what can be merged with
the in-tree DSCC4 driver and 2) to integrate the driver with the existing
hdlc stack. Imho there is some duplicated work/code.

--
Ueimor

<Prev in Thread] Current Thread [Next in Thread>