* [PATCH v2] media: atmel-isi: parse the DT parameters for vsync/hsync/pixclock polarity
@ 2015-08-04 6:47 Josh Wu
2015-08-04 8:33 ` Laurent Pinchart
0 siblings, 1 reply; 3+ messages in thread
From: Josh Wu @ 2015-08-04 6:47 UTC (permalink / raw)
To: linux-arm-kernel
This patch will get the DT parameters of vsync/hsync/pixclock polarity, and
pass to driver.
Also add a debug information for test purpose.
Signed-off-by: Josh Wu <josh.wu@atmel.com>
---
Changes in v2:
- rewrite the debug message and add pix clock polarity setup thanks to
Laurent.
- update the commit log.
drivers/media/platform/soc_camera/atmel-isi.c | 12 ++++++++++++
1 file changed, 12 insertions(+)
diff --git a/drivers/media/platform/soc_camera/atmel-isi.c b/drivers/media/platform/soc_camera/atmel-isi.c
index fead841..d6f8def 100644
--- a/drivers/media/platform/soc_camera/atmel-isi.c
+++ b/drivers/media/platform/soc_camera/atmel-isi.c
@@ -1061,6 +1061,11 @@ static int isi_camera_set_bus_param(struct soc_camera_device *icd)
if (common_flags & V4L2_MBUS_PCLK_SAMPLE_FALLING)
cfg1 |= ISI_CFG1_PIXCLK_POL_ACTIVE_FALLING;
+ dev_dbg(icd->parent, "vsync active %s, hsync active %s, sampling on pix clock %s\n",
+ common_flags & V4L2_MBUS_VSYNC_ACTIVE_LOW ? "low" : "high",
+ common_flags & V4L2_MBUS_HSYNC_ACTIVE_LOW ? "low" : "high",
+ common_flags & V4L2_MBUS_PCLK_SAMPLE_FALLING ? "falling" : "rising");
+
if (isi->pdata.has_emb_sync)
cfg1 |= ISI_CFG1_EMB_SYNC;
if (isi->pdata.full_mode)
@@ -1148,6 +1153,13 @@ static int atmel_isi_parse_dt(struct atmel_isi *isi,
return -EINVAL;
}
+ if (ep.bus.parallel.flags & V4L2_MBUS_HSYNC_ACTIVE_LOW)
+ isi->pdata.hsync_act_low = true;
+ if (ep.bus.parallel.flags & V4L2_MBUS_VSYNC_ACTIVE_LOW)
+ isi->pdata.vsync_act_low = true;
+ if (ep.bus.parallel.flags & V4L2_MBUS_PCLK_SAMPLE_FALLING)
+ isi->pdata.pclk_act_falling = true;
+
return 0;
}
--
1.9.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH v2] media: atmel-isi: parse the DT parameters for vsync/hsync/pixclock polarity
2015-08-04 6:47 [PATCH v2] media: atmel-isi: parse the DT parameters for vsync/hsync/pixclock polarity Josh Wu
@ 2015-08-04 8:33 ` Laurent Pinchart
2015-08-04 8:41 ` Josh Wu
0 siblings, 1 reply; 3+ messages in thread
From: Laurent Pinchart @ 2015-08-04 8:33 UTC (permalink / raw)
To: linux-arm-kernel
Hi Josh,
Thank you for the patch.
On Tuesday 04 August 2015 14:47:07 Josh Wu wrote:
> This patch will get the DT parameters of vsync/hsync/pixclock polarity, and
> pass to driver.
>
> Also add a debug information for test purpose.
>
> Signed-off-by: Josh Wu <josh.wu@atmel.com>
> ---
>
> Changes in v2:
> - rewrite the debug message and add pix clock polarity setup thanks to
> Laurent.
> - update the commit log.
>
> drivers/media/platform/soc_camera/atmel-isi.c | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
> diff --git a/drivers/media/platform/soc_camera/atmel-isi.c
> b/drivers/media/platform/soc_camera/atmel-isi.c index fead841..d6f8def
> 100644
> --- a/drivers/media/platform/soc_camera/atmel-isi.c
> +++ b/drivers/media/platform/soc_camera/atmel-isi.c
> @@ -1061,6 +1061,11 @@ static int isi_camera_set_bus_param(struct
> soc_camera_device *icd) if (common_flags & V4L2_MBUS_PCLK_SAMPLE_FALLING)
> cfg1 |= ISI_CFG1_PIXCLK_POL_ACTIVE_FALLING;
>
> + dev_dbg(icd->parent, "vsync active %s, hsync active %s, sampling on pix
> clock %s\n",
You forgot " edge" right before "\n" :-)
> + common_flags & V4L2_MBUS_VSYNC_ACTIVE_LOW ? "low" : "high",
> + common_flags & V4L2_MBUS_HSYNC_ACTIVE_LOW ? "low" : "high",
> + common_flags & V4L2_MBUS_PCLK_SAMPLE_FALLING ? "falling" : "rising");
> +
> if (isi->pdata.has_emb_sync)
> cfg1 |= ISI_CFG1_EMB_SYNC;
> if (isi->pdata.full_mode)
> @@ -1148,6 +1153,13 @@ static int atmel_isi_parse_dt(struct atmel_isi *isi,
> return -EINVAL;
> }
>
> + if (ep.bus.parallel.flags & V4L2_MBUS_HSYNC_ACTIVE_LOW)
> + isi->pdata.hsync_act_low = true;
> + if (ep.bus.parallel.flags & V4L2_MBUS_VSYNC_ACTIVE_LOW)
> + isi->pdata.vsync_act_low = true;
> + if (ep.bus.parallel.flags & V4L2_MBUS_PCLK_SAMPLE_FALLING)
> + isi->pdata.pclk_act_falling = true;
> +
How about has_emb_sync, which can be set from ep.bus_type ?
> return 0;
> }
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH v2] media: atmel-isi: parse the DT parameters for vsync/hsync/pixclock polarity
2015-08-04 8:33 ` Laurent Pinchart
@ 2015-08-04 8:41 ` Josh Wu
0 siblings, 0 replies; 3+ messages in thread
From: Josh Wu @ 2015-08-04 8:41 UTC (permalink / raw)
To: linux-arm-kernel
Hi, Laurent
On 8/4/2015 4:33 PM, Laurent Pinchart wrote:
> Hi Josh,
>
> Thank you for the patch.
>
> On Tuesday 04 August 2015 14:47:07 Josh Wu wrote:
>> This patch will get the DT parameters of vsync/hsync/pixclock polarity, and
>> pass to driver.
>>
>> Also add a debug information for test purpose.
>>
>> Signed-off-by: Josh Wu <josh.wu@atmel.com>
>> ---
>>
>> Changes in v2:
>> - rewrite the debug message and add pix clock polarity setup thanks to
>> Laurent.
>> - update the commit log.
>>
>> drivers/media/platform/soc_camera/atmel-isi.c | 12 ++++++++++++
>> 1 file changed, 12 insertions(+)
>>
>> diff --git a/drivers/media/platform/soc_camera/atmel-isi.c
>> b/drivers/media/platform/soc_camera/atmel-isi.c index fead841..d6f8def
>> 100644
>> --- a/drivers/media/platform/soc_camera/atmel-isi.c
>> +++ b/drivers/media/platform/soc_camera/atmel-isi.c
>> @@ -1061,6 +1061,11 @@ static int isi_camera_set_bus_param(struct
>> soc_camera_device *icd) if (common_flags & V4L2_MBUS_PCLK_SAMPLE_FALLING)
>> cfg1 |= ISI_CFG1_PIXCLK_POL_ACTIVE_FALLING;
>>
>> + dev_dbg(icd->parent, "vsync active %s, hsync active %s, sampling on pix
>> clock %s\n",
> You forgot " edge" right before "\n" :-)
oops, I missed the word.
>
>> + common_flags & V4L2_MBUS_VSYNC_ACTIVE_LOW ? "low" : "high",
>> + common_flags & V4L2_MBUS_HSYNC_ACTIVE_LOW ? "low" : "high",
>> + common_flags & V4L2_MBUS_PCLK_SAMPLE_FALLING ? "falling" : "rising");
>> +
>> if (isi->pdata.has_emb_sync)
>> cfg1 |= ISI_CFG1_EMB_SYNC;
>> if (isi->pdata.full_mode)
>> @@ -1148,6 +1153,13 @@ static int atmel_isi_parse_dt(struct atmel_isi *isi,
>> return -EINVAL;
>> }
>>
>> + if (ep.bus.parallel.flags & V4L2_MBUS_HSYNC_ACTIVE_LOW)
>> + isi->pdata.hsync_act_low = true;
>> + if (ep.bus.parallel.flags & V4L2_MBUS_VSYNC_ACTIVE_LOW)
>> + isi->pdata.vsync_act_low = true;
>> + if (ep.bus.parallel.flags & V4L2_MBUS_PCLK_SAMPLE_FALLING)
>> + isi->pdata.pclk_act_falling = true;
>> +
> How about has_emb_sync, which can be set from ep.bus_type ?
yes, right. I forgot this as well.
I'll generate a new version of this patch soon. Thanks for the catching.
Best Regards,
Josh Wu
>
>> return 0;
>> }
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-08-04 8:41 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-08-04 6:47 [PATCH v2] media: atmel-isi: parse the DT parameters for vsync/hsync/pixclock polarity Josh Wu
2015-08-04 8:33 ` Laurent Pinchart
2015-08-04 8:41 ` Josh Wu
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).