[GIT PULL] Mediatek basic support for v3.17

[GIT PULL] Mediatek basic support for v3.17

[Matthias Brugger]

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

for you to fetch changes up to 747eec721f99b3d2eb3985c3199bfb54888c716d:

  arm: mediatek: add dts for Aquaris5 mobile phone (2014-07-15 11:15:00 +0200)

----------------------------------------------------------------
This patch set adds basic support for the Mediatek Cortex-A7 SoCs.
Support is quite basic, as the only component working up to now are the
timers.

----------------------------------------------------------------
Matthias Brugger (7):
      of: Provide function to request and map memory
      clocksource: Add support for the Mediatek SoCs
      dt-bindings: add mtk-timer bindings
      vendor-prefixes: add prefix for Mediatek Inc.
      arm: add basic support for Mediatek MT6589 boards
      dt-bindings: add documentation for Mediatek SoC
      arm: mediatek: add dts for Aquaris5 mobile phone

 Documentation/devicetree/bindings/arm/mediatek.txt |    8 +
 .../bindings/timer/mediatek,mtk-timer.txt          |   17 ++
 .../devicetree/bindings/vendor-prefixes.txt        |    1 +
 arch/arm/Kconfig                                   |    2 +
 arch/arm/Makefile                                  |    1 +
 arch/arm/boot/dts/mt6589-aquaris5.dts              |   25 ++
 arch/arm/boot/dts/mt6589.dtsi                      |   94 +++++++
 arch/arm/mach-mediatek/Kconfig                     |    6 +
 arch/arm/mach-mediatek/Makefile                    |    1 +
 arch/arm/mach-mediatek/mediatek.c                  |   27 ++
 drivers/clocksource/Kconfig                        |    5 +
 drivers/clocksource/Makefile                       |    1 +
 drivers/clocksource/mtk_timer.c                    |  261 ++++++++++++++++++++
 drivers/of/address.c                               |   36 +++
 include/linux/io.h                                 |    2 +
 include/linux/of_address.h                         |   11 +
 lib/devres.c                                       |    2 -
 17 files changed, 498 insertions(+), 2 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/arm/mediatek.txt
 create mode 100644
Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt
 create mode 100644 arch/arm/boot/dts/mt6589-aquaris5.dts
 create mode 100644 arch/arm/boot/dts/mt6589.dtsi
 create mode 100644 arch/arm/mach-mediatek/Kconfig
 create mode 100644 arch/arm/mach-mediatek/Makefile
 create mode 100644 arch/arm/mach-mediatek/mediatek.c
 create mode 100644 drivers/clocksource/mtk_timer.c

Re: [GIT PULL] Mediatek basic support for v3.17

[Greg KH]

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.

greg k-h

Re: [GIT PULL] Mediatek basic support for v3.17

[Arnd Bergmann]

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

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

Re: [GIT PULL] Mediatek basic support for v3.17

[Matthias Brugger]

2014-07-15 17:17 GMT+02:00 Arnd Bergmann <arnd@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.

Sorry for the noise.

Matthias

[0] http://thread.gmane.org/gmane.linux.documentation/24562/focus=24665
[1] http://thread.gmane.org/gmane.linux.documentation/24447/focus=331382
[2] https://lkml.org/lkml/2014/6/20/676


-- 
motzblog.wordpress.com

Re: [GIT PULL] Mediatek basic support for v3.17

[Rob Herring]

On Wed, Jul 16, 2014 at 8:47 AM, Matthias Brugger
<matthias.bgg@gmail.com> wrote:
> 2014-07-15 17:17 GMT+02:00 Arnd Bergmann <arnd@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.

Rob

Re: [GIT PULL] Mediatek basic support for v3.17

[Matthias Brugger]

2014-07-16 16:40 GMT+02:00 Rob Herring <robherring2@gmail.com>:
> On Wed, Jul 16, 2014 at 8:47 AM, Matthias Brugger
> <matthias.bgg@gmail.com> wrote:
>> 2014-07-15 17:17 GMT+02:00 Arnd Bergmann <arnd@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