From: Stephen Hemminger <stephen@networkplumber.org>
To: Thomas Monjalon <thomas@monjalon.net>
Cc: dev@dpdk.org
Subject: Re: [PATCH v3 0/2] standardize SPDX tag
Date: Wed, 4 Feb 2026 17:41:40 -0800 [thread overview]
Message-ID: <20260204174140.39e39ec1@phoenix.local> (raw)
In-Reply-To: <20260204204945.3893117-1-thomas@monjalon.net>
On Wed, 4 Feb 2026 21:46:04 +0100
Thomas Monjalon <thomas@monjalon.net> wrote:
> SPDX lines are fixed to be more standard,
> and the checker is improved to detect some of the fixed issues.
> For readability, the script changes are in a separate patch,
> but for the sake of atomicity, the patches should be squashed.
>
> v2: update the script and README, and remove useless parentheses
> v3: keep Linux uAPI imported file untouched
>
> Thomas Monjalon (2):
> license: standardize SPDX tag
> devtools: check SPDX syntax
>
> devtools/check-spdx-tag.sh | 33 +++++++++++++++++--
> doc/guides/contributing/patches.rst | 2 +-
> doc/guides/eventdevs/dpaa.rst | 2 +-
> drivers/bus/dpaa/base/fman/fman.c | 2 +-
> drivers/bus/dpaa/base/fman/netcfg_layer.c | 2 +-
> drivers/bus/dpaa/base/qbman/bman.c | 2 +-
> drivers/bus/dpaa/base/qbman/bman.h | 2 +-
> drivers/bus/dpaa/base/qbman/bman_driver.c | 2 +-
> drivers/bus/dpaa/base/qbman/bman_priv.h | 2 +-
> drivers/bus/dpaa/base/qbman/dpaa_alloc.c | 2 +-
> drivers/bus/dpaa/base/qbman/dpaa_sys.c | 2 +-
> drivers/bus/dpaa/base/qbman/dpaa_sys.h | 2 +-
> drivers/bus/dpaa/base/qbman/process.c | 2 +-
> drivers/bus/dpaa/base/qbman/qman.c | 2 +-
> drivers/bus/dpaa/base/qbman/qman.h | 2 +-
> drivers/bus/dpaa/base/qbman/qman_driver.c | 2 +-
> drivers/bus/dpaa/base/qbman/qman_priv.h | 2 +-
> drivers/bus/dpaa/include/fman.h | 2 +-
> drivers/bus/dpaa/include/fsl_bman.h | 2 +-
> drivers/bus/dpaa/include/fsl_fman.h | 2 +-
> drivers/bus/dpaa/include/fsl_fman_crc64.h | 2 +-
> drivers/bus/dpaa/include/fsl_qman.h | 2 +-
> drivers/bus/dpaa/include/fsl_usd.h | 2 +-
> drivers/bus/dpaa/include/netcfg.h | 2 +-
> drivers/bus/dpaa/include/process.h | 2 +-
> drivers/bus/fslmc/mc/dpbp.c | 2 +-
> drivers/bus/fslmc/mc/dpci.c | 2 +-
> drivers/bus/fslmc/mc/dpcon.c | 2 +-
> drivers/bus/fslmc/mc/dpio.c | 2 +-
> drivers/bus/fslmc/mc/dpmng.c | 2 +-
> drivers/bus/fslmc/mc/dprc.c | 2 +-
> drivers/bus/fslmc/mc/fsl_dpbp.h | 2 +-
> drivers/bus/fslmc/mc/fsl_dpbp_cmd.h | 2 +-
> drivers/bus/fslmc/mc/fsl_dpci.h | 2 +-
> drivers/bus/fslmc/mc/fsl_dpci_cmd.h | 2 +-
> drivers/bus/fslmc/mc/fsl_dpcon.h | 2 +-
> drivers/bus/fslmc/mc/fsl_dpcon_cmd.h | 2 +-
> drivers/bus/fslmc/mc/fsl_dpio.h | 2 +-
> drivers/bus/fslmc/mc/fsl_dpio_cmd.h | 2 +-
> drivers/bus/fslmc/mc/fsl_dpmng.h | 2 +-
> drivers/bus/fslmc/mc/fsl_dpmng_cmd.h | 2 +-
> drivers/bus/fslmc/mc/fsl_dpopr.h | 2 +-
> drivers/bus/fslmc/mc/fsl_dprc.h | 2 +-
> drivers/bus/fslmc/mc/fsl_dprc_cmd.h | 2 +-
> drivers/bus/fslmc/mc/fsl_mc_cmd.h | 2 +-
> drivers/bus/fslmc/mc/fsl_mc_sys.h | 2 +-
> drivers/bus/fslmc/mc/mc_sys.c | 2 +-
> drivers/common/dpaax/caamflib.c | 2 +-
> drivers/common/dpaax/caamflib/compat.h | 2 +-
> drivers/common/dpaax/caamflib/desc.h | 2 +-
> drivers/common/dpaax/caamflib/desc/algo.h | 2 +-
> drivers/common/dpaax/caamflib/desc/common.h | 2 +-
> drivers/common/dpaax/caamflib/desc/ipsec.h | 2 +-
> drivers/common/dpaax/caamflib/desc/pdcp.h | 2 +-
> drivers/common/dpaax/caamflib/rta.h | 2 +-
> .../dpaax/caamflib/rta/fifo_load_store_cmd.h | 2 +-
> .../common/dpaax/caamflib/rta/header_cmd.h | 2 +-
> drivers/common/dpaax/caamflib/rta/jump_cmd.h | 2 +-
> drivers/common/dpaax/caamflib/rta/key_cmd.h | 2 +-
> drivers/common/dpaax/caamflib/rta/load_cmd.h | 2 +-
> drivers/common/dpaax/caamflib/rta/math_cmd.h | 2 +-
> drivers/common/dpaax/caamflib/rta/move_cmd.h | 2 +-
> drivers/common/dpaax/caamflib/rta/nfifo_cmd.h | 2 +-
> .../common/dpaax/caamflib/rta/operation_cmd.h | 2 +-
> .../common/dpaax/caamflib/rta/protocol_cmd.h | 2 +-
> .../dpaax/caamflib/rta/sec_run_time_asm.h | 2 +-
> .../dpaax/caamflib/rta/seq_in_out_ptr_cmd.h | 2 +-
> .../common/dpaax/caamflib/rta/signature_cmd.h | 2 +-
> drivers/common/dpaax/caamflib/rta/store_cmd.h | 2 +-
> drivers/common/dpaax/compat.h | 2 +-
> drivers/common/dpaax/dpaa_of.c | 2 +-
> drivers/common/dpaax/dpaa_of.h | 2 +-
> drivers/common/qat/qat_adf/adf_pf2vf_msg.h | 2 +-
> .../qat/qat_adf/adf_transport_access_macros.h | 2 +-
> .../adf_transport_access_macros_gen4.h | 2 +-
> .../adf_transport_access_macros_gen4vf.h | 2 +-
> .../adf_transport_access_macros_gen_lce.h | 2 +-
> .../adf_transport_access_macros_gen_lcevf.h | 2 +-
> drivers/common/qat/qat_adf/icp_qat_fw.h | 2 +-
> drivers/common/qat/qat_adf/icp_qat_fw_comp.h | 2 +-
> drivers/common/qat/qat_adf/icp_qat_fw_la.h | 2 +-
> drivers/common/qat/qat_adf/icp_qat_hw.h | 2 +-
> .../common/qat/qat_adf/icp_qat_hw_gen4_comp.h | 2 +-
> .../qat/qat_adf/icp_qat_hw_gen4_comp_defs.h | 2 +-
> drivers/crypto/dpaa2_sec/mc/dpseci.c | 2 +-
> drivers/crypto/dpaa2_sec/mc/fsl_dpseci.h | 2 +-
> drivers/crypto/dpaa2_sec/mc/fsl_dpseci_cmd.h | 2 +-
> drivers/crypto/qat/qat_sym_session.c | 2 +-
> drivers/event/dpaa/dpaa_eventdev.c | 2 +-
> drivers/event/dpaa/dpaa_eventdev.h | 2 +-
> drivers/net/atlantic/atl_hw_regs.c | 5 +--
> drivers/net/atlantic/atl_hw_regs.h | 5 +--
> drivers/net/atlantic/hw_atl/hw_atl_b0.c | 5 +--
> drivers/net/atlantic/hw_atl/hw_atl_b0.h | 5 +--
> .../net/atlantic/hw_atl/hw_atl_b0_internal.h | 5 +--
> drivers/net/atlantic/hw_atl/hw_atl_llh.c | 5 +--
> drivers/net/atlantic/hw_atl/hw_atl_llh.h | 5 +--
> .../net/atlantic/hw_atl/hw_atl_llh_internal.h | 5 +--
> drivers/net/atlantic/hw_atl/hw_atl_utils.c | 5 +--
> drivers/net/atlantic/hw_atl/hw_atl_utils.h | 5 +--
> .../net/atlantic/hw_atl/hw_atl_utils_fw2x.c | 5 +--
> drivers/net/avp/rte_avp_common.h | 2 +-
> drivers/net/avp/rte_avp_fifo.h | 2 +-
> drivers/net/bnxt/bnxt_nvm_defs.h | 2 +-
> drivers/net/dpaa/fmlib/dpaa_integration.h | 2 +-
> drivers/net/dpaa/fmlib/ncsw_ext.h | 2 +-
> drivers/net/dpaa/fmlib/net_ext.h | 2 +-
> drivers/net/dpaa2/dpaa2_ethdev.c | 8 ++---
> drivers/net/dpaa2/dpaa2_recycle.c | 6 ++--
> drivers/net/dpaa2/dpaa2_sparser.h | 4 +--
> drivers/net/dpaa2/mc/dpdmux.c | 2 +-
> drivers/net/dpaa2/mc/dpkg.c | 2 +-
> drivers/net/dpaa2/mc/dpni.c | 2 +-
> drivers/net/dpaa2/mc/dprtc.c | 2 +-
> drivers/net/dpaa2/mc/fsl_dpdmux.h | 2 +-
> drivers/net/dpaa2/mc/fsl_dpdmux_cmd.h | 2 +-
> drivers/net/dpaa2/mc/fsl_dpkg.h | 2 +-
> drivers/net/dpaa2/mc/fsl_dpmac.h | 2 +-
> drivers/net/dpaa2/mc/fsl_dpni.h | 2 +-
> drivers/net/dpaa2/mc/fsl_dpni_cmd.h | 2 +-
> drivers/net/dpaa2/mc/fsl_dprtc.h | 2 +-
> drivers/net/dpaa2/mc/fsl_dprtc_cmd.h | 2 +-
> drivers/net/dpaa2/mc/fsl_net.h | 2 +-
> drivers/net/mlx5/mlx5_flow_meter.c | 3 +-
> lib/eal/include/rte_pci_dev_feature_defs.h | 2 +-
> lib/eal/include/rte_pci_dev_features.h | 2 +-
> license/README | 4 +--
> 127 files changed, 183 insertions(+), 150 deletions(-)
>
LGTM.
Maybe spdx check script would be easier to manage in python?
Could also use it to flag anything outside the expected.
next prev parent reply other threads:[~2026-02-05 1:41 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-11-12 9:13 [PATCH] license: standardize SPDX tag for 3-Clause BSD Thomas Monjalon
2025-11-12 9:31 ` Bruce Richardson
2025-11-12 9:38 ` Thomas Monjalon
2025-11-12 9:41 ` Bruce Richardson
2025-11-12 9:43 ` Thomas Monjalon
2026-01-14 6:00 ` Stephen Hemminger
2026-01-28 16:20 ` [PATCH v2] license: standardize SPDX tag Thomas Monjalon
2026-01-28 16:30 ` Bruce Richardson
2026-01-28 17:44 ` Thomas Monjalon
2026-01-28 17:46 ` Bruce Richardson
2026-02-04 20:46 ` [PATCH v3 0/2] " Thomas Monjalon
2026-02-04 20:46 ` [PATCH v3 1/2] license: " Thomas Monjalon
2026-02-05 5:34 ` Hemant Agrawal
2026-02-04 20:46 ` [PATCH v3 2/2] devtools: check SPDX syntax Thomas Monjalon
2026-02-05 1:41 ` Stephen Hemminger [this message]
2026-02-05 8:44 ` [PATCH v3 0/2] standardize SPDX tag Bruce Richardson
2026-02-05 14:30 ` Thomas Monjalon
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=20260204174140.39e39ec1@phoenix.local \
--to=stephen@networkplumber.org \
--cc=dev@dpdk.org \
--cc=thomas@monjalon.net \
/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