On 3/29/2018 5:40 PM, Ajit Khaparde wrote: > From: Scott Branden <scott.bran...@broadcom.com> > > Update the license header on bnxt files to be the standard > BSD-3-Clause license used for the rest of DPDK, > bring the files in compliance with the DPDK licensing policy. > > Signed-off-by: Scott Branden <scott.bran...@broadcom.com> > Signed-off-by: Ajit Khaparde <ajit.khapa...@broadcom.com> > Cc:hemant.agra...@nxp.com > ---- > v1->v2: modify first line of SPDX identifer as per review comment. > --- > doc/guides/nics/bnxt.rst | 30 ++---------------------------- > drivers/net/bnxt/bnxt.h | 32 ++------------------------------ > drivers/net/bnxt/bnxt_cpr.c | 32 ++------------------------------ > drivers/net/bnxt/bnxt_cpr.h | 32 ++------------------------------ > drivers/net/bnxt/bnxt_ethdev.c | 32 ++------------------------------ > drivers/net/bnxt/bnxt_filter.c | 32 ++------------------------------ > drivers/net/bnxt/bnxt_filter.h | 32 ++------------------------------ > drivers/net/bnxt/bnxt_hwrm.c | 32 ++------------------------------ > drivers/net/bnxt/bnxt_hwrm.h | 32 ++------------------------------ > drivers/net/bnxt/bnxt_irq.c | 32 ++------------------------------ > drivers/net/bnxt/bnxt_irq.h | 32 ++------------------------------ > drivers/net/bnxt/bnxt_nvm_defs.h | 11 +++-------- > drivers/net/bnxt/bnxt_ring.c | 32 ++------------------------------ > drivers/net/bnxt/bnxt_ring.h | 32 ++------------------------------ > drivers/net/bnxt/bnxt_rxq.c | 32 ++------------------------------ > drivers/net/bnxt/bnxt_rxq.h | 32 ++------------------------------ > drivers/net/bnxt/bnxt_rxr.c | 32 ++------------------------------ > drivers/net/bnxt/bnxt_rxr.h | 32 ++------------------------------ > drivers/net/bnxt/bnxt_stats.c | 32 ++------------------------------ > drivers/net/bnxt/bnxt_stats.h | 32 ++------------------------------ > drivers/net/bnxt/bnxt_txq.c | 32 ++------------------------------ > drivers/net/bnxt/bnxt_txq.h | 32 ++------------------------------ > drivers/net/bnxt/bnxt_txr.c | 32 ++------------------------------ > drivers/net/bnxt/bnxt_txr.h | 32 ++------------------------------ > drivers/net/bnxt/bnxt_vnic.c | 32 ++------------------------------ > drivers/net/bnxt/bnxt_vnic.h | 32 ++------------------------------ > drivers/net/bnxt/hsi_struct_def_dpdk.h | 32 ++------------------------------ > drivers/net/bnxt/rte_pmd_bnxt.c | 32 ++------------------------------ > drivers/net/bnxt/rte_pmd_bnxt.h | 32 ++------------------------------ > 29 files changed, 59 insertions(+), 846 deletions(-)
Can you please update Makefile too, but I can see it has multiple coypright, it can be better to send it as a separate patch to not block this one. <...> > @@ -1,34 +1,6 @@ > -/*- > - * BSD LICENSE > - * > - * Copyright(c) Broadcom Limited. > +/* SPDX-License-Identifier: BSD-3-Clause > + * Copyright(c) 2014-2018 Broadcom > * All rights reserved. It might be nit-picking but we don't have leading spaces [1] in format, are they intentional? If not it is easy to fix when introduced otherwise will stay as it is forever J [1] /* SPDX-License-Identifier: BSD-3-Clause * Copyright(c) 2014-2018 Broadcom * All rights reserved. */ <...>
