On 20/03/2024 09:24, Ghennadi Procopciuc wrote: > 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. submitting-patches describes the process, thus also patches, but not the Linux code. Whatever is in the C/H/DTS/SH/COCCI code, is independent. Best regards, Krzysztof
