Re: [GIT PULL v2] Initial i.MX5/CODA7 support for the CODA driver

2012-09-20 Thread Richard Zhao
On Thu, Sep 20, 2012 at 09:10:46AM +0200, javier Martin wrote: > Hi Richard, > > On 20 September 2012 05:32, Richard Zhao wrote: > > why is it a request-pull? > > After 5 version of Philipp's patches we have agreed they are good > enough to be merged; they don&#x

Re: [GIT PULL v2] Initial i.MX5/CODA7 support for the CODA driver

2012-09-19 Thread Richard Zhao
why is it a request-pull? Thanks Richard -- 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

Re: [PATCH v3 03/16] media: coda: fix IRAM/AXI handling for i.MX53

2012-09-10 Thread Richard Zhao
On Fri, Aug 31, 2012 at 10:10:57AM +0200, Philipp Zabel wrote: > This uses the ARCH_MXC specific iram_alloc API to allocate a work > buffer in the SoC's on-chip SRAM and sets up the AXI_SRAM_USE > register. In the future, the allocation will be converted to use > the genalloc API. > > Signed-off-b

Re: [PATCH v2 0/14] Initial i.MX5/CODA7 support for the CODA driver

2012-08-28 Thread Richard Zhao
On Tue, Aug 28, 2012 at 12:53:47PM +0200, Philipp Zabel wrote: > These patches contain initial firmware loading and encoding support for the > CODA7 series VPU contained in i.MX51 and i.MX53 SoCs, and fix some > multi-instance > issues. The last two patches touching files in arch/arm/* are include

[PATCH] media: coda: remove duplicated call of fh_to_ctx in vidioc_s_fmt_vid_out

2012-08-20 Thread Richard Zhao
Signed-off-by: Richard Zhao --- drivers/media/platform/coda.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/platform/coda.c b/drivers/media/platform/coda.c index 6908514..69ff0d3 100644 --- a/drivers/media/platform/coda.c +++ b/drivers/media/platform/coda.c

Re: [v7] media: coda: Add driver for Coda video codec.

2012-08-19 Thread Richard Zhao
Hi Javier, Did the patch get picked? I didn't see it on git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media.git Still, how did you test this v4l2 device? Thanks Richard On Fri, Aug 03, 2012 at 01:21:02PM +0200, javier Martin wrote: > Hi Richard, > thank you for your review. > >

Re: __video_register_device: warning cannot be reached if warn_if_nr_in_use

2012-08-09 Thread Richard Zhao
Hans Verkuil wrote: >On Thu August 9 2012 14:55:02 Richard Zhao wrote: >> In file drivers/media/video/v4l2-dev.c >> >> int __video_register_device(struct video_device *vdev, int type, int >nr, >> int warn_if_nr_in_use, struct module *owner) >>

__video_register_device: warning cannot be reached if warn_if_nr_in_use

2012-08-09 Thread Richard Zhao
In file drivers/media/video/v4l2-dev.c int __video_register_device(struct video_device *vdev, int type, int nr, int warn_if_nr_in_use, struct module *owner) { [...] vdev->minor = i + minor_offset; 878:vdev->num = nr; vdev->num is set to nr here. [...] if (nr !

Re: [v7] media: coda: Add driver for Coda video codec.

2012-08-03 Thread Richard Zhao
On Fri, Aug 03, 2012 at 10:47:01AM +0200, Hans Verkuil wrote: > On Fri August 3 2012 10:24:43 Richard Zhao wrote: > > Hi Javier, > > > > Glad to see the vpu patch. I'd like to try it on imx6. What else > > do I need to do besides add vpu devices in dts? Do you ha

Re: [v7] media: coda: Add driver for Coda video codec.

2012-08-03 Thread Richard Zhao
Hi Javier, Glad to see the vpu patch. I'd like to try it on imx6. What else do I need to do besides add vpu devices in dts? Do you have a gst plugin or any other test program to test it? Please also see below comments. On Mon, Jul 23, 2012 at 11:31:01AM +, Javier Martin wrote: > Coda is a ra