On Fri, 5 Oct 2012, Stephen Warren wrote:
> On 10/04/2012 03:35 PM, Guennadi Liakhovetski wrote:
> > Hi Steffen
> >
> > Sorry for chiming in so late in the game, but I've long been wanting to
> > have a look at this and compare with what we do for V4L2, so, thi
On Fri, 5 Oct 2012, Sascha Hauer wrote:
> On Fri, Oct 05, 2012 at 05:41:00PM +0200, Guennadi Liakhovetski wrote:
> > Hi Sascha
> >
> > > > +
> > > > + ceu0: ceu@0xfe91 {
> > > > + compatible = "renesas,sh-mob
On Fri, 5 Oct 2012, Anatolij Gustschin wrote:
> On Fri, 5 Oct 2012 16:31:58 +0200 (CEST)
> Guennadi Liakhovetski wrote:
>
> > Hi Anatolij
> >
> > On Fri, 5 Oct 2012, Anatolij Gustschin wrote:
> >
> > > Hi Guennadi,
> > >
> >
user-provided line length by the image height to obtain a frame buffer
size estimate. The original version of the "media: soc_camera: don't clear
pix->sizeimage in JPEG mode" patch was correct and my amendment, adding
these two checks was superfluous.
Signed-off-by: Guennadi Liak
Hi Sascha
On Fri, 5 Oct 2012, Sascha Hauer wrote:
> Hi Guennadi,
>
> Some comments inline.
>
>
> On Thu, Sep 27, 2012 at 04:07:23PM +0200, Guennadi Liakhovetski wrote:
> > This patch adds a document, describing common V4L2 device tree bindings.
> >
> >
Update soc-camera documentation to reflect the current camera host API and
the use of the common V4L2 subdev API.
Signed-off-by: Guennadi Liakhovetski
---
Documentation/video4linux/soc-camera.txt | 146 +++---
1 files changed, 75 insertions(+), 71 deletions(-)
diff
e.
> Patch failed at 0007 media: soc-camera: support deferred probing of clients
> When you have resolved this problem run "git am --resolved".
> If you would prefer to skip this patch, instead run "git am --skip".
> To restore the original branch and stop patching ru
Hi Anatolij
On Fri, 5 Oct 2012, Anatolij Gustschin wrote:
> Hi Guennadi,
>
> On Wed, 3 Oct 2012 00:09:29 +0200 (CEST)
> Guennadi Liakhovetski wrote:
>
> > Hi Anatolij
> >
> > > > > +#if !defined(CONFIG_VIDEO_MPC52xx_CSI) && \
>
On Fri, 5 Oct 2012, Hans Verkuil wrote:
> On Fri October 5 2012 11:43:27 Guennadi Liakhovetski wrote:
> > On Wed, 3 Oct 2012, Rob Herring wrote:
> >
> > > On 10/02/2012 09:33 AM, Guennadi Liakhovetski wrote:
> > > > Hi Rob
> > > >
On Fri, 5 Oct 2012, Hans Verkuil wrote:
> On Fri October 5 2012 12:58:21 Guennadi Liakhovetski wrote:
> > On Fri, 5 Oct 2012, Hans Verkuil wrote:
[snip]
> > > One area that I do not yet completely understand is the i2c bus
> > > notifications
> > > (or
On Fri, 5 Oct 2012, Hans Verkuil wrote:
> On Tue October 2 2012 12:13:20 Sylwester Nawrocki wrote:
> > Hi Guennadi,
> >
> > On 10/02/2012 11:49 AM, Guennadi Liakhovetski wrote:
> > >>> + if (!of_property_read_u32_
for VEU. So far only conversion from NV12 to RGB565 is
supported. Further functionality shall be added in the future.
This driver is based on a VEU vidix driver by Magnus Damm.
Signed-off-by: Guennadi Liakhovetski
---
v4:
- remove redundant checks
- embed struct video_device
- move
On Wed, 3 Oct 2012, Rob Herring wrote:
> On 10/02/2012 09:33 AM, Guennadi Liakhovetski wrote:
> > Hi Rob
> >
> > On Tue, 2 Oct 2012, Rob Herring wrote:
> >
> >> On 09/27/2012 09:07 AM, Guennadi Liakhovetski wrote:
> >>> This patch adds a docum
for VEU. So far only conversion from NV12 to RGB565 is
supported. Further functionality shall be added in the future.
This driver is based on a VEU vidix driver by Magnus Damm.
Signed-off-by: Guennadi Liakhovetski
---
v3:
- compliance test is now happy apart from 2 warnings - CREATE_BUFS not
i.e. of the board? Can the
same display controller on one board require interlaced data and on
another board - progressive? BTW, I'm not very familiar with display
interfaces, but for interlaced you probably sometimes use a field signal,
whose polarity you also want to specify here? We use a &quo
omplish this.
Sorry, I don't know what file you mean. In any case you'd have to
implement overlay support in the mx3fb.c framebuffer driver.
Thanks
Guennadi
> Cheers,
> Alex
>
> -Original Message-
> From: Guennadi Liakhovetski [mailto:g.liakhovet...@gmx.de]
&
but the
> support "community" is a bit quiet.
>
> Thanks
>
> Alex Pollard
> Software Engineer
>
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
--
To unsubscribe from this list: send the line "unsubscr
) But
if anyone thinks, that I missed their mail, please, re-send.
Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a me
r does in it is
totally its own business. Nobody says, that you have to issue I2C commands
in it.
Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
--
To unsubscribe from this list: send the line "unsubscribe linux-medi
Hi Rob
On Tue, 2 Oct 2012, Rob Herring wrote:
> On 09/27/2012 09:07 AM, Guennadi Liakhovetski wrote:
> > This patch adds a document, describing common V4L2 device tree bindings.
> >
> > Co-authored-by: Sylwester Nawrocki
> > Signed-off-by: Guennadi Liakhovetski
Hi Hans
On Tue, 2 Oct 2012, Hans Verkuil wrote:
> On Mon 1 October 2012 19:51:18 Guennadi Liakhovetski wrote:
> > Hi Hans
> >
> > Thanks for the review. As you might have seen, I just posted v2 of this
> > driver. In it I addressed almost all your comments. As for th
On Tue, 2 Oct 2012, Sylwester Nawrocki wrote:
> Hi Guennadi,
>
> On 10/02/2012 11:49 AM, Guennadi Liakhovetski wrote:
> >>> + if (!of_property_read_u32_array(node, "data-lanes", data_lanes,
> >>> + ARRAY
Hi Sylwester
On Mon, 1 Oct 2012, Sylwester Nawrocki wrote:
> On 09/27/2012 04:07 PM, Guennadi Liakhovetski wrote:
> > Add a V4L2 OF parser, implementing bindings, documented in
> > Documentation/devicetree/bindings/media/v4l2.txt.
> >
> > Signed-of
Hi Hans
Thanks for the review. As you might have seen, I just posted v2 of this
driver. In it I addressed almost all your comments. As for the rest:
On Tue, 11 Sep 2012, Hans Verkuil wrote:
> On Tue 11 September 2012 15:01:19 Guennadi Liakhovetski wrote:
[snip]
> > +struct s
for VEU. So far only conversion from NV12 to RGB565 is
supported. Further functionality shall be added in the future.
This driver is based on a VEU vidix driver by Magnus Damm.
Signed-off-by: Guennadi Liakhovetski
---
v2: addressed comments from Hans (thanks!), namely:
- taking into account
for ‘sh_mobile_ceu_host_ops.set_livecrop’)
> [enabled by default
>
> --
>
> Regards,
> Sylwester
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majord...@vger.kernel.org
> More majordomo inf
oc_host = &mcam->soc_host;
> +#else
> + struct mmp_camera_platform_data *pdata = cam->pdev->dev.platform_data;
> +#endif
>
> mmpcam_remove_device(cam);
> - free_irq(cam->irq, mcam);
> + devm_free_irq(&cam->pdev->dev, cam->irq,
nd re-use the common stuff.
> That, I think, is how I'd like to go toward a cleaner, more reviewable,
> more maintainable solution. Make sense?
Definitely!
Thanks
Guennadi
> Thanks,
>
> jon
>
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://
SDH0APMU_REG(0x054)
> #define APMU_SDH1APMU_REG(0x058)
> #define APMU_USB APMU_REG(0x05c)
> --
> 1.7.0.4
>
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
--
To unsubscribe from this list: send the line "unsubsc
buffers support in
> DMA_CONTIG mode
>
>
> Thanks
> Albert Wang
>
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord
On Fri, 28 Sep 2012, Anatolij Gustschin wrote:
> Hi Guennadi,
>
> On Fri, 28 Sep 2012 14:33:34 +0200 (CEST)
> Guennadi Liakhovetski wrote:
> ...
> > > @@ -235,12 +238,32 @@ static int mt9v022_s_stream(struct v4l2_subdev *sd,
> > > int enable)
A recent commit "[media] v4l2: make vidioc_s_crop const" introduced
warnings in mx2_camera. Fix them by cleanly separating writable and
read-only variables in cropping operations.
Signed-off-by: Guennadi Liakhovetski
---
drivers/media/platform/soc_camera/mx2_camera.c |5 +++-
A recent commit "[media] v4l2: make vidioc_s_crop const" introduced
warnings in sh_mobile_ceu_camera. Fix them by cleanly separating writable
and read-only variables in cropping operations.
Signed-off-by: Guennadi Liakhovetski
---
.../platform/soc_camera/sh_mobile_ceu_camera.c
.set_crop() implementation in mx1_camera is identical with the default.
Remove the copy to switch to using the default stab.
Signed-off-by: Guennadi Liakhovetski
---
drivers/media/platform/soc_camera/mx1_camera.c |9 -
1 files changed, 0 insertions(+), 9 deletions(-)
diff --git a
pull request.
Thanks
Guennadi
Guennadi Liakhovetski (7):
media: sh_mobile_ceu_camera: fix const cropping related warnings
media: pxa_camera: fix const cropping related warnings
media: sh_vou: fix const cropping related warnings
media: mx3_camera: fix const cropping related warnings
A recent commit "[media] v4l2: make vidioc_s_crop const" introduced
warnings in omap1_camera. Fix them by adjusting a function declaration.
Signed-off-by: Guennadi Liakhovetski
---
drivers/media/platform/soc_camera/omap1_camera.c |4 ++--
1 files changed, 2 insertions(+), 2
A recent commit "[media] v4l2: make vidioc_s_crop const" introduced
warnings in sh_vou. Fix them by cleanly separating writable and
read-only variables in cropping operations.
Signed-off-by: Guennadi Liakhovetski
---
drivers/media/platform/sh_vou.c |3 ++-
1 files changed, 2
A recent commit "[media] v4l2: make vidioc_s_crop const" introduced
warnings in mx3_camera. Fix them by cleanly separating writable and
read-only variables in cropping operations.
Signed-off-by: Guennadi Liakhovetski
---
drivers/media/platform/soc_camera/mx3_camera.c |5 +++-
A recent commit "[media] v4l2: make vidioc_s_crop const" introduced
warnings in pxa_camera.c. Fix them by adjusting a function declaration.
Signed-off-by: Guennadi Liakhovetski
---
drivers/media/platform/soc_camera/pxa_camera.c |4 ++--
1 files changed, 2 insertions(+), 2 deletion
ring running
streaming, right? Then how does this patch help you with that?
Thanks
Guennadi
>
> if (reg_write(client, MT9V022_CHIP_CONTROL, mt9v022->chip_control) < 0)
> return -EIO;
> @@ -652,6 +675,8 @@ static int mt9v022_video_probe(struct i2c_client *client)
Hi Eric
On Fri, 28 Sep 2012, Guennadi Liakhovetski wrote:
> Hi Anatolij
>
> I can take this patch, but we need an ack from a PXA / ARM maintainer.
Could we have your ack, please?
Thanks
Guennadi
> On Fri, 28 Sep 2012, Anatolij Gustschin wrote:
>
> > Set "y_sk
0..4056180
> --- /dev/null
> +++ b/include/media/mt9v022.h
> @@ -0,0 +1,16 @@
> +/*
> + * mt9v022 sensor
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * publis
any
further I2C commands, while streaming, their drivers have to do that
themselves. And not only for stream-on, there are many operations, that
are allowed during streaming.
Thanks
Guennadi
> if (!ret)
> v4l2_subdev_call(sd, video, s_stream, 1);
> +#endif
>
of_*_cmp() macros do not depend on any OF functions and can be defined also
if CONFIG_OF isn't set. Also include linux/string.h, required by those
macros.
Signed-off-by: Guennadi Liakhovetski
---
These definitions are also required for "# CONFIG_OF is not set" builds.
inc
Mauro, that should have been "3.6-rc7" of course... Please don't delay
until 3.7-rc7;-) I think, I'll rename the branch _after_ you pull it.
Thanks
Guennadi
On Tue, 25 Sep 2012, Guennadi Liakhovetski wrote:
> Hi Mauro
>
> There go the 3 fixes, that popped up (
The "struct device_node *" argument of of_parse_phandle_*() can be const.
Signed-off-by: Guennadi Liakhovetski
---
drivers/of/base.c |4 ++--
include/linux/of.h |6 +++---
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/of/base.c b/drivers/of/base.c
ind
If CONFIG_OF isn't defined, no declaration of of_get_parent will be found
and compilation can fail. This patch adds a dummy inline function
definition to fix the problem.
Signed-off-by: Guennadi Liakhovetski
---
include/linux/of.h |5 +
1 files changed, 5 insertions(+), 0 dele
If CONFIG_OF_I2C and CONFIG_OF_I2C_MODULE are undefined no declaration of
of_find_i2c_device_by_node and of_find_i2c_adapter_by_node will be
available. Add dummy inline functions to avoid compilation breakage.
Signed-off-by: Guennadi Liakhovetski
Cc: Ben Dooks
---
include/linux/of_i2c.h | 12
nt to depend on the clock
framework progress, we could implement V4L2 generic functions to do the same
until clocks become available.
2. the two driver-specific DT properties in patch 13/14 are just an RFC and
can be modified / discussed.
Thanks
Guennadi
Guennadi Liakhovetski (14):
i2c: add dummy i
In both error and clean up cases there is no need to wait for runtime PM
to finish suspending the device.
Signed-off-by: Guennadi Liakhovetski
---
.../platform/soc_camera/sh_mobile_ceu_camera.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/media
If .probe() is called without our private data, this might mean, that the
host driver hasn't probed yet. Defer probing in this case.
Signed-off-by: Guennadi Liakhovetski
---
drivers/media/i2c/soc_camera/mt9t112.c |5 -
1 files changed, 4 insertions(+), 1 deletions(-)
diff --
rface.
Signed-off-by: Guennadi Liakhovetski
---
drivers/media/platform/soc_camera/soc_camera.c | 255 ++--
include/media/soc_camera.h |2 +
2 files changed, 197 insertions(+), 60 deletions(-)
diff --git a/drivers/media/platform/soc_camera/soc_camera.c
b/dr
be
probed and / or used with it. However, if a client probes at a different
time, there is no way to know when to attach it to and when to detach it
from the host. To solve this we allow the client to decide itself, when it
hae to be attached and detached.
Signed-off-by: Guennadi Liakhovetski
--
ff-by: Guennadi Liakhovetski
---
drivers/media/platform/soc_camera/soc_camera.c | 337 ++--
include/media/soc_camera.h |5 +
2 files changed, 326 insertions(+), 16 deletions(-)
diff --git a/drivers/media/platform/soc_camera/soc_camera.c
b/drivers/media/pla
This patch adds a document, describing common V4L2 device tree bindings.
Co-authored-by: Sylwester Nawrocki
Signed-off-by: Guennadi Liakhovetski
---
Documentation/devicetree/bindings/media/v4l2.txt | 162 ++
1 files changed, 162 insertions(+), 0 deletions(-)
create mode
Additionally to the basic DT support, added to the driver in previous
patches, this patch implements complete interface configuration from DT.
Signed-off-by: Guennadi Liakhovetski
---
.../platform/soc_camera/sh_mobile_ceu_camera.c | 126
1 files changed, 78 insertions
Some CEU implementations have non-standard (larger) maximum supported
width and height values. Add two OF properties to specify them.
Signed-off-by: Guennadi Liakhovetski
---
.../platform/soc_camera/sh_mobile_ceu_camera.c | 19 +++
1 files changed, 19 insertions(+), 0
Using device-managed devm_regulator_bulk_get() eliminates the need to
release regulators explicitly.
Signed-off-by: Guennadi Liakhovetski
---
drivers/media/platform/soc_camera/soc_camera.c |8 ++--
1 files changed, 2 insertions(+), 6 deletions(-)
diff --git a/drivers/media/platform
Add a V4L2 OF parser, implementing bindings, documented in
Documentation/devicetree/bindings/media/v4l2.txt.
Signed-off-by: Guennadi Liakhovetski
---
drivers/media/v4l2-core/Makefile |3 +
drivers/media/v4l2-core/v4l2-of.c | 190 +
include/media/v4l2
Add an OF hook to sh_mobile_ceu_camera.c, no properties so far. Booting
with DT also requires platform data to be optional.
Signed-off-by: Guennadi Liakhovetski
---
.../platform/soc_camera/sh_mobile_ceu_camera.c | 34 ++--
1 files changed, 24 insertions(+), 10 deletions
Hi Sylwester
Ok, looks like there are no more comments coming, so, we can submit a new
version:-)
Thanks for your comments:
On Tue, 11 Sep 2012, Sylwester Nawrocki wrote:
> Hi Guennadi,
>
> On 09/11/2012 05:51 PM, Guennadi Liakhovetski wrote:
> > This patch adds a document, de
ivers/media/video/soc_camera.c: fix error return code
drivers/media/video/mx2_camera.c |4 +++-
drivers/media/video/ov2640.c |5 +
drivers/media/video/soc_camera.c |3 ++-
3 files changed, 10 insertions(+), 2 deletions(-)
Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freel
On Sun, 23 Sep 2012, Frank Schäfer wrote:
> Am 24.09.2012 00:33, schrieb Guennadi Liakhovetski:
> > On Sun, 23 Sep 2012, Frank SchÀfer wrote:
> >
> >> We currently don't select the register bank in ov2640_s_ctrl, so we can
> >> end up
> >> writing
);
> + if (ret < 0)
> + return ret;
>
> switch (ctrl->id) {
> case V4L2_CID_VFLIP:
> --
> 1.7.10.4
>
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
--
To unsubscribe from this li
;s occurring too often in
real life, so, I didn't bother sending a separate fix:-) But yes, let's
fix it properly. Please, update the other patch to mx2_camera and I'll
send a "fixes" pull request with these two and an ov2640 fix.
Thanks
Guennadi
>
> /* At
On Sun, 23 Sep 2012, Frank Schäfer wrote:
> Am 23.09.2012 23:43, schrieb Guennadi Liakhovetski:
> > On Sun, 23 Sep 2012, Frank Schäfer wrote:
> >
> >> Signed-off-by: Frank Schäfer
> >> ---
> >> drivers/media/i2c/soc_camera/ov2640.c | 17
}
> }
>
> pcdev->soc_host.drv_name = MX2_CAM_DRV_NAME,
>
> --
> 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://vger.kernel.
On Sun, 23 Sep 2012, Frank Schäfer wrote:
> Am 23.09.2012 23:21, schrieb Guennadi Liakhovetski:
> > Hi Frank
> >
> > On Sun, 23 Sep 2012, Frank Schäfer wrote:
> >
> >> We currently don't select the register bank in ov2640_s_ctrl, so we can
> >> e
u8 val;
>
> - regval.reg_num = BANK_SEL;
> - regval.value = BANK_SEL_SENS;
> - ret = ov2640_write_array(client, ®val);
> + ret = ov2640_write_single(client, BANK_SEL, BANK_SEL_SENS);
> if (ret < 0)
> return ret;
>
> -
9,11 +942,13 @@ static int ov2640_try_fmt(struct v4l2_subdev *sd,
> mf->field = V4L2_FIELD_NONE;
>
> switch (mf->code) {
> + case V4L2_MBUS_FMT_RGB565_2X8_BE:
> case V4L2_MBUS_FMT_RGB565_2X8_LE:
> mf->colorspace = V4L2_COLO
Guennadi
> + if (ret < 0)
> + return ret;
> +
> switch (ctrl->id) {
> case V4L2_CID_VFLIP:
> val = ctrl->val ? REG04_VFLIP_IMG : 0x00;
> --
> 1.7.10.4
>
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://ww
e you
more specifically.
> Thanks, Pete
Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
--
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://vger.kernel.org/majordomo-info.html
unction 'sh_vou_poll':
drivers/media/platform/sh_vou.c:1242: error: 'vdev' undeclared (first use in
this function)
make[2]: *** [drivers/media/platform/sh_vou.o] Error 1
Add missing variable definitions.
Signed-off-by: Guennadi Liakhovetski
---
This is a fix for 3.7
drivers/me
ramefmt::colorspace field
> to select between capture/preview in a device or using 32-bit integer
> control where upper 16-bits are used for pixel width and lower 16 for
> pixel height. This may suggest there something missing at the API.
>
> Any suggestions, critics, please ?... :)
>
On Mon, 17 Sep 2012, javier Martin wrote:
> Hi Shawn,
>
> On 17 September 2012 11:21, Guennadi Liakhovetski
> wrote:
> > On Mon, 17 Sep 2012, Shawn Guo wrote:
> >
> >> It changes the driver to use platform_device_id rather than cpu_is_xxx
> >> to dete
On Mon, 17 Sep 2012, Shawn Guo wrote:
> The header ipu.h really belongs to dma subsystem rather than imx
> platform. Rename it to ipu-dma.h and put it into include/linux/dma/.
>
> Signed-off-by: Shawn Guo
> Cc: Vinod Koul
> Cc: Guennadi Liakhovetski
> Cc: Florian To
ned-off-by: Shawn Guo
> Cc: Guennadi Liakhovetski
> Cc: linux-media@vger.kernel.org
Acked-by: Guennadi Liakhovetski
Thanks
Guennadi
> ---
> arch/arm/mach-imx/clk-imx25.c |6 +-
> arch/arm/mach-imx/clk-imx27.c |6 +-
> arc
On Mon, 17 Sep 2012, Shawn Guo wrote:
> Use managed functions to clean up the error handling code and function
> mx2_camera_remove(). Along with the change, a few variables get removed
> from struct mx2_camera_dev.
>
> Signed-off-by: Shawn Guo
> Cc: Guennadi Liakhovetski
known the breakage [1], but
> no one shows the interest to bring it back to work. Thus it becomes
> a piece of unmaintained code, so let's remove it.
>
> [1] https://lkml.org/lkml/2012/2/9/171
>
> Signed-off-by: Shawn Guo
> Cc: Paulius Zaleckas
> Cc: Guennadi L
e EMMA
> mandatory). It should have been removed together with the commit.
> Remove it now.
>
> Signed-off-by: Shawn Guo
> Cc: Guennadi Liakhovetski
> Cc: linux-media@vger.kernel.org
Acked-by: Guennadi Liakhovetski
Thanks
Guennadi
> ---
> drivers/media/video/mx2_
On Wed, 12 Sep 2012, Stephen Warren wrote:
> On 09/12/2012 01:28 PM, Guennadi Liakhovetski wrote:
> > Hi Stephen
> >
> > On Wed, 12 Sep 2012, Stephen Warren wrote:
> >
> >> On 09/11/2012 09:51 AM, Guennadi Liakhovetski wrote:
> >>> This patch add
Hi Stephen
On Wed, 12 Sep 2012, Stephen Warren wrote:
> On 09/11/2012 09:51 AM, Guennadi Liakhovetski wrote:
> > This patch adds a document, describing common V4L2 device tree bindings.
> >
> > Co-authored-by: Sylwester Nawrocki
> > Signed-off-by: Guennadi Liakhove
This patch adds a document, describing common V4L2 device tree bindings.
Co-authored-by: Sylwester Nawrocki
Signed-off-by: Guennadi Liakhovetski
---
Documentation/devicetree/bindings/media/v4l2.txt | 143 ++
1 files changed, 143 insertions(+), 0 deletions(-)
create mode
Hi Stephen
Thanks for the review.
On Wed, 5 Sep 2012, Stephen Warren wrote:
> On 09/05/2012 04:57 AM, Guennadi Liakhovetski wrote:
> > Hi all
> >
> > Version 5 of this RFC is a result of a discussion of its version 4, which
> > took place during the recent Linux
for VEU. So far only conversion from NV12 to RGB565 is
supported. Further functionality shall be added in the future.
This driver is based on a VEU vidix driver by Magnus Damm.
Signed-off-by: Guennadi Liakhovetski
---
Tested on Linux and Android ;-)
drivers/media/platform/Kconfig |9
The mem2mem core doesn't change struct m2m_ops, provided by the driver,
make references to it const.
Signed-off-by: Guennadi Liakhovetski
---
drivers/media/v4l2-core/v4l2-mem2mem.c |4 ++--
include/media/v4l2-mem2mem.h |2 +-
2 files changed, 3 insertions(+), 3 dele
Using list_first_entry() improves code readability and makes it less
error-prone.
Signed-off-by: Guennadi Liakhovetski
---
drivers/media/v4l2-core/v4l2-mem2mem.c |6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/media/v4l2-core/v4l2-mem2mem.c
b/drivers/media
form/soc_camera/mx2_camera.c |6 ++--
drivers/media/platform/soc_camera/soc_camera.c | 37 ---
2 files changed, 10 insertions(+), 33 deletions(-)
Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
--
To unsubscribe
On Fri, 24 Aug 2012, Anatolij Gustschin wrote:
> On Fri, 24 Aug 2012 13:23:22 +0200 (CEST)
> Guennadi Liakhovetski wrote:
>
> > On Fri, 24 Aug 2012, Anatolij Gustschin wrote:
> >
> > > Set "y_skip_top" to zero and remove comment as I do not see thi
Hi Anatolij
On Tue, 28 Aug 2012, Anatolij Gustschin wrote:
> Hi Guennadi,
>
> On Fri, 24 Aug 2012 23:23:37 +0200 (CEST)
> Guennadi Liakhovetski wrote:
> ...
> > > Every time the sensor is reset, it resets this register. Without setting
> > > the register after
ude
Do you really need the soc_camera.h header? From a quick glance I didn't
find any uses of the soc-camera API. If I missed them and you really are
using the API, the driver should probably go under
drivers/media/i2c/soc_camera/ and be submitted to the mainline via my
tree. Since you
era_device_register(icd);
> }
>
> /*
> @@ -1572,8 +1563,6 @@ static int __devexit soc_camera_pdrv_remove(struct
> platform_device *pdev)
>
> list_del(&icd->list);
>
> - kfree(icd);
> -
> return 0;
> }
>
> --
> 1.7.4.1
>
---
gt; + s->target == V4L2_SEL_TGT_COMPOSE) {
> icd->user_width = s->r.width;
> icd->user_height = s->r.height;
> if (!icd->streamer)
> --
> 1.7.4.1
>
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software
to the
parent.
(8) use standard names for "SoC-external" (e.g., i2c) devices on their
respective busses. "Sensor" has been proposed, maybe "camera" is a better
match though.
Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
htt
On Wed, 5 Sep 2012, Guennadi Liakhovetski wrote:
> Hi Javier
>
> On Mon, 3 Sep 2012, javier Martin wrote:
>
> > Hi,
> > Guennadi,did you pick this one?
>
> Wanted to do so, but
I've applied this your patch with only that "memset()" line additiona
o have to be removed?
Thanks
Guennadi
> > if ((mf.width != pix->width || mf.height != pix->height) &&
> > - pcdev->emma_prp->cfg.in_fmt == PRP_CNTL_DATA_IN_YUV422) {
> > + emma_prp->cfg.in_fmt == PRP_CNTL_DATA_IN_YUV422) {
>
On Thu, 30 Aug 2012, Hans Verkuil wrote:
> Fine by me!
Good, then let's meet at 11:55 in front of Nautilus 3 / 4 and then see
what's available.
Thanks
Guennadi
>
> Hans
>
> Guennadi Liakhovetski wrote:
>
> >On Fri, 31 Aug 2012, Laurent Pi
On Fri, 31 Aug 2012, Laurent Pinchart wrote:
> Hi Guennadi,
>
> On Thursday 30 August 2012 22:58:17 Guennadi Liakhovetski wrote:
> > Hi all
> >
> > Do we still want to try to organise a short discussion of this while at
> > plumbers? Maybe tomorrow during or ar
Hi all
Do we still want to try to organise a short discussion of this while at
plumbers? Maybe tomorrow during or around lunch? Or any other time for
that matter?
Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
--
To
this going to work when this driver is build and as a module
> and its loading is deferred past system booting ? At that time the driver's
> platform data may be well discarded.
Right, it will be gone, I think.
> You may wan't to duplicate it on the
> running boards in boa
901 - 1000 of 2736 matches
Mail list logo