Re: [PATCH linux-next v2 1/1] spi: imx: dynamic burst length adjust for PIO mode

2017-05-18 Thread Leonard Crestez
On Wed, 2017-05-17 at 18:50 -0700, Jiada Wang wrote: > Hello Leonard > > Thanks for the report, can you help to check if the following change  > address the issue? > > diff --git a/drivers/spi/spi-imx.c b/drivers/spi/spi-imx.c > index 782045f..19b30cf 100644 > --- a/drivers/spi/spi-imx.c > +++

Re: [PATCH linux-next v2 1/1] spi: imx: dynamic burst length adjust for PIO mode

2017-05-18 Thread Leonard Crestez
On Wed, 2017-05-17 at 18:50 -0700, Jiada Wang wrote: > Hello Leonard > > Thanks for the report, can you help to check if the following change  > address the issue? > > diff --git a/drivers/spi/spi-imx.c b/drivers/spi/spi-imx.c > index 782045f..19b30cf 100644 > --- a/drivers/spi/spi-imx.c > +++

Re: [PATCH linux-next v2 1/1] spi: imx: dynamic burst length adjust for PIO mode

2017-05-17 Thread Jiada Wang
Hello Leonard Thanks for the report, can you help to check if the following change address the issue? diff --git a/drivers/spi/spi-imx.c b/drivers/spi/spi-imx.c index 782045f..19b30cf 100644 --- a/drivers/spi/spi-imx.c +++ b/drivers/spi/spi-imx.c @@ -288,6 +288,9 @@ static void

Re: [PATCH linux-next v2 1/1] spi: imx: dynamic burst length adjust for PIO mode

2017-05-17 Thread Jiada Wang
Hello Leonard Thanks for the report, can you help to check if the following change address the issue? diff --git a/drivers/spi/spi-imx.c b/drivers/spi/spi-imx.c index 782045f..19b30cf 100644 --- a/drivers/spi/spi-imx.c +++ b/drivers/spi/spi-imx.c @@ -288,6 +288,9 @@ static void

Re: [PATCH linux-next v2 1/1] spi: imx: dynamic burst length adjust for PIO mode

2017-05-17 Thread Leonard Crestez
On Mon, 2017-05-01 at 03:31 -0700, jiada_w...@mentor.com wrote: > From: Jiada Wang > > previously burst length (BURST_LENGTH) is always set to equal > to bits_per_word, causes a 10us gap between each word in > transfer, which significantly affects performance. > > This

Re: [PATCH linux-next v2 1/1] spi: imx: dynamic burst length adjust for PIO mode

2017-05-17 Thread Leonard Crestez
On Mon, 2017-05-01 at 03:31 -0700, jiada_w...@mentor.com wrote: > From: Jiada Wang > > previously burst length (BURST_LENGTH) is always set to equal > to bits_per_word, causes a 10us gap between each word in > transfer, which significantly affects performance. > > This patch uses 32 bits

[PATCH linux-next v2 1/1] spi: imx: dynamic burst length adjust for PIO mode

2017-05-01 Thread jiada_wang
From: Jiada Wang previously burst length (BURST_LENGTH) is always set to equal to bits_per_word, causes a 10us gap between each word in transfer, which significantly affects performance. This patch uses 32 bits transfer to simulate lower bits transfer, and adjusts burst

[PATCH linux-next v2 1/1] spi: imx: dynamic burst length adjust for PIO mode

2017-05-01 Thread jiada_wang
From: Jiada Wang previously burst length (BURST_LENGTH) is always set to equal to bits_per_word, causes a 10us gap between each word in transfer, which significantly affects performance. This patch uses 32 bits transfer to simulate lower bits transfer, and adjusts burst length runtimely to use