Yevgeny Petrilin wrote:
[...]
quoted text
> +#ifdef CONFIG_MLX4_EN_DEBUG
> +#define mlx4_dbg(mdev, format, arg...)	\
> +	printk(KERN_DEBUG "%s %s: " format , DRV_NAME ,\
> +		(&mdev->pdev->dev)->bus_id , ## arg)

CONFIG_MLX4_EN_DEBUG isn't in Kconfig.  If you expect debugging to be
enabled through tweaking EXTRA_CFLAGS then the macro should be called
something else.

[...]
quoted text
> +#ifdef CONFIG_MLX4_EN_PREFETCH
> +	#define PREFETCH(x)		prefetch(x)

Neither is CONFIG_MLX4_EN_PREFETCH in Kconfig.  Why don't you do this
unconditionally?

There are several more CONFIG macros tested here which should be removed
or renamed (or possibly added to Kconfig).

Ben.

-- 
Ben Hutchings, Senior Software Engineer, Solarflare Communications
Not speaking for my employer; that's the marketing department's job.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html