Re: [PATCH] [media] staging: allow omap4iss to be modular

2014-06-13 Thread Laurent Pinchart
Hi Tony, On Thursday 12 June 2014 22:30:44 Tony Lindgren wrote: * Laurent Pinchart laurent.pinch...@ideasonboard.com [140612 08:32]: On Thursday 12 June 2014 08:15:35 Tony Lindgren wrote: * Laurent Pinchart laurent.pinch...@ideasonboard.com [140612 07:52]: On Wednesday 11 June 2014

Re: [PATCH] [media] staging: allow omap4iss to be modular

2014-06-13 Thread Tony Lindgren
* Laurent Pinchart laurent.pinch...@ideasonboard.com [140612 23:48]: On Thursday 12 June 2014 22:30:44 Tony Lindgren wrote: 1. They live in separate hardware modules that can be clocked separately Actually I don't think that's true. The CSI2 PHY is part of the camera device, with all

Re: [PATCH] [media] staging: allow omap4iss to be modular

2014-06-13 Thread Laurent Pinchart
Hi Tony, On Friday 13 June 2014 00:53:25 Tony Lindgren wrote: * Laurent Pinchart laurent.pinch...@ideasonboard.com [140612 23:48]: On Thursday 12 June 2014 22:30:44 Tony Lindgren wrote: 1. They live in separate hardware modules that can be clocked separately Actually I don't think

Re: [PATCH] [media] staging: allow omap4iss to be modular

2014-06-13 Thread Tony Lindgren
* Laurent Pinchart laurent.pinch...@ideasonboard.com [140613 03:30]: Hi Tony, On Friday 13 June 2014 00:53:25 Tony Lindgren wrote: * Laurent Pinchart laurent.pinch...@ideasonboard.com [140612 23:48]: On Thursday 12 June 2014 22:30:44 Tony Lindgren wrote: 1. They live in separate

Re: [PATCH] [media] staging: allow omap4iss to be modular

2014-06-13 Thread Laurent Pinchart
Hi Tony, On Friday 13 June 2014 04:10:12 Tony Lindgren wrote: * Laurent Pinchart laurent.pinch...@ideasonboard.com [140613 03:30]: On Friday 13 June 2014 00:53:25 Tony Lindgren wrote: * Laurent Pinchart laurent.pinch...@ideasonboard.com [140612 23:48]: On Thursday 12 June 2014 22:30:44

Re: [PATCH] [media] staging: allow omap4iss to be modular

2014-06-12 Thread Laurent Pinchart
Hi Arnd, On Wednesday 11 June 2014 16:49:31 Arnd Bergmann wrote: On Wednesday 11 June 2014 09:42:04 Nishanth Menon wrote: On 06/11/2014 09:35 AM, Arnd Bergmann wrote: The OMAP4 camera support depends on I2C and VIDEO_V4L2, both of which can be loadable modules. This causes build failures

Re: [PATCH] [media] staging: allow omap4iss to be modular

2014-06-12 Thread Arnd Bergmann
On Thursday 12 June 2014 16:12:17 Laurent Pinchart wrote: From 3a965f4fd5a6b3ef4a66aa4e7c916cfd34fd5706 Mon Sep 17 00:00:00 2001 From: Arnd Bergmann a...@arndb.de Date: Tue, 21 Jan 2014 09:32:43 +0100 Subject: [PATCH] [media] staging: tighten omap4iss dependencies The OMAP4 camera

Re: [PATCH] [media] staging: allow omap4iss to be modular

2014-06-12 Thread Greg KH
On Thu, Jun 12, 2014 at 04:15:32PM +0200, Arnd Bergmann wrote: On Thursday 12 June 2014 16:12:17 Laurent Pinchart wrote: From 3a965f4fd5a6b3ef4a66aa4e7c916cfd34fd5706 Mon Sep 17 00:00:00 2001 From: Arnd Bergmann a...@arndb.de Date: Tue, 21 Jan 2014 09:32:43 +0100 Subject: [PATCH]

Re: [PATCH] [media] staging: allow omap4iss to be modular

2014-06-12 Thread Arnd Bergmann
On Thursday 12 June 2014 07:25:15 Greg KH wrote: On Thu, Jun 12, 2014 at 04:15:32PM +0200, Arnd Bergmann wrote: On Thursday 12 June 2014 16:12:17 Laurent Pinchart wrote: From 3a965f4fd5a6b3ef4a66aa4e7c916cfd34fd5706 Mon Sep 17 00:00:00 2001 From: Arnd Bergmann a...@arndb.de Date:

Re: [PATCH] [media] staging: allow omap4iss to be modular

2014-06-12 Thread Laurent Pinchart
Hi Tony, On Wednesday 11 June 2014 07:47:54 Tony Lindgren wrote: * Arnd Bergmann a...@arndb.de [140611 07:37]: The OMAP4 camera support depends on I2C and VIDEO_V4L2, both of which can be loadable modules. This causes build failures if we want the camera driver to be built-in. That's

Re: [PATCH] [media] staging: allow omap4iss to be modular

2014-06-12 Thread Laurent Pinchart
Hi Arnd, On Thursday 12 June 2014 16:28:39 Arnd Bergmann wrote: On Thursday 12 June 2014 07:25:15 Greg KH wrote: On Thu, Jun 12, 2014 at 04:15:32PM +0200, Arnd Bergmann wrote: On Thursday 12 June 2014 16:12:17 Laurent Pinchart wrote: From 3a965f4fd5a6b3ef4a66aa4e7c916cfd34fd5706 Mon

Re: [PATCH] [media] staging: allow omap4iss to be modular

2014-06-12 Thread Tony Lindgren
* Laurent Pinchart laurent.pinch...@ideasonboard.com [140612 07:52]: On Wednesday 11 June 2014 07:47:54 Tony Lindgren wrote: These should just use either pinctrl-single.c instead for muxing. Or if they are not mux registers, we do have the syscon mapping available in omap4.dtsi that

Re: [PATCH] [media] staging: allow omap4iss to be modular

2014-06-12 Thread Laurent Pinchart
Hi Tony, On Thursday 12 June 2014 08:15:35 Tony Lindgren wrote: * Laurent Pinchart laurent.pinch...@ideasonboard.com [140612 07:52]: On Wednesday 11 June 2014 07:47:54 Tony Lindgren wrote: These should just use either pinctrl-single.c instead for muxing. Or if they are not mux registers,

Re: [PATCH] [media] staging: allow omap4iss to be modular

2014-06-12 Thread Greg KH
On Thu, Jun 12, 2014 at 04:28:39PM +0200, Arnd Bergmann wrote: On Thursday 12 June 2014 07:25:15 Greg KH wrote: On Thu, Jun 12, 2014 at 04:15:32PM +0200, Arnd Bergmann wrote: On Thursday 12 June 2014 16:12:17 Laurent Pinchart wrote: From 3a965f4fd5a6b3ef4a66aa4e7c916cfd34fd5706 Mon Sep

Re: [PATCH] [media] staging: allow omap4iss to be modular

2014-06-12 Thread Tony Lindgren
* Laurent Pinchart laurent.pinch...@ideasonboard.com [140612 08:32]: Hi Tony, On Thursday 12 June 2014 08:15:35 Tony Lindgren wrote: * Laurent Pinchart laurent.pinch...@ideasonboard.com [140612 07:52]: On Wednesday 11 June 2014 07:47:54 Tony Lindgren wrote: These should just use

[PATCH] [media] staging: allow omap4iss to be modular

2014-06-11 Thread Arnd Bergmann
The OMAP4 camera support depends on I2C and VIDEO_V4L2, both of which can be loadable modules. This causes build failures if we want the camera driver to be built-in. This can be solved by turning the option into tristate, which unfortunately causes another problem, because the driver incorrectly

Re: [PATCH] [media] staging: allow omap4iss to be modular

2014-06-11 Thread Tony Lindgren
* Arnd Bergmann a...@arndb.de [140611 07:37]: The OMAP4 camera support depends on I2C and VIDEO_V4L2, both of which can be loadable modules. This causes build failures if we want the camera driver to be built-in. That's good news, but let's not fix it this way. This can be solved by turning

Re: [PATCH] [media] staging: allow omap4iss to be modular

2014-06-11 Thread Arnd Bergmann
On Wednesday 11 June 2014 09:42:04 Nishanth Menon wrote: On 06/11/2014 09:35 AM, Arnd Bergmann wrote: The OMAP4 camera support depends on I2C and VIDEO_V4L2, both of which can be loadable modules. This causes build failures if we want the camera driver to be built-in. This can be

Re: [PATCH] [media] staging: allow omap4iss to be modular

2014-06-11 Thread Nishanth Menon
On 06/11/2014 09:49 AM, Arnd Bergmann wrote: On Wednesday 11 June 2014 09:42:04 Nishanth Menon wrote: On 06/11/2014 09:35 AM, Arnd Bergmann wrote: The OMAP4 camera support depends on I2C and VIDEO_V4L2, both of which can be loadable modules. This causes build failures if we want the camera

Re: [PATCH] [media] staging: allow omap4iss to be modular

2014-06-11 Thread Nishanth Menon
On 06/11/2014 09:35 AM, Arnd Bergmann wrote: The OMAP4 camera support depends on I2C and VIDEO_V4L2, both of which can be loadable modules. This causes build failures if we want the camera driver to be built-in. This can be solved by turning the option into tristate, which unfortunately

Re: [PATCH] [media] staging: allow omap4iss to be modular

2014-06-11 Thread Tony Lindgren
* Arnd Bergmann a...@arndb.de [140611 07:51]: On Wednesday 11 June 2014 09:42:04 Nishanth Menon wrote: On 06/11/2014 09:35 AM, Arnd Bergmann wrote: The OMAP4 camera support depends on I2C and VIDEO_V4L2, both of which can be loadable modules. This causes build failures if we want the