All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kevin Hilman <khilman@kernel.org>
To: Tobias Jakobi <liquid.acid@gmx.net>
Cc: Javier Martinez Canillas <javier@dowhile0.org>,
	Kukjin Kim <kgene@kernel.org>,
	Krzysztof Kozlowski <k.kozlowski@samsung.com>,
	Linux Kernel <linux-kernel@vger.kernel.org>,
	"linux-samsung-soc@vger.kernel.org"
	<linux-samsung-soc@vger.kernel.org>,
	Russell King <linux@arm.linux.org.uk>,
	Arnd Bergmann <arnd@arndb.de>, Olof Johansson <olof@lixom.net>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH] MAINTAINERS: ARM: EXYNOS: Add Krzysztof Kozlowski asco-maintainer
Date: Thu, 14 May 2015 08:52:07 -0700	[thread overview]
Message-ID: <7hfv6z17iw.fsf@deeprootsystems.com> (raw)
In-Reply-To: <5554ABF4.80002@gmx.net> (Tobias Jakobi's message of "Thu, 14 May 2015 16:06:44 +0200")

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

WARNING: multiple messages have this Message-ID (diff)
From: khilman@kernel.org (Kevin Hilman)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] MAINTAINERS: ARM: EXYNOS: Add Krzysztof Kozlowski asco-maintainer
Date: Thu, 14 May 2015 08:52:07 -0700	[thread overview]
Message-ID: <7hfv6z17iw.fsf@deeprootsystems.com> (raw)
In-Reply-To: <5554ABF4.80002@gmx.net> (Tobias Jakobi's message of "Thu, 14 May 2015 16:06:44 +0200")

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

WARNING: multiple messages have this Message-ID (diff)
From: Kevin Hilman <khilman@kernel.org>
To: Tobias Jakobi <liquid.acid@gmx.net>
Cc: Javier Martinez Canillas <javier@dowhile0.org>,
	Kukjin Kim <kgene@kernel.org>,
	Krzysztof Kozlowski <k.kozlowski@samsung.com>,
	Linux Kernel <linux-kernel@vger.kernel.org>,
	"linux-samsung-soc\@vger.kernel.org" 
	<linux-samsung-soc@vger.kernel.org>,
	Russell King <linux@arm.linux.org.uk>,
	Arnd Bergmann <arnd@arndb.de>, Olof Johansson <olof@lixom.net>,
	"linux-arm-kernel\@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH] MAINTAINERS: ARM: EXYNOS: Add Krzysztof Kozlowski asco-maintainer
Date: Thu, 14 May 2015 08:52:07 -0700	[thread overview]
Message-ID: <7hfv6z17iw.fsf@deeprootsystems.com> (raw)
In-Reply-To: <5554ABF4.80002@gmx.net> (Tobias Jakobi's message of "Thu, 14 May 2015 16:06:44 +0200")

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

  reply	other threads:[~2015-05-14 15:52 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-05-14 12:15 [PATCH] MAINTAINERS: ARM: EXYNOS: Add Krzysztof Kozlowski as co-maintainer Krzysztof Kozlowski
2015-05-14 12:15 ` Krzysztof Kozlowski
2015-05-14 13:00 ` [PATCH] MAINTAINERS: ARM: EXYNOS: Add Krzysztof Kozlowski asco-maintainer Kukjin Kim
2015-05-14 13:00   ` Kukjin Kim
2015-05-14 13:20   ` Javier Martinez Canillas
2015-05-14 13:20     ` Javier Martinez Canillas
2015-05-14 14:06     ` Tobias Jakobi
2015-05-14 14:06       ` Tobias Jakobi
2015-05-14 15:52       ` Kevin Hilman [this message]
2015-05-14 15:52         ` Kevin Hilman
2015-05-14 15:52         ` Kevin Hilman
2015-05-14 16:45         ` Olof Johansson
2015-05-14 16:45           ` Olof Johansson
2015-05-14 23:50           ` Krzysztof Kozlowski
2015-05-14 23:50             ` Krzysztof Kozlowski
2015-05-15 14:15             ` Kevin Hilman
2015-05-15 14:15               ` Kevin Hilman

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=7hfv6z17iw.fsf@deeprootsystems.com \
    --to=khilman@kernel.org \
    --cc=arnd@arndb.de \
    --cc=javier@dowhile0.org \
    --cc=k.kozlowski@samsung.com \
    --cc=kgene@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=liquid.acid@gmx.net \
    --cc=olof@lixom.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.