On 3/20/24 09:49, Krzysztof Kozlowski wrote: > On 20/03/2024 08:00, Ghennadi Procopciuc wrote: >> On 3/20/24 00:16, Wadim Mueller wrote: >> [...] >>> +++ b/arch/arm64/boot/dts/freescale/s32g3.dtsi >>> @@ -0,0 +1,236 @@ >>> +// SPDX-License-Identifier: GPL-2.0-or-later >>> +/* >>> + * Copyright 2021-2023 NXP >>> + * >>> + * Authors: Ghennadi Procopciuc <ghennadi.procopciuc@xxxxxxx> >>> + * Ciprian Costea <ciprianmarian.costea@xxxxxxx> >>> + * Andra-Teodora Ilie <andra.ilie@xxxxxxx> >> >> This pollutes the content of the file. Please make them part of the >> commit description using 'Signed-off-by' tags. >> > > No, that's not how process works. SoB are part of DCO and Ghennadi is > allowed to skip them. Just read the DCO. Don't add fake SoB tags just > because someone wants... > > Nothing is polluted in the file. That's what this section you have (if > someone wants). > I apologize for not getting it right earlier. After reviewing the information available at [0], I noticed that it suggests using Co-developed-by and Signed-off-by when a patch has multiple contributors. However, I could not find any mention of the 'Authors' section in the file although it seems to be a common practice. [0] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst#n491 Regards, Ghennadi
