[GIT PULL FOR v3.11]

2013-06-26 Thread Hans Verkuil
(Same as my previous git pull message, but with more cleanup patches and fixes from Prabhakar.) Some async/OF work from Prabhakar (the correct version this time) and assorted improvements and fixes for compiler warnings. Also some fixes for some fallout from the new requirement that v4l2_dev must

Re: [PATCH 1/2] media: davinci: vpif: capture: add V4L2-async support

2013-06-26 Thread Hans Verkuil
On Thu June 27 2013 08:13:40 Prabhakar Lad wrote: > Hi Hans, > > Thanks for the review. > > On Thu, Jun 27, 2013 at 11:27 AM, Hans Verkuil wrote: > > On Tue June 25 2013 17:17:34 Prabhakar Lad wrote: > >> From: "Lad, Prabhakar" > >> > >> Add support for asynchronous subdevice probing, using the

Re: Samsung i2c subdev drivers that set sd->name

2013-06-26 Thread Hans Verkuil
On Wed June 26 2013 11:00:51 Sakari Ailus wrote: > Hi Sylwester and Hans, > > On Tue, Jun 25, 2013 at 06:55:49PM +0200, Sylwester Nawrocki wrote: > > Hi Hans, > > > > Cc: Laurent and Sakari > > > > On 06/24/2013 10:54 AM, Hans Verkuil wrote: > > > Hi Sylwester, > > > > > > It came to my attenti

Re: [PATCH v2 1/5] phy: Add driver for Exynos MIPI CSIS/DSIM DPHYs

2013-06-26 Thread Felipe Balbi
On Wed, Jun 26, 2013 at 05:00:34PM +0200, Sylwester Nawrocki wrote: > Hi, > > On 06/25/2013 05:06 PM, Felipe Balbi wrote: > >> +static struct platform_driver exynos_video_phy_driver = { > >> > +.probe = exynos_video_phy_probe, > > > > you *must* provide a remove method. drivers with NULL

Re: [PATCH 1/2] media: davinci: vpif: capture: add V4L2-async support

2013-06-26 Thread Prabhakar Lad
Hi Hans, Thanks for the review. On Thu, Jun 27, 2013 at 11:27 AM, Hans Verkuil wrote: > On Tue June 25 2013 17:17:34 Prabhakar Lad wrote: >> From: "Lad, Prabhakar" >> >> Add support for asynchronous subdevice probing, using the v4l2-async API. >> The legacy synchronous mode is still supported t

Re: [PATCH 1/2] media: davinci: vpif: capture: add V4L2-async support

2013-06-26 Thread Hans Verkuil
On Tue June 25 2013 17:17:34 Prabhakar Lad wrote: > From: "Lad, Prabhakar" > > Add support for asynchronous subdevice probing, using the v4l2-async API. > The legacy synchronous mode is still supported too, which allows to > gradually update drivers and platforms. > > Signed-off-by: Prabhakar La

PRIVATE MESSAGE

2013-06-26 Thread JMW SOLICITORS LLP
LEGAL NOTICE; I once again try to notify you that my late client Mr James Campbell made his inheritance of 3.8 million pounds in your favour. Barr Colin Lee -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordom

cron job: media_tree daily build: WARNINGS

2013-06-26 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: Wed Jun 26 19:00:16 CEST 2013 git branch: test git hash: 188af63c0af2d7ef395bc94e3efa173f34dae03d gcc versi

Re: [RFC PATCH v2] dmabuf-sync: Introduce buffer synchronization framework

2013-06-26 Thread Russell King - ARM Linux
On Tue, Jun 25, 2013 at 11:23:21AM +0200, Daniel Vetter wrote: > Just a quick question on your assertion that we need all four > functions: Since we already have begin/end_cpu_access functions > (intention here was to allow the dma_buf exporter to ensure the memory > is pinned, e.g. for swapable ge

Re: [RFC PATCH] dmabuf-sync: Introduce buffer synchronization framework

2013-06-26 Thread Inki Dae
2013/6/25 Jerome Glisse : > On Tue, Jun 25, 2013 at 10:17 AM, Inki Dae wrote: >> 2013/6/25 Rob Clark : >>> On Tue, Jun 25, 2013 at 5:09 AM, Inki Dae wrote: > that > should be the role of kernel memory management which of course needs > synchronization btw A and B. But in no case this

Re: [PATCH v2] staging: lirc: clean error handling in probe()

2013-06-26 Thread Dan Carpenter
On Wed, Jun 26, 2013 at 06:29:12PM +0300, Andy Shevchenko wrote: > On Wed, 2013-06-26 at 18:10 +0300, Dan Carpenter wrote: > > On Wed, Jun 26, 2013 at 05:37:36PM +0300, Andy Shevchenko wrote: > > > From: Dan Carpenter > > > > > > We have reorganized the error handling into a simpler and more can

Re: [PATCH v2] staging: lirc: clean error handling in probe()

2013-06-26 Thread Andy Shevchenko
On Wed, 2013-06-26 at 18:10 +0300, Dan Carpenter wrote: > On Wed, Jun 26, 2013 at 05:37:36PM +0300, Andy Shevchenko wrote: > > From: Dan Carpenter > > > > We have reorganized the error handling into a simpler and more canonical > > format. > > > > Additionally we removed extra empty lines, swit

Re: [PATCH v2] staging: lirc: clean error handling in probe()

2013-06-26 Thread Dan Carpenter
On Wed, Jun 26, 2013 at 05:37:36PM +0300, Andy Shevchenko wrote: > From: Dan Carpenter > > We have reorganized the error handling into a simpler and more canonical > format. > > Additionally we removed extra empty lines, switched to devm_kzalloc(), and > substitute 'minor' by 'ret' in the igorpl

[PATCH v3 3/5] video: exynos_mipi_dsim: Use the generic PHY driver

2013-06-26 Thread Sylwester Nawrocki
Use the generic PHY API instead of the platform callback to control the MIPI DSIM DPHY. The 'phy_label' field is added to the platform data structure to allow PHY lookup on non-dt platforms. Signed-off-by: Sylwester Nawrocki Signed-off-by: Kyungmin Park Acked-by: Felipe Balbi --- drivers/video

[PATCH v3 4/5] exynos4-is: Use the generic MIPI CSIS PHY driver

2013-06-26 Thread Sylwester Nawrocki
Use the generic PHY API instead of the platform callback to control the MIPI CSIS DPHY. The 'phy_label' field is added to the platform data structure to allow PHY lookup on non-dt platforms Signed-off-by: Sylwester Nawrocki Signed-off-by: Kyungmin Park Acked-by: Felipe Balbi --- drivers/media/

[PATCH v3 2/5] ARM: dts: Add MIPI PHY node to exynos4.dtsi

2013-06-26 Thread Sylwester Nawrocki
Add PHY provider node for the MIPI CSIS and MIPI DSIM PHYs. Signed-off-by: Sylwester Nawrocki Signed-off-by: Kyungmin Park --- arch/arm/boot/dts/exynos4.dtsi | 10 ++ 1 file changed, 10 insertions(+) diff --git a/arch/arm/boot/dts/exynos4.dtsi b/arch/arm/boot/dts/exynos4.dtsi index 4

[PATCH v3 5/5] ARM: Samsung: Remove the MIPI PHY setup code

2013-06-26 Thread Sylwester Nawrocki
Generic PHY drivers are used to handle the MIPI CSIS and MIPI DSIM DPHYs so we can remove now unused code at arch/arm/plat-samsung. In case there is any board file for S5PV210 platforms using MIPI CSIS/DSIM (not any upstream currently) it should use the generic PHY API to bind the PHYs to respectiv

[PATCH v3 1/5] phy: Add driver for Exynos MIPI CSIS/DSIM DPHYs

2013-06-26 Thread Sylwester Nawrocki
Add a PHY provider driver for the Samsung S5P/Exynos SoC MIPI CSI-2 receiver and MIPI DSI transmitter DPHYs. Signed-off-by: Sylwester Nawrocki Signed-off-by: Kyungmin Park --- Changes since v2: - adapted to the generic PHY API v9: use phy_set/get_drvdata(), - fixed of_xlate callback to return

[PATCH v3 0/5] Generic PHY driver for the Exynos SoC MIPI CSI-2/DSI DPHYs

2013-06-26 Thread Sylwester Nawrocki
This patch series adds a simple driver for the Samsung S5P/Exynos SoC series MIPI CSI-2 receiver (MIPI CSIS) and MIPI DSI transmitter (MIPI DSIM) DPHYs, using the generic PHY framework [1]. Previously the MIPI CSIS and MIPI DSIM used a platform callback to control the PHY power enable and reset bit

Re: [PATCH v2 1/5] phy: Add driver for Exynos MIPI CSIS/DSIM DPHYs

2013-06-26 Thread Sylwester Nawrocki
Hi, On 06/25/2013 05:06 PM, Felipe Balbi wrote: >> +static struct platform_driver exynos_video_phy_driver = { >> > + .probe = exynos_video_phy_probe, > > you *must* provide a remove method. drivers with NULL remove are > non-removable :-) Actually the remove() callback can be NULL, it's just mi

[PATCH v2] staging: lirc: clean error handling in probe()

2013-06-26 Thread Andy Shevchenko
From: Dan Carpenter We have reorganized the error handling into a simpler and more canonical format. Additionally we removed extra empty lines, switched to devm_kzalloc(), and substitute 'minor' by 'ret' in the igorplugusb_remote_probe() function. Signed-off-by: Dan Carpenter Signed-off-by: An

Re: [PATCH v3 7/8] [media] V4L: Add VP8 encoder controls

2013-06-26 Thread Arun Kumar K
Hi Hans, On Wed, Jun 26, 2013 at 12:23 PM, Hans Verkuil wrote: > On Tue June 25 2013 12:57:14 Arun Kumar K wrote: >> This patch adds new V4L controls for VP8 encoding. >> > > FYI: I plan on reviewing this as soon as I have some time (should be this > weekend at the latest). > Thank you. Will wai

Re: [PATCH v3 7/8] [media] V4L: Add VP8 encoder controls

2013-06-26 Thread Arun Kumar K
Hi Kamil, >> --- a/drivers/media/v4l2-core/v4l2-ctrls.c >> +++ b/drivers/media/v4l2-core/v4l2-ctrls.c >> @@ -424,6 +424,12 @@ const char * const *v4l2_ctrl_get_menu(u32 id) >> NULL, >> }; >> >> + static const char * const vpx_golden_frame_sel[] = { >> + "Use Pre

Re: [PATCH v3 5/8] [media] s5p-mfc: Update driver for v7 firmware

2013-06-26 Thread Arun Kumar K
Hi Kamil, On Tue, Jun 25, 2013 at 7:33 PM, Kamil Debski wrote: > Hi Arun, > > This patch sets looks very good. > Please check my comments below. > >> From: Arun Kumar K [mailto:arun...@samsung.com] >> >> Firmware version v7 is mostly similar to v6 in terms of hardware >> specific controls and com

Re: [PATCH v3 4/8] [media] s5p-mfc: Core support for MFC v7

2013-06-26 Thread Arun Kumar K
Hi Kamil, On Wed, Jun 26, 2013 at 1:50 PM, Kamil Debski wrote: > Hi, > >> -Original Message- >> From: Sachin Kamat [mailto:sachin.ka...@linaro.org] >> Sent: Wednesday, June 26, 2013 9:05 AM >> To: Arun Kumar K >> Cc: Arun Kumar K; LMML; Kamil Debski; jtp.p...@samsung.com; Sylwester >> Naw

Re: [PATCH v2 1/5] phy: Add driver for Exynos MIPI CSIS/DSIM DPHYs

2013-06-26 Thread Kishon Vijay Abraham I
On Wednesday 26 June 2013 05:52 PM, Felipe Balbi wrote: On Wed, Jun 26, 2013 at 02:03:42PM +0200, Sylwester Nawrocki wrote: On 06/26/2013 01:21 PM, Kishon Vijay Abraham I wrote: +static int exynos_video_phy_probe(struct platform_device *pdev) +{ + struct exynos_video_phy *state; +

Re: [PATCH v2 1/5] phy: Add driver for Exynos MIPI CSIS/DSIM DPHYs

2013-06-26 Thread Kishon Vijay Abraham I
On Wednesday 26 June 2013 05:33 PM, Sylwester Nawrocki wrote: On 06/26/2013 01:21 PM, Kishon Vijay Abraham I wrote: +static int exynos_video_phy_probe(struct platform_device *pdev) +{ + struct exynos_video_phy *state; + struct device *dev = &pdev->dev; + struct resource *res;

Re: [PATCH v2 1/5] phy: Add driver for Exynos MIPI CSIS/DSIM DPHYs

2013-06-26 Thread Felipe Balbi
On Wed, Jun 26, 2013 at 02:03:42PM +0200, Sylwester Nawrocki wrote: > On 06/26/2013 01:21 PM, Kishon Vijay Abraham I wrote: > > +static int exynos_video_phy_probe(struct platform_device *pdev) > > +{ > > + struct exynos_video_phy *state; > > + struct device *dev = &

Re: [PATCH v2 1/5] phy: Add driver for Exynos MIPI CSIS/DSIM DPHYs

2013-06-26 Thread Sylwester Nawrocki
On 06/26/2013 01:21 PM, Kishon Vijay Abraham I wrote: > +static int exynos_video_phy_probe(struct platform_device *pdev) > +{ > +struct exynos_video_phy *state; > +struct device *dev = &pdev->dev; > +struct resource *res; > +struct ph

Re: [PATCH v2 1/5] phy: Add driver for Exynos MIPI CSIS/DSIM DPHYs

2013-06-26 Thread Kishon Vijay Abraham I
Hi, On Wednesday 26 June 2013 02:24 AM, Felipe Balbi wrote: Hi, On Tue, Jun 25, 2013 at 07:44:52PM +0200, Sylwester Nawrocki wrote: +struct exynos_video_phy { + spinlock_t slock; + struct phy *phys[NUM_PHYS]; more than one phy ? This means you should instantiate driver multiple d

Re: [patch] [media] staging: lirc: clean error handling in probe()

2013-06-26 Thread Dan Carpenter
On Wed, Jun 26, 2013 at 11:00:40AM +0300, Andy Shevchenko wrote: > On Wed, 2013-06-26 at 10:53 +0300, Dan Carpenter wrote: > > I have reorganized the error handling into a simpler and more canonical > > format. > > Since you reorganize error handling, might be worth to convert it to > devm_*? >

Re: [REVIEW PATCH 2/3] bfin_capture: fix compiler warning

2013-06-26 Thread Scott Jiang
2013/6/21 Hans Verkuil : > From: Hans Verkuil > > media-git/drivers/media/platform/blackfin/bfin_capture.c: In function > ‘bcap_probe’: > media-git/drivers/media/platform/blackfin/bfin_capture.c:1007:16: warning: > ignoring return value of ‘vb2_queue_init’, declared with attribute > warn_unused

Re: Samsung i2c subdev drivers that set sd->name

2013-06-26 Thread Sakari Ailus
Hi Sylwester and Hans, On Tue, Jun 25, 2013 at 06:55:49PM +0200, Sylwester Nawrocki wrote: > Hi Hans, > > Cc: Laurent and Sakari > > On 06/24/2013 10:54 AM, Hans Verkuil wrote: > > Hi Sylwester, > > > > It came to my attention that several i2c subdev drivers overwrite the > > sd->name > > as s

RE: [PATCH] mem2mem_testdev: fix missing v4l2_dev assignment

2013-06-26 Thread Kamil Debski
Hi Hans, > From: Hans Verkuil [mailto:hverk...@xs4all.nl] > Sent: Wednesday, June 26, 2013 8:31 AM > > Hi Kamil, > > This patch adds the missing v4l2_dev assignment as reported by > Fengguang. > > It also fixes a poorly formatted message: > > m2m-testdev m2m-testdev.0: mem2mem-testdevDevice re

RE: [PATCH v3 4/8] [media] s5p-mfc: Core support for MFC v7

2013-06-26 Thread Kamil Debski
Hi, > -Original Message- > From: Sachin Kamat [mailto:sachin.ka...@linaro.org] > Sent: Wednesday, June 26, 2013 9:05 AM > To: Arun Kumar K > Cc: Arun Kumar K; LMML; Kamil Debski; jtp.p...@samsung.com; Sylwester > Nawrocki; Hans Verkuil; avnd.ki...@samsung.com > Subject: Re: [PATCH v3 4/8]

Re: [patch] [media] staging: lirc: clean error handling in probe()

2013-06-26 Thread Andy Shevchenko
On Wed, 2013-06-26 at 10:53 +0300, Dan Carpenter wrote: > I have reorganized the error handling into a simpler and more canonical > format. Since you reorganize error handling, might be worth to convert it to devm_*? If you want I could do the patch. -- Andy Shevchenko Intel Finland Oy -- T

[patch] [media] staging: lirc: clean error handling in probe()

2013-06-26 Thread Dan Carpenter
I have reorganized the error handling into a simpler and more canonical format. Signed-off-by: Dan Carpenter diff --git a/drivers/staging/media/lirc/lirc_igorplugusb.c b/drivers/staging/media/lirc/lirc_igorplugusb.c index 2faa391..4cf3933 100644 --- a/drivers/staging/media/lirc/lirc_igorplugusb

Re: [PATCH v7] V4L2: soc_camera: Renesas R-Car VIN driver

2013-06-26 Thread Guennadi Liakhovetski
Hi Sergei On Sat, 22 Jun 2013, Sergei Shtylyov wrote: > From: Vladimir Barinov > > Add Renesas R-Car VIN (Video In) V4L2 driver. > > Based on the patch by Phil Edworthy . > > Signed-off-by: Vladimir Barinov > [Sergei: removed deprecated IRQF_DISABLED flag, reordered/renamed 'enum > chip_id'

Re: [RFC v2 05/10] exynos5-fimc-is: Adds the sensor subdev

2013-06-26 Thread Hans Verkuil
On Fri May 31 2013 15:03:23 Arun Kumar K wrote: > FIMC-IS uses certain sensors which are exclusively controlled > from the IS firmware. This patch adds the sensor subdev for the > fimc-is sensors. > > Signed-off-by: Arun Kumar K > Signed-off-by: Kilyeon Im Not surprisingly I really hate the ide

[GIT PULL] #3 for 3.11 - clk / async API documentation, fixes

2013-06-26 Thread Guennadi Liakhovetski
Hi Mauro A couple of fixes on top of my clock / async work plus the so much desired documentation :-) The following changes since commit 05959be7b646e8755a9339ad13e3b87849249f90: [media] uvc: Depend on VIDEO_V4L2 (2013-06-24 22:54:40 -0300) are available in the git repository at: git://lin

Re: [RFC v2 06/10] exynos5-fimc-is: Adds isp subdev

2013-06-26 Thread Hans Verkuil
On Fri May 31 2013 15:03:24 Arun Kumar K wrote: > fimc-is driver takes video data input from the ISP video node > which is added in this patch. This node accepts Bayer input > buffers which is given from the IS sensors. > > Signed-off-by: Arun Kumar K > Signed-off-by: Kilyeon Im > --- > drivers

Re: [RFC v2 07/10] exynos5-fimc-is: Adds scaler subdev

2013-06-26 Thread Hans Verkuil
On Fri May 31 2013 15:03:25 Arun Kumar K wrote: > FIMC-IS has two hardware scalers named as scaler-codec and > scaler-preview. This patch adds the common code handling the > video nodes and subdevs of both the scalers. > > Signed-off-by: Arun Kumar K > Signed-off-by: Kilyeon Im > --- > drivers/

Re: [PATCH v3 4/8] [media] s5p-mfc: Core support for MFC v7

2013-06-26 Thread Sachin Kamat
Hi Arun, On 26 June 2013 12:18, Arun Kumar K wrote: > Hi Sachin, > > On Tue, Jun 25, 2013 at 4:54 PM, Sachin Kamat wrote: >> Hi Arun, >> >>> @@ -684,5 +685,6 @@ void set_work_bit_irqsave(struct s5p_mfc_ctx *ctx); >>> (dev->variant->port_num ? 1 : 0) : 0) : 0) >>>