All of lore.kernel.org
 help / color / mirror / Atom feed
From: thomas.petazzoni@free-electrons.com (Thomas Petazzoni)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/8] net: mvpp2: add TX interrupts support
Date: Fri, 28 Jul 2017 17:03:20 +0200	[thread overview]
Message-ID: <20170728170320.4e4bd683@windsurf.lan> (raw)
In-Reply-To: <20170726.133533.262039701450895012.davem@davemloft.net>

Hello,

On Wed, 26 Jul 2017 13:35:33 -0700 (PDT), David Miller wrote:

> >  - This series depends on the previous series sent by Antoine T?nart
> >    "net: mvpp2: MAC/GoP configuration and optional PHYs". Functionally
> >    speaking there is no real dependency, but we touch in a few areas
> >    the same piece of code, so I based my patch series on top of
> >    Antoine's.
> > 
> >  - Please do not apply the last patch of this series "arm64: dts:
> >    marvell: add TX interrupts for PPv2.2", it will be taken by the ARM
> >    mvebu maintainers.  
> 
> Please don't do things this way.
> 
> Patiently wait for Antione's series to make it into my tree, then
> submit your's.

I'll resubmit when Antoine's series is merged. The idea of submitting
my patches was to allow others to review/test them, not necessarily to
have them merged right away. Perhaps I should have marked them RFC to
make it clear that I don't expect you to merge them right away.

> Also, if we're continually going to elide the DTS file patches, just

It's the dutty of the sub-architecture maintainers to merge the DTS
changes. Merging them through your tree doesn't work really well, as
the changes might very likely conflict with other DTS changes merged by
the sub-architecture maintainer.

But it makes sense to have such DTS changes in the same patch series,
since they are really related. It's very common to have patch series
that contain patches that will be merged by different maintainers.

> don't bother adding them to the series.  That way you don't have to
> give me special instructions, and I don't have the possibility of
> making a mistake and applying it accidently.

... but OK, I won't resend them, so you don't get potentially confused
by such patches.

Thanks!

Thomas Petazzoni
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

WARNING: multiple messages have this Message-ID (diff)
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: David Miller <davem@davemloft.net>
Cc: netdev@vger.kernel.org, linux@arm.linux.org.uk,
	antoine.tenart@free-electrons.com,
	miquel.raynal@free-electrons.com,
	linux-arm-kernel@lists.infradead.org, jason@lakedaemon.net,
	andrew@lunn.ch, sebastian.hesselbarth@gmail.com,
	gregory.clement@free-electrons.com, nadavh@marvell.com,
	hannah@marvell.com, yehuday@marvell.com, stefanc@marvell.com,
	mw@semihalf.com
Subject: Re: [PATCH 0/8] net: mvpp2: add TX interrupts support
Date: Fri, 28 Jul 2017 17:03:20 +0200	[thread overview]
Message-ID: <20170728170320.4e4bd683@windsurf.lan> (raw)
In-Reply-To: <20170726.133533.262039701450895012.davem@davemloft.net>

Hello,

On Wed, 26 Jul 2017 13:35:33 -0700 (PDT), David Miller wrote:

> >  - This series depends on the previous series sent by Antoine Ténart
> >    "net: mvpp2: MAC/GoP configuration and optional PHYs". Functionally
> >    speaking there is no real dependency, but we touch in a few areas
> >    the same piece of code, so I based my patch series on top of
> >    Antoine's.
> > 
> >  - Please do not apply the last patch of this series "arm64: dts:
> >    marvell: add TX interrupts for PPv2.2", it will be taken by the ARM
> >    mvebu maintainers.  
> 
> Please don't do things this way.
> 
> Patiently wait for Antione's series to make it into my tree, then
> submit your's.

I'll resubmit when Antoine's series is merged. The idea of submitting
my patches was to allow others to review/test them, not necessarily to
have them merged right away. Perhaps I should have marked them RFC to
make it clear that I don't expect you to merge them right away.

> Also, if we're continually going to elide the DTS file patches, just

It's the dutty of the sub-architecture maintainers to merge the DTS
changes. Merging them through your tree doesn't work really well, as
the changes might very likely conflict with other DTS changes merged by
the sub-architecture maintainer.

But it makes sense to have such DTS changes in the same patch series,
since they are really related. It's very common to have patch series
that contain patches that will be merged by different maintainers.

> don't bother adding them to the series.  That way you don't have to
> give me special instructions, and I don't have the possibility of
> making a mistake and applying it accidently.

... but OK, I won't resend them, so you don't get potentially confused
by such patches.

Thanks!

Thomas Petazzoni
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

  reply	other threads:[~2017-07-28 15:03 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-07-25 15:55 [PATCH 0/8] net: mvpp2: add TX interrupts support Thomas Petazzoni
2017-07-25 15:55 ` Thomas Petazzoni
2017-07-25 15:55 ` [PATCH 1/8] net: mvpp2: fix MVPP21_ISR_RXQ_GROUP_REG definition Thomas Petazzoni
2017-07-25 15:55   ` Thomas Petazzoni
2017-07-25 15:55 ` [PATCH 2/8] net: mvpp2: remove RX queue group reset code Thomas Petazzoni
2017-07-25 15:55   ` Thomas Petazzoni
2017-07-25 15:55 ` [PATCH 3/8] net: mvpp2: introduce per-port nrxqs/ntxqs variables Thomas Petazzoni
2017-07-25 15:55   ` Thomas Petazzoni
2017-07-25 15:55 ` [PATCH 4/8] net: mvpp2: move from cpu-centric naming to "software thread" naming Thomas Petazzoni
2017-07-25 15:55   ` Thomas Petazzoni
2017-07-25 15:55 ` [PATCH 5/8] net: mvpp2: introduce queue_vector concept Thomas Petazzoni
2017-07-25 15:55   ` Thomas Petazzoni
2017-07-25 15:55 ` [PATCH 6/8] net: mvpp2: add support for TX interrupts and RX queue distribution modes Thomas Petazzoni
2017-07-25 15:55   ` Thomas Petazzoni
2017-07-25 15:55 ` [PATCH 7/8] dt-bindings: net: marvell-pp2: update interrupt-names with TX interrupts Thomas Petazzoni
2017-07-25 15:55   ` Thomas Petazzoni
2017-07-25 15:55 ` [PATCH 8/8] arm64: dts: marvell: add TX interrupts for PPv2.2 Thomas Petazzoni
2017-07-25 15:55   ` Thomas Petazzoni
2017-07-26  6:18 ` [PATCH 0/8] net: mvpp2: add TX interrupts support Marcin Wojtas
2017-07-26  6:18   ` Marcin Wojtas
2017-07-26 20:35 ` David Miller
2017-07-26 20:35   ` David Miller
2017-07-28 15:03   ` Thomas Petazzoni [this message]
2017-07-28 15:03     ` Thomas Petazzoni

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20170728170320.4e4bd683@windsurf.lan \
    --to=thomas.petazzoni@free-electrons.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.