[07:42] <jmondi> sailus: pinchartl: thansk for review
[07:43] <jmondi> if we create an helper to register controls, the argument that setting up an integer menu control for drivers is a bit more work is now moot
[07:43] <jmondi> so I would actually go for a (menu control + helper)
[07:45] <jmondi> sailus: pinchartl: in the DT bindings, one seems to want to remove "camera" from "camera device", the other wants "camera sensor" in place of "device".. should we make the property camera specific, or stay generic, as it could apply to lenses and flash devices too ?
[07:50] <sailus> X-)
[07:50] <sailus> Let's wait for pinchartl to wake up.
[07:53] <jmondi> :)
[08:00] <sailus> It is certainly possible to start with cameras and make the property more generic later on.
[13:02] <hverkuil> mchehab: syoung: b-rad: (or other dvb specialists) please note this patch: https://patchwork.linuxtv.org/patch/58184/
[13:03] <hverkuil> I think that driver has been broken for over a year. Does anyone have that HW? Have there been bug reports about this device?
[13:05] <mchehab> I *might* have one device with gl861, but crope has for sure
[13:05] <mchehab> better to ping him at #linuxtv
[13:05] * mchehab has not seeing this for years
[13:14] <mchehab> hverkuil: yeah, that sounds broken...
[13:14] <mchehab> it seems that this new memcpy() is used only on some ISDB cards
[13:14] <mchehab> as this was added by Akihiro
[13:15] <mchehab> Friio White ISDB-T receiver
[13:15] <mchehab> I don't have such devices here
[13:15] <mchehab> it won't hurt pinging Akihiro for his ack, but for me the fix is brain-dead
[13:16] <mchehab> s/brain-dead/nobrain/
[13:36] <hverkuil> ezequielg: v6 looks good. If there are no more comments, then I'll make a PR for it tomorrow.
[13:46] <jmondi> sailus: Hi Sakari, you there?
[13:46] <ezequielg> hverkuil: sounds good :-)
[13:47] <ezequielg> hverkuil: btw, the vivid race condition sounds familiar. we talked about drivers taking locks before.
[13:47] <ezequielg> I wish we could clean that and avoid any drivers from doing that.
[14:39] <b-rad> I think I lack anything using that driver mchehab
[15:13] <sailus> jmondi: Yes.
[15:18] <jmondi> sailus: are you happy with Hans and Laurent conclusions?
[15:19] <jmondi> using an integer control, and define the location as the "camera location" to be expanded to "flash location" eventually ?
[15:21] <jmondi> I've just seen you have replied
[15:30] <mfelsch> pinchartl: Hi Laurent as you said, this way is faster for the connector discussion ^^
[15:31] <sailus> I'm fine with starting with starting with "camera location", with an integer control.
[15:31] <jmondi> sailus: thanks
[20:31] <b-rad> ping hverkuil, i just mailed out a patch related to your recent device_caps mismatch warning. I think it should sort out cx231xx