All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
To: Lars-Peter Clausen <lars@metafoo.de>, Vinod Koul <vinod.koul@intel.com>
Cc: Boris Brezillon <boris.brezillon@free-electrons.com>,
	alsa-devel@alsa-project.org,
	Nicolas Ferre <nicolas.ferre@atmel.com>,
	linux-kernel@vger.kernel.org, Liam Girdwood <lgirdwood@gmail.com>,
	Mark Brown <broonie@kernel.org>,
	dmaengine@vger.kernel.org,
	Hans-Christian Noren Egtvedt <egtvedt@samfundet.no>,
	Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com>,
	Haavard Skinnemoen <hskinnemoen@gmail.com>
Subject: Re: [PATCH v2 2/2] dmaengine: dw: Remove AVR32 bits from the driver
Date: Mon, 15 May 2017 12:42:30 +0300	[thread overview]
Message-ID: <1494841350.6967.55.camel@linux.intel.com> (raw)
In-Reply-To: <45cb3a0a-28b5-a64c-90ac-1abf0c59d70f@metafoo.de>

On Mon, 2017-05-15 at 11:25 +0200, Lars-Peter Clausen wrote:
> On 05/15/2017 10:43 AM, Andy Shevchenko wrote:
> > On Sun, 2017-05-14 at 18:34 +0530, Vinod Koul wrote:
> > > On Tue, May 09, 2017 at 07:18:37PM +0300, Andy Shevchenko wrote:
> > > > AVR32 is gone. Now it's time to clean up the driver by removing
> > > > leftovers that was used by AVR32 related code.
> > > 
> > > Since the cover didn't mention any dependency, I went ahead and
> > > applied this
> > > now
> > > 
> > 
> > Sorry, I forgot to mention that it's supposed to go via either tree,
> > though together. Since first we remove users of non-standard DMA
> > callbacks.
> > 
> 
> Since there is no hurry maybe split it over multiple releases. Audio
> patch
> now through the audio tree and once it has made it to a upstream
> release
> apply the DMA patch through the DMA tree.

While there is indeed no hurry (I'm not expecting long patches against
dw_dmac), we may do that, though OTOH better to do it at once to
minimize a probability that the second change might be forgotten for
longer.

-- 
Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Intel Finland Oy

WARNING: multiple messages have this Message-ID (diff)
From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
To: Lars-Peter Clausen <lars@metafoo.de>, Vinod Koul <vinod.koul@intel.com>
Cc: Hans-Christian Noren Egtvedt <egtvedt@samfundet.no>,
	linux-kernel@vger.kernel.org,
	Haavard Skinnemoen <hskinnemoen@gmail.com>,
	Nicolas Ferre <nicolas.ferre@atmel.com>,
	Boris Brezillon <boris.brezillon@free-electrons.com>,
	Mark Brown <broonie@kernel.org>,
	Liam Girdwood <lgirdwood@gmail.com>,
	alsa-devel@alsa-project.org, dmaengine@vger.kernel.org,
	Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com>
Subject: Re: [PATCH v2 2/2] dmaengine: dw: Remove AVR32 bits from the driver
Date: Mon, 15 May 2017 12:42:30 +0300	[thread overview]
Message-ID: <1494841350.6967.55.camel@linux.intel.com> (raw)
In-Reply-To: <45cb3a0a-28b5-a64c-90ac-1abf0c59d70f@metafoo.de>

On Mon, 2017-05-15 at 11:25 +0200, Lars-Peter Clausen wrote:
> On 05/15/2017 10:43 AM, Andy Shevchenko wrote:
> > On Sun, 2017-05-14 at 18:34 +0530, Vinod Koul wrote:
> > > On Tue, May 09, 2017 at 07:18:37PM +0300, Andy Shevchenko wrote:
> > > > AVR32 is gone. Now it's time to clean up the driver by removing
> > > > leftovers that was used by AVR32 related code.
> > > 
> > > Since the cover didn't mention any dependency, I went ahead and
> > > applied this
> > > now
> > > 
> > 
> > Sorry, I forgot to mention that it's supposed to go via either tree,
> > though together. Since first we remove users of non-standard DMA
> > callbacks.
> > 
> 
> Since there is no hurry maybe split it over multiple releases. Audio
> patch
> now through the audio tree and once it has made it to a upstream
> release
> apply the DMA patch through the DMA tree.

While there is indeed no hurry (I'm not expecting long patches against
dw_dmac), we may do that, though OTOH better to do it at once to
minimize a probability that the second change might be forgotten for
longer.

-- 
Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Intel Finland Oy

  reply	other threads:[~2017-05-15  9:45 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-05-09 16:18 [PATCH v2 0/2] ALSA, dmaengine: dw: Remove AVR32 bits in drivers Andy Shevchenko
2017-05-09 16:18 ` [PATCH v2 1/2] ALSA: atmel: Remove AVR32 bits from the driver Andy Shevchenko
2017-05-09 16:18 ` [PATCH v2 2/2] dmaengine: dw: " Andy Shevchenko
2017-05-14 13:04   ` Vinod Koul
2017-05-14 13:04     ` Vinod Koul
2017-05-15  8:43     ` Andy Shevchenko
2017-05-15  8:43       ` Andy Shevchenko
2017-05-15  9:25       ` Lars-Peter Clausen
2017-05-15  9:42         ` Andy Shevchenko [this message]
2017-05-15  9:42           ` Andy Shevchenko
2017-05-15 14:49       ` Vinod Koul
2017-05-15 14:49         ` [alsa-devel] " Vinod Koul
2017-05-15 15:09         ` Takashi Iwai
2017-05-16  4:28           ` Vinod Koul
2017-05-17  8:20             ` Andy Shevchenko

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=1494841350.6967.55.camel@linux.intel.com \
    --to=andriy.shevchenko@linux.intel.com \
    --cc=Eugeniy.Paltsev@synopsys.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=boris.brezillon@free-electrons.com \
    --cc=broonie@kernel.org \
    --cc=dmaengine@vger.kernel.org \
    --cc=egtvedt@samfundet.no \
    --cc=hskinnemoen@gmail.com \
    --cc=lars@metafoo.de \
    --cc=lgirdwood@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nicolas.ferre@atmel.com \
    --cc=vinod.koul@intel.com \
    /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.