Hi Richard,
thank you for your review.

This patch has been reviewed and acked by several people:

    Reviewed-by: Philipp Zabel<p.za...@pengutronix.de>
    Reviewed-by: Sylwester Nawrocki <s.nawro...@samsung.com>
    Acked-by: Hans Verkuil <hans.verk...@cisco.com>

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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to