| 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> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH] /driver/net/wan/sbs520, Matthias-Christian Ott |
|---|---|
| Next by Date: | [PATCH PKT_SCHED 0/22]: tc action cleanups, Patrick McHardy |
| Previous by Thread: | [PATCH] /driver/net/wan/sbs520, Adam Anthony |
| Next by Thread: | Re: [PATCH] /driver/net/wan/sbs520, Alexey Dobriyan |
| Indexes: | [Date] [Thread] [Top] [All Lists] |