netdev - Re: [PATCH net-next v8 2/4] net: netcp: Add Keystone NetCP core ethernet driver

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: <1421741935.9655.73.camel@x220>
Date:	Tue, 20 Jan 2015 09:18:55 +0100
From:	Paul Bolle <pebolle@...cali.nl>
To:	Murali Karicheri <m-karicheri2@...com>,
	Wingman Kwok <w-kwok2@...com>
Cc:	Valentin Rothberg <valentinrothberg@...il.com>,
	davem@...emloft.net, devicetree@...r.kernel.org,
	linux-kernel@...r.kernel.org, netdev@...r.kernel.org
Subject: Re: [PATCH net-next v8 2/4] net: netcp: Add Keystone NetCP core
 ethernet driver

Murali, Wingman,

On Thu, 2015-01-15 at 19:12 -0500, Murali Karicheri wrote:
> The network coprocessor (NetCP) is a hardware accelerator available in
> Keystone SoCs that processes Ethernet packets. NetCP consists of following
> hardware components
> 
>  1 Gigabit Ethernet (GbE) subsystem with a Ethernet switch sub-module to
>    send and receive packets.
>  2 Packet Accelerator (PA) module to perform packet classification
>    operations such as header matching, and packet modification operations
>    such as checksum generation.
>  3 Security Accelerator(SA) capable of performing IPSec operations on
>    ingress/egress packets.
>  4 An optional 10 Gigabit Ethernet Subsystem (XGbE) which includes a
>    3-port Ethernet switch sub-module capable of 10Gb/s and 1Gb/s rates
>    per Ethernet port.
>  5 Packet DMA and Queue Management Subsystem (QMSS) to enqueue and dequeue
>    packets and DMA the packets between memory and NetCP hardware components
>    described above.
> 
> NetCP core driver make use of the Keystone Navigator driver API to allocate
> DMA channel for the Ethenet device and to handle packet queue/de-queue,
> Please refer API's in include/linux/soc/ti/knav_dma.h and
> drivers/soc/ti/knav_qmss.h for details.
> 
> NetCP driver consists of NetCP core driver and at a minimum Gigabit
> Ethernet (GBE) module (1) driver to implement the Network device function.
> Other modules (2,3) can be optionally added to achieve supported hardware
> acceleration function. The initial version of the driver include NetCP
> core driver and GBE driver modules.
> 
> Please refer Documentation/devicetree/bindings/net/keystone-netcp.txt
> for design of the driver.
> 
>  Cc: David Miller <davem@...emloft.net>
>  Cc: Rob Herring <robh+dt@...nel.org>
>  Cc: Grant Likely <grant.likely@...aro.org>
>  Cc: Santosh Shilimkar <santosh.shilimkar@...nel.org>
>  Cc: Pawel Moll <pawel.moll@....com>
>  Cc: Mark Rutland <mark.rutland@....com>
>  Cc: Ian Campbell <ijc+devicetree@...lion.org.uk>
>  Cc: Kumar Gala <galak@...eaurora.org>
> 
> Signed-off-by: Murali Karicheri <m-karicheri2@...com>
> Signed-off-by: Wingman Kwok <w-kwok2@...com>

This patch is included as commit 84640e27f230 ("net: netcp: Add Keystone
NetCP core ethernet driver") in today's linux-next. I noticed because a
script I run on linux-next spotted a problem in it.

> ---
>  MAINTAINERS                          |    7 +
>  drivers/net/ethernet/ti/Kconfig      |   11 +
>  drivers/net/ethernet/ti/Makefile     |    3 +
>  drivers/net/ethernet/ti/netcp.h      |  229 ++++
>  drivers/net/ethernet/ti/netcp_core.c | 2141 ++++++++++++++++++++++++++++++++++
>  5 files changed, 2391 insertions(+)
>  create mode 100644 drivers/net/ethernet/ti/netcp.h
>  create mode 100644 drivers/net/ethernet/ti/netcp_core.c
> 
>[...]
> diff --git a/drivers/net/ethernet/ti/Kconfig b/drivers/net/ethernet/ti/Kconfig
> index 605dd90..e11bcfa 100644
> --- a/drivers/net/ethernet/ti/Kconfig
> +++ b/drivers/net/ethernet/ti/Kconfig
> @@ -73,12 +73,23 @@ config TI_CPSW
>  config TI_CPTS
>  	boolean "TI Common Platform Time Sync (CPTS) Support"
>  	depends on TI_CPSW
> +	depends on TI_CPSW || TI_KEYSTONE_NET

You probably meant to add
    || TI_KEYSTONE_NETCP

here. Ie, add CP. But as this slipped through testing it _might_ not be
needed at all.

>  	select PTP_1588_CLOCK
>  	---help---
>  	  This driver supports the Common Platform Time Sync unit of
>  	  the CPSW Ethernet Switch. The unit can time stamp PTP UDP/IPv4
>  	  and Layer 2 packets, and the driver offers a PTP Hardware Clock.
>  
> +config TI_KEYSTONE_NETCP
> +	tristate "TI Keystone NETCP Ethernet subsystem Support"
> +	depends on OF
> +	depends on KEYSTONE_NAVIGATOR_DMA && KEYSTONE_NAVIGATOR_QMSS
> +	---help---
> +	  This driver supports TI's Keystone NETCP Ethernet subsystem.
> +
> +	  To compile this driver as a module, choose M here: the module
> +	  will be called keystone_netcp.
> +
>  config TLAN
>  	tristate "TI ThunderLAN support"
>  	depends on (PCI || EISA)

Thanks,


Paul Bolle

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html