Re: [PATCH] dmaengine: pl330: fix the race condition in pl330 driver.

2015-03-31 Thread Scott Branden
Hi Jassi, Thanks for taking the time to comment on this patch and provide additional solution. We have went back to reproduce the problem using the dmatest. I am glad you asked for more info as we discovered the problem does not happen in the current code. The problem only happens when we

Re: [PATCH] dmaengine: pl330: fix the race condition in pl330 driver.

2015-03-31 Thread Scott Branden
Hi Jassi, Thanks for taking the time to comment on this patch and provide additional solution. We have went back to reproduce the problem using the dmatest. I am glad you asked for more info as we discovered the problem does not happen in the current code. The problem only happens when we

Re: [PATCH] dmaengine: pl330: fix the race condition in pl330 driver.

2015-03-30 Thread Jassi Brar
On Tue, Mar 31, 2015 at 9:10 AM, Scott Branden wrote: > Hi Vinod, Jassi, > > Some details on the problem encountered. > > > On 15-03-30 10:25 AM, Vinod Koul wrote: >> >> On Mon, Mar 30, 2015 at 10:17:17PM +0530, Jassi Brar wrote: >>> >>> On Fri, Mar 27, 2015 at 5:25 AM, Scott Branden >>> wrote:

Re: [PATCH] dmaengine: pl330: fix the race condition in pl330 driver.

2015-03-30 Thread Scott Branden
Hi Vinod, Jassi, Some details on the problem encountered. On 15-03-30 10:25 AM, Vinod Koul wrote: On Mon, Mar 30, 2015 at 10:17:17PM +0530, Jassi Brar wrote: On Fri, Mar 27, 2015 at 5:25 AM, Scott Branden wrote: From: ismail Update the thread running index before issuing the GO command to

Re: [PATCH] dmaengine: pl330: fix the race condition in pl330 driver.

2015-03-30 Thread Vinod Koul
On Mon, Mar 30, 2015 at 10:17:17PM +0530, Jassi Brar wrote: > On Fri, Mar 27, 2015 at 5:25 AM, Scott Branden wrote: > > From: ismail > > > > Update the thread running index before issuing the > > GO command to the DMAC. > > > > Tested-by: Mohamed Ismail Abdul Packir Mohamed > > Reviewed-by: Ray

Re: [PATCH] dmaengine: pl330: fix the race condition in pl330 driver.

2015-03-30 Thread Jassi Brar
On Fri, Mar 27, 2015 at 5:25 AM, Scott Branden wrote: > From: ismail > > Update the thread running index before issuing the > GO command to the DMAC. > > Tested-by: Mohamed Ismail Abdul Packir Mohamed > Reviewed-by: Ray Jui > Reviewed-by: Arun Parameswaran > Reviewed-by: Scott Branden >

Re: [PATCH] dmaengine: pl330: fix the race condition in pl330 driver.

2015-03-30 Thread Vinod Koul
On Mon, Mar 30, 2015 at 10:17:17PM +0530, Jassi Brar wrote: On Fri, Mar 27, 2015 at 5:25 AM, Scott Branden sbran...@broadcom.com wrote: From: ismail ism...@broadcom.com Update the thread running index before issuing the GO command to the DMAC. Tested-by: Mohamed Ismail Abdul Packir

Re: [PATCH] dmaengine: pl330: fix the race condition in pl330 driver.

2015-03-30 Thread Jassi Brar
On Fri, Mar 27, 2015 at 5:25 AM, Scott Branden sbran...@broadcom.com wrote: From: ismail ism...@broadcom.com Update the thread running index before issuing the GO command to the DMAC. Tested-by: Mohamed Ismail Abdul Packir Mohamed ism...@broadcom.com Reviewed-by: Ray Jui r...@broadcom.com

Re: [PATCH] dmaengine: pl330: fix the race condition in pl330 driver.

2015-03-30 Thread Jassi Brar
On Tue, Mar 31, 2015 at 9:10 AM, Scott Branden sbran...@broadcom.com wrote: Hi Vinod, Jassi, Some details on the problem encountered. On 15-03-30 10:25 AM, Vinod Koul wrote: On Mon, Mar 30, 2015 at 10:17:17PM +0530, Jassi Brar wrote: On Fri, Mar 27, 2015 at 5:25 AM, Scott Branden

Re: [PATCH] dmaengine: pl330: fix the race condition in pl330 driver.

2015-03-30 Thread Scott Branden
Hi Vinod, Jassi, Some details on the problem encountered. On 15-03-30 10:25 AM, Vinod Koul wrote: On Mon, Mar 30, 2015 at 10:17:17PM +0530, Jassi Brar wrote: On Fri, Mar 27, 2015 at 5:25 AM, Scott Branden sbran...@broadcom.com wrote: From: ismail ism...@broadcom.com Update the thread

Re: [PATCH] dmaengine: pl330: fix the race condition in pl330 driver.

2015-03-29 Thread Scott Branden
Adding original author Jaswinder Singh (3 email addresses) to email list. On 15-03-26 04:55 PM, Scott Branden wrote: From: ismail Update the thread running index before issuing the GO command to the DMAC. Tested-by: Mohamed Ismail Abdul Packir Mohamed Reviewed-by: Ray Jui Reviewed-by: Arun

Re: [PATCH] dmaengine: pl330: fix the race condition in pl330 driver.

2015-03-29 Thread Scott Branden
Adding original author Jaswinder Singh (3 email addresses) to email list. On 15-03-26 04:55 PM, Scott Branden wrote: From: ismail ism...@broadcom.com Update the thread running index before issuing the GO command to the DMAC. Tested-by: Mohamed Ismail Abdul Packir Mohamed ism...@broadcom.com

[PATCH] dmaengine: pl330: fix the race condition in pl330 driver.

2015-03-26 Thread Scott Branden
From: ismail Update the thread running index before issuing the GO command to the DMAC. Tested-by: Mohamed Ismail Abdul Packir Mohamed Reviewed-by: Ray Jui Reviewed-by: Arun Parameswaran Reviewed-by: Scott Branden Signed-off-by: Scott Branden Signed-off-by: Mohamed Ismail Abdul Packir

[PATCH] dmaengine: pl330: fix the race condition in pl330 driver.

2015-03-26 Thread Scott Branden
From: ismail ism...@broadcom.com Update the thread running index before issuing the GO command to the DMAC. Tested-by: Mohamed Ismail Abdul Packir Mohamed ism...@broadcom.com Reviewed-by: Ray Jui r...@broadcom.com Reviewed-by: Arun Parameswaran apara...@broadcom.com Reviewed-by: Scott Branden