[GIT PULL] Mediatek basic support for v3.17

[Matthias Brugger]

2014-07-16 16:40 GMT+02:00 Rob Herring <robherring2 at gmail.com>:
> On Wed, Jul 16, 2014 at 8:47 AM, Matthias Brugger
> <matthias.bgg at gmail.com> wrote:
>> 2014-07-15 17:17 GMT+02:00 Arnd Bergmann <arnd at arndb.de>:
>>> On Tuesday 15 July 2014 06:59:13 Greg KH wrote:
>>>> On Tue, Jul 15, 2014 at 01:11:42PM +0200, Matthias Brugger wrote:
>>>> > The following changes since commit 7171511eaec5bf23fb06078f59784a3a0626b38f:
>>>> >
>>>> >   Linux 3.16-rc1 (2014-06-15 17:45:28 -1000)
>>>> >
>>>> > are available in the git repository at:
>>>> >
>>>> >   https://github.com/mbgg/linux-mediatek.git tags/v3.17-next-basic-support
>>>>
>>>> As these are only 7 patches, why do a git pull?  And I know I can't take
>>>> pull requests from github, I'm not sure what the rules are for the ARM
>>>> SoC developers either...
>>>>
>>>> Just send patches, it's easier that way.
>>>
>>> We generally prefer pull requests, but patches also work for us.
>>>
>>> However, one note about the text in the pull request: Matthias,
>>> please always try to be explicit about who you want to pull or
>>> merge things when you are ready and list in the pull request the
>>> other maintainers that have agreed to merging it (or whether they
>>> have not sent an Ack yet, which is even more important).
>>
>> Ok, thank you for the advice. I will do so in the next pull request.
>>
>>>
>>> In this case, the clocksource driver needs an Ack from the clocksource
>>> maintainers, and the drivers/of/address.c changes need an Ack from the
>>> OF maintainers. I assume you have those, but I'd have to first look
>>> at the patches to be sure.
>>>
>>>         arnd
>>
>> For the drivers/of/address.c patch I used to have an Ack [1] from
>> Grant Likely who told me to merge it through arm-soc. But I deleted
>> the Ack after changing the patch because of some comments from Russel
>> King [2].
>> So at the moment I don't have any Ack anymore.
>>
>> I have an Ack for the clocksource driver from Thomas [0]. Daniel
>> wanted to merge it through his branch, but because of the changes in
>> the OF patch he never did.
>>
>> So I suppose I have to get some comments from the OF maintainer about
>> the patch, before we can go on with the pull request.
>
> I've now acked the patch in question. If there's no other change to
> the pull req, there's no reason to respin the branch to add the ack.

I talked with Daniel Lezcano on IRC and he told me, that he will take
the clocksource patches through his repository.
For the rest of the patches I will send a new pull request.

Best regards,
Matthias

>
> Rob



-- 
motzblog.wordpress.com