netdev - Re: [PATCH v3 01/14] net-next/yunsilicon: Add xsc driver basic framework

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20250116105358.GA6206@kernel.org>
Date: Thu, 16 Jan 2025 10:53:58 +0000
From: Simon Horman <horms@...nel.org>
To: tianx <tianx@...silicon.com>
Cc: netdev@...r.kernel.org, leon@...nel.org, andrew+netdev@...n.ch,
	kuba@...nel.org, pabeni@...hat.com, edumazet@...gle.com,
	davem@...emloft.net, jeff.johnson@....qualcomm.com,
	przemyslaw.kitszel@...el.com, weihg@...silicon.com,
	wanry@...silicon.com
Subject: Re: [PATCH v3 01/14] net-next/yunsilicon: Add xsc driver basic
 framework

On Thu, Jan 16, 2025 at 04:04:47PM +0800, tianx wrote:
> On 2025/1/16 0:04, Simon Horman wrote:
> > On Wed, Jan 15, 2025 at 06:22:44PM +0800, Xin Tian wrote:
> >> Add yunsilicon xsc driver basic framework, including xsc_pci driver
> >> and xsc_eth driver
> >>
> >> Co-developed-by: Honggang Wei <weihg@...silicon.com>
> >> Signed-off-by: Honggang Wei <weihg@...silicon.com>
> >> Co-developed-by: Lei Yan <jacky@...silicon.com>
> >> Signed-off-by: Lei Yan <jacky@...silicon.com>
> >> Signed-off-by: Xin Tian <tianx@...silicon.com>
> > ...
> >
> >> diff --git a/drivers/net/ethernet/yunsilicon/xsc/pci/Makefile b/drivers/net/ethernet/yunsilicon/xsc/pci/Makefile
> >> new file mode 100644
> >> index 000000000..709270df8
> >> --- /dev/null
> >> +++ b/drivers/net/ethernet/yunsilicon/xsc/pci/Makefile
> >> @@ -0,0 +1,9 @@
> >> +# SPDX-License-Identifier: GPL-2.0
> >> +# Copyright (C) 2021-2025, Shanghai Yunsilicon Technology Co., Ltd.
> >> +# All rights reserved.
> >> +
> >> +ccflags-y += -I$(srctree)/drivers/net/ethernet/yunsilicon/xsc
> >> +
> >> +obj-$(CONFIG_YUNSILICON_XSC_PCI) += xsc_pci.o
> >> +
> >> +xsc_pci-y := main.o
> >> diff --git a/drivers/net/ethernet/yunsilicon/xsc/pci/main.c b/drivers/net/ethernet/yunsilicon/xsc/pci/main.c
> >> new file mode 100644
> >> index 000000000..4859be58f
> >> --- /dev/null
> >> +++ b/drivers/net/ethernet/yunsilicon/xsc/pci/main.c
> >> @@ -0,0 +1,251 @@
> >> +// SPDX-License-Identifier: GPL-2.0
> >> +/* Copyright (C) 2021-2025, Shanghai Yunsilicon Technology Co., Ltd.
> >> + * All rights reserved.
> >> + */
> >> +
> >> +#include "common/xsc_core.h"
> > Hi Xin Tian, all,
> >
> > Sorry for not noticing this before sending my previous email.
> >
> > Please consider a relative include like the following,
> > rather than the above combined with a -I directive in the Makefile.
> >
> > #include "../common/xsc_core.h"
> >
> > This is common practice in Networking code.
> > And, for one thing, allows the following to work:
> >
> > make drivers/net/ethernet/yunsilicon/xsc/pci/main.o
> 
> Hi, Simon
> 
> I don't fully understand the benefit of using relative includes in this case,
> as|"make drivers/net/ethernet/yunsilicon/xsc/pci/main.o|" already works with the current setup.

Thanks Xin Tian,

Sorry about this. There was an error on my part in exercising this yesterday.

I agree that the above make invocation does indeed work. And I do not have
a strong objection to the current #include scheme at this time.