Re: Subject: [PATCH 1/9] bna: Brocade 10Gb Ethernet device driver

[David Miller <davem@davemloft.net>]

You're going to have to remove all of the
__ESX_COS__, __VMKERNEL_MODULE__, __VMKNETDDI_QUEUEOPS__
ifdef tests and protected code in this driver.

Nothing sets those CPP defines, and the Linux kernel is not
a place where you can maintain your compatability code for
building the driver in other environments.

Next, you have defined your own implementation of a driver
debug logging facility, when we have a generic one already
for network drivers via netif_msg_*() and associated
NETIF_MSG_* debugging masks.  See other drivers for details.
If it doesn't fully suit your needs, fix the existing
facilities to do so.

These are just the most obvious things I noticed by glancing
quickly at your driver.  I'm sure there a many other things
you will need to fix up before we can add your driver.

The code is 1MB in size, which almost guarentees there are lots of
things which will need to be removed :-) Even the largest 10GB drivers
with tons of firmware are ~500K in size and I even consider that
bordering on excessive.