↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |
Who | What | When |
---|---|---|
*** | ChanServ sets mode: +v mchehab` | [06:27] |
...... (idle for 29mn) | ||
hverkuil1 has left | [06:56] | |
................... (idle for 1h33mn) | ||
hverkuil | mchehab: 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) | ||
mchehab | hverkuil: whom do you believe should be handling it? | [12:01] |
hverkuil | pinchartl
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] |
mchehab | XILINX 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] |
pinchartl | mchehab: 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] |
mchehab | from 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] |
hverkuil | I 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] |
mchehab | hverkuil: 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] |
hverkuil | sailus: I agree with the comments you made for the v13 request API. I'll put them in v14. | [12:20] |
sailus | hverkuil: Thanks. I'll review the rest of the set as well. | [12:32] |
pinchartl | mchehab: I agree, I'll get the patches merged, no worries | [12:39] |
.......... (idle for 46mn) | ||
mchehab | pinchartl: 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) |