Thread (6 messages) 6 messages, 3 authors, 2016-12-30

Re: [PATCH] ARM: dts: vf610-zii-dev-rev-b: Add missing newline

From: Shawn Guo <hidden>
Date: 2016-12-30 07:07:19
Also in: linux-arm-kernel, lkml

On Sun, Nov 27, 2016 at 10:35:16PM +0100, Andrew Lunn wrote:
On Sun, Nov 27, 2016 at 10:30:54PM +0100, Andreas Färber wrote:
quoted
Hi,

Am 27.11.2016 um 22:17 schrieb Andrew Lunn:
quoted
On Sun, Nov 27, 2016 at 08:54:44PM +0100, Andreas Färber wrote:
quoted
Found while reviewing Marvell dsa bindings usage.
Hi Andreas

It is good practice to put the maintainer you expect to accept the
patch on the To: line. You have at least two different maintainers on
Cc: so it is currently ambiguous. And these lists can be high volume,
so without a copy in the maintainers inbox, patches can be overlooked.
As a vf610 DT patch with LAKML in To I am expecting it to be handled by
Shawn or anyone from the NXP/ARM side.
So please have Shawn as To:

The patch you are fixing went through Dave Miller, who is also on Cc:,
hence the ambiguity.
The good practice was broken from the beginning when patch "arm: vf610:
zii devel b: Add support for switch interrupts") went through net tree,
even without me on copy.

Shawn
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help