linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [GIT PULL v2.1] Mediatek basic support for v3.17
       [not found] <CABuKBeLc=Bbpqrksdjoc_0Sx_rVpZLuY1aUY1WtK5Vp20Jiv8g@mail.gmail.com>
@ 2014-07-22 18:18 ` Russell King - ARM Linux
  2014-07-23 20:59   ` Arnd Bergmann
  0 siblings, 1 reply; 2+ messages in thread
From: Russell King - ARM Linux @ 2014-07-22 18:18 UTC (permalink / raw)
  To: linux-arm-kernel

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.

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

* [GIT PULL v2.1] Mediatek basic support for v3.17
  2014-07-22 18:18 ` [GIT PULL v2.1] Mediatek basic support for v3.17 Russell King - ARM Linux
@ 2014-07-23 20:59   ` Arnd Bergmann
  0 siblings, 0 replies; 2+ messages in thread
From: Arnd Bergmann @ 2014-07-23 20:59 UTC (permalink / raw)
  To: linux-arm-kernel

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

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

end of thread, other threads:[~2014-07-23 20:59 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <CABuKBeLc=Bbpqrksdjoc_0Sx_rVpZLuY1aUY1WtK5Vp20Jiv8g@mail.gmail.com>
2014-07-22 18:18 ` [GIT PULL v2.1] Mediatek basic support for v3.17 Russell King - ARM Linux
2014-07-23 20:59   ` Arnd Bergmann

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