Hi Cédric, Thank you for the quick review and for suggesting the SPDX approach.
On Sat, Jun 14, 2025 at 1:36 AM Cédric Le Goater <c...@kaod.org> wrote: > > Instead, I would remove the license boiler plate and add : > > SPDX-License-Identifier: GPL-2.0-or-later > Before we respin the patch, could you please confirm the scope below? 1. For the two headers currently under review (aspeed_i2c.h and aspeed_timer.h) I will drop the multi-line GPL boiler-plate and replace it with SPDX line, while keeping the copyright statements. 2. For the other ASPEED-maintained files hw/*/*aspeed* include/hw/*/*aspeed* tests/*/*aspeed* Would you prefer to convert them as well? If so, I'd prefer to send a dedicated patch series after this one. 3. Replacement rule: * multi-line GPL boiler-plate -> replace with SPDX * existing short two-line notice This code is licensed under the GPL version 2 or later. See the COPYING file in the top-level directory. -> leave unchanged Is that correct? 4. If we do the wider sweep, should MIT-licensed ASPEED files also be updated by replacing their boiler-plate with SPDX identifier in the same patch series? Please let me know what works best for you and I will send v2 accordingly. Thanks again for the guidance, Sean Wei