Re: [PATCH 1/3] bnx2: Add support for CNIC driver.

+static void
+bnx2_cnic_stop(struct bnx2 *bp)

The preceding functions follow this format:
static void <fnc name>(<args)

While this and some below follow the
static void
<fnc name>(<args>)

You probably want to follow one style.