diff mbox

MAINTAINERS: ARM: EXYNOS: Add Krzysztof Kozlowski as co-maintainer

Message ID 1431605720-828-1-git-send-email-k.kozlowski@samsung.com (mailing list archive)
State New, archived
Headers show

Commit Message

Krzysztof Kozlowski May 14, 2015, 12:15 p.m. UTC
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.

Cc: Russell King <linux@arm.linux.org.uk>
Cc: Kukjin Kim <kgene@kernel.org>
Cc: Kevin Hilman <khilman@kernel.org>
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: Olof Johansson <olof@lixom.net>
Cc: linux-samsung-soc@vger.kernel.org
Cc: linux-arm-kernel@lists.infradead.org
Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

Comments

kgene@kernel.org May 14, 2015, 1 p.m. UTC | #1
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.

Thanks,
Kukjin


> Cc: Russell King <linux@arm.linux.org.uk>
> Cc: Kukjin Kim <kgene@kernel.org>
> Cc: Kevin Hilman <khilman@kernel.org>
> Cc: Arnd Bergmann <arnd@arndb.de>
> Cc: Olof Johansson <olof@lixom.net>
> Cc: linux-samsung-soc@vger.kernel.org
> Cc: linux-arm-kernel@lists.infradead.org
> Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
> ---
>  MAINTAINERS | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 2dc517cd4835..a3cb77c0ae23 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1380,6 +1380,7 @@ N:	rockchip
>  
>  ARM/SAMSUNG EXYNOS ARM ARCHITECTURES
>  M:	Kukjin Kim <kgene@kernel.org>
> +M:	Krzysztof Kozlowski <k.kozlowski@samsung.com>
>  L:	linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
>  L:	linux-samsung-soc@vger.kernel.org (moderated for non-subscribers)
>  S:	Maintained
Javier Martinez Canillas May 14, 2015, 1:20 p.m. UTC | #2
Hello Kukjin,

On Thu, May 14, 2015 at 3:00 PM, Kukjin Kim <kgene@kernel.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.

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 ;-)

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
Tobias Jakobi May 14, 2015, 2:06 p.m. UTC | #3
Javier Martinez Canillas wrote:
> Hello Kukjin,
> 
> On Thu, May 14, 2015 at 3:00 PM, Kukjin Kim <kgene@kernel.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@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
Kevin Hilman May 14, 2015, 3:52 p.m. UTC | #4
Tobias Jakobi <liquid.acid@gmx.net> writes:

> Javier Martinez Canillas wrote:
>> Hello Kukjin,
>> 
>> On Thu, May 14, 2015 at 3:00 PM, Kukjin Kim <kgene@kernel.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.

I'll take this as an Acked-by then.

FWIW, I agree with Tobias and Javier, extra help for exynos has been
sorely needed for a long time, and I am very happy to see someone
stepping up to help.  Thank you Krzysztof!  (and I apologize in advance
for the many times I'm likely to misspell your name ;)

I'll be applying this to the arm-soc tree,

Thanks,

Kevin
Olof Johansson May 14, 2015, 4:45 p.m. UTC | #5
On Thu, May 14, 2015 at 8:52 AM, Kevin Hilman <khilman@kernel.org> wrote:
> Tobias Jakobi <liquid.acid@gmx.net> writes:
>
>> Javier Martinez Canillas wrote:
>>> Hello Kukjin,
>>>
>>> On Thu, May 14, 2015 at 3:00 PM, Kukjin Kim <kgene@kernel.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.
>
> I'll take this as an Acked-by then.
>
> FWIW, I agree with Tobias and Javier, extra help for exynos has been
> sorely needed for a long time, and I am very happy to see someone
> stepping up to help.  Thank you Krzysztof!  (and I apologize in advance
> for the many times I'm likely to misspell your name ;)
>
> I'll be applying this to the arm-soc tree,

I agree, I think it's a great improvement to get another official
maintainer of Samsung's platforms that can share the load and send us
code.

While I often also use the argument that anyone can help review code
without being named a maintainer, in the particular case of Exynos
platforms there's a dire need for more close attention: They're
frequently broken for long periods of time and there's been high
latency for fixes, which has made bisecting hard or impossible for new
failures. Naming Krzysztof as co-maintainer makes it more clear that
we're happy to merge code from him directly as appropriate for latency
reasons and so on.

So:

Acked-by: Olof Johansson <olof@lixom.net>


-Olof
Krzysztof Kozlowski May 14, 2015, 11:50 p.m. UTC | #6
2015-05-15 1:45 GMT+09:00 Olof Johansson <olof@lixom.net>:
> On Thu, May 14, 2015 at 8:52 AM, Kevin Hilman <khilman@kernel.org> wrote:
>> Tobias Jakobi <liquid.acid@gmx.net> writes:
>>
>>> Javier Martinez Canillas wrote:
>>>> Hello Kukjin,
>>>>
>>>> On Thu, May 14, 2015 at 3:00 PM, Kukjin Kim <kgene@kernel.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.
>>
>> I'll take this as an Acked-by then.
>>
>> FWIW, I agree with Tobias and Javier, extra help for exynos has been
>> sorely needed for a long time, and I am very happy to see someone
>> stepping up to help.  Thank you Krzysztof!  (and I apologize in advance
>> for the many times I'm likely to misspell your name ;)
>>
>> I'll be applying this to the arm-soc tree,
>
> I agree, I think it's a great improvement to get another official
> maintainer of Samsung's platforms that can share the load and send us
> code.
>
> While I often also use the argument that anyone can help review code
> without being named a maintainer, in the particular case of Exynos
> platforms there's a dire need for more close attention: They're
> frequently broken for long periods of time and there's been high
> latency for fixes, which has made bisecting hard or impossible for new
> failures. Naming Krzysztof as co-maintainer makes it more clear that
> we're happy to merge code from him directly as appropriate for latency
> reasons and so on.
>
> So:
>
> Acked-by: Olof Johansson <olof@lixom.net>

Thank you Javier, Tobias, Kevin and Olof for your support.
Receiving patches directly to the mailbox should help in smooth review
although my gmail filters work quite good to grab them from the lists.

P.S. Kevin, the hardest part would be to pronounce my name, but
fortunately there is a nice English version of it - Christopher. :)

Best regards,
Krzysztof
Kevin Hilman May 15, 2015, 2:15 p.m. UTC | #7
On Thu, May 14, 2015 at 4:50 PM, Krzysztof Kozlowski
<k.kozlowski@samsung.com> wrote:
> 2015-05-15 1:45 GMT+09:00 Olof Johansson <olof@lixom.net>:
>> On Thu, May 14, 2015 at 8:52 AM, Kevin Hilman <khilman@kernel.org> wrote:
>>> Tobias Jakobi <liquid.acid@gmx.net> writes:
>>>
>>>> Javier Martinez Canillas wrote:
>>>>> Hello Kukjin,
>>>>>
>>>>> On Thu, May 14, 2015 at 3:00 PM, Kukjin Kim <kgene@kernel.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.
>>>
>>> I'll take this as an Acked-by then.
>>>
>>> FWIW, I agree with Tobias and Javier, extra help for exynos has been
>>> sorely needed for a long time, and I am very happy to see someone
>>> stepping up to help.  Thank you Krzysztof!  (and I apologize in advance
>>> for the many times I'm likely to misspell your name ;)
>>>
>>> I'll be applying this to the arm-soc tree,
>>
>> I agree, I think it's a great improvement to get another official
>> maintainer of Samsung's platforms that can share the load and send us
>> code.
>>
>> While I often also use the argument that anyone can help review code
>> without being named a maintainer, in the particular case of Exynos
>> platforms there's a dire need for more close attention: They're
>> frequently broken for long periods of time and there's been high
>> latency for fixes, which has made bisecting hard or impossible for new
>> failures. Naming Krzysztof as co-maintainer makes it more clear that
>> we're happy to merge code from him directly as appropriate for latency
>> reasons and so on.
>>
>> So:
>>
>> Acked-by: Olof Johansson <olof@lixom.net>
>
> Thank you Javier, Tobias, Kevin and Olof for your support.
> Receiving patches directly to the mailbox should help in smooth review
> although my gmail filters work quite good to grab them from the lists.
>
> P.S. Kevin, the hardest part would be to pronounce my name,

Hopefully we get to meet in person some day and you can teach me. :)

Anyways, this patch has now been applied to arm-soc/fixes and merged
into arm-soc/for-next.  Thanks again for stepping up to help!

Kevin
diff mbox

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index 2dc517cd4835..a3cb77c0ae23 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1380,6 +1380,7 @@  N:	rockchip
 
 ARM/SAMSUNG EXYNOS ARM ARCHITECTURES
 M:	Kukjin Kim <kgene@kernel.org>
+M:	Krzysztof Kozlowski <k.kozlowski@samsung.com>
 L:	linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
 L:	linux-samsung-soc@vger.kernel.org (moderated for non-subscribers)
 S:	Maintained