| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |
| Who | What | When |
|---|---|---|
| hverkuil | tfiga: gnurou: posted RFCv2 of the request API. Take a look, I think I incorporated all comments and clarified a number of corner cases. | [08:47] |
| tfiga | hverkuil: thanks! at the first sight it looks good to me
will give it one more thought in my head and post Acked-by or something if I don't find any problems | [08:53] |
| ...... (idle for 29mn) | ||
| gnurou | I'm probably not qualified to have an opinion on that, but it looks fine | [09:22] |
| hverkuil | Just let me know if there are any questions or if I missed something. | [09:26] |
| ............................................ (idle for 3h39mn) | ||
| mchehab: I see that there is still a 4.16 fix pending: https://patchwork.linuxtv.org/patch/47491/
It's in the fixes branch, so I assume you'll make a pull request for Linus soon? | [13:05] | |
| mchehab | thanks for remininding me... I'll be sending the fixes there right now | [13:11] |
| gah, I ended by pushing the warning fixes too :-( | [13:16] | |
| syoung | mchehab: the warning fixes looked fine to me :) | [13:24] |
| mchehab | thanks for reviewing it | [13:25] |
| pull request sent
if someone finds an issue on the warning fixes, I can revert or apply a fixup I really hate when things like that happens :-( | [13:31] | |
| hverkuil | Just a missing space for the v4l2-tpg fix. | [13:33] |
| mchehab | two bugs on one of my script | [13:33] |
| hverkuil: thanks for noticing. If it is just a white space issue, please add a fixup patch on your tree to be applied after the merge window. no rush on that ;-) | [13:38] | |
| ............................. (idle for 2h21mn) | ||
| *** | prabhakarlad has left | [15:59] |
| benjiG has left | [16:09] | |
| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |