Re: [PATCH v2 0/7] Add support of OV9655 camera

2017-07-20 Thread H. Nikolaus Schaller
Hi, > Am 18.07.2017 um 21:52 schrieb Sakari Ailus <sakari.ai...@iki.fi>: > > On Tue, Jul 18, 2017 at 12:53:12PM +, Hugues FRUCHET wrote: >> >> >> On 07/18/2017 02:17 PM, H. Nikolaus Schaller wrote: >>> Hi, >>> >>>>

Re: [PATCH v2 0/7] [PATCH v2 0/7] Add support of OV9655 camera

2017-07-18 Thread H. Nikolaus Schaller
Hi, > Am 18.07.2017 um 13:59 schrieb Hans Verkuil : > > On 12/07/17 22:01, Sylwester Nawrocki wrote: >> Hi Hugues, >> >> On 07/03/2017 11:16 AM, Hugues Fruchet wrote: >>> This patchset enables OV9655 camera support. >>> >>> OV9655 support has been tested using

Re: [PATCH v1 1/6] DT bindings: add bindings for ov965x camera module

2017-06-28 Thread H. Nikolaus Schaller
> Am 28.06.2017 um 12:50 schrieb Sylwester Nawrocki <s.nawro...@samsung.com>: > > On 06/28/2017 11:12 AM, H. Nikolaus Schaller wrote: >>> Am 28.06.2017 um 00:57 schrieb Sylwester Nawrocki <snawro...@kernel.org>: >>> On 06/27/2017 07:48 AM, H. Nikolaus Schal

Re: [PATCH v1 1/6] DT bindings: add bindings for ov965x camera module

2017-06-28 Thread H. Nikolaus Schaller
> Am 28.06.2017 um 00:57 schrieb Sylwester Nawrocki <snawro...@kernel.org>: > > On 06/27/2017 07:48 AM, H. Nikolaus Schaller wrote: >>> Am 26.06.2017 um 22:04 schrieb Sylwester Nawrocki <snawro...@kernel.org>: >>> >>> On 06/26/2017 12:35 P

Re: omap3isp camera was Re: [PATCH v1 0/6] Add support of OV9655 camera

2017-06-26 Thread H. Nikolaus Schaller
> Am 26.06.2017 um 13:16 schrieb Pavel Machek : > > Hi! > >>> You may want to try this one: >>> >>> commit 0eae9d2a8f096f703cbc8f9a0ab155cd3cc14cef >>> Author: Pavel >>> Date: Mon Feb 13 21:26:51 2017 +0100 >>> >>> omap3isp: fix VP2SDR bit so capture (not

Re: [PATCH v1 1/6] DT bindings: add bindings for ov965x camera module

2017-06-26 Thread H. Nikolaus Schaller
> Am 26.06.2017 um 22:04 schrieb Sylwester Nawrocki : > > On 06/26/2017 12:35 PM, Hugues FRUCHET wrote: >>> What I am missing to support the GTA04 camera is the control of the >>> optional "vana-supply". >>> So the driver does not power up the camera module when needed and

Re: [PATCH] media: omap3isp: handle NULL return of omap3isp_video_format_info() in ccdc_is_shiftable().

2017-06-26 Thread H. Nikolaus Schaller
Hi, > Am 26.06.2017 um 22:12 schrieb Sakari Ailus <sakari.ai...@iki.fi>: > > Hi Nikolaus, > > On Mon, Jun 26, 2017 at 07:54:19PM +0200, H. Nikolaus Schaller wrote: >> If a camera module driver specifies a format that is not >> supported by omap3isp this ends

[PATCH] media: omap3isp: handle NULL return of omap3isp_video_format_info() in ccdc_is_shiftable().

2017-06-26 Thread H. Nikolaus Schaller
If a camera module driver specifies a format that is not supported by omap3isp this ends in a NULL pointer dereference instead of a simple fail. Signed-off-by: H. Nikolaus Schaller <h...@goldelico.com> --- drivers/media/platform/omap3isp/ispccdc.c | 3 +++ 1 file changed, 3 insertions(+)

Re: [PATCH v1 2/6] [media] ov9650: add device tree support

2017-06-26 Thread H. Nikolaus Schaller
Hi, > Am 26.06.2017 um 18:31 schrieb Sakari Ailus : > > Hi Hugues, > > On Thu, Jun 22, 2017 at 05:05:38PM +0200, Hugues Fruchet wrote: >> @@ -1545,15 +1577,22 @@ static int ov965x_remove(struct i2c_client *client) >> } >> >> static const struct i2c_device_id ov965x_id[] =

Re: omap3isp camera was Re: [PATCH v1 0/6] Add support of OV9655 camera

2017-06-26 Thread H. Nikolaus Schaller
Hi Pavel, > Am 26.06.2017 um 10:39 schrieb Pavel Machek <pa...@ucw.cz>: > > On Mon 2017-06-26 08:05:04, H. Nikolaus Schaller wrote: >> Hi Pavel, >> >>> Am 25.06.2017 um 11:18 schrieb Pavel Machek <pa...@ucw.cz>: >>> >

Re: omap3isp camera was Re: [PATCH v1 0/6] Add support of OV9655 camera

2017-06-26 Thread H. Nikolaus Schaller
Hi Pavel, > Am 25.06.2017 um 11:18 schrieb Pavel Machek : > > Hi! > >> * unfortunately we still get no image :( >> >> The latter is likely a setup issue of our camera interface (OMAP3 ISP = >> Image Signal Processor) which >> we were not yet able to solve. Oscilloscoping signals

Re: [PATCH v1 1/6] DT bindings: add bindings for ov965x camera module

2017-06-26 Thread H. Nikolaus Schaller
> Am 24.06.2017 um 00:24 schrieb Suman Anna <s-a...@ti.com>: > > On 06/23/2017 01:59 PM, H. Nikolaus Schaller wrote: >> Hi Suman, >> >>> Am 23.06.2017 um 20:05 schrieb Suman Anna <s-a...@ti.com>: >>> >>>>>> >>>&

Re: [PATCH v1 6/6] [media] ov9650: add support of OV9655 variant

2017-06-26 Thread H. Nikolaus Schaller
ited to VGA, QVGA, QQVGA. > Supported format is limited to RGB565. > Controls are limited to color bar test pattern for test purpose. > > Signed-off-by: H. Nikolaus Schaller <h...@goldelico.com> > Signed-off-by: Hugues Fruchet <hugues.fruc...@st.com> > --- > drivers

Re: [PATCH v1 0/6] Add support of OV9655 camera

2017-06-26 Thread H. Nikolaus Schaller
Hi Hugues, > Am 23.06.2017 um 12:25 schrieb H. Nikolaus Schaller <h...@goldelico.com>: > > Hi Hugues, > >> Am 22.06.2017 um 17:41 schrieb H. Nikolaus Schaller <h...@goldelico.com>: >> >> >>> Am 22.06.2017 um 17:05 schrieb Hugues Fruchet <

Re: [PATCH v1 1/6] DT bindings: add bindings for ov965x camera module

2017-06-23 Thread H. Nikolaus Schaller
Hi Suman, > Am 23.06.2017 um 20:05 schrieb Suman Anna : > Or does it just mean that it defines the property name? >>> >>> Please read the documentation link I sent - it's in the very bottom and >>> should have an example. >> >> I have seen it but it does not give me a

Re: [PATCH v1 1/6] DT bindings: add bindings for ov965x camera module

2017-06-23 Thread H. Nikolaus Schaller
Hi, > Am 23.06.2017 um 16:57 schrieb Andreas Färber <afaer...@suse.de>: > > Am 23.06.2017 um 16:53 schrieb H. Nikolaus Schaller: >> Hi Laurent, >> >>> Am 23.06.2017 um 13:58 schrieb Laurent Pinchart >>> <laurent.pinch...@ideasonboard.com>: &

Re: [PATCH v1 1/6] DT bindings: add bindings for ov965x camera module

2017-06-23 Thread H. Nikolaus Schaller
Hi Laurent, > Am 23.06.2017 um 13:58 schrieb Laurent Pinchart > <laurent.pinch...@ideasonboard.com>: > > Hi Nikolaus, > > On Friday 23 Jun 2017 12:59:24 H. Nikolaus Schaller wrote: >> Am 23.06.2017 um 12:46 schrieb Andreas Färber <afaer...@suse.de>:

Re: [PATCH v1 1/6] DT bindings: add bindings for ov965x camera module

2017-06-23 Thread H. Nikolaus Schaller
> Am 23.06.2017 um 12:46 schrieb Andreas Färber <afaer...@suse.de>: > > Hi, > > Am 23.06.2017 um 12:25 schrieb H. Nikolaus Schaller: >>> diff --git a/Documentation/devicetree/bindings/media/i2c/ov965x.txt >>> b/Documentation/devicetree/bindings/media/

Re: [PATCH v1 1/6] DT bindings: add bindings for ov965x camera module

2017-06-23 Thread H. Nikolaus Schaller
Hi Hugues, > Am 22.06.2017 um 17:05 schrieb Hugues Fruchet <hugues.fruc...@st.com>: > > From: "H. Nikolaus Schaller" <h...@goldelico.com> > > This adds documentation of device tree bindings > for the OV965X family camera sensor module. >

Re: [PATCH v1 0/6] Add support of OV9655 camera

2017-06-23 Thread H. Nikolaus Schaller
Hi Hugues, > Am 22.06.2017 um 17:41 schrieb H. Nikolaus Schaller <h...@goldelico.com>: > > >> Am 22.06.2017 um 17:05 schrieb Hugues Fruchet <hugues.fruc...@st.com>: >> >> This patchset enables OV9655 camera support. >> >> OV9655 support h

Re: [PATCH v1 0/6] Add support of OV9655 camera

2017-06-22 Thread H. Nikolaus Schaller
650/9652, > not all of the driver features are supported (controls). Supported > resolutions are limited to VGA, QVGA, QQVGA. > Supported format is limited to RGB565. > Controls are limited to color bar test pattern for test purpose. > > OV9655 initial support is based on a driver written

Re: [PATCH] [media] omap3isp: don't call of_node_put

2016-10-25 Thread H. Nikolaus Schaller
Hi, has it been merged somewhere? I can't find it in linux-next yet. BR and thanks, Nikolaus > Am 30.09.2016 um 10:52 schrieb Laurent Pinchart > <laurent.pinch...@ideasonboard.com>: > > Hello Nikolaus, > > On Friday 30 Sep 2016 10:28:40 H. Nikolaus Schaller wrote: &g

Re: [PATCH] [media] omap3isp: don't call of_node_put

2016-09-30 Thread H. Nikolaus Schaller
Hi Laurent, > Am 29.09.2016 um 10:54 schrieb Laurent Pinchart > <laurent.pinch...@ideasonboard.com>: > > Hi Nikolaus, > > On Thursday 08 Sep 2016 17:48:33 H. Nikolaus Schaller wrote: >> of_node_put() has already been called inside of_graph_get_next_endpoint(

Re: [PATCH] [media] omap3isp: don't call of_node_put

2016-09-29 Thread H. Nikolaus Schaller
ping ping. > Am 19.09.2016 um 11:55 schrieb H. Nikolaus Schaller <h...@goldelico.com>: > > ping. > >> Am 08.09.2016 um 17:48 schrieb H. Nikolaus Schaller <h...@goldelico.com>: >> >> of_node_put() has already been called inside of_graph_get_next_endpoi

Re: [PATCH] [media] ov9650: add support for asynchronous probing

2016-09-08 Thread H. Nikolaus Schaller
> Am 08.09.2016 um 15:05 schrieb Javier Martinez Canillas > : > > Allow the sub-device to be probed asynchronously so a bridge driver that's > waiting for the device can be notified and its .bound callback executed. > > Signed-off-by: Javier Martinez Canillas

[PATCH] [media] omap3isp: don't call of_node_put

2016-09-08 Thread H. Nikolaus Schaller
[] (ret_fast_syscall+0x0/0x1c) Signed-off-by: H. Nikolaus Schaller <h...@goldelico.com> --- drivers/media/platform/omap3isp/isp.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/media/platform/omap3isp/isp.c b/drivers/media/platform/omap3isp/isp.c index 5d54e2c..6e2624e