Re: [REVIEW PATCH v3 1/2] media: Change media device link_notify behaviour

2013-06-10 Thread Laurent Pinchart
Hi Sylwester, Should I take the series in my tree, or would you like to push it yourself to avoid conflicts with other Exynos patches ? On Monday 10 June 2013 01:10:30 Sakari Ailus wrote: Hi Sylwester, Thanks for the update. Sylwester Nawrocki wrote: ... diff --git

Re: [REVIEW PATCH v3 1/2] media: Change media device link_notify behaviour

2013-06-10 Thread Sylwester Nawrocki
Hi Laurent, On 06/10/2013 11:46 AM, Laurent Pinchart wrote: Hi Sylwester, Should I take the series in my tree, or would you like to push it yourself to avoid conflicts with other Exynos patches ? My plan was to handle this series together with the other Exynos patches it depends on. I

Re: [REVIEW PATCH v3 1/2] media: Change media device link_notify behaviour

2013-06-10 Thread Laurent Pinchart
On Monday 10 June 2013 12:20:15 Sylwester Nawrocki wrote: On 06/10/2013 11:46 AM, Laurent Pinchart wrote: Hi Sylwester, Should I take the series in my tree, or would you like to push it yourself to avoid conflicts with other Exynos patches ? My plan was to handle this series together

Re: [REVIEW PATCH v3 1/2] media: Change media device link_notify behaviour

2013-06-10 Thread Sylwester Nawrocki
Hi Sakari, On 06/10/2013 12:10 AM, Sakari Ailus wrote: Sylwester Nawrocki wrote: ... diff --git a/drivers/media/platform/omap3isp/isp.c b/drivers/media/platform/omap3isp/isp.c index 1d7dbd5..1a2d25c 100644 --- a/drivers/media/platform/omap3isp/isp.c +++

Re: [REVIEW PATCH v3 1/2] media: Change media device link_notify behaviour

2013-06-10 Thread Laurent Pinchart
On Monday 10 June 2013 12:47:02 Sylwester Nawrocki wrote: On 06/10/2013 12:10 AM, Sakari Ailus wrote: Sylwester Nawrocki wrote: ... diff --git a/drivers/media/platform/omap3isp/isp.c b/drivers/media/platform/omap3isp/isp.c index 1d7dbd5..1a2d25c 100644 ---

Re: [REVIEW PATCH v3 1/2] media: Change media device link_notify behaviour

2013-06-10 Thread Sylwester Nawrocki
On 06/10/2013 12:49 PM, Laurent Pinchart wrote: -static int isp_pipeline_link_notify(struct media_pad *source, - struct media_pad *sink, u32 flags) +static int isp_pipeline_link_notify(struct media_link *link, u32 flags, +

Re: [REVIEW PATCH v3 1/2] media: Change media device link_notify behaviour

2013-06-09 Thread Laurent Pinchart
Hi Sylwester, Thanks for the patch. On Sunday 09 June 2013 22:14:37 Sylwester Nawrocki wrote: Currently the media device link_notify callback is invoked before the actual change of state of a link when the link is being enabled, and after the actual change of state when the link is being

Re: [REVIEW PATCH v3 1/2] media: Change media device link_notify behaviour

2013-06-09 Thread Sakari Ailus
Hi Sylwester, Thanks for the update. Sylwester Nawrocki wrote: ... diff --git a/drivers/media/platform/omap3isp/isp.c b/drivers/media/platform/omap3isp/isp.c index 1d7dbd5..1a2d25c 100644 --- a/drivers/media/platform/omap3isp/isp.c +++ b/drivers/media/platform/omap3isp/isp.c @@ -792,9 +792,9