#media-maint 2018-05-07,Mon

↑back Search ←Prev date Next date→ Show only urls(Click on time to select a line by its url)

WhoWhatWhen
***ChanServ sets mode: +v mchehab` [06:27]
...... (idle for 29mn)
hverkuil1 has left [06:56]
................... (idle for 1h33mn)
hverkuilmchehab: pinchartl: Why is this xilinx patch series delegated to me? https://patchwork.linuxtv.org/patch/49174/
Is this a mistake?
[08:29]
.................. (idle for 1h29mn)
I undelegated this xilinx patch series. [09:58]
......................... (idle for 2h3mn)
mchehabhverkuil: whom do you believe should be handling it? [12:01]
hverkuilpinchartl
his driver.
or perhaps Kieran, I don't know if he also supports that driver.
(well, he's not in the delegates list at the moment)
[12:02]
mchehabXILINX VIDEO IP CORES
M: Hyun Kwon <hyun.kwon@xilinx.com>
M: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
L: linux-media@vger.kernel.org
T: git git://linuxtv.org/media_tree.git
S: Supported
F: Documentation/devicetree/bindings/media/xilinx/
F: drivers/media/platform/xilinx/
F: include/uapi/linux/xilinx-v4l2-controls.h
yeah, delegating to pinchartl makes sense to me
[12:05]
pinchartlmchehab: you can delegate it to me, although at the moment Xilinx should maintain the driver
this should change again soon hopefully, and I'll resume work on Xilinx drivers
[12:07]
mchehabfrom what I'm seeing, at the ML, Xilinx is indeed reviewing the patchset... Hyun sent several comments against version 4 and Satish sent a version 5 after his comments...
so, it is not a matter of supporting the dirver...
just a matter about who will apply it once the patchset is in good shape
in other words, it is part of our volunteering work as maintainers
[12:12]
hverkuilI don't think the patch series looks right. It looks like they think formats with contiguous planes in memory need the _MPLANE API, when that's not needed at all
Our fault for not calling it _MBUFFER API or something like that :-(
I'll comment on it once I've reviewed the fence patch series.
[12:17]
mchehabhverkuil: thanks!
btw, my plan for today is to review the properties API patchset - have to finish some internal duties first
s/properties/request/
so many APIs :-)
[12:18]
hverkuilsailus: I agree with the comments you made for the v13 request API. I'll put them in v14. [12:20]
sailushverkuil: Thanks. I'll review the rest of the set as well. [12:32]
pinchartlmchehab: I agree, I'll get the patches merged, no worries [12:39]
.......... (idle for 46mn)
mchehabpinchartl: thanks! [13:25]
......................................................................................... (idle for 7h23mn)
***ChanServ sets mode: +v pinchart1 [20:48]
pinchartl has quit IRC (*.net *.split) [20:53]

↑back Search ←Prev date Next date→ Show only urls(Click on time to select a line by its url)