* [PATCH 1/1] dmaengine: s3c24xx-dma: use DMA_COMPLETE for dma completion status @ 2013-10-31 5:18 Sachin Kamat 2013-10-31 13:42 ` Vinod Koul 2013-10-31 15:00 ` Heiko Stübner 0 siblings, 2 replies; 8+ messages in thread From: Sachin Kamat @ 2013-10-31 5:18 UTC (permalink / raw) To: dmaengine Cc: linux-samsung-soc, dan.j.williams, kgene.kim, sachin.kamat, Heiko Stuebner, Vinod Koul Use the recently introduced DMA_COMPLETE instead of DMA_SUCCESS. Without this patch we get the following build error: drivers/dma/s3c24xx-dma.c: In function ‘s3c24xx_dma_tx_status’: drivers/dma/s3c24xx-dma.c:798:13: error: ‘DMA_SUCCESS’ undeclared (first use in this function) Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Cc: Heiko Stuebner <heiko@sntech.de> Cc: Vinod Koul <vinod.koul@intel.com> --- drivers/dma/s3c24xx-dma.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/dma/s3c24xx-dma.c b/drivers/dma/s3c24xx-dma.c index 4cb127978636..085da4fe6613 100644 --- a/drivers/dma/s3c24xx-dma.c +++ b/drivers/dma/s3c24xx-dma.c @@ -795,7 +795,7 @@ static enum dma_status s3c24xx_dma_tx_status(struct dma_chan *chan, spin_lock_irqsave(&s3cchan->vc.lock, flags); ret = dma_cookie_status(chan, cookie, txstate); - if (ret == DMA_SUCCESS) { + if (ret == DMA_COMPLETE) { spin_unlock_irqrestore(&s3cchan->vc.lock, flags); return ret; } -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH 1/1] dmaengine: s3c24xx-dma: use DMA_COMPLETE for dma completion status 2013-10-31 5:18 [PATCH 1/1] dmaengine: s3c24xx-dma: use DMA_COMPLETE for dma completion status Sachin Kamat @ 2013-10-31 13:42 ` Vinod Koul 2013-11-12 10:28 ` Kukjin Kim 2013-10-31 15:00 ` Heiko Stübner 1 sibling, 1 reply; 8+ messages in thread From: Vinod Koul @ 2013-10-31 13:42 UTC (permalink / raw) To: Sachin Kamat Cc: dmaengine, linux-samsung-soc, dan.j.williams, kgene.kim, Heiko Stuebner On Thu, Oct 31, 2013 at 10:48:09AM +0530, Sachin Kamat wrote: > Use the recently introduced DMA_COMPLETE instead of DMA_SUCCESS. > Without this patch we get the following build error: > drivers/dma/s3c24xx-dma.c: In function ‘s3c24xx_dma_tx_status’: > drivers/dma/s3c24xx-dma.c:798:13: error: ‘DMA_SUCCESS’ undeclared > (first use in this function) I missed it because this is not in my tree! This needs to go thru kgene's tree and for that you can merge dma_complete branch from my tree. I will not rebase that Acked-by: Vinod Koul <vinod.koul@intel.com> -- ~Vinod > > Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> > Cc: Heiko Stuebner <heiko@sntech.de> > Cc: Vinod Koul <vinod.koul@intel.com> > --- > drivers/dma/s3c24xx-dma.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/dma/s3c24xx-dma.c b/drivers/dma/s3c24xx-dma.c > index 4cb127978636..085da4fe6613 100644 > --- a/drivers/dma/s3c24xx-dma.c > +++ b/drivers/dma/s3c24xx-dma.c > @@ -795,7 +795,7 @@ static enum dma_status s3c24xx_dma_tx_status(struct dma_chan *chan, > > spin_lock_irqsave(&s3cchan->vc.lock, flags); > ret = dma_cookie_status(chan, cookie, txstate); > - if (ret == DMA_SUCCESS) { > + if (ret == DMA_COMPLETE) { > spin_unlock_irqrestore(&s3cchan->vc.lock, flags); > return ret; > } > -- > 1.7.9.5 > -- ^ permalink raw reply [flat|nested] 8+ messages in thread
* RE: [PATCH 1/1] dmaengine: s3c24xx-dma: use DMA_COMPLETE for dma completion status 2013-10-31 13:42 ` Vinod Koul @ 2013-11-12 10:28 ` Kukjin Kim 2013-11-13 8:58 ` Vinod Koul 0 siblings, 1 reply; 8+ messages in thread From: Kukjin Kim @ 2013-11-12 10:28 UTC (permalink / raw) To: 'Vinod Koul', 'Sachin Kamat' Cc: dmaengine, linux-samsung-soc, dan.j.williams, 'Heiko Stuebner' Vinod Koul wrote: > > On Thu, Oct 31, 2013 at 10:48:09AM +0530, Sachin Kamat wrote: > > Use the recently introduced DMA_COMPLETE instead of DMA_SUCCESS. > > Without this patch we get the following build error: > > drivers/dma/s3c24xx-dma.c: In function ‘s3c24xx_dma_tx_status’: > > drivers/dma/s3c24xx-dma.c:798:13: error: ‘DMA_SUCCESS’ undeclared > > (first use in this function) > I missed it because this is not in my tree! This needs to go thru kgene's > tree > and for that you can merge dma_complete branch from my tree. I will not > rebase > that > > Acked-by: Vinod Koul <vinod.koul@intel.com> > Oops, I missed :( Vinod, since the s3c24xx-dma is in Linus' tree now, you can go ahead with my ack into your fixes. If you want me to take this into samsung tree, let me know :-) Thanks, Kukjin > -- > ~Vinod > > > > Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> > > Cc: Heiko Stuebner <heiko@sntech.de> > > Cc: Vinod Koul <vinod.koul@intel.com> > > --- > > drivers/dma/s3c24xx-dma.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/dma/s3c24xx-dma.c b/drivers/dma/s3c24xx-dma.c > > index 4cb127978636..085da4fe6613 100644 > > --- a/drivers/dma/s3c24xx-dma.c > > +++ b/drivers/dma/s3c24xx-dma.c > > @@ -795,7 +795,7 @@ static enum dma_status s3c24xx_dma_tx_status(struct > dma_chan *chan, > > > > spin_lock_irqsave(&s3cchan->vc.lock, flags); > > ret = dma_cookie_status(chan, cookie, txstate); > > - if (ret == DMA_SUCCESS) { > > + if (ret == DMA_COMPLETE) { > > spin_unlock_irqrestore(&s3cchan->vc.lock, flags); > > return ret; > > } > > -- > > 1.7.9.5 > > > > -- ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/1] dmaengine: s3c24xx-dma: use DMA_COMPLETE for dma completion status 2013-11-12 10:28 ` Kukjin Kim @ 2013-11-13 8:58 ` Vinod Koul 2013-11-28 7:25 ` Vinod Koul 0 siblings, 1 reply; 8+ messages in thread From: Vinod Koul @ 2013-11-13 8:58 UTC (permalink / raw) To: Kukjin Kim Cc: 'Sachin Kamat', dmaengine, linux-samsung-soc, dan.j.williams, 'Heiko Stuebner' On Tue, Nov 12, 2013 at 07:28:10PM +0900, Kukjin Kim wrote: > Vinod Koul wrote: > > > > On Thu, Oct 31, 2013 at 10:48:09AM +0530, Sachin Kamat wrote: > > > Use the recently introduced DMA_COMPLETE instead of DMA_SUCCESS. > > > Without this patch we get the following build error: > > > drivers/dma/s3c24xx-dma.c: In function ‘s3c24xx_dma_tx_status’: > > > drivers/dma/s3c24xx-dma.c:798:13: error: ‘DMA_SUCCESS’ undeclared > > > (first use in this function) > > I missed it because this is not in my tree! This needs to go thru kgene's > > tree > > and for that you can merge dma_complete branch from my tree. I will not > > rebase > > that > > > > Acked-by: Vinod Koul <vinod.koul@intel.com> > > > > Oops, I missed :( > > Vinod, since the s3c24xx-dma is in Linus' tree now, you can go ahead with my ack into your fixes. > > If you want me to take this into samsung tree, let me know :-) Okay, I will apply this after merge window -- ~Vinod > > Thanks, > Kukjin > > > -- > > ~Vinod > > > > > > Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> > > > Cc: Heiko Stuebner <heiko@sntech.de> > > > Cc: Vinod Koul <vinod.koul@intel.com> > > > --- > > > drivers/dma/s3c24xx-dma.c | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/drivers/dma/s3c24xx-dma.c b/drivers/dma/s3c24xx-dma.c > > > index 4cb127978636..085da4fe6613 100644 > > > --- a/drivers/dma/s3c24xx-dma.c > > > +++ b/drivers/dma/s3c24xx-dma.c > > > @@ -795,7 +795,7 @@ static enum dma_status s3c24xx_dma_tx_status(struct > > dma_chan *chan, > > > > > > spin_lock_irqsave(&s3cchan->vc.lock, flags); > > > ret = dma_cookie_status(chan, cookie, txstate); > > > - if (ret == DMA_SUCCESS) { > > > + if (ret == DMA_COMPLETE) { > > > spin_unlock_irqrestore(&s3cchan->vc.lock, flags); > > > return ret; > > > } > > > -- > > > 1.7.9.5 > > > > > > > -- > > -- > To unsubscribe from this list: send the line "unsubscribe dmaengine" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/1] dmaengine: s3c24xx-dma: use DMA_COMPLETE for dma completion status 2013-11-13 8:58 ` Vinod Koul @ 2013-11-28 7:25 ` Vinod Koul 2013-11-28 22:33 ` Dan Williams 0 siblings, 1 reply; 8+ messages in thread From: Vinod Koul @ 2013-11-28 7:25 UTC (permalink / raw) To: Kukjin Kim Cc: 'Sachin Kamat', dmaengine, linux-samsung-soc, dan.j.williams, 'Heiko Stuebner' On Wed, Nov 13, 2013 at 02:28:58PM +0530, Vinod Koul wrote: > On Tue, Nov 12, 2013 at 07:28:10PM +0900, Kukjin Kim wrote: > > Vinod Koul wrote: > > > > > > On Thu, Oct 31, 2013 at 10:48:09AM +0530, Sachin Kamat wrote: > > > > Use the recently introduced DMA_COMPLETE instead of DMA_SUCCESS. > > > > Without this patch we get the following build error: > > > > drivers/dma/s3c24xx-dma.c: In function ‘s3c24xx_dma_tx_status’: > > > > drivers/dma/s3c24xx-dma.c:798:13: error: ‘DMA_SUCCESS’ undeclared > > > > (first use in this function) > > > I missed it because this is not in my tree! This needs to go thru kgene's > > > tree > > > and for that you can merge dma_complete branch from my tree. I will not > > > rebase > > > that > > > > > > Acked-by: Vinod Koul <vinod.koul@intel.com> > > > > > > > Oops, I missed :( > > > > Vinod, since the s3c24xx-dma is in Linus' tree now, you can go ahead with my ack into your fixes. > > > > If you want me to take this into samsung tree, let me know :-) > Okay, I will apply this after merge window Applied now, Thanks ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/1] dmaengine: s3c24xx-dma: use DMA_COMPLETE for dma completion status 2013-11-28 7:25 ` Vinod Koul @ 2013-11-28 22:33 ` Dan Williams 2013-11-29 5:11 ` Vinod Koul 0 siblings, 1 reply; 8+ messages in thread From: Dan Williams @ 2013-11-28 22:33 UTC (permalink / raw) To: Vinod Koul Cc: Kukjin Kim, Sachin Kamat, dmaengine@vger.kernel.org, linux-samsung-soc, Heiko Stuebner On Wed, Nov 27, 2013 at 11:25 PM, Vinod Koul <vinod.koul@intel.com> wrote: > On Wed, Nov 13, 2013 at 02:28:58PM +0530, Vinod Koul wrote: >> On Tue, Nov 12, 2013 at 07:28:10PM +0900, Kukjin Kim wrote: >> > Vinod Koul wrote: >> > > >> > > On Thu, Oct 31, 2013 at 10:48:09AM +0530, Sachin Kamat wrote: >> > > > Use the recently introduced DMA_COMPLETE instead of DMA_SUCCESS. >> > > > Without this patch we get the following build error: >> > > > drivers/dma/s3c24xx-dma.c: In function ‘s3c24xx_dma_tx_status’: >> > > > drivers/dma/s3c24xx-dma.c:798:13: error: ‘DMA_SUCCESS’ undeclared >> > > > (first use in this function) >> > > I missed it because this is not in my tree! This needs to go thru kgene's >> > > tree >> > > and for that you can merge dma_complete branch from my tree. I will not >> > > rebase >> > > that >> > > >> > > Acked-by: Vinod Koul <vinod.koul@intel.com> >> > > >> > >> > Oops, I missed :( >> > >> > Vinod, since the s3c24xx-dma is in Linus' tree now, you can go ahead with my ack into your fixes. >> > >> > If you want me to take this into samsung tree, let me know :-) >> Okay, I will apply this after merge window > Applied now, > This driver also missed the unmap rework. Fixing now. Vinod, I'll send you the pile after I finish my build regression suite. -- Dan ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/1] dmaengine: s3c24xx-dma: use DMA_COMPLETE for dma completion status 2013-11-28 22:33 ` Dan Williams @ 2013-11-29 5:11 ` Vinod Koul 0 siblings, 0 replies; 8+ messages in thread From: Vinod Koul @ 2013-11-29 5:11 UTC (permalink / raw) To: Dan Williams Cc: Kukjin Kim, Sachin Kamat, dmaengine@vger.kernel.org, linux-samsung-soc, Heiko Stuebner On Thu, Nov 28, 2013 at 02:33:22PM -0800, Dan Williams wrote: > On Wed, Nov 27, 2013 at 11:25 PM, Vinod Koul <vinod.koul@intel.com> wrote: > > On Wed, Nov 13, 2013 at 02:28:58PM +0530, Vinod Koul wrote: > >> On Tue, Nov 12, 2013 at 07:28:10PM +0900, Kukjin Kim wrote: > >> > Vinod Koul wrote: > >> > > > >> > > On Thu, Oct 31, 2013 at 10:48:09AM +0530, Sachin Kamat wrote: > >> > > > Use the recently introduced DMA_COMPLETE instead of DMA_SUCCESS. > >> > > > Without this patch we get the following build error: > >> > > > drivers/dma/s3c24xx-dma.c: In function ‘s3c24xx_dma_tx_status’: > >> > > > drivers/dma/s3c24xx-dma.c:798:13: error: ‘DMA_SUCCESS’ undeclared > >> > > > (first use in this function) > >> > > I missed it because this is not in my tree! This needs to go thru kgene's > >> > > tree > >> > > and for that you can merge dma_complete branch from my tree. I will not > >> > > rebase > >> > > that > >> > > > >> > > Acked-by: Vinod Koul <vinod.koul@intel.com> > >> > > > >> > > >> > Oops, I missed :( > >> > > >> > Vinod, since the s3c24xx-dma is in Linus' tree now, you can go ahead with my ack into your fixes. > >> > > >> > If you want me to take this into samsung tree, let me know :-) > >> Okay, I will apply this after merge window > > Applied now, > > > > This driver also missed the unmap rework. Fixing now. yup, this is a new driver and got merged thru samsung tree > Vinod, I'll send you the pile after I finish my build regression suite. sure, i think these should go into fixes -- ~Vinod ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/1] dmaengine: s3c24xx-dma: use DMA_COMPLETE for dma completion status 2013-10-31 5:18 [PATCH 1/1] dmaengine: s3c24xx-dma: use DMA_COMPLETE for dma completion status Sachin Kamat 2013-10-31 13:42 ` Vinod Koul @ 2013-10-31 15:00 ` Heiko Stübner 1 sibling, 0 replies; 8+ messages in thread From: Heiko Stübner @ 2013-10-31 15:00 UTC (permalink / raw) To: Sachin Kamat Cc: dmaengine, linux-samsung-soc, dan.j.williams, kgene.kim, Vinod Koul Am Donnerstag, 31. Oktober 2013, 06:18:09 schrieb Sachin Kamat: > Use the recently introduced DMA_COMPLETE instead of DMA_SUCCESS. > Without this patch we get the following build error: > drivers/dma/s3c24xx-dma.c: In function ‘s3c24xx_dma_tx_status’: > drivers/dma/s3c24xx-dma.c:798:13: error: ‘DMA_SUCCESS’ undeclared > (first use in this function) > > Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> thanks for the catch! Acked-by: Heiko Stuebner <heiko@sntech.de> > Cc: Heiko Stuebner <heiko@sntech.de> > Cc: Vinod Koul <vinod.koul@intel.com> > --- > drivers/dma/s3c24xx-dma.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/dma/s3c24xx-dma.c b/drivers/dma/s3c24xx-dma.c > index 4cb127978636..085da4fe6613 100644 > --- a/drivers/dma/s3c24xx-dma.c > +++ b/drivers/dma/s3c24xx-dma.c > @@ -795,7 +795,7 @@ static enum dma_status s3c24xx_dma_tx_status(struct > dma_chan *chan, > > spin_lock_irqsave(&s3cchan->vc.lock, flags); > ret = dma_cookie_status(chan, cookie, txstate); > - if (ret == DMA_SUCCESS) { > + if (ret == DMA_COMPLETE) { > spin_unlock_irqrestore(&s3cchan->vc.lock, flags); > return ret; > } ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2013-11-29 6:08 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-10-31 5:18 [PATCH 1/1] dmaengine: s3c24xx-dma: use DMA_COMPLETE for dma completion status Sachin Kamat 2013-10-31 13:42 ` Vinod Koul 2013-11-12 10:28 ` Kukjin Kim 2013-11-13 8:58 ` Vinod Koul 2013-11-28 7:25 ` Vinod Koul 2013-11-28 22:33 ` Dan Williams 2013-11-29 5:11 ` Vinod Koul 2013-10-31 15:00 ` Heiko Stübner
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.