<!-- Some styling for better description lists --><style type='text/css'>dt { font-weight: bold;float: left;display:inline;margin-right: 1em} dd { display:block; margin-left: 2em}</style> mchehab: (today I'm a bit short of time... I have just ~20 mins for discussions) hverkuil: hi! mchehab: btw, I won't be able to join our meeting next week... will be on a biztrip without Internet access <br> <u>hverkuil</u>: I still have some patches from one of your PRs on my queue (the ones after that /4 patch series we've discussed yesterday) <br> I intend to handle it later today <br> hverkuil, pinchartl, sailus, syoung: anything for today's discussion? <br> <u>syoung</u>: I also intend to handle your DVB patch series today hverkuil: ok. I think it's best to just drop the /4 series and apply everything else from that PR. <br> I'll likely make a new PR tomorrow for the staging rkisp1 driver. <br> v13 seems to be the lucky version :-) mchehab: :-) <br> as I won't have internet access next week, I hope not many PRs will arrive... <br> as otherwise they may be applied only for 5.7 hverkuil: I get try to push that PR out today, if you prefer? mchehab: (I'll likely apply pending PRs on Jan, 20 - but if there are too many things, some stuff may end being postponed) <br> tomorrow should be fine, if you can do it during the morning hverkuil: are you back in Brazil, or still in Germany? mchehab: still in DE <br> anything else for today's meeting? hverkuil: Not from me. mchehab: ok. if someone pops up, I'll read the backlog and reply after lunch.] pinchartl: <u>hverkuil</u>: regarding the rkisp1 driver <br> what's your opinion on how they handle ISP parameters ? <br> they're put in a buffer queued to a parameters queue <br> but it's then entirely consumed by the driver to write registers, the buffer isn't passed to the hardware hverkuil: It's a valid choice, but not necessarily the best choice. I'd have chosen compound controls for this. <br> The only advantage of using a buffer is that can do zero-copy, whereas a control always involves a memcpy. pinchartl: there are performance concerns, and the fact that we have no format support in the request API hverkuil: Performance concerns with what? Using buffers or using controls? mchehab: I suspect that the uapi for the driver need improvements... as it is now at staging, I'm blindly accepting it <br> but I want to understand and see it properly documented before moving out of staging <br> anyway, the pending PRs were applied pinchartl: <u>hverkuil</u>: with using controls <br> <u>mchehab</u>: agreed, for staging it's fine