Re: [PATCH 3/6] ARM: OMAP2+: Move plat/iovmm.h to include/linux/omap-iommu.h

2012-10-25 Thread Laurent Pinchart
Hi Tony, Thanks for the patch. On Wednesday 24 October 2012 17:20:56 Tony Lindgren wrote: Looks like the iommu framework does not have generic functions exported for all the needs yet. The hardware specific functions are defined in files like intel-iommu.h and amd-iommu.h. Follow the same

Re: [media-workshop] Tentative Agenda for the November workshop

2012-10-25 Thread Sylwester Nawrocki
Hi Laurent, On 10/25/2012 12:42 AM, Laurent Pinchart wrote: Sylwester, would Samsung be able to prepare for a brainstorming session on Monday or Tuesday? Both Laurent and myself have presentations on Wednesday, so that's not the best day for such a session. Kamil has presentation on Tuesday

Re: [media-workshop] Tentative Agenda for the November workshop

2012-10-25 Thread Hans Verkuil
On Thu October 25 2012 10:43:59 Sylwester Nawrocki wrote: Hi Laurent, On 10/25/2012 12:42 AM, Laurent Pinchart wrote: Sylwester, would Samsung be able to prepare for a brainstorming session on Monday or Tuesday? Both Laurent and myself have presentations on Wednesday, so that's not the

Re: [GIT PULL FOR v3.8] OMAP3 ISP patches

2012-10-25 Thread Laurent Pinchart
Hi Mauro, Please cancel this pull request. Two of the patches need to be pushed to v3.7. I've sent a separate pull request for them, and I'll resend this one without those two patches. -- Regards, Laurent Pinchart -- To unsubscribe from this list: send the line unsubscribe linux-media in

[PATCH] s5p-fimc: Fix platform entities registration

2012-10-25 Thread Sylwester Nawrocki
Make sure there is no v4l2_device_unregister_subdev() call on a subdev which wasn't registered. Signed-off-by: Sylwester Nawrocki s.nawro...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- drivers/media/platform/s5p-fimc/fimc-mdevice.c | 25 1

[PATCH] Add MFC device tree support

2012-10-25 Thread Arun Kumar K
This patch adds device tree support for MFC driver. The arch side patch for Exynos5250 adding MFC DT node is applied to linux-samsung tree. [1] [1]http://www.mail-archive.com/linux-samsung-soc@vger.kernel.org/msg12672.html Arun Kumar K (1): s5p-mfc: Add device tree support

[PATCH] s5p-mfc: Add device tree support

2012-10-25 Thread Arun Kumar K
This patch will add the device tree support for MFC driver. Signed-off-by: Arun Kumar K arun...@samsung.com --- drivers/media/platform/s5p-mfc/s5p_mfc.c | 100 +- 1 files changed, 84 insertions(+), 16 deletions(-) diff --git

Re: [PATCH] s5p-fimc: Fix platform entities registration

2012-10-25 Thread Laurent Pinchart
Hi Sylwester, On Thursday 25 October 2012 11:06:56 Sylwester Nawrocki wrote: Make sure there is no v4l2_device_unregister_subdev() call on a subdev which wasn't registered. I'm not implying that this fix is bad, but doesn't the V4L2 core already handle this ? v4l2_device_unregister_subdev()

Re: [PATCH] s5p-fimc: Fix platform entities registration

2012-10-25 Thread Sylwester Nawrocki
Hi Laurent, On 10/25/2012 01:35 PM, Laurent Pinchart wrote: On Thursday 25 October 2012 11:06:56 Sylwester Nawrocki wrote: Make sure there is no v4l2_device_unregister_subdev() call on a subdev which wasn't registered. I'm not implying that this fix is bad, but doesn't the V4L2 core already

[PATCH] [media] fix tua6034 pll bandwich configuration

2012-10-25 Thread Arne Fitzenreiter
commit 80d8d4985f280dca3c395286d13b49f910a029e7 introduce a bug that set the wrong bandwich bit on the tua6034 pll. Signed-off-by: Arne Fitzenreiter arne.fitzenrei...@ipfire.org diff -Naur a/drivers/media/dvb-frontends/dvb-pll.c b/drivers/media/dvb-frontends/dvb-pll.c ---

Re: [RESEND-PATCH] media:davinci: clk - {prepare/unprepare} for common clk

2012-10-25 Thread Prabhakar Lad
Hi Murali, Thanks for the patch. I'll queue this patch for 3.8. On Mon, Oct 22, 2012 at 9:06 PM, Murali Karicheri m-kariche...@ti.com wrote: As a first step towards migrating davinci platforms to use common clock framework, replace all instances of clk_enable() with clk_prepare_enable() and

Re: [PATCH v2] media: davinci: vpbe: fix build warning

2012-10-25 Thread Laurent Pinchart
Hi Prabhakar, Thanks for the patch. On Tuesday 23 October 2012 18:54:37 Prabhakar Lad wrote: From: Lad, Prabhakar prabhakar@ti.com Warnings were generated because of the following commit changed data type for address pointer 195bbca ARM: 7500/1: io: avoid writeback addressing modes

Re: [PATCH v2 0/5] media: ov7670: driver cleanup and support for ov7674.

2012-10-25 Thread javier Martin
Hi Mauro, do you have any problems with this series? Regards. On 29 September 2012 21:20, Jonathan Corbet cor...@lwn.net wrote: On Thu, 27 Sep 2012 17:38:20 +0200 Javier Martin javier.mar...@vista-silicon.com wrote: The following series includes all the changes discussed in [1] that don't

Re: [PATCH v3 0/4] ov7670: migrate this sensor and its users to ctrl framework.

2012-10-25 Thread javier Martin
Hi Mauro, Jon, any more issues related with this series? Regards. On 29 September 2012 21:25, Jonathan Corbet cor...@lwn.net wrote: On Fri, 28 Sep 2012 13:26:39 +0200 Javier Martin javier.mar...@vista-silicon.com wrote: The following series migrate ov7670 sensor and current users to ctrl

Re: [PATCH] Fixed list_del corruption in videobuf-core.c : videobuf_queue_cancel()

2012-10-25 Thread Mauro Carvalho Chehab
Hi Vaibhav/Andrei, Em Tue, 9 Oct 2012 17:32:25 + Hiremath, Vaibhav hvaib...@ti.com escreveu: On Tue, Oct 09, 2012 at 14:08:18, Andrey Mandychev wrote: Hi, Please find below some additional comments. Actually it's not a real issue. It's more warning than issue. When you are

Re: [PATCH 1/2] ARM: clk-imx27: Add missing clock for mx2-camera

2012-10-25 Thread Mauro Carvalho Chehab
Hi Fábio, Em Fri, 5 Oct 2012 18:53:01 -0300 Fabio Estevam fabio.este...@freescale.com escreveu: During the clock conversion for mx27 the per4_gate clock was missed to get registered as a dependency of mx2-camera driver. In the old mx27 clock driver we used to have: DEFINE_CLOCK1(csi_clk,

Re: [PATCH 1/2] ARM: clk-imx27: Add missing clock for mx2-camera

2012-10-25 Thread Mauro Carvalho Chehab
Em Thu, 25 Oct 2012 11:38:41 -0200 Mauro Carvalho Chehab mche...@redhat.com escreveu: Hi Fábio, Em Fri, 5 Oct 2012 18:53:01 -0300 Fabio Estevam fabio.este...@freescale.com escreveu: During the clock conversion for mx27 the per4_gate clock was missed to get registered as a dependency of

Re: [PATCH] omap_vout: Set DSS overlay_info only if paddr is non zero

2012-10-25 Thread Tomi Valkeinen
On 2012-03-09 10:03, Hiremath, Vaibhav wrote: On Fri, Mar 09, 2012 at 05:17:41, Laurent Pinchart wrote: Hi Archit, On Wednesday 07 March 2012 14:31:16 Archit Taneja wrote: The omap_vout driver tries to set the DSS overlay_info using set_overlay_info() when the physical address for the

[PATCH] s5p-fimc: Fix platform entities registration

2012-10-25 Thread Sylwester Nawrocki
Make sure the platform sub-devices are registered to the media device driver only when v4l2_device_unregister_subdev() succeeds. Signed-off-by: Sylwester Nawrocki s.nawro...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- drivers/media/platform/s5p-fimc/fimc-mdevice.c |

Re: [PATCH 1/2] ARM: clk-imx27: Add missing clock for mx2-camera

2012-10-25 Thread Fabio Estevam
Hi Sascha, On Thu, Oct 25, 2012 at 11:46 AM, Mauro Carvalho Chehab mche...@redhat.com wrote: As this patch is for arch/arm, I'm understanding that it will be merged via arm tree. So, Acked-by: Mauro Carvalho Chehab mche...@redhat.com Forgot to comment: as patch 2 relies on this change, the

[PATCH] pvr2: fix minor storage

2012-10-25 Thread Alan Cox
From: Alan Cox a...@linux.intel.com This should have break statements in it. Signed-off-by: Alan Cox a...@linux.intel.com --- drivers/media/usb/pvrusb2/pvrusb2-hdw.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/media/usb/pvrusb2/pvrusb2-hdw.c

[PATCH] v4l2: sn9c102 incorrectly blocks FMT_SN9C10X

2012-10-25 Thread Alan Cox
From: Alan Cox a...@linux.intel.com Missing break Signed-off-by: Alan Cox a...@linux.intel.com --- drivers/media/usb/sn9c102/sn9c102_core.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/media/usb/sn9c102/sn9c102_core.c b/drivers/media/usb/sn9c102/sn9c102_core.c index

[PATCH v3] media: davinci: vpbe: fix build warning

2012-10-25 Thread Prabhakar Lad
From: Lad, Prabhakar prabhakar@ti.com Warnings were generated because of the following commit changed data type for address pointer 195bbca ARM: 7500/1: io: avoid writeback addressing modes for __raw_ accessors add __iomem annotation to fix following warnings

[PATCH] Rotate the videout via V4L2_CID_ROTATE based on fb_var_screeninfo.rotate

2012-10-25 Thread Florian Neuhaus
Hi Laurent You are very fast on implementing new features in omap3-isp-live. I appreciate much the new panning-feature - thank you! On my beagleboard I use omap-vout with vrfb-rotation. As I already configured omapfb to rotate the screen upon start (kernel-argument omapfb.rotate), I thought it

Re: [PATCH 1/2] ARM: clk-imx27: Add missing clock for mx2-camera

2012-10-25 Thread Sascha Hauer
On Fri, Oct 05, 2012 at 06:53:01PM -0300, Fabio Estevam wrote: During the clock conversion for mx27 the per4_gate clock was missed to get registered as a dependency of mx2-camera driver. In the old mx27 clock driver we used to have: DEFINE_CLOCK1(csi_clk, 0, NULL, 0, parent, csi_clk1,

Re: [PATCH v3 0/4] ov7670: migrate this sensor and its users to ctrl framework.

2012-10-25 Thread Mauro Carvalho Chehab
Hi Javier, Em Thu, 25 Oct 2012 15:27:08 +0200 javier Martin javier.mar...@vista-silicon.com escreveu: Hi Mauro, Jon, any more issues related with this series? Patch doesn't apply anymore: patching file drivers/media/i2c/ov7670.c Hunk #2 succeeded at 191 (offset -32 lines). Hunk #3 succeeded

[PATCH] [media] drxk_hard: fix a few warnings

2012-10-25 Thread Mauro Carvalho Chehab
drivers/media/dvb-frontends/drxk_hard.c:68:6: warning: no previous prototype for 'IsA1WithPatchCode' [-Wmissing-prototypes] drivers/media/dvb-frontends/drxk_hard.c:73:6: warning: no previous prototype for 'IsA1WithRomCode' [-Wmissing-prototypes] drivers/media/dvb-frontends/drxk_hard.c:192:12:

[GIT PULL FOR 3.7] Samsung media drivers fixes

2012-10-25 Thread Sylwester Nawrocki
Hi Mauro, please pull following fixes for v3.7-rc. The following changes since commit 1fdead8ad31d3aa833bc37739273fcde89ace93c: [media] m5mols: Add missing #include linux/sizes.h (2012-10-10 08:17:16 -0300) are available in the git repository at:

Re: [PATCH v2 0/5] media: ov7670: driver cleanup and support for ov7674.

2012-10-25 Thread Mauro Carvalho Chehab
Em Thu, 25 Oct 2012 15:24:49 +0200 javier Martin javier.mar...@vista-silicon.com escreveu: Hi Mauro, do you have any problems with this series? It didn't apply here, maybe due to its dependency from your previous ov7670 (v3) series. From my side, feel free to rebase them and re-submit.

Re: [GIT PULL FOR 3.7] Samsung media drivers fixes

2012-10-25 Thread Mauro Carvalho Chehab
Em Thu, 25 Oct 2012 18:01:17 +0200 Sylwester Nawrocki s.nawro...@samsung.com escreveu: Hi Mauro, please pull following fixes for v3.7-rc. The following changes since commit 1fdead8ad31d3aa833bc37739273fcde89ace93c: [media] m5mols: Add missing #include linux/sizes.h (2012-10-10

Re: [PATCH 3/6] ARM: OMAP2+: Move plat/iovmm.h to include/linux/omap-iommu.h

2012-10-25 Thread Tony Lindgren
* Laurent Pinchart laurent.pinch...@ideasonboard.com [121025 01:39]: I still think you should split this in two files, omap-iommu.h and omap- iovmm.h. The later would just be arch/arm/plat-omap/include/plat/iovmm.h moved to include/linux.h. Can you please explain a bit more why you're

Re: [media-workshop] Tentative Agenda for the November workshop

2012-10-25 Thread Mauro Carvalho Chehab
Hi Hans, Em Mon, 22 Oct 2012 10:35:56 +0200 Hans Verkuil hverk...@xs4all.nl escreveu: Hi all, This is the tentative agenda for the media workshop on November 8, 2012. If you have additional things that you want to discuss, or something is wrong or incomplete in this list, please let me

Re: [GIT PULL FOR 3.7] Samsung media drivers fixes

2012-10-25 Thread Sylwester Nawrocki
On 10/25/2012 06:46 PM, Mauro Carvalho Chehab wrote: Em Thu, 25 Oct 2012 18:01:17 +0200 Sylwester Nawrockis.nawro...@samsung.com escreveu: Hi Mauro, please pull following fixes for v3.7-rc. The following changes since commit 1fdead8ad31d3aa833bc37739273fcde89ace93c: [media] m5mols:

[PATCH] MAINTAINERS: update email and git tree

2012-10-25 Thread Mauro Carvalho Chehab
While mche...@infradead.org is valid, I prefer to use just one email for all patches upstream, instead of receiving some things on one emails, and others on some other place. While here, also update the main linux-media development tree, as the one at kernel.org is used mainly for patch merging

Pinnacle PCTV pcmcia through yenta pci brighe

2012-10-25 Thread André EBERSOLD
Hi, I have a Pinnacle PCTV (310c) board that I was trying to make work on a Quad core PC tower with Ubuntu 12.04 kernel 3.0.0-26 Unfortunately there is a kernel panic and the computer resets has soon as the card is plugged in the yenta pcmcia to pci adapter. Even after reset, the kernel raises

Re: [PATCH v3 2/6] Add the main bulk of core driver for SI476x code

2012-10-25 Thread Mark Brown
On Tue, Oct 23, 2012 at 11:44:28AM -0700, Andrey Smirnov wrote: + core-regmap = devm_regmap_init_si476x(core); + if (IS_ERR(core-regmap)) { This really makes little sense to me, why are you doing this? Does the device *really* layer a byte stream on top of I2C for sending messages

cron job: media_tree daily build: WARNINGS

2012-10-25 Thread Hans Verkuil
This message is generated daily by a cron job that builds media_tree for the kernels and architectures in the list below. Results of the daily build of media_tree: date:Thu Oct 25 19:00:22 CEST 2012 git hash:542fb082b9c76af8e71e9ea286d2dafdfba27099 gcc version: i686-linux-gcc

Re: [PATCH 3/6] ARM: OMAP2+: Move plat/iovmm.h to include/linux/omap-iommu.h

2012-10-25 Thread Laurent Pinchart
Hi Tony, On Thursday 25 October 2012 09:56:44 Tony Lindgren wrote: * Laurent Pinchart laurent.pinch...@ideasonboard.com [121025 01:39]: I still think you should split this in two files, omap-iommu.h and omap- iovmm.h. The later would just be arch/arm/plat-omap/include/plat/iovmm.h moved to

[PATCH v2] omap3isp: Remove unneeded module memory address definitions

2012-10-25 Thread Laurent Pinchart
The OMAP3ISP_*_REG_OFFSET, OMAP3ISP_*_REG_BASE and OMAP3ISP_*_REG macros are not needed. Remove them. The only expection is the OMAP3ISP_HIST_REG_BASE address. Replace it with the memory address received through platform resources. Signed-off-by: Laurent Pinchart

Re: [Linaro-mm-sig] [PATCH] dma-buf: Use EXPORT_SYMBOL

2012-10-25 Thread Dave Airlie
Unlikely as most of the code I've written belongs to Intel or Red Hat. I also have better things to do with life than sue Nvidia and start an all out copyright and patent war in Linuxspace. I forgot to ask, but after your petty G+ trolling, if most of the code belings to Intel or Red Hat, why

[PATCH 1/2] omap3isp: Prepare/unprepare clocks before/after enable/disable

2012-10-25 Thread Laurent Pinchart
Clock enable (disable) is split in two operations, prepare and enable (disable and unprepare). Perform both when enabling/disabling the ISP clocks. Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com --- drivers/media/platform/omap3isp/isp.c | 25 + 1

[PATCH 2/2] omap3isp: video: Merge two pipeline search operations at streamon time

2012-10-25 Thread Laurent Pinchart
The current code first iterates over entities with external connectivity to locate the external source, and then walks the pipeline to verify its connectivity. Merge both search operations. This has the side effect of removing the bogus can't find source, failing now warning message printed when

Re: [PATCH 3/6] ARM: OMAP2+: Move plat/iovmm.h to include/linux/omap-iommu.h

2012-10-25 Thread Tony Lindgren
* Laurent Pinchart laurent.pinch...@ideasonboard.com [121025 13:23]: Hi Tony, On Thursday 25 October 2012 09:56:44 Tony Lindgren wrote: * Laurent Pinchart laurent.pinch...@ideasonboard.com [121025 01:39]: I still think you should split this in two files, omap-iommu.h and omap- iovmm.h.

Re: [PATCH 1/2] omap3isp: Prepare/unprepare clocks before/after enable/disable

2012-10-25 Thread Sakari Ailus
Hi Laurent, On Thu, Oct 25, 2012 at 11:39:42PM +0200, Laurent Pinchart wrote: Clock enable (disable) is split in two operations, prepare and enable (disable and unprepare). Perform both when enabling/disabling the ISP clocks. Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com

Re: [PATCH v3 2/6] Add the main bulk of core driver for SI476x code

2012-10-25 Thread Andrey Smirnov
On 10/25/2012 12:45 PM, Mark Brown wrote: On Tue, Oct 23, 2012 at 11:44:28AM -0700, Andrey Smirnov wrote: +core-regmap = devm_regmap_init_si476x(core); +if (IS_ERR(core-regmap)) { This really makes little sense to me, why are you doing this? Does the device *really* layer a byte

Re: [PATCH 1/2] media: V4L2: add temporary clock helpers

2012-10-25 Thread Laurent Pinchart
Hello, On Sunday 21 October 2012 20:52:21 Sylwester Nawrocki wrote: On 10/20/2012 12:20 AM, Guennadi Liakhovetski wrote: Typical video devices like camera sensors require an external clock source. Many such devices cannot even access their hardware registers without a running clock. These

Re: [PATCH] pvr2: fix minor storage

2012-10-25 Thread Mike Isely
Completely agree! Thanks for spotting that one. Signed-off-by: Mike Isely is...@pobox.com -Mike On Thu, 25 Oct 2012, Alan Cox wrote: From: Alan Cox a...@linux.intel.com This should have break statements in it. Signed-off-by: Alan Cox a...@linux.intel.com ---

Re: [PATCH 3/6] ARM: OMAP2+: Move plat/iovmm.h to include/linux/omap-iommu.h

2012-10-25 Thread Ohad Ben-Cohen
On Thu, Oct 25, 2012 at 11:39 PM, Tony Lindgren t...@atomide.com wrote: Joerg and Ohad, do you have any opinions on this? I agree that there's some merit in having a separate header file for IOVMM, since it's a different layer from the IOMMU API. But in reality it's tightly coupled with OMAP's