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

2013-06-25 Thread Hans Verkuil
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). Regards, Hans -- To unsubscribe from this list: send the line "unsubscribe linux-medi

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

2013-06-25 Thread Arun Kumar K
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) >> #define IS_TWOPORT(dev)(dev->variant->port_num

Re: [PATCH] mem2mem_testdev: fix missing v4l2_dev assignment

2013-06-25 Thread Hans Verkuil
On Wed June 26 2013 08:31:14 Hans Verkuil wrote: > 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 registered as /dev/video0 > > Is it OK if I take this thr

[PATCH] mem2mem_testdev: fix missing v4l2_dev assignment

2013-06-25 Thread Hans Verkuil
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 registered as /dev/video0 Is it OK if I take this through my tree? I have similar fix as well in another driver. Regard

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

2013-06-25 Thread Felipe Balbi
On Tue, Jun 25, 2013 at 11:47:13PM +0200, Sylwester Nawrocki wrote: > Hi, > > On 06/25/2013 10:54 PM, Felipe Balbi wrote: > +static int exynos_video_phy_probe(struct platform_device *pdev) > > >> +{ > > >> + struct exynos_video_phy *state; > > >> + struct device *dev =&pdev

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

2013-06-25 Thread Sylwester Nawrocki
Hi, On 06/25/2013 10:54 PM, Felipe Balbi 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 phy_provider *phy_pr

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

2013-06-25 Thread Felipe Balbi
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 > > drivers. Each phy id should call probe again. >

cron job: media_tree daily build: WARNINGS

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

[GIT PULL] platform/exynos4-is updates

2013-06-25 Thread Sylwester Nawrocki
Hi Mauro, This change set includes mostly fixes to the exynos4-is and patches to add support for some IPs of the Exynos5 SoC series at this driver. If it looks OK and it is too late for 3.11 already, please just queue for 3.12. The following changes since commit 188af63c0af2d7ef395bc94e3efa173

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

2013-06-25 Thread Tomasz Figa
Hi Sylwester, Felipe, On Tuesday 25 of June 2013 19:44:52 Sylwester Nawrocki wrote: > Hi Felipe, > > Thanks for the review. > > On 06/25/2013 05:06 PM, Felipe Balbi wrote: > > On Tue, Jun 25, 2013 at 04:21:46PM +0200, Sylwester Nawrocki wrote: > >> +enum phy_id { > >> + PHY_CSIS0, > >> + PHY_D

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

2013-06-25 Thread Sylwester Nawrocki
Hi Felipe, Thanks for the review. On 06/25/2013 05:06 PM, Felipe Balbi wrote: > On Tue, Jun 25, 2013 at 04:21:46PM +0200, Sylwester Nawrocki wrote: >> +enum phy_id { >> +PHY_CSIS0, >> +PHY_DSIM0, >> +PHY_CSIS1, >> +PHY_DSIM1, >> +NUM_PHYS > > please prepend these with EXYNOS_

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

2013-06-25 Thread Sylwester Nawrocki
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 set by v4l2_i2c_subdev_init with a custom name. > > This is wrong if it is possible that there are multiple identic

[PATCH 0/2] media: davinci: vpif: capture/display support for async subdevice probing

2013-06-25 Thread Prabhakar Lad
From: "Lad, Prabhakar" This patch series adds support for vpif capture and display driver to support asynchronously register subdevices. Need for this support: Currently bridge device drivers register devices for all subdevices synchronously, typically, during their probing. E.g. if an I2C CMOS

[PATCH 2/2] media: davinci: vpif: display: add V4L2-async support

2013-06-25 Thread Prabhakar Lad
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: Lad, Prabhakar Cc: Guennadi Liakhovetski Cc: Hans Verkuil Cc: Laurent Pin

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

2013-06-25 Thread Prabhakar Lad
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 Lad Cc: Guennadi Liakhovetski Cc: Hans Verkuil Cc: Laurent Pinc

Re: [PATCH v2 5/5] ARM: Samsung: Remove MIPI PHY setup code

2013-06-25 Thread Felipe Balbi
On Tue, Jun 25, 2013 at 04:21:50PM +0200, Sylwester Nawrocki wrote: > 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 upstrea

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

2013-06-25 Thread Felipe Balbi
On Tue, Jun 25, 2013 at 04:21:49PM +0200, Sylwester Nawrocki wrote: > 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 Nawro

Re: [PATCH v2 3/5] video: exynos_mipi_dsim: Use generic PHY driver

2013-06-25 Thread Felipe Balbi
Hi, On Tue, Jun 25, 2013 at 04:21:48PM +0200, Sylwester Nawrocki wrote: > 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

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

2013-06-25 Thread Felipe Balbi
Hi, On Tue, Jun 25, 2013 at 04:21:46PM +0200, Sylwester Nawrocki wrote: > +enum phy_id { > + PHY_CSIS0, > + PHY_DSIM0, > + PHY_CSIS1, > + PHY_DSIM1, > + NUM_PHYS please prepend these with EXYNOS_PHY_ or EXYNOS_MIPI_PHY_ > +struct exynos_video_phy { > + spinlock_t slock; >

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

2013-06-25 Thread 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 should be done using dma-buf

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

2013-06-25 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 --- drivers/media/platform/exynos4-is/mipi

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

2013-06-25 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 v2 2/5] ARM: dts: Add MIPI PHY node to exynos4.dtsi

2013-06-25 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 v2 3/5] video: exynos_mipi_dsim: Use generic PHY driver

2013-06-25 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 --- drivers/video/exynos/exynos_mipi_dsi.

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

2013-06-25 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 v1: - enabled build as module and with CONFIG_OF disabled - added phy_id enum, - of_address_to_resource

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

2013-06-25 Thread Inki Dae
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 should be done using >>> dma-buf. dma-buf is for sharing content btw different devices not

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

2013-06-25 Thread Sylwester Nawrocki
This patch series adds a simple driver for the Samsung S5P/Exynos SoC series MIPI CSI-2 receiver (MIPI DSIM) 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 v3 7/8] [media] V4L: Add VP8 encoder controls

2013-06-25 Thread Kamil Debski
Hi Arun, This patch sets looks very good. Please check my comments below. > From: Arun Kumar K [mailto:arun...@samsung.com] > > This patch adds new V4L controls for VP8 encoding. > > Signed-off-by: Kiran AVND > Signed-off-by: Arun Kumar K > --- > Documentation/DocBook/media/v4l/controls.xml

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

2013-06-25 Thread Kamil Debski
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 commands. So the hardware specific opr_v6 and > cmd_v6 are re-used for v7 also.

UBS Benachrichtigung - Ihre Internet-Banking gesperrt‏‏‏‏

2013-06-25 Thread UBS AG
Sehr geehrter Kunde, Kürzlich zeigen unsere Aufzeichnungen, dass Ihr UBS-Konto möglich durch einen Dritten unbefugten Zutritt. Die Sicherheit Ihres Kontos ist unser wichtigstes Anliegen, deshalb haben wir beschlossen, den Zugang zu Ihrem Konto vorübergehend zu begrenzen. Für den vollen Zugang

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

2013-06-25 Thread 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 should be done using >> dma-buf. dma-buf is for sharing content btw different devices not >> sharing resources. >> >

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

2013-06-25 Thread Sachin Kamat
Hi Arun, > @@ -684,5 +685,6 @@ void set_work_bit_irqsave(struct s5p_mfc_ctx *ctx); > (dev->variant->port_num ? 1 : 0) : 0) : 0) > #define IS_TWOPORT(dev)(dev->variant->port_num == 2 ? 1 : 0) > #define IS_MFCV6_PLUS(dev) (dev->variant->version >

[PATCH v2 3/6] exynos4-is: Fix format propagation on FIMC-IS-ISP subdev

2013-06-25 Thread Sylwester Nawrocki
Ensure TRY formats are propagated from the sink pad to the source pads of the FIMC-IS-ISP subdev and the TRY and ACTIVE formats are separated. Signed-off-by: Sylwester Nawrocki Signed-off-by: Kyungmin Park --- drivers/media/platform/exynos4-is/fimc-isp.c | 94 ++ drive

[PATCH v2 6/6] exynos4-is: Correct colorspace handling at FIMC-LITE

2013-06-25 Thread Sylwester Nawrocki
Ensure the colorspace is properly adjusted by the driver for YUV and Bayer image formats. The subdev try_fmt helper is simplified. Signed-off-by: Sylwester Nawrocki Signed-off-by: Kyungmin Park --- drivers/media/platform/exynos4-is/fimc-lite.c | 17 + include/media/s5p_fimc.h

[PATCH v2 4/6] exynos4-is: Set valid initial format on FIMC-IS-ISP subdev pads

2013-06-25 Thread Sylwester Nawrocki
Ensure there is a valid initial resolution and pixel format set at the FIMC-IS-ISP subdev pads. Signed-off-by: Sylwester Nawrocki Signed-off-by: Kyungmin Park --- drivers/media/platform/exynos4-is/fimc-isp.c | 18 ++ 1 file changed, 18 insertions(+) diff --git a/drivers/media

[PATCH v2 5/6] exynos4-is: Set valid initial format on FIMC.n subdevs

2013-06-25 Thread Sylwester Nawrocki
Ensure there are valid initial image formats on the FIMC.n subdev pads. Signed-off-by: Sylwester Nawrocki Signed-off-by: Kyungmin Park --- drivers/media/platform/exynos4-is/fimc-capture.c | 19 +-- drivers/media/platform/exynos4-is/fimc-core.h|2 ++ 2 files changed, 19

[PATCH v2 1/6] exynos4-is: Fix format propagation on FIMC-LITE.n subdevs

2013-06-25 Thread Sylwester Nawrocki
FIMC-LITE subdevs have one sink pad and two source pads on which the image formats are always same. This patch implements missing format propagation from the sink pad to the source pads, to allow user space to negotiate TRY format on whole media pipeline involving FIMC-LITE.n subdevs. The subdev tr

[PATCH v2 2/6] exynos4-is: Set valid initial format at FIMC-LITE

2013-06-25 Thread Sylwester Nawrocki
Ensure the image resolution and crop rectangle on the FIMC-LITE.n subdevs and fimc-lite.n.capture video nodes is properly configured upon the driver's initialization. Signed-off-by: Sylwester Nawrocki Signed-off-by: Kyungmin Park --- drivers/media/platform/exynos4-is/fimc-lite.c | 23

[PATCH v2 0/6] exynos4-is: TRY format propagation fixes

2013-06-25 Thread Sylwester Nawrocki
This series includes fixes for issues found while working on a libv4l plugin for the exynos4-is driver, i.e. TRY format propagation, colorspace handling and initial format setting on some exynos4-is subdevs. Sylwester Nawrocki (6): exynos4-is: Fix format propagation on FIMC-LITE.n subdevs exyn

[PATCH v3 2/8] [media] s5p-mfc: Rename IS_MFCV6 macro

2013-06-25 Thread Arun Kumar K
The MFC v6 specific code holds good for MFC v7 also as the v7 version is a superset of v6 and the HW interface remains more or less similar. This patch renames the macro IS_MFCV6() to IS_MFCV6_PLUS() so that it can be used for v7 also. Signed-off-by: Arun Kumar K --- drivers/media/platform/s5p-m

[PATCH v3 0/8] Add support for MFC v7 firmware

2013-06-25 Thread Arun Kumar K
This patch series adds MFC v7 firmware support to the Exynos MFC driver. MFC v7 is present in 5420 SoC which has support for VP8 encoding and many other features. Changes from v2: - Addressed review comments from Sylwester http://www.mail-archive.com/linux-media@vger.kernel.org/msg63676.html http:

[PATCH v3 6/8] [media] V4L: Add support for integer menu controls with standard menu items

2013-06-25 Thread Arun Kumar K
From: Sylwester Nawrocki The patch modifies the helper function v4l2_ctrl_new_std_menu to accept integer menu controls with standard menu items. Signed-off-by: Sylwester Nawrocki Signed-off-by: Arun Kumar K --- Documentation/video4linux/v4l2-controls.txt | 21 ++-- drivers/m

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

2013-06-25 Thread Arun Kumar K
Adds variant data and core support for the MFC v7 firmware Signed-off-by: Arun Kumar K --- .../devicetree/bindings/media/s5p-mfc.txt |1 + drivers/media/platform/s5p-mfc/s5p_mfc.c | 32 drivers/media/platform/s5p-mfc/s5p_mfc_common.h|2 ++ 3

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

2013-06-25 Thread Arun Kumar K
Firmware version v7 is mostly similar to v6 in terms of hardware specific controls and commands. So the hardware specific opr_v6 and cmd_v6 are re-used for v7 also. This patch updates the v6 files to handle v7 version also. Signed-off-by: Arun Kumar K --- drivers/media/platform/s5p-mfc/s5p_mfc_e

[PATCH v3 8/8] [media] s5p-mfc: Add support for VP8 encoder

2013-06-25 Thread Arun Kumar K
MFC v7 supports VP8 encoding and this patch adds support for it in the driver. Signed-off-by: Arun Kumar K --- drivers/media/platform/s5p-mfc/s5p_mfc_cmd_v6.c |3 + drivers/media/platform/s5p-mfc/s5p_mfc_common.h | 19 - drivers/media/platform/s5p-mfc/s5p_mfc_enc.c| 90 ++

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

2013-06-25 Thread Arun Kumar K
This patch adds new V4L controls for VP8 encoding. Signed-off-by: Kiran AVND Signed-off-by: Arun Kumar K --- Documentation/DocBook/media/v4l/controls.xml | 150 ++ drivers/media/v4l2-core/v4l2-ctrls.c | 33 ++ include/uapi/linux/v4l2-controls.h |

[PATCH v3 3/8] [media] s5p-mfc: Add register definition file for MFC v7

2013-06-25 Thread Arun Kumar K
The patch adds the register definition file for new firmware version v7 for MFC. New firmware supports VP8 encoding along with many other features. Signed-off-by: Arun Kumar K --- drivers/media/platform/s5p-mfc/regs-mfc-v7.h | 58 ++ 1 file changed, 58 insertions(+) cr

[PATCH v3 1/8] [media] s5p-mfc: Update v6 encoder buffer sizes

2013-06-25 Thread Arun Kumar K
The patch updates few encoder buffer sizes for MFC v6.5 as per the udpdated user manual. The same buffer sizes holds good for v7 firmware also. Signed-off-by: Arun Kumar K Signed-off-by: Kiran AVND --- drivers/media/platform/s5p-mfc/regs-mfc-v6.h |4 ++-- 1 file changed, 2 insertions(+), 2

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

2013-06-25 Thread Daniel Vetter
On Tue, Jun 18, 2013 at 12:46 PM, Russell King - ARM Linux wrote: >> > Note: the existing stuff does have the nice side effect of being able >> > to pass buffers which do not have a struct page * associated with them >> > through the dma_buf API - I think we can still preserve that by having >> >

Re: [PATCH v2] V4L2: add documentation for V4L2 clock helpers and asynchronous probing

2013-06-25 Thread Guennadi Liakhovetski
Hi Sylwester On Mon, 24 Jun 2013, Sylwester Nawrocki wrote: > Hi Guennadi, > > On 06/24/2013 01:20 PM, Guennadi Liakhovetski wrote: > > Add documentation for the V4L2 clock and V4L2 asynchronous probing APIs > > to v4l2-framework.txt. > > > > Signed-off-by: Guennadi Liakhovetski > > --- > > >

Re: [PATCH RFC v3] media: OF: add video sync endpoint property

2013-06-25 Thread Prabhakar Lad
Hi Hans, On Mon, Jun 24, 2013 at 1:21 PM, Hans Verkuil wrote: > Hi Prabhakar, > > On Sat June 22 2013 17:03:03 Prabhakar Lad wrote: >> From: "Lad, Prabhakar" >> [snip] >> +#ifndef _DT_BINDINGS_VIDEO_INTERFACES_H >> +#define _DT_BINDINGS_VIDEO_INTERFACES_H >> + >> +#define V4L2_MBUS_VIDEO_SEPARAT

[PATCH] V4L2: soc-camera: remove several CEU references in the generic scaler

2013-06-25 Thread Guennadi Liakhovetski
The scaling / cropping library, that has been extracted from the CEU driver still contained a couple of references to the original hardware. Clean them up. Signed-off-by: Guennadi Liakhovetski --- A cosmetic fix, goes on top of https://patchwork.linuxtv.org/patch/18209/ https://patchwork.linuxt

Re: Question: interaction between selection API, ENUM_FRAMESIZES and S_FMT?

2013-06-25 Thread Hans Verkuil
On Tue 25 June 2013 10:21:19 Sakari Ailus wrote: > Hi Hans, > > On Mon, Jun 24, 2013 at 02:48:15PM +0200, Hans Verkuil wrote: > > Hi all, > > > > While working on extending v4l2-compliance with cropping/selection test > > cases > > I decided to add support for that to vivi as well (this would gi

Re: Question: interaction between selection API, ENUM_FRAMESIZES and S_FMT?

2013-06-25 Thread Sakari Ailus
Hi Hans, On Mon, Jun 24, 2013 at 02:48:15PM +0200, Hans Verkuil wrote: > Hi all, > > While working on extending v4l2-compliance with cropping/selection test cases > I decided to add support for that to vivi as well (this would give > applications > a good test driver to work with). > > However,

RE: [PATCH 2/7] marvell-ccic: add clock tree support for marvell-ccic driver

2013-06-25 Thread Libin Yang
Hi Jonathan, Do you mean using IS_ERR() here directly? I think it should be OK. I will change to IS_ERR() in next version. Regards, Libin >-Original Message- >From: Jonathan Corbet [mailto:cor...@lwn.net] >Sent: Saturday, June 22, 2013 1:02 AM >To: Libin Yang >Cc: g.liakhovet...@gmx.

RE: [PATCH 1/7] marvell-ccic: add MIPI support for marvell-ccic driver

2013-06-25 Thread Libin Yang
Hi Jonathan, Sorry for delay reply. Please see the below comments. Regards, Libin >-Original Message- >From: Jonathan Corbet [mailto:cor...@lwn.net] >Sent: Saturday, June 22, 2013 1:00 AM >To: Libin Yang >Cc: g.liakhovet...@gmx.de; mche...@redhat.com; >linux-media@vger.kernel.org; al