| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |
| Who | What | When |
|---|---|---|
| *** | lfam has left "Leaving" | [02:47] |
| .......................................................................................... (idle for 7h25mn) | ||
| pinchartl | javier__: can I assume that the tvp5150 patches that come from me have not been modified, or should I review them too ? | [10:12] |
| javier__ | pinchartl: yes, they have not been modified besides simple changes to fix build issues due API changes | [10:21] |
| pinchartl | javier__: I'll trust you on that :-)
thank you | [10:21] |
| javier__ | pinchartl: thanks to you | [10:22] |
| .......................... (idle for 2h7mn) | ||
| *** | zahari has quit IRC (Quit: WeeChat 1.3) | [12:29] |
| ..................................... (idle for 3h0mn) | ||
| ndufresne | pinchartl: I started working on fixing bad stride with planar formats in uvc driver, I'd be interested to know if that approach is acceptable or not, http://paste.fedoraproject.org/307811/94120145/ | [15:29] |
| ......................... (idle for 2h0mn) | ||
| *** | benjiG has left | [17:29] |
| ................ (idle for 1h16mn) | ||
| tharvey has quit IRC (Ping timeout: 250 seconds) | [18:45] | |
| ...... (idle for 29mn) | ||
| awalls has left | [19:14] | |
| ...................... (idle for 1h45mn) | ||
| pinchartl | ndufresne: yes that looks good to me | [20:59] |
| ndufresne | ok, will give it proper testing and then submit
pinchartl: about the probing performance, in the end the vendor provided us a new firmware, it take 200ms now to start, so not a priority anymore ;-P | [21:01] |
| pinchartl | I'd make the two parameters const
:-) that's good because I don't remember the details of the issue ;-) | [21:03] |
| ndufresne | ok, taking note
pinchartl: haha (for the record, when doing TRY_FMT with different parameter, we endup asking the same thing to the HW multiple time, because UVC parameteres are a subset of v4l2 parameters, solution could be more enumeration types in v4l2, or caching ... | [21:03] |
| .......... (idle for 46mn) | ||
| javier__ | pinchartl: I'm waiting for your answer to my latest comments about the delay / sleep API usage before posting a v2 | [21:50] |
| pinchartl | javier__: no strong opinion, I'll let you decide | [21:53] |
| shuah | pinchartl | [21:54] |
| pinchartl | shuah: hi | [21:55] |
| javier__ | pinchartl: Ok, I'll post the patches tomorrow then. Thanks a lot for your feedback! | [21:55] |
| shuah | We discussed this (I think before) - Managed media API need to hold struct device reference I think | [21:55] |
| pinchartl | shuah: yes, I wanted to discuss that with you. the current implementation is broken I believe
as media_device will be freed when the last device is unbound which will race with userspace | [21:55] |
| shuah | I vaguely recall talking about it in Finland and I ran into it while I was testing the patch series I sent out | [21:56] |
| pinchartl | I think we'll have to do without devm_* and manage the reference count explicitly | [21:56] |
| shuah | We have to solve two issues I would think - one when media device is a non-devm resource
We need to be devm resource for alsa and media drivers to share it | [21:57] |
| pinchartl | why ?
can't we have a central media_device allocator/registry without devm ? | [21:57] |
| shuah | because ALSA and au0828 don't share any data structures
take a look at the patch series I just sent out and see what you think | [21:58] |
| pinchartl | which patch in particular ? | [21:59] |
| shuah | patch 18
and 26 | [22:00] |
| pinchartl | ok, let me prototype what I'm talking about otherwise we'll spend hours discussing it without understanding each other :-)
(I have a feeling the night will be long...) | [22:01] |
| shuah | right - I will send you my current work in progress for you to check as well
It might be late for you so - we don't have to talk about it now | [22:02] |
| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |