> +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. -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html