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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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.
> 
> This patch has been reviewed and acked by several people:
> 
>     Reviewed-by: Philipp Zabel<p.zabel@xxxxxxxxxxxxxx>
>     Reviewed-by: Sylwester Nawrocki <s.nawrocki@xxxxxxxxxxx>
>     Acked-by: Hans Verkuil <hans.verkuil@xxxxxxxxx>
> 
> For this reason a pull request has already been sent to Mauro recently:
> 
> http://patchwork.linuxtv.org/patch/13483/
> 
> 
> >> In this case I personally don't think it will be easier to read if this line is split up.
> > My point here is checkpatch.
> > total: 2 errors, 30 warnings, 2086 lines checked
> 
> Thank you for noticing this. I have solved it in my tree so that Mauro
> can pull for 3.7.
> 
> You can find it here:
> 
> https://github.com/jmartinc/video_visstrim.git  for_3.6
> 
> Regarding your i.MX6 question, maybe Philippe will be able to help you
> since I am only interested on i.MX27. However, the driver was
> developed considering much of his suggestions so that adding support
> for different chips later is as straightforward as possible.
> 
> -- 
> Javier Martin
> Vista Silicon S.L.
> CDTUC - FASE C - Oficina S-345
> Avda de los Castros s/n
> 39005- Santander. Cantabria. Spain
> +34 942 25 32 60
> www.vista-silicon.com
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux