linux-kernel - Re: [PATCH] MAINTAINERS: ARM: EXYNOS: Add Krzysztof Kozlowski asco-maintainer

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 for Android: free password hash cracker in your pocket

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

Date:	Thu, 14 May 2015 16:06:44 +0200
From:	Tobias Jakobi <liquid.acid@....net>
To:	Javier Martinez Canillas <javier@...hile0.org>,
	Kukjin Kim <kgene@...nel.org>
CC:	Krzysztof Kozlowski <k.kozlowski@...sung.com>,
	Linux Kernel <linux-kernel@...r.kernel.org>,
	"linux-samsung-soc@...r.kernel.org" 
	<linux-samsung-soc@...r.kernel.org>,
	Russell King <linux@....linux.org.uk>,
	Arnd Bergmann <arnd@...db.de>,
	Kevin Hilman <khilman@...nel.org>,
	Olof Johansson <olof@...om.net>,
	"linux-arm-kernel@...ts.infradead.org" 
	<linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH] MAINTAINERS: ARM: EXYNOS: Add Krzysztof Kozlowski
 asco-maintainer

Javier Martinez Canillas wrote:
> Hello Kukjin,
> 
> On Thu, May 14, 2015 at 3:00 PM, Kukjin Kim <kgene@...nel.org> wrote:
>> On 05/14/15 21:15, Krzysztof Kozlowski wrote:
>>> Add Krzysztof Kozlowski as a co-maintainer of Samsung Exynos ARM
>>> architecture to review the patches. Patches will go as usual - picked up
>>> by Kukjin Kim.
>>>
>> Hi Krzysztof,
>>
>> Sorry, I don't agree with this patch because you can review any exynos
>> patches without maintainer title. And I can focus on the maintainer'r
>> role more than now because of change of role ;-) Of course your review
>> would be helpful to me.
>>
> 
> Sorry but I disagree. Krzysztof has been very active reviewing Exynos
> patches and even picking/queuing important fixes while it seems you
> were too busy and couldn't pay close attention.
I would even go further than that. We're in this situation because the
current maintainer didn't do his job properly.

I want to point out what gregkh once said about this matter:
https://youtu.be/iiED1K98lnw?t=1745



> It's really great that due a role change you will have more time to
> focus on the maintainership but I don't see why having more
> maintainers could be something negative. As long as you work in a
> coordinate manner, the more maintainers we have the better IMHO. It
> may be that in the future you change your role at your company again
> and then it will be good to have something else to take care of the
> posted patches ;-)
I very much agree with Javier here. Krzysztof promptly responds to
patches and usually provides detailed feedback when things need changes.
It would be stupid not to accept him as an additional maintainer.



With best wishes,
Tobias


> It's true that Krzysztof can review any Exynos patches (like anyone
> subscribed to the lists) without a co-maintainer title, but being
> listed in MAINTAINERS will make sure that people know that he is
> willing to review and ack Exynos patches so they should add him to the
> cc list.
> 
>> Thanks,
>> Kukjin
>>
> 
> Best regards,
> Javier
> --
> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/