[07:42] *** paulk-leonov has quit IRC (Ping timeout: 272 seconds)
[12:53] *** mfelsch is now known as mfe[home]
[13:45] <montjoie> hello, I still work on the zoran driver, and the buffer seems touched, but still displayed as green
[13:45] <montjoie> even if I poison it after dma operation (just before vb2_buffer_done)
[13:49] <montjoie> could someone said if use vb2 right in https://github.com/montjoie/linux/blob/zoran-5.3.16/drivers/staging/media/zoran/zoran_driver.c ?
[14:10] <dagmcr> hans: mauro: and don't know if I should mention someone else. 2 weeks ago I sent an RFC patch to change the framerate at the v4l2-subdev level but I didn't get any response. The idea is to add this support for the driver I want to upstream. Besides that, I also add a simple v4l2 cid for the temperature of the sensor. Did you guys check it?
[14:10] <dagmcr> Thanks!
[14:10] <dagmcr> here the link: https://patchwork.linuxtv.org/patch/63047/
[14:12] <hverkuil> sailus: ^^^
[14:12] <dagmcr> sorry, correct nicknames: hverkuil and mchehab, could you check my above comment?
[14:13] <hverkuil> dagmcr: it's something that sailus should look at. He's the sensor expert :-)
[14:15] <hverkuil> BTW, what do you mean with the temperature control? Is that the sensor's temperature in celcius? Or does it refer to color temperature in kelvin? It certainly needs to be documented.
[14:16] <dagmcr> hverkuil: thanks, I will wait for him to have a look. I guess it will be needed to adapt the v4l-utils (v4l2-compliance and v4l-ctl) but that would if you guys agree :P
[14:17] <dagmcr> hverkuil: in regards of the temperature is the celsius temperature of the sensor. I think there is not control for that one.
[14:19] <hverkuil> dagmcr: regarding patch 1/3: I'd leave that out for the initial implementation of this sensor driver. It's more complex than you think. See e.g. here for a previous attempt: https://patchwork.linuxtv.org/patch/51974/
[14:20] <hverkuil> The temperature control shouldn't be an issue, it just needs to add documentation to the v4l2 spec.
[14:22] <dagmcr> okay, I didn't know about the above patch. What I did was exactly the same you are doing with the frame size so, I thought it was going to be an easy one :P
[14:22] <dagmcr> okay about the temperature, I will add it then.
[14:37] <montjoie> missing vb2_set_plane_payload()... now it works\o/
[16:02] *** benjiG has left 
[19:31] <sailus> dagmcr: Could you cc me going forward?
[19:32] <sailus> hverkuil: Thanks for pinging.
[20:46] <ezequielg> montjoie: woohoo! progress!
[22:23] *** sailus has quit IRC (Quit: leaving)