devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [GIT PULL] Mediatek basic support for v3.17
@ 2014-07-15 11:11 Matthias Brugger
  2014-07-15 13:59 ` Greg KH
  0 siblings, 1 reply; 6+ messages in thread
From: Matthias Brugger @ 2014-07-15 11:11 UTC (permalink / raw)
  To: linux-kernel@vger.kernel.org
  Cc: Mark Rutland, Andrew Lunn, linux-doc@vger.kernel.org,
	devicetree@vger.kernel.org, Linus Walleij, Gregory CLEMENT,
	Thierry Reding, Heiko Stübner, ak, Russell King,
	Arnd Bergmann, Daniel Lezcano, shawn.guo@freescale.com,
	u.kleine-koenig, Grant Likely, Florian Vaussard,
	Sebastian Hesselbarth, anders.berg@lsi.com, Jason Cooper,
	Pawel Moll

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [GIT PULL] Mediatek basic support for v3.17
  2014-07-15 11:11 [GIT PULL] Mediatek basic support for v3.17 Matthias Brugger
@ 2014-07-15 13:59 ` Greg KH
  2014-07-15 15:17   ` Arnd Bergmann
  0 siblings, 1 reply; 6+ messages in thread
From: Greg KH @ 2014-07-15 13:59 UTC (permalink / raw)
  To: Matthias Brugger
  Cc: Mark Rutland, Andrew Lunn, linux-doc@vger.kernel.org,
	devicetree@vger.kernel.org, Linus Walleij, Thierry Reding,
	Heiko Stübner, ak, Russell King, Arnd Bergmann,
	Daniel Lezcano, shawn.guo@freescale.com, u.kleine-koenig,
	Grant Likely, Florian Vaussard, Sebastian Hesselbarth,
	anders.berg@lsi.com, Jason Cooper, Pawel Moll, Ian Campbell,
	Marc Zyngier

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [GIT PULL] Mediatek basic support for v3.17
  2014-07-15 13:59 ` Greg KH
@ 2014-07-15 15:17   ` Arnd Bergmann
  2014-07-16 13:47     ` Matthias Brugger
  0 siblings, 1 reply; 6+ messages in thread
From: Arnd Bergmann @ 2014-07-15 15:17 UTC (permalink / raw)
  To: Greg KH
  Cc: Mark Rutland, Andrew Lunn, linux-doc@vger.kernel.org,
	devicetree@vger.kernel.org, Linus Walleij, Gregory CLEMENT,
	Thierry Reding, Heiko Stübner, ak, Russell King,
	Daniel Lezcano, shawn.guo@freescale.com, u.kleine-koenig,
	Grant Likely, Florian Vaussard, Sebastian Hesselbarth,
	anders.berg@lsi.com, Jason Cooper, Pawel Moll, Ian Campbell

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [GIT PULL] Mediatek basic support for v3.17
  2014-07-15 15:17   ` Arnd Bergmann
@ 2014-07-16 13:47     ` Matthias Brugger
  2014-07-16 14:40       ` Rob Herring
  0 siblings, 1 reply; 6+ messages in thread
From: Matthias Brugger @ 2014-07-16 13:47 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: Mark Rutland, Andrew Lunn, linux-doc@vger.kernel.org,
	devicetree@vger.kernel.org, Linus Walleij, Thierry Reding,
	Heiko Stübner, ak, Russell King, Daniel Lezcano,
	shawn.guo@freescale.com, u.kleine-koenig, Grant Likely,
	Florian Vaussard, Sebastian Hesselbarth, anders.berg@lsi.com,
	Jason Cooper, Pawel Moll, Ian Campbell, Marc Zyngier, rostedt,
	fabf

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [GIT PULL] Mediatek basic support for v3.17
  2014-07-16 13:47     ` Matthias Brugger
@ 2014-07-16 14:40       ` Rob Herring
  2014-07-18  9:48         ` Matthias Brugger
  0 siblings, 1 reply; 6+ messages in thread
From: Rob Herring @ 2014-07-16 14:40 UTC (permalink / raw)
  To: Matthias Brugger
  Cc: Mark Rutland, Andrew Lunn, linux-doc@vger.kernel.org,
	devicetree@vger.kernel.org, Linus Walleij, Thierry Reding,
	Heiko Stübner, ak, Russell King, Pawel Moll, Daniel Lezcano,
	shawn.guo@freescale.com, Uwe Kleine-König, Grant Likely,
	Florian Vaussard, Sebastian Hesselbarth, anders.berg@lsi.com,
	Jason Cooper, Arnd Bergmann, Ian Campbell

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [GIT PULL] Mediatek basic support for v3.17
  2014-07-16 14:40       ` Rob Herring
@ 2014-07-18  9:48         ` Matthias Brugger
  0 siblings, 0 replies; 6+ messages in thread
From: Matthias Brugger @ 2014-07-18  9:48 UTC (permalink / raw)
  To: Rob Herring
  Cc: Mark Rutland, Andrew Lunn, linux-doc@vger.kernel.org,
	devicetree@vger.kernel.org, Linus Walleij, Thierry Reding,
	Heiko Stübner, ak, Russell King, Pawel Moll, Daniel Lezcano,
	shawn.guo@freescale.com, Uwe Kleine-König, Grant Likely,
	Florian Vaussard, Sebastian Hesselbarth, anders.berg@lsi.com,
	Jason Cooper, Arnd Bergmann, Ian Campbell

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2014-07-18  9:48 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-07-15 11:11 [GIT PULL] Mediatek basic support for v3.17 Matthias Brugger
2014-07-15 13:59 ` Greg KH
2014-07-15 15:17   ` Arnd Bergmann
2014-07-16 13:47     ` Matthias Brugger
2014-07-16 14:40       ` Rob Herring
2014-07-18  9:48         ` Matthias Brugger

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).