netdev - RE: [PATCH 2/2] net: ethernet: Add driver for Sunplus SP7021

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: <36d5bc6d40734ae0a9c1fb26d258f49f@sphcmbx02.sunplus.com.tw>
Date:   Thu, 4 Nov 2021 05:31:57 +0000
From:   Wells Lu 呂芳騰 <wells.lu@...plus.com>
To:     Andrew Lunn <andrew@...n.ch>
CC:     Randy Dunlap <rdunlap@...radead.org>,
        Wells Lu <wellslutw@...il.com>,
        "davem@...emloft.net" <davem@...emloft.net>,
        "kuba@...nel.org" <kuba@...nel.org>,
        "robh+dt@...nel.org" <robh+dt@...nel.org>,
        "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "p.zabel@...gutronix.de" <p.zabel@...gutronix.de>
Subject: RE: [PATCH 2/2] net: ethernet: Add driver for Sunplus SP7021

Hi,

Thanks a lot for review.

> 
> > config NET_VENDOR_SUNPLUS
> > 	bool "Sunplus devices"
> > 	default y
> > 	depends on ARCH_SUNPLUS
> 
> Does it actually depend on ARCH_SUNPLUS? What do you make use of?

ARCH_SUNPLUS will be defined for Sunplus family series SoC.
Ethernet devices of Sunplus are designed and used for Sunplus SoC.
So far, only two SoC of Sunplus have the network device.
I'd like to show up the selection only for Sunplus SoC.

> 
> Ideally, you want it to also build with COMPILE_TEST, so that the driver gets
> build by 0-day and all the other build bots.

I am not sure if this is mandatory or not.
Should I add COMPILE_TEST as below?

	depends on ARCH_SUNPLUS | COMPILE_TEST

> 
> > 	---help---
> > 	  If you have a network (Ethernet) card belonging to this
> > 	  class, say Y here.
> >
> > 	  Note that the answer to this question doesn't directly
> > 	  affect the kernel: saying N will just cause the configurator
> > 	  to skip all the questions about Sunplus cards. If you say Y,
> > 	  you will be asked for your specific card in the following
> > 	  questions.
> >
> > if NET_VENDOR_SUNPLUS
> >
> > config SP7021_EMAC
> > 	tristate "Sunplus Dual 10M/100M Ethernet (with L2 switch) devices"
> > 	depends on ETHERNET && SOC_SP7021
> 
> Does it actually depend on SOC_SP7021 to build?
> 
>      Andrew

Yes, the device is now only for Sunplus SP7021 SoC.
Devices in each SoC may have a bit difference because of adding new 
function or improving something.