[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20160714053530.GJ10079@leon.nu>
Date: Thu, 14 Jul 2016 08:35:31 +0300
From: Leon Romanovsky <leon@...nel.org>
To: Lijun Ou <oulijun@...wei.com>
Cc: dledford@...hat.com, sean.hefty@...el.com,
hal.rosenstock@...il.com, davem@...emloft.net,
jeffrey.t.kirsher@...el.com, jiri@...lanox.com,
ogerlitz@...lanox.com, linux-rdma@...r.kernel.org,
linux-kernel@...r.kernel.org, netdev@...r.kernel.org,
gongyangming@...wei.com, xiaokun@...wei.com,
tangchaofei@...wei.com, haifeng.wei@...wei.com,
yisen.zhuang@...wei.com, yankejian@...wei.com,
charles.chenxin@...wei.com, linuxarm@...wei.com
Subject: Re: [PATCH v11 21/22] IB/hns: Kconfig and Makefile for RoCE module
On Sat, Jul 02, 2016 at 05:39:23PM +0800, Lijun Ou wrote:
> This patch added Kconfig and Makefile for building RoCE module.
>
> Signed-off-by: Wei Hu <xavier.huwei@...wei.com>
> Signed-off-by: Nenglong Zhao <zhaonenglong@...ilicon.com>
> Signed-off-by: Lijun Ou <oulijun@...wei.com>
> ---
> PATCH v11:
> hns_roce_icm.o -> hns_roce_hem.o
>
> PATCH v10/v9/v8/v7/v6/v5:
> - No change over the PATCH v4
>
> PATCH v4:
> This fixes the comments given by Christoph Hellwig over the PATCH v3:
> Link: https://lkml.org/lkml/2016/3/22/609
>
> PATCH V3:
> This fixes the comments given by Leon Romanovsky over the PATCH v2:
> Link: https://lkml.org/lkml/2016/3/20/5
>
> PATCH v2:
> This fixes the comments given by Leon Romanovsky over the PATCH v1:
> Link: https://lkml.org/lkml/2016/3/6/94
> Fixes the error tested by kbuild test robot over the PATCH v1:
> Link: https://lkml.org/lkml/2016/3/4/343
>
> PATCH v1:
> - The initial patch
> ---
> ---
> drivers/infiniband/Kconfig | 1 +
> drivers/infiniband/hw/Makefile | 1 +
> drivers/infiniband/hw/hns/Kconfig | 10 ++++++++++
> drivers/infiniband/hw/hns/Makefile | 8 ++++++++
> 4 files changed, 20 insertions(+)
> create mode 100644 drivers/infiniband/hw/hns/Kconfig
> create mode 100644 drivers/infiniband/hw/hns/Makefile
>
> diff --git a/drivers/infiniband/Kconfig b/drivers/infiniband/Kconfig
> index 2137adf..767f92b 100644
> --- a/drivers/infiniband/Kconfig
> +++ b/drivers/infiniband/Kconfig
> @@ -74,6 +74,7 @@ source "drivers/infiniband/hw/mlx5/Kconfig"
> source "drivers/infiniband/hw/nes/Kconfig"
> source "drivers/infiniband/hw/ocrdma/Kconfig"
> source "drivers/infiniband/hw/usnic/Kconfig"
> +source "drivers/infiniband/hw/hns/Kconfig"
>
> source "drivers/infiniband/ulp/ipoib/Kconfig"
>
> diff --git a/drivers/infiniband/hw/Makefile b/drivers/infiniband/hw/Makefile
> index c0c7cf8..2ad851d 100644
> --- a/drivers/infiniband/hw/Makefile
> +++ b/drivers/infiniband/hw/Makefile
> @@ -9,3 +9,4 @@ obj-$(CONFIG_INFINIBAND_NES) += nes/
> obj-$(CONFIG_INFINIBAND_OCRDMA) += ocrdma/
> obj-$(CONFIG_INFINIBAND_USNIC) += usnic/
> obj-$(CONFIG_INFINIBAND_HFI1) += hfi1/
> +obj-$(CONFIG_INFINIBAND_HISILICON_HNS) += hns/
--------------------------^^^^^^^--
There is no need in HISILICON word here.
> diff --git a/drivers/infiniband/hw/hns/Kconfig b/drivers/infiniband/hw/hns/Kconfig
> new file mode 100644
> index 0000000..c47c168
> --- /dev/null
> +++ b/drivers/infiniband/hw/hns/Kconfig
> @@ -0,0 +1,10 @@
> +config INFINIBAND_HISILICON_HNS
> + tristate "Hisilicon Hns ROCE Driver"
And you are still inconsistent with the names
Hisilicon/HiSilicon/hisilicon/HISILICON/e.t.c., ROCE/roce/RoCE/e.t.c.
Download attachment "signature.asc" of type "application/pgp-signature" (820 bytes)
Powered by blists - more mailing lists