Thread (51 messages) 51 messages, 10 authors, 2017-12-28

Re: [PATCH v2 1/3] media: V3s: Add support for Allwinner CSI.

From: Maxime Ripard <hidden>
Date: 2017-12-15 22:21:14
Also in: linux-arm-kernel, linux-media, lkml

Hi,

On Fri, Dec 15, 2017 at 07:01:40PM +0800, Yong wrote:
Hi Maxime,

On Fri, 15 Dec 2017 11:50:47 +0100
Maxime Ripard [off-list ref] wrote:
quoted
Hi Yong,

On Mon, Dec 04, 2017 at 05:45:11PM +0800, Yong wrote:
quoted
I just noticed that you are using the second iteration?
Have you received my third iteration?
Sorry for the late reply, and for not coming back to you yet about
that test. No, this is still in your v2. I've definitely received your
v3, I just didn't have time to update to it yet.

But don't worry, my mail was mostly to know if you had tested that
setup on your side to try to nail down the issue on my end, not really
a review or comment that would prevent your patch from going in.
I mean,
The v2 exactly has a bug which may cause the CSI writing frame to 
a wrong memory address.
Ah, sorry I misunderstood you then. I'll definitely test your v3..
BTW, should I send a new version. I have made some improve sine v3.
.. or your v4 :)

Yes, please send a new version.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help