On Mon, Jan 25, 2021 at 09:29:01AM +0000, Daire.McNamara@xxxxxxxxxxxxx wrote: > Hi Leon, > > thanks. > > Just: > > +builtin_platform_driver(mc_pcie_driver); > > +MODULE_LICENSE("GPL v2"); > to > > +builtin_platform_driver(mc_pcie_driver); > > +MODULE_LICENSE("GPL"); > > or does > +// SPDX-License-Identifier: GPL-2.0 > need to change as well, to: > +// SPDX-License-Identifier: GPL You should change only MODULE_LICENSE(..) and leave SPDX as is. See Documentation/process/license-rules.rst 448 "GPL v2" Same as "GPL". It exists for historic 449 reasons. Thanks