hint needed : how to integrate sublvds to mipi csi-2 converter

2019-02-20 Thread Philippe De Muyter
-length that were configured in the sensor. I woul like to avoid the burden for the user to configure the fpga properly when changing bpp or line-length in the sensor. How should I implement that. Is there an example somewhere ? TIA Philippe -- Philippe De Muyter +32 2 6101532 Macq SA rue de

Re: [PATCH v3 1/2] [media] imx214: device tree binding

2018-10-02 Thread Philippe De Muyter
On Tue, Oct 02, 2018 at 03:46:48PM +0200, Ricardo Ribalda Delgado wrote: > Document bindings for imx214 camera sensor ... > +Optional Properties: > +- flash-leds: See ../video-interfaces.txt > +- lens-focus: See ../video-interfaces.txt > + > +The imx274 device node should contain one 'port' child

Re: [PATCH v5] media: imx208: Add imx208 camera sensor driver

2018-10-01 Thread Philippe De Muyter
e mantissa, giving values from 1 to (1 + 63/64) * (1, 2, 4 or 8). I currently combine them and indicate the unit in the name of the control "gain (64th)", but a more robust solution would be welcome. the other sensor provides an analog gain which is expressed in tenth of dB (cB ?) from 0 t

? recommended way to implement gain control

2018-09-21 Thread Philippe De Muyter
Hi, I see there's an ongoing discussion about implementing gain control for imx208. I have a similar problem (in the sense that the documentation of V4L2_CID_GAIN does not answer my question). The IMX264 has only one gain control, ranging from 0 to 480, but the unit is 'tenth of dB'. How can I m

Re: [PATCH v3 1/2] media: v4l2-subdev.h: allow V4L2_FRMIVAL_TYPE_CONTINUOUS & _STEPWISE

2018-09-20 Thread Philippe De Muyter
On Thu, Sep 20, 2018 at 05:11:50PM +0300, Laurent Pinchart wrote: > Hi Philippe, > > Thank you for the patch. > > On Tuesday, 11 September 2018 20:06:32 EEST Philippe De Muyter wrote: > > add V4L2_FRMIVAL_TYPE_CONTINUOUS and V4L2_FRMIVAL_TYPE_STEPWISE for > > subdev&#x

[PATCH v3 2/2] media: imx: capture: use 'v4l2_fill_frmivalenum_from_subdev'

2018-09-11 Thread Philippe De Muyter
--- drivers/staging/media/imx/imx-media-capture.c | 18 +- 1 file changed, 1 insertion(+), 17 deletions(-) diff --git a/drivers/staging/media/imx/imx-media-capture.c b/drivers/staging/media/imx/imx-media-capture.c index 256039c..688dd7a 100644 --- a/drivers/staging/media/imx/imx-

[PATCH v3 1/2] media: v4l2-subdev.h: allow V4L2_FRMIVAL_TYPE_CONTINUOUS & _STEPWISE

2018-09-11 Thread Philippe De Muyter
e.interval; fival->stepwise.max = fie.max_interval; fival->stepwise.step = fie.step_interval; } Kernel users should use the new 'v4l2_fill_frmivalenum_from_subdev' helper function. Signed-off-by: Philippe De Muyter --- .../uapi/v4l/vidioc-

[PATCH v3 0/2] media: v4l2-subdev.h: allow V4L2_FRMIVAL_TYPE_CONTINUOUS & _STEPWISE

2018-09-11 Thread Philippe De Muyter
Convert a driver to use the new helper function (asked by Hans) Initialize 'which' to V4L2_SUBDEV_FORMAT_ACTIVE in helper Philippe De Muyter (2): media: v4l2-subdev.h: allow V4L2_FRMIVAL_TYPE_CONTINUOUS & _STEPWISE media: imx: capture: use 'v4l2_fill_frmiv

[PATCH v2] media: v4l2-subdev.h: allow V4L2_FRMIVAL_TYPE_CONTINUOUS & _STEPWISE

2018-09-11 Thread Philippe De Muyter
e.interval; fival->stepwise.max = fie.max_interval; fival->stepwise.step = fie.step_interval; } Kernel users should use the new 'v4l2_fill_frmivalenum_from_subdev' helper function. Signed-off-by: Philippe De Muyter --- v2: Add a

Re: [PATCH v3 1/4] dt-bindings: media: Add i.MX Pixel Pipeline binding

2018-09-06 Thread Philippe De Muyter
uld be "fsl,-pxp", where SoC can be one of imx23, > imx28, > + imx6dl, imx6sl, imx6ul, imx6sx, imx6ull, or imx7d. Is imx6q also compatible ? Best regards Philippe -- Philippe De Muyter +32 2 6101532 Macq SA rue de l'Aeronef 2 B-1140 Bruxelles

Re: [PATCH] media: v4l2-subdev.h: allow V4L2_FRMIVAL_TYPE_CONTINUOUS & _STEPWISE

2018-08-28 Thread Philippe De Muyter
Hi Hans, On Tue, Aug 28, 2018 at 12:29:21PM +0200, Hans Verkuil wrote: > On 28/08/18 12:26, Philippe De Muyter wrote: > > Hi Hans, > > > > On Tue, Aug 28, 2018 at 12:03:25PM +0200, Hans Verkuil wrote: > >> This is a bit too magical for my tastes. I'd

Re: [PATCH] media: v4l2-subdev.h: allow V4L2_FRMIVAL_TYPE_CONTINUOUS & _STEPWISE

2018-08-28 Thread Philippe De Muyter
Hi Hans, On Tue, Aug 28, 2018 at 12:03:25PM +0200, Hans Verkuil wrote: > Hi Philippe, > > On 28/08/18 09:55, Philippe De Muyter wrote: > > add max_interval and step_interval to struct > > v4l2_subdev_frame_interval_enum. > > Yeah, I never understood why this wasn&#x

[PATCH] media: v4l2-subdev.h: allow V4L2_FRMIVAL_TYPE_CONTINUOUS & _STEPWISE

2018-08-28 Thread Philippe De Muyter
se.step = fie.step_interval; } Signed-off-by: Philippe De Muyter --- .../uapi/v4l/vidioc-subdev-enum-frame-interval.rst | 39 +- include/uapi/linux/v4l2-subdev.h | 4 ++- 2 files changed, 41 insertions(+), 2 deletions(-) diff --git a/Documentation/m

Re: [PATCH 2/2] media: v4l2-common: simplify v4l2_i2c_subdev_init name generation

2018-08-08 Thread Philippe De Muyter
On Fri, Aug 03, 2018 at 03:43:15PM +0300, Sakari Ailus wrote: > Hi Philippe, > > On Wed, Aug 01, 2018 at 11:20:57PM +0200, Philippe De Muyter wrote: > > When v4l2_i2c_subdev_init is called, dev_name(&client->dev) has already > > been set. Use it to generate subdev

Re: [PATCH 2/2] media: v4l2-common: simplify v4l2_i2c_subdev_init name generation

2018-08-03 Thread Philippe De Muyter
Hi Sakari, On Fri, Aug 03, 2018 at 03:43:15PM +0300, Sakari Ailus wrote: > Hi Philippe, > > On Wed, Aug 01, 2018 at 11:20:57PM +0200, Philippe De Muyter wrote: > > When v4l2_i2c_subdev_init is called, dev_name(&client->dev) has already > > been set. Use it to gener

[PATCH 2/2] media: v4l2-common: simplify v4l2_i2c_subdev_init name generation

2018-08-01 Thread Philippe De Muyter
When v4l2_i2c_subdev_init is called, dev_name(&client->dev) has already been set. Use it to generate subdev's name instead of recreating it with "%d-%04x". This improves the similarity in subdev's name creation between v4l2_i2c_subdev_init and v4l2_spi_subdev_init.

[PATCH 1/2] media: v4l2-common: v4l2_spi_subdev_init : generate unique name

2018-08-01 Thread Philippe De Muyter
quot;, but missing the "spi1.1" part, and not generating a unique name. Fix that. Signed-off-by: Philippe De Muyter --- drivers/media/v4l2-core/v4l2-common.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/media/v4l2-core/v4l2-common.c b/drivers/media/v4l2-core

BUG ? v4l2_spi_subdev_init does not produce unique names

2018-08-01 Thread Philippe De Muyter
is set as "xxx %d-%04x", giving a unique name to the subdev. with spi, subdev name is set as "xxx", giving the same name to all similar subdevs on the same host Is that intentional or an oversight, and if so, how should that be fixed ? Best regards Philippe -- Philippe De Muyter +32 2 6101532 Macq SA rue de l'Aeronef 2 B-1140 Bruxelles

v4l2_spi_subdev_init vs v4l2_i2c_subdev_init

2018-07-31 Thread Philippe De Muyter
x", giving the same name to all similar subdevs on the same host Is that intentional or an oversight, and if so, how should that be fixed ? Best regards Philippe -- Philippe De Muyter +32 2 6101532 Macq SA rue de l'Aeronef 2 B-1140 Bruxelles

linux and sony pregius cmos sensors

2017-07-25 Thread Philippe De Muyter
, they don't have a MIPI-CSI2 interface but a LVDS interface. Fortunately there is ia chip from a FPGA vendor that can convert LVDS to MIPI-CSI2. Any advice or info ? TIA Philippe -- Philippe De Muyter +32 2 6101532 Macq SA rue de l'Aeronef 2 B-1140 Bruxelles

Re: [PATCH 4/4] media: imx-media-capture: add frame sizes/interval enumeration

2017-03-20 Thread Philippe De Muyter
On Mon, Mar 20, 2017 at 09:05:25AM +, Russell King - ARM Linux wrote: > On Mon, Mar 20, 2017 at 09:55:12AM +0100, Philippe De Muyter wrote: > > Hi Russel, > > > > On Sun, Mar 19, 2017 at 10:49:08AM +, Russell King wrote: > > > Add support for enumerating f

Re: [PATCH 4/4] media: imx-media-capture: add frame sizes/interval enumeration

2017-03-20 Thread Philippe De Muyter
frame interval is possible, and hence type should be V4L2_FRMIVAL_TYPE_CONTINUOUS. see also https://www.spinics.net/lists/linux-media/msg98622.html, https://patchwork.kernel.org/patch/9171201/ and https://patchwork.kernel.org/patch/9171199/ Philippe -- Philippe De Muyter +32 2 6101532 Macq SA rue de l'Aeronef 2 B-1140 Bruxelles

Re: media / v4l2-mc: wishlist for complex cameras (was Re: [PATCH v4 14/36] [media] v4l2-mc: add a function to inherit controls from a pipeline)

2017-03-16 Thread Philippe De Muyter
On Thu, Mar 16, 2017 at 11:01:56AM +0100, Philipp Zabel wrote: > On Thu, 2017-03-16 at 10:47 +0100, Philippe De Muyter wrote: > > On Thu, Mar 16, 2017 at 10:26:00AM +0100, Philipp Zabel wrote: > > > On Wed, 2017-03-15 at 14:55 -0400, Nicolas Dufresne wrote: > > > > Le

Re: media / v4l2-mc: wishlist for complex cameras (was Re: [PATCH v4 14/36] [media] v4l2-mc: add a function to inherit controls from a pipeline)

2017-03-16 Thread Philippe De Muyter
On Thu, Mar 16, 2017 at 10:26:00AM +0100, Philipp Zabel wrote: > On Wed, 2017-03-15 at 14:55 -0400, Nicolas Dufresne wrote: > > Le mercredi 15 mars 2017 à 11:50 +0100, Philippe De Muyter a écrit : > > > > I would say: camorama, xawtv3, zbar, google talk, skype. If it runs &

Re: media / v4l2-mc: wishlist for complex cameras (was Re: [PATCH v4 14/36] [media] v4l2-mc: add a function to inherit controls from a pipeline)

2017-03-15 Thread Philippe De Muyter
art 'imxv4l2videosrc' from the gstreamer-imx package at https://github.com/Freescale/gstreamer-imx, and 'v4l2-ctl'. Philippe -- Philippe De Muyter +32 2 6101532 Macq SA rue de l'Aeronef 2 B-1140 Bruxelles

[PATCH 1/2] [media] v4l2-subdev.h: allow V4L2_FRMIVAL_TYPE_CONTINUOUS & _STEPWISE

2016-06-11 Thread Philippe De Muyter
se.step = fie.step_interval; } Signed-off-by: Philippe De Muyter --- include/uapi/linux/v4l2-subdev.h | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/include/uapi/linux/v4l2-subdev.h b/include/uapi/linux/v4l2-subdev.h index dbce2b554..846dd36 100644 --- a/include/uap

[PATCH 2/2] media: mx6-camif: add V4L2_FRMIVAL_TYPE_STEPWISE & _CONTINUOUS

2016-06-11 Thread Philippe De Muyter
Signed-off-by: Philippe De Muyter --- drivers/staging/media/imx6/capture/mx6-camif.c | 15 +-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --git a/drivers/staging/media/imx6/capture/mx6-camif.c b/drivers/staging/media/imx6/capture/mx6-camif.c index b36f9d1..137733e 100644

Re: subdev sensor driver and V4L2_FRMIVAL_TYPE_CONTINUOUS/V4L2_FRMIVAL_TYPE_STEPWISE

2016-03-15 Thread Philippe De Muyter
My post is about framerate, not framesize :) I agree the macro's names are misleading. On Tue, Mar 15, 2016 at 12:54:22PM +0100, Hans Verkuil wrote: > On 03/15/16 12:06, Sakari Ailus wrote: > > Hi Philippe, > > > > On Tue, Mar 15, 2016 at 11:14:17AM +0100, Philipp

subdev sensor driver and V4L2_FRMIVAL_TYPE_CONTINUOUS/V4L2_FRMIVAL_TYPE_STEPWISE

2016-03-15 Thread Philippe De Muyter
ctionality not considered relevant anymorea ? Best regards Philippe -- Philippe De Muyter +32 2 6101532 Macq SA rue de l'Aeronef 2 B-1140 Bruxelles -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More m

Re: i.mx6 camera interface (CSI) and mainline kernel

2016-03-14 Thread Philippe De Muyter
internal mux in > > the SoC, > > or can control an external mux via gpio. Our driver only supports the > > internal mux, > > and does it with a platform data function. > > > > But like I said, I don't what the latest status is of the Pengutronix video > &g

Re: i.mx6 camera interface (CSI) and mainline kernel

2016-03-03 Thread Philippe De Muyter
On Thu, Mar 03, 2016 at 09:45:08AM -0800, Steve Longerbeam wrote: > Hi Philippe, > > On 03/03/2016 12:36 AM, Philippe De Muyter wrote: > > > > Just to be sure : do you mean https://github.com/slongerbeam/mediatree.git > > or something else ? > > Sorry, yes I mea

Re: i.mx6 camera interface (CSI) and mainline kernel

2016-03-03 Thread Philippe De Muyter
or this device upstream would be > very nice indeed. > > I'd really like to see some upstream support for this SoC soon. There are a bunch of people expecting that (and trying to help) :) Best regards Philippe > > Regards, > > Hans > > > > >