linux-kernel - Re: [PATCH] MAINTAINERS: Add entry for Samsung Exynos850 SoC

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <d66e2153-d07d-40a9-98d5-742bc04ee2d8@linaro.org>
Date: Mon, 29 Jul 2024 08:27:58 +0200
From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To: Alim Akhtar <alim.akhtar@...sung.com>,
 'Sam Protsenko' <semen.protsenko@...aro.org>
Cc: 'Peter Griffin' <peter.griffin@...aro.org>,
 linux-arm-kernel@...ts.infradead.org, linux-samsung-soc@...r.kernel.org,
 linux-kernel@...r.kernel.org, 'Chanwoo Choi' <cw00.choi@...sung.com>,
 'Marek Szyprowski' <m.szyprowski@...sung.com>,
 'Jaewon Kim' <jaewon02.kim@...sung.com>,
 'Mateusz Majewski' <m.majewski2@...sung.com>,
 'Henrik Grimler' <henrik@...mler.se>,
 'David Virag' <virag.david003@...il.com>,
 'Artur Weber' <aweber.kernel@...il.com>,
 'Raymond Hackley' <raymondhackley@...tonmail.com>
Subject: Re: [PATCH] MAINTAINERS: Add entry for Samsung Exynos850 SoC

On 26/07/2024 18:08, Alim Akhtar wrote:
> 
> 
>> -----Original Message-----
>> From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
>> Sent: Friday, July 26, 2024 8:47 PM
>> To: Sam Protsenko <semen.protsenko@...aro.org>
>> Cc: Alim Akhtar <alim.akhtar@...sung.com>; Peter Griffin
>> <peter.griffin@...aro.org>; linux-arm-kernel@...ts.infradead.org; linux-
>> samsung-soc@...r.kernel.org; linux-kernel@...r.kernel.org; Chanwoo Choi
>> <cw00.choi@...sung.com>; Marek Szyprowski
>> <m.szyprowski@...sung.com>; Peter Griffin <peter.griffin@...aro.org>;
>> Jaewon Kim <jaewon02.kim@...sung.com>; Mateusz Majewski
>> <m.majewski2@...sung.com>; Henrik Grimler <henrik@...mler.se>; David
>> Virag <virag.david003@...il.com>; Artur Weber <aweber.kernel@...il.com>;
>> Raymond Hackley <raymondhackley@...tonmail.com>
>> Subject: Re: [PATCH] MAINTAINERS: Add entry for Samsung Exynos850 SoC
>>
>> On 30/01/2024 08:29, Krzysztof Kozlowski wrote:
>>> On 29/01/2024 21:47, Sam Protsenko wrote:
>>>> Add maintainers entry for the Samsung Exynos850 SoC based platforms.
>>>>
>>>> Signed-off-by: Sam Protsenko <semen.protsenko@...aro.org>
>>>> ---
>>>>  MAINTAINERS | 10 ++++++++++
>>>>  1 file changed, 10 insertions(+)
>>>>
>>>> diff --git a/MAINTAINERS b/MAINTAINERS index
>>>> 939f6dd0ef6a..77c10cc669f8 100644
>>>> --- a/MAINTAINERS
>>>> +++ b/MAINTAINERS
>>>> @@ -19281,6 +19281,16 @@ B:	mailto:linux-samsung-
>> soc@...r.kernel.org
>>>>  F:	Documentation/devicetree/bindings/sound/samsung*
>>>>  F:	sound/soc/samsung/
>>>>
>>>> +SAMSUNG EXYNOS850 SoC SUPPORT
>>>> +M:	Sam Protsenko <semen.protsenko@...aro.org>
>>>> +L:	linux-arm-kernel@...ts.infradead.org (moderated for non-subscribers)
>>>> +L:	linux-samsung-soc@...r.kernel.org
>>>
>>> Sorry, but I am still against individual SoC entries in maintainers,
>>> like I replied multiple times and pointed to the updated
>>> get_maintainers.pl script to fetch emails from boards.
>>
>> I retract my earlier statement.
>>
>> Some background: I was really hoping that scripts/get_maintainers.pl patch,
>> which adds fetching emails from files (e.g. DTS), will be picked up, but it has
>> been few years, few resends and there is no conclusion. I don't think it will be
>> ever merged, thus this email.
>>
>> Since C files do not have in-file "maintainer" entry and particular drivers have
>> MAINTAINERS-file entries, then why DTS should be different?
>>
> Not sure what was the discussion in past, do you think this need some discussion in LPC? 
> 
>> I'll take the patch after merge window.
>>
>> +Cc few other folks,
>>
>> I understand that with lei/lore filters one can easily track patches sent for
>> particular boards or SoCs, but being listed in MAINTAINERS have a bit bigger
>> meaning. Therefore if any of you consider / want to add themself to
>> MAINTAINERS for particular DTS, then go ahead. By DTS I
>> mean: particular boards (e.g. Galaxy Tab 3 family) or particular SoC (e.g.
>> Exynos850 like here).
>>
> Actually it is being confusing. Completely agreed that  "MAINTAINERS have a bit bigger".
> By adding each DTS/SoC/Board MAINTAINERS entry, are those members expected to send pull requests?
> If so, that leads to more confusion and overhead. 

No, just like maintainers of drivers do not send them. The way of
handling patches won't change.

Best regards,
Krzysztof