Re: [PATCH V7 3/3] dma: add Qualcomm Technologies HIDMA channel driver

From	Vinod Koul <vinod.koul@intel.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH V7 3/3] dma: add Qualcomm Technologies HIDMA channel driver
Date	2015-12-05 09:00 +0100
Message-ID	<qCfV8-6f3-9@gated-at.bofh.it> (permalink)
References	(2 earlier) <qAsts-1kv-1@gated-at.bofh.it> <qACVQ-8dD-15@gated-at.bofh.it> <qARrQ-Ak-13@gated-at.bofh.it> <qB0v7-6w7-5@gated-at.bofh.it> <qBkWS-2Tg-7@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Wed, Dec 02, 2015 at 02:04:05PM -0500, Sinan Kaya wrote:
> >> > You are missing the point. Channel can be paused, yes but the descriptor
> >> > is in queue and is not paused. The descriptor running is paused, yes.
> >> > There is subtle difference between these
> > I'll follow your recommendation. PAUSE for the currently active
> > descriptor and DMA_IN_PROGRESS for the rest.
> > 
> 
> I'm now confused.
> 
> I looked at several DMA driver implementations.
> 
> 1. They call dma_cookie_status function to see if the job is done.
> 2. If done, they return right ahead.
> 3. Otherwise, dma_cookie_status returns DMA_IN_PROGRESS.
> 4. Next the code checks if the channel is paused and return value is
> DMA_IN_PROGRESS. The code changes return code to DMA_PAUSED.
> 
> Whereas, I was returning paused first before even checking if the
> descriptor is done. Are you OK with the sequence 1..4 above?

Yes am okay with this with slight change in 4.

You should set to PAUSED only for current descriptor and not for the ones in
queue

-- 
~Vinod
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Thread

Re: [PATCH V7 3/3] dma: add Qualcomm Technologies HIDMA channel  driver Sinan Kaya <okaya@codeaurora.org> - 2015-11-30 21:10 +0100
  Re: [PATCH V7 3/3] dma: add Qualcomm Technologies HIDMA channel  driver Vinod Koul <vinod.koul@intel.com> - 2015-12-01 12:40 +0100
    Re: [PATCH V7 3/3] dma: add Qualcomm Technologies HIDMA channel  driver Sinan Kaya <okaya@codeaurora.org> - 2015-12-01 22:20 +0100
      Re: [PATCH V7 3/3] dma: add Qualcomm Technologies HIDMA channel  driver Sinan Kaya <okaya@codeaurora.org> - 2015-12-02 20:10 +0100
        Re: [PATCH V7 3/3] dma: add Qualcomm Technologies HIDMA channel  driver Vinod Koul <vinod.koul@intel.com> - 2015-12-05 09:00 +0100
          Re: [PATCH V7 3/3] dma: add Qualcomm Technologies HIDMA channel  driver Sinan Kaya <okaya@codeaurora.org> - 2015-12-08 15:40 +0100
          Re: [PATCH V7 3/3] dma: add Qualcomm Technologies HIDMA channel  driver Sinan Kaya <okaya@codeaurora.org> - 2015-12-10 21:20 +0100
            Re: [PATCH V7 3/3] dma: add Qualcomm Technologies HIDMA channel  driver Vinod Koul <vinod.koul@intel.com> - 2015-12-11 10:40 +0100

csiph-web