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

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

 



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