[Review request]: Attansic L2 ethernet driver

John Baldwin jhb at freebsd.org
Mon Sep 8 20:12:37 UTC 2008


On Sunday 31 August 2008 02:28:22 pm Stanislav Sedov wrote:
> Hi!
>
> Seems that I've implemented the most stuff I wanted
> to see in the driver for Attansic L2 ethernet controller.
> I haven't received any bugreports since the last public
> version and I think it's in a pretty good form and ready
> to be committed in HEAD. Before that, I'd like to ask
> you to review the code of the driver, since it's my
> first work in this area, and I'm not sure I've done
> everything correctly.
>
> The latest diff against HEAD is available here:
> http://www.SpringDaemons.com/stas/if_ae.diff.2008083100

A few style notes:
- return (FILTER_STRAY) (missing ()'s)
- Extra indentation for one of the TASK_INIT()'s
- Are you using 4 space indent or 8 space indent?

-- 
John Baldwin


More information about the freebsd-drivers mailing list