RE: [PATCH v2] dmaengine: fsl-edma: add PM suspend/resume support

2015-09-08 Thread Li Leo
> On Thu, Aug 20, 2015 at 12:08:00PM +0800, Vinod Koul wrote: > > On Mon, Aug 17, 2015 at 02:10:46PM -0500, Li Yang wrote: > > > >> Think of it from the end user perspective. Would you like your > > > >> laptop (or > > > >> whatever) to refuse to suspend because of this condition? The > > > >> use

RE: [PATCH v2] dmaengine: fsl-edma: add PM suspend/resume support

2015-09-07 Thread Yao Yuan
On Thu, Aug 20, 2015 at 12:08:00PM +0800, Vinod Koul wrote: > On Mon, Aug 17, 2015 at 02:10:46PM -0500, Li Yang wrote: > > >> Think of it from the end user perspective. Would you like your > > >> laptop (or > > >> whatever) to refuse to suspend because of this condition? The user > > >> may well ex

Re: [PATCH v2] dmaengine: fsl-edma: add PM suspend/resume support

2015-08-19 Thread Vinod Koul
On Mon, Aug 17, 2015 at 02:10:46PM -0500, Li Yang wrote: > >> Think of it from the end user perspective. Would you like your laptop (or > >> whatever) to refuse to suspend because of this condition? The user may well > >> expect that closing the lid on their laptop will reliably lead to it > >> su

Re: [PATCH v2] dmaengine: fsl-edma: add PM suspend/resume support

2015-08-17 Thread Li Yang
On Mon, Aug 17, 2015 at 2:22 AM, Yao Yuan wrote: > Hi Nigel, > > On Mon, Aug 17, 2015 at 2:49 PM, Nigel Cunningham < > ni...@nigelcunningham.com.au > wrote: >> On 17/08/15 13:59, Yao Yuan wrote: >> > On Sat, Aug 15, 2015 at 7:48 AM, pku.leo < pku@gmail.com > wrote: >> >> On Fri, Aug 14, 2015

RE: [PATCH v2] dmaengine: fsl-edma: add PM suspend/resume support

2015-08-17 Thread Yao Yuan
Hi Nigel, On Mon, Aug 17, 2015 at 2:49 PM, Nigel Cunningham < ni...@nigelcunningham.com.au > wrote: > On 17/08/15 13:59, Yao Yuan wrote: > > On Sat, Aug 15, 2015 at 7:48 AM, pku.leo < pku@gmail.com > wrote: > >> On Fri, Aug 14, 2015 at 1:24 AM, Yao Yuan wrote: > >>> Hi Leo, > >>> > >>> Thank

Re: [PATCH v2] dmaengine: fsl-edma: add PM suspend/resume support

2015-08-16 Thread Nigel Cunningham
Hi Yao. On 17/08/15 13:59, Yao Yuan wrote: > On Sat, Aug 15, 2015 at 7:48 AM, pku.leo < pku@gmail.com > wrote: >> On Fri, Aug 14, 2015 at 1:24 AM, Yao Yuan wrote: >>> Hi Leo, >>> >>> Thanks for your review. >>> About those two methods for DMA suspend that you have mentioned. We >> have a lot

RE: [PATCH v2] dmaengine: fsl-edma: add PM suspend/resume support

2015-08-16 Thread Yao Yuan
> > > Best Regards, > > Yuan Yao > > > >> -Original Message- > >> From: pku@gmail.com [mailto:pku@gmail.com] On Behalf Of Li > >> Yang > >> Sent: Friday, August 14, 2015 4:58 AM > >> To: Yuan Yao-B46683 > >

Re: [PATCH v2] dmaengine: fsl-edma: add PM suspend/resume support

2015-08-14 Thread Li Yang
>> Cc: Vinod Koul; ste...@agner.ch; Arnd Bergmann; Dan Williams; >> dmaeng...@vger.kernel.org; lkml; linux-arm-ker...@lists.infradead.org; linux- >> p...@vger.kernel.org >> Subject: Re: [PATCH v2] dmaengine: fsl-edma: add PM suspend/resume >> support >> >>

RE: [PATCH v2] dmaengine: fsl-edma: add PM suspend/resume support

2015-08-13 Thread Yao Yuan
te...@agner.ch; Arnd Bergmann; Dan Williams; > dmaeng...@vger.kernel.org; lkml; linux-arm-ker...@lists.infradead.org; linux- > p...@vger.kernel.org > Subject: Re: [PATCH v2] dmaengine: fsl-edma: add PM suspend/resume > support > > On Tue, Jul 21, 2015 at 3:56 AM, Yuan Yao wrote:

Re: [PATCH v2] dmaengine: fsl-edma: add PM suspend/resume support

2015-08-13 Thread Li Yang
On Tue, Jul 21, 2015 at 3:56 AM, Yuan Yao wrote: > This add power management suspend/resume support for the fsl-edma > driver. > > eDMA acted as a basic function used by others. What it needs to do is > the two steps below to support power management. > > In fsl_edma_suspend_late: > Check whether

RE: [PATCH v2] dmaengine: fsl-edma: add PM suspend/resume support

2015-08-11 Thread Yao Yuan
ao > Sent: Tuesday, July 21, 2015 4:57 PM > To: vinod.k...@intel.com; ste...@agner.ch; a...@arndb.de > Cc: dmaeng...@vger.kernel.org; dan.j.willi...@intel.com; linux- > ker...@vger.kernel.org; linux-arm-ker...@lists.infradead.org > Subject: [PATCH v2] dmaengine: fsl-edma: add P

[PATCH v2] dmaengine: fsl-edma: add PM suspend/resume support

2015-07-21 Thread Yuan Yao
This add power management suspend/resume support for the fsl-edma driver. eDMA acted as a basic function used by others. What it needs to do is the two steps below to support power management. In fsl_edma_suspend_late: Check whether the DMA chan is idle and if it is not idle, stop PM operation.