Re: [PATCH V1] add huawei ibma driver modules The driver is used for communication between in-band management agent(iBMA) and out-of-band management controller(iBMC) via pcie bus in Huawei V3 server. The driver provides character device,VNIC and black box interface for application layer.

[Randy Dunlap]

Hi,

Please fix the Subject: and patch description.

(more below)

On 08/29/2018 06:11 PM, xiongsujuan wrote:
> diff --git a/drivers/net/ethernet/huawei/Kconfig b/drivers/net/ethernet/huawei/Kconfig
> index c1a95ae..68748e9 100644
> --- a/drivers/net/ethernet/huawei/Kconfig
> +++ b/drivers/net/ethernet/huawei/Kconfig
> @@ -4,7 +4,7 @@
>  
>  config NET_VENDOR_HUAWEI
>  	bool "Huawei devices"
> -	default y
> +	default y 

Don't add a space after the 'y'.

>  	---help---
>  	  If you have a network (Ethernet) card belonging to this class, say Y.
>  	  Note that the answer to this question doesn't directly affect the
> @@ -16,4 +16,7 @@ if NET_VENDOR_HUAWEI
>  
>  source "drivers/net/ethernet/huawei/hinic/Kconfig"
>  
> +source "drivers/net/ethernet/huawei/ibma/Kconfig"
> +
> +
>  endif # NET_VENDOR_HUAWEI

> diff --git a/drivers/net/ethernet/huawei/ibma/Kconfig b/drivers/net/ethernet/huawei/ibma/Kconfig
> new file mode 100644
> index 0000000..810cc60
> --- /dev/null
> +++ b/drivers/net/ethernet/huawei/ibma/Kconfig
> @@ -0,0 +1,17 @@
> +#
> +# Huawei driver configuration
> +#
> +
> +config IBMANIC
> +	tristate "Huawei IBMA PCIE Network Interface Card"
> +	depends on (PCI_MSI && X86)
> +	---help---
> +	  This driver supports IBMANIC PCIE Ethernet cards.
> +	  To compile this driver as part of the kernel, choose Y here.
> +	  If unsure, choose N.
> +	  The default is compiled as module.
> +
> +
> +
> +
> +

Drop those trailing blank lines.

What makes the default be compiled as module?

thanks,
-- 
~Randy