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
(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 to 480. Clearly this is difficult to comunicate to the u

? 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

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's

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

2018-09-11 Thread Philippe De Muyter
se.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-subdev-enum-frame-interval.rst | 69 +- drivers/media/v4l2-core/v4l2-common.c | 33

[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 ---

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

2018-09-11 Thread Philippe De Muyter
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_frmivalenum_from_subdev' .../uapi

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

2018-09-11 Thread Philippe De Muyter
se.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 'type' field and a helper function, as asked by Hans .../uapi/v4l/vidioc-subdev-enum-frame-interval.rst | 46

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 add a

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 was

[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(>dev) has already > > been set. Use it to generate subdev's na

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(>dev) has already > > been set. Use it to genera

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

2018-08-01 Thread Philippe De Muyter
mx185", 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

[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(>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. Signed-off-by: Phili

BUG ? v4l2_spi_subdev_init does not produce unique names

2018-08-01 Thread Philippe De Muyter
ev name 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
t;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

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
V ev76c560) "any" 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 ru

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
ounterpart '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 <p...@macqel.be> --- 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..846dd3

[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 <p...@macqel.be> --- 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 b

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, Philippe De

subdev sensor driver and V4L2_FRMIVAL_TYPE_CONTINUOUS/V4L2_FRMIVAL_TYPE_STEPWISE

2016-03-15 Thread Philippe De Muyter
dered 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 majordomo info at http://

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

2016-03-14 Thread Philippe De Muyter
> > 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 > > capture support. > > > &

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
eam 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 > > > > > > > Steve >