From: Greg KH <gregkh@linuxfoundation.org>
To: Razvan Stefanescu <razvan.stefanescu@nxp.com>
Cc: devel@driverdev.osuosl.org, stuyoder@gmail.com, arnd@arndb.de,
netdev@vger.kernel.org, alexandru.marginean@nxp.com,
linux-kernel@vger.kernel.org, agraf@suse.de,
ioana.ciornei@nxp.com, laurentiu.tudor@nxp.com
Subject: Re: [PATCH v5 1/6] staging: fsl-dpaa2/ethsw: Add APIs for DPSW object
Date: Wed, 14 Mar 2018 13:17:59 +0100 [thread overview]
Message-ID: <20180314121759.GA22695@kroah.com> (raw)
In-Reply-To: <20180313135156.3322-2-razvan.stefanescu@nxp.com>
On Tue, Mar 13, 2018 at 08:51:51AM -0500, Razvan Stefanescu wrote:
> Add the command build/parse APIs for operating on DPSW objects through
> the DPAA2 Management Complex.
>
> Signed-off-by: Razvan Stefanescu <razvan.stefanescu@nxp.com>
> ---
> Changelog:
> v2:
> - use u8 for en parameter of dpsw_if_set_flooding/broadcast()
> v3:
> - no changes
> v4:
> - adjust to moving MC-bus out of staging
> - fix sparse warnings
> v5:
> - no changes
>
> drivers/staging/fsl-dpaa2/Kconfig | 8 +
> drivers/staging/fsl-dpaa2/Makefile | 1 +
> drivers/staging/fsl-dpaa2/ethsw/Makefile | 7 +
> drivers/staging/fsl-dpaa2/ethsw/dpsw-cmd.h | 358 +++++++++
> drivers/staging/fsl-dpaa2/ethsw/dpsw.c | 1116 ++++++++++++++++++++++++++++
> drivers/staging/fsl-dpaa2/ethsw/dpsw.h | 579 +++++++++++++++
> 6 files changed, 2069 insertions(+)
> create mode 100644 drivers/staging/fsl-dpaa2/ethsw/Makefile
> create mode 100644 drivers/staging/fsl-dpaa2/ethsw/dpsw-cmd.h
> create mode 100644 drivers/staging/fsl-dpaa2/ethsw/dpsw.c
> create mode 100644 drivers/staging/fsl-dpaa2/ethsw/dpsw.h
>
> diff --git a/drivers/staging/fsl-dpaa2/Kconfig b/drivers/staging/fsl-dpaa2/Kconfig
> index dfff675..8a508ef 100644
> --- a/drivers/staging/fsl-dpaa2/Kconfig
> +++ b/drivers/staging/fsl-dpaa2/Kconfig
> @@ -16,3 +16,11 @@ config FSL_DPAA2_ETH
> ---help---
> Ethernet driver for Freescale DPAA2 SoCs, using the
> Freescale MC bus driver
> +
> +config FSL_DPAA2_ETHSW
> + tristate "Freescale DPAA2 Ethernet Switch"
> + depends on FSL_DPAA2
> + depends on NET_SWITCHDEV
> + ---help---
> + Driver for Freescale DPAA2 Ethernet Switch. Select
> + BRIDGE to have support for bridge tools.
> diff --git a/drivers/staging/fsl-dpaa2/Makefile b/drivers/staging/fsl-dpaa2/Makefile
> index 0836ba8..6cfd76b 100644
> --- a/drivers/staging/fsl-dpaa2/Makefile
> +++ b/drivers/staging/fsl-dpaa2/Makefile
> @@ -3,3 +3,4 @@
> #
>
> obj-$(CONFIG_FSL_DPAA2_ETH) += ethernet/
> +obj-$(CONFIG_FSL_DPAA2_ETHSW) += ethsw/
> diff --git a/drivers/staging/fsl-dpaa2/ethsw/Makefile b/drivers/staging/fsl-dpaa2/ethsw/Makefile
> new file mode 100644
> index 0000000..db137f7
> --- /dev/null
> +++ b/drivers/staging/fsl-dpaa2/ethsw/Makefile
> @@ -0,0 +1,7 @@
> +#
> +# Makefile for the Freescale DPAA2 Ethernet Switch
> +#
> +
> +obj-$(CONFIG_FSL_DPAA2_ETHSW) += dpaa2-ethsw.o
> +
> +dpaa2-ethsw-objs := dpsw.o
> diff --git a/drivers/staging/fsl-dpaa2/ethsw/dpsw-cmd.h b/drivers/staging/fsl-dpaa2/ethsw/dpsw-cmd.h
> new file mode 100644
> index 0000000..36edef6
> --- /dev/null
> +++ b/drivers/staging/fsl-dpaa2/ethsw/dpsw-cmd.h
> @@ -0,0 +1,358 @@
> +/* Copyright 2013-2016 Freescale Semiconductor Inc.
> + * Copyright 2017-2018 NXP
> + *
> + * Redistribution and use in source and binary forms, with or without
> + * modification, are permitted provided that the following conditions are met:
> + * * Redistributions of source code must retain the above copyright
> + * notice, this list of conditions and the following disclaimer.
> + * * Redistributions in binary form must reproduce the above copyright
> + * notice, this list of conditions and the following disclaimer in the
> + * documentation and/or other materials provided with the distribution.
> + * * Neither the name of the above-listed copyright holders nor the
> + * names of any contributors may be used to endorse or promote products
> + * derived from this software without specific prior written permission.
> + *
> + *
> + * ALTERNATIVELY, this software may be distributed under the terms of the
> + * GNU General Public License ("GPL") as published by the Free Software
> + * Foundation, either version 2 of that License or (at your option) any
> + * later version.
> + *
> + * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
> + * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
> + * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
> + * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDERS OR CONTRIBUTORS BE
> + * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
> + * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
> + * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
> + * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
> + * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
> + * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
> + * POSSIBILITY OF SUCH DAMAGE.
> + */
Can you resend this series and just use the correct SPDX identifiers for
all of the new files, instead of all of this horrid boiler-plate code?
That will save me time when I have to go delete all of this in the near
future :)
Also, why dual license it? Are you _SURE_ you want to do that, and are
totally aware of all of the crazy issues surrounding it? Hint, it
almost never means what you might think it does, and I have yet to know
of anyone doing anything "real" with any dual-licensed Linux kernel
code.
Remember, BSD is not the license you want/need for GPL header files that
are exposed to userspace...
thanks,
greg k-h
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
next prev parent reply other threads:[~2018-03-14 12:17 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-13 13:51 [PATCH v5 0/6] staging: Introduce DPAA2 Ethernet Switch driver Razvan Stefanescu
2018-03-13 13:51 ` [PATCH v5 1/6] staging: fsl-dpaa2/ethsw: Add APIs for DPSW object Razvan Stefanescu
2018-03-14 12:17 ` Greg KH [this message]
2018-03-14 14:30 ` Razvan Stefanescu
2018-03-13 13:51 ` [PATCH v5 2/6] staging: fsl-dpaa2/ethsw: Add Freescale DPAA2 Ethernet Switch driver Razvan Stefanescu
2018-03-13 14:22 ` Andrew Lunn
2018-03-13 15:20 ` Razvan Stefanescu
2018-03-13 15:26 ` Andrew Lunn
2018-03-14 14:27 ` Razvan Stefanescu
2018-03-13 13:51 ` [PATCH v5 3/6] staging: fsl-dpaa2/ethsw: Add ethtool support Razvan Stefanescu
2018-03-13 13:51 ` [PATCH v5 4/6] staging: fsl-dpaa2/ethsw: Add maintainer for Ethernet Switch driver Razvan Stefanescu
2018-03-13 13:51 ` [PATCH v5 5/6] staging: fsl-dpaa2/ethsw: Add README Razvan Stefanescu
2018-03-13 13:51 ` [PATCH v5 6/6] staging: fsl-dpaa2/ethsw: Add TODO Razvan Stefanescu
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180314121759.GA22695@kroah.com \
--to=gregkh@linuxfoundation.org \
--cc=agraf@suse.de \
--cc=alexandru.marginean@nxp.com \
--cc=arnd@arndb.de \
--cc=devel@driverdev.osuosl.org \
--cc=ioana.ciornei@nxp.com \
--cc=laurentiu.tudor@nxp.com \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=razvan.stefanescu@nxp.com \
--cc=stuyoder@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).