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

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

[Matthias Brugger]

The patches 1-4 from the v10 series [0] of the patch set have been
merged by Daniel Lezcano [1]. This pull request has just the remaining
patches 5-7

There is an Ack from Rob Herring for the SoC device tree binding patch.
The other two patches lack an Ack from the maintainers.

[0] http://thread.gmane.org/gmane.linux.documentation/25231
[1] https://lkml.org/lkml/2014/7/21/291

---

Changes for v2.1:
- fix screwed up explanation

Changes for v2:
- request includes just patches 5-7 from the series

----------------------------------------------------------------
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-mediatek-support

for you to fetch changes up to afaedea451149ab8f08c37ce688c8021ed347811:

  arm: mediatek: add dts for Aquaris5 mobile phone (2014-07-22 17:52:38 +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.

Signed-off-by: Matthias Brugger matthias.bgg@gmail.com

----------------------------------------------------------------
Matthias Brugger (3):
      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 ++
 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 ++++++
 8 files changed, 164 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/arm/mediatek.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

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

[Russell King - ARM Linux]

Your To: header contains many tens of people.  Which person out of
all those do you expect to pull your changes?

Email has a way to identify whom you are talking _to_ and whom you wish
to _copy_ for information purposes.

These are the "To:" and "Cc:" fields on email.  Those on the "To:" list
are those who you are directly addressing your message.  Those on the
Cc: list are "Carbon Copy" recipients, in other words, ones you are
notifying for information purposes.

Let's say that you're an HR department.  You're firing an employee.
You send your message "To" the employee and "To" the employee's managers.
Congratulations, you've just fired the employee along with his managers.

If instead, you send your message "To" the employee and "Cc" the
employee's managers, it's then clear that the employee's managers
are being copied for information purposes.

The same thing applies here.  Send your pull request "To" the person
or group of people who you wish to action your request, and include
on the "Cc" those whom may be interested in seeing the pull request
for information purposes.

Please be more careful with your email addressing technique in the
future, thanks.

[No, I haven't pulled anything]

On Tue, Jul 22, 2014 at 06:34:17PM +0200, Matthias Brugger wrote:
> The patches 1-4 from the v10 series [0] of the patch set have been
> merged by Daniel Lezcano [1]. This pull request has just the remaining
> patches 5-7
> 
> There is an Ack from Rob Herring for the SoC device tree binding patch.
> The other two patches lack an Ack from the maintainers.
> 
> [0] http://thread.gmane.org/gmane.linux.documentation/25231
> [1] https://lkml.org/lkml/2014/7/21/291
> 
> ---
> 
> Changes for v2.1:
> - fix screwed up explanation
> 
> Changes for v2:
> - request includes just patches 5-7 from the series
> 
> ----------------------------------------------------------------
> 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-mediatek-support
> 
> for you to fetch changes up to afaedea451149ab8f08c37ce688c8021ed347811:
> 
>   arm: mediatek: add dts for Aquaris5 mobile phone (2014-07-22 17:52:38 +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.
> 
> Signed-off-by: Matthias Brugger matthias.bgg@gmail.com
> 
> ----------------------------------------------------------------
> Matthias Brugger (3):
>       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 ++
>  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 ++++++
>  8 files changed, 164 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/arm/mediatek.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

-- 
FTTC broadband for 0.8mile line: currently at 9.5Mbps down 400kbps up
according to speedtest.net.

[GIT PULL v2.1] Mediatek basic support for v3.17

[Russell King - ARM Linux]

Your To: header contains many tens of people.  Which person out of
all those do you expect to pull your changes?

Email has a way to identify whom you are talking _to_ and whom you wish
to _copy_ for information purposes.

These are the "To:" and "Cc:" fields on email.  Those on the "To:" list
are those who you are directly addressing your message.  Those on the
Cc: list are "Carbon Copy" recipients, in other words, ones you are
notifying for information purposes.

Let's say that you're an HR department.  You're firing an employee.
You send your message "To" the employee and "To" the employee's managers.
Congratulations, you've just fired the employee along with his managers.

If instead, you send your message "To" the employee and "Cc" the
employee's managers, it's then clear that the employee's managers
are being copied for information purposes.

The same thing applies here.  Send your pull request "To" the person
or group of people who you wish to action your request, and include
on the "Cc" those whom may be interested in seeing the pull request
for information purposes.

Please be more careful with your email addressing technique in the
future, thanks.

[No, I haven't pulled anything]

On Tue, Jul 22, 2014 at 06:34:17PM +0200, Matthias Brugger wrote:
> The patches 1-4 from the v10 series [0] of the patch set have been
> merged by Daniel Lezcano [1]. This pull request has just the remaining
> patches 5-7
> 
> There is an Ack from Rob Herring for the SoC device tree binding patch.
> The other two patches lack an Ack from the maintainers.
> 
> [0] http://thread.gmane.org/gmane.linux.documentation/25231
> [1] https://lkml.org/lkml/2014/7/21/291
> 
> ---
> 
> Changes for v2.1:
> - fix screwed up explanation
> 
> Changes for v2:
> - request includes just patches 5-7 from the series
> 
> ----------------------------------------------------------------
> 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-mediatek-support
> 
> for you to fetch changes up to afaedea451149ab8f08c37ce688c8021ed347811:
> 
>   arm: mediatek: add dts for Aquaris5 mobile phone (2014-07-22 17:52:38 +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.
> 
> Signed-off-by: Matthias Brugger matthias.bgg at gmail.com
> 
> ----------------------------------------------------------------
> Matthias Brugger (3):
>       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 ++
>  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 ++++++
>  8 files changed, 164 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/arm/mediatek.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

-- 
FTTC broadband for 0.8mile line: currently at 9.5Mbps down 400kbps up
according to speedtest.net.

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

[Arnd Bergmann]

On Tuesday 22 July 2014, Russell King - ARM Linux wrote:
> Your To: header contains many tens of people.  Which person out of
> all those do you expect to pull your changes?
> 
> Email has a way to identify whom you are talking _to_ and whom you wish
> to _copy_ for information purposes.
> 
> These are the "To:" and "Cc:" fields on email.  Those on the "To:" list
> are those who you are directly addressing your message.  Those on the
> Cc: list are "Carbon Copy" recipients, in other words, ones you are
> notifying for information purposes.

Absolutely. In addition, I'd recommend always addressing the maintainer
by name, with a friendly greeting, to ensure they really know they
are meant.

I would also add that you should be careful with the number of people
on Cc in any email. If the Cc list exceeds 1024 characters, the kernel.org
mailing lists will reject your mail. It normally isn't necessary to
have all those people on Cc.

As a special case for arm-soc, please send the patches "To: arm@kernel.org"
so it ends up in the right folders for Olof and me.

> > ----------------------------------------------------------------
> > 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-mediatek-support
> > 
> > for you to fetch changes up to afaedea451149ab8f08c37ce688c8021ed347811:
> > 
> >   arm: mediatek: add dts for Aquaris5 mobile phone (2014-07-22 17:52:38 +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.
> > 
> > Signed-off-by: Matthias Brugger matthias.bgg@gmail.com
> > 
> > ----------------------------------------------------------------
> > Matthias Brugger (3):
> >       arm: add basic support for Mediatek MT6589 boards
> >       dt-bindings: add documentation for Mediatek SoC
> >       arm: mediatek: add dts for Aquaris5 mobile phone

Patches look good, I've pulled them into the next/soc tree. It would be
good for the future to sign your tags with a gpg key that is signed by
other kernel developers you know.

Two other patches I'd like to see are:

- adding the dts file to ls arch/arm/boot/dts/Makefile
- adding your platform and all drivers you need to multi_v7_defconfig
  so they are included in regular build tests.

Thanks,

	Arnd

[GIT PULL v2.1] Mediatek basic support for v3.17

[Arnd Bergmann]

On Tuesday 22 July 2014, Russell King - ARM Linux wrote:
> Your To: header contains many tens of people.  Which person out of
> all those do you expect to pull your changes?
> 
> Email has a way to identify whom you are talking _to_ and whom you wish
> to _copy_ for information purposes.
> 
> These are the "To:" and "Cc:" fields on email.  Those on the "To:" list
> are those who you are directly addressing your message.  Those on the
> Cc: list are "Carbon Copy" recipients, in other words, ones you are
> notifying for information purposes.

Absolutely. In addition, I'd recommend always addressing the maintainer
by name, with a friendly greeting, to ensure they really know they
are meant.

I would also add that you should be careful with the number of people
on Cc in any email. If the Cc list exceeds 1024 characters, the kernel.org
mailing lists will reject your mail. It normally isn't necessary to
have all those people on Cc.

As a special case for arm-soc, please send the patches "To: arm at kernel.org"
so it ends up in the right folders for Olof and me.

> > ----------------------------------------------------------------
> > 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-mediatek-support
> > 
> > for you to fetch changes up to afaedea451149ab8f08c37ce688c8021ed347811:
> > 
> >   arm: mediatek: add dts for Aquaris5 mobile phone (2014-07-22 17:52:38 +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.
> > 
> > Signed-off-by: Matthias Brugger matthias.bgg at gmail.com
> > 
> > ----------------------------------------------------------------
> > Matthias Brugger (3):
> >       arm: add basic support for Mediatek MT6589 boards
> >       dt-bindings: add documentation for Mediatek SoC
> >       arm: mediatek: add dts for Aquaris5 mobile phone

Patches look good, I've pulled them into the next/soc tree. It would be
good for the future to sign your tags with a gpg key that is signed by
other kernel developers you know.

Two other patches I'd like to see are:

- adding the dts file to ls arch/arm/boot/dts/Makefile
- adding your platform and all drivers you need to multi_v7_defconfig
  so they are included in regular build tests.

Thanks,

	Arnd