mbox

[GIT,PULL,for,3.17] pull request for adding Hisilicon SoC MAINTAINERS

Message ID 53D1F235.2080607@hisilicon.com
State New
Headers show

Pull-request

git://github.com/hisilicon/linux-hisi.git tags/misc-for-3.17

Message

Wei Xu July 25, 2014, 5:59 a.m. UTC 
Hi Olof, Hi Kevin, Hi Arnd,

Resend it again and add arm@kernel.org into the CC list.
Please help to merge pull request for adding Hisilicon SoC maintainer. 


Best Regards,

Wei


The following changes since commit 4dfad0696da39f4c05ac5cadde9648a650830993:

  MAINTAINERS:ARM:hisi: add Hisilicon SoC family (2014-07-07 10:41:53 +0800)

are available in the git repository at:

  git://github.com/hisilicon/linux-hisi.git tags/misc-for-3.17

for you to fetch changes up to 4dfad0696da39f4c05ac5cadde9648a650830993:

  MAINTAINERS:ARM:hisi: add Hisilicon SoC family (2014-07-07 10:41:53 +0800)

----------------------------------------------------------------
update MAINTAINERS to add Hisilicon SoC maintainer

----------------------------------------------------------------

Comments

Arnd Bergmann July 26, 2014, 10:33 a.m. UTC | #1 
On Friday 25 July 2014 13:59:17 xuwei wrote:
> Hi Olof, Hi Kevin, Hi Arnd,
> 
> Resend it again and add arm@kernel.org into the CC list.
> Please help to merge pull request for adding Hisilicon SoC maintainer. 
> 

Merged into next/soc, thanks!

I think Olof's previous comment was that this didn't need to be
a separate pull request but could just be part of the "pull
request for hisilicon soc".

	Arnd
Olof Johansson July 26, 2014, 4:41 p.m. UTC | #2 
On Sat, Jul 26, 2014 at 12:33:48PM +0200, Arnd Bergmann wrote:
> On Friday 25 July 2014 13:59:17 xuwei wrote:
> > Hi Olof, Hi Kevin, Hi Arnd,
> > 
> > Resend it again and add arm@kernel.org into the CC list.
> > Please help to merge pull request for adding Hisilicon SoC maintainer. 
> > 
> 
> Merged into next/soc, thanks!
> 
> I think Olof's previous comment was that this didn't need to be
> a separate pull request but could just be part of the "pull
> request for hisilicon soc".

In particular, it seemed early to me to merge a maintainer update for a new
contributor that we haven't seen code from yet, and the pull requests we've
seen have been problematic. More often we label maintainers after they've
gotten used to the role.

It just seemed backwards the way it came in.


-Olof