[PATCH] media: atmel-isi: increase the burst length to improve the performance

Hi Josh,

On Mon, 2 Feb 2015, Josh Wu wrote:

Hi, Guennadi

Ping? what about the status of this patch?

Right, got lost, sorry... Added to the queue now.

Thanks
Guennadi

Best Regards,
Josh Wu

On 11/25/2014 5:30 PM, Josh Wu wrote:

quoted

The burst length could be BEATS_4/8/16. Before this patch, isi use default
value BEATS_4. To imporve the performance we could set it to BEATS_16.

Otherwise sometime it would cause the ISI overflow error.

Reported-by: Bo Shen <redacted>
Signed-off-by: Josh Wu <redacted>
---
  drivers/media/platform/soc_camera/atmel-isi.c | 2 ++
  include/media/atmel-isi.h                     | 4 ++++
  2 files changed, 6 insertions(+)

diff --git a/drivers/media/platform/soc_camera/atmel-isi.c

b/drivers/media/platform/soc_camera/atmel-isi.c
index ee5650f..fda587b 100644

--- a/drivers/media/platform/soc_camera/atmel-isi.c
+++ b/drivers/media/platform/soc_camera/atmel-isi.c

@@ -839,6 +839,8 @@ static int isi_camera_set_bus_param(struct

soc_camera_device *icd)
  	if (isi->pdata.full_mode)
  		cfg1 |= ISI_CFG1_FULL_MODE;
  +	cfg1 |= ISI_CFG1_THMASK_BEATS_16;
+
  	isi_writel(isi, ISI_CTRL, ISI_CTRL_DIS);
  	isi_writel(isi, ISI_CFG1, cfg1);
  diff --git a/include/media/atmel-isi.h b/include/media/atmel-isi.h
index c2e5703..6008b09 100644

--- a/include/media/atmel-isi.h
+++ b/include/media/atmel-isi.h

@@ -59,6 +59,10 @@
  #define		ISI_CFG1_FRATE_DIV_MASK		(7 << 8)
  #define ISI_CFG1_DISCR				(1 << 11)
  #define ISI_CFG1_FULL_MODE			(1 << 12)
+/* Definition for THMASK(ISI_V2) */
+#define		ISI_CFG1_THMASK_BEATS_4		(0 << 13)
+#define		ISI_CFG1_THMASK_BEATS_8		(1 << 13)
+#define		ISI_CFG1_THMASK_BEATS_16	(2 << 13)
    /* Bitfields in CFG2 */
  #define ISI_CFG2_GRAYSCALE			(1 << 13)