| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |
| Who | What | When |
|---|---|---|
| pinchartl | hverkuil: ping
you've acked bd3e275f3ec0 ("[media] media: i2c: adv7604: Use v4l2-dv-timings helpers") but the patch seems plain wrong to me it ignores the cap->pad value in adv76xx_dv_timings_cap() and returns the dv timing caps for the currently selected input, not for the requested pad | [11:25] |
| hverkuil | pinchartl: you're right, I completely missed that. | [11:36] |
| pinchartl | glad to know that you at least agree :-)
neg: would you like to submit a fix or should I do it ? | [11:36] |
| neg | pinchartl: please do, I'm swamped with rcar-vin :) | [11:37] |
| pinchartl | ok | [11:38] |
| ....... (idle for 31mn) | ||
| patch sent | [12:09] | |
| Renault | hverkuil: nohous is connected now :) | [12:10] |
| hverkuil | Renault: got his email yesterday :-) | [12:10] |
| Renault | ok, nice, what is the next step now? | [12:11] |
| hverkuil | Me finding time and organizing a meeting.
Problem is that I would like to read up a bit on SDI first and that I am traveling starting mid-next-week. | [12:12] |
| Renault | ok | [12:15] |
| hverkuil | I tentatively plan to mail you both on Friday with some proposed dates. | [12:16] |
| Renault | ok, no problem, I have the time :) | [12:17] |
| pinchartl | hverkuil: thanks for your ack. will you take the patch in your tree ?
mchehab: ping | [12:19] |
| hverkuil | pinchartl: Can you delegate it to me? It's for 4.7, right? I have another patch for 4.7, so I can make a pull request for those two on Friday. | [12:21] |
| pinchartl | the regression got merged in v4.6, so strictly speaking there's no urgency to get this fix in v4.7, but it would be nice
patch delegated | [12:23] |
| mchehab | pinchartl: pong | [12:27] |
| ..... (idle for 21mn) | ||
| jmleo | pinchartl, thx for catching the error with the pad value... | [12:48] |
| pinchartl | jmleo: you're welcome
mchehab: how are you doing ? | [12:51] |
| mchehab | fine. and you? | [12:52] |
| pinchartl | are you feeling better now or still recovering ?
I'm doing good | [12:52] |
| mchehab | I'm OK
recovered from the cold | [12:52] |
| pinchartl | is it too hot now ? :-) | [12:53] |
| mchehab | no. it is rainy
:) | [12:53] |
| pinchartl | :-)
I was wondering what your plan was for the last vsp pull request I've sent the one that missed v4.7 and that contains two drm patches | [12:54] |
| mchehab | I need to review the patches first
the idea is that, once reviewed, to add it on a separate topic branch this way, drm maintainers can use it | [12:56] |
| pinchartl | and Dave could merge that branch, yes
would you be able to handle them soon ? benjiG needs that branch to rebase one of his DRM/KMS patch series on for v4.8 | [12:57] |
| mchehab | right now, I'm without e-mail access at samsung, due to some VPN issues | [12:58] |
| pinchartl | nice | [12:58] |
| mchehab | yes :( | [12:58] |
| pinchartl | that must feel like half-holidays :-) | [12:58] |
| mchehab | I'm actually taking the time to improve a DVB userspace app
anyway, after having my access restored, the priority is to merge the patches fixing the MC, as they need to be backported to -stable btw, I'm still waiting for a comment from you on that thread ;) | [12:59] |
| pinchartl | yes, there's that too
which thread ? there were several of them :-) | [13:00] |
| mchehab | Subject: Re: [PATCH 2/2] [media] media-device: dynamically allocate struct media_devnode
Shuah patches fixing the lifetime issue depend on it | [13:01] |
| pinchartl | ok I'll check that
for the vsp patches, it would be good if Dave could merge your branch early to avoid blocking other developers. do you think you would be able to handle them next week ? | [13:02] |
| mchehab | I expect so | [13:02] |
| pinchartl | thank you | [13:02] |
| mchehab | that would depend if there are any pending issues due to the merge window (with I can't check without the VPN access)
although I guess everything should be ok, as we send few patches on this merge window... only ~160 patches | [13:03] |
| shuah | mchehab, pinchartl: yes lifetime fix depends on Mauro's devnode dynamic alloc. Also my Media dev allocator api is lot simpler now with the fix in place. | [13:04] |
| mchehab | so, if everything happens as expected, I guess I should be reviewing the patches next week and applying, if everything's ok | [13:04] |
| shuah | hverkuil: sent me comments and I am going to respond to him today. | [13:04] |
| ribalda | hverkuil: Good evening! | [13:10] |
| benjiG | pinchartl: Hell,o have you been able to test zpos patches ? | [13:13] |
| pinchartl | benjiG: I have a conflicting patch series for rcar-du-drm :-/
would it help if I rebased your latest version on top of it myself and tested it ? it only conflicts in the rcar-du-drm driver | [13:15] |
| benjiG | pinchartl: an ACK could help ;-) | [13:16] |
| pinchartl | if your series gets merged first I'll likely miss another merge window for mine :-/
let me show you I have this branch https://git.linuxtv.org/pinchartl/media.git/log/?h=vsp1/next that contains two rcar-du-drm patches in the middle it should be merged by Mauro next week in his tree in a branch that Dave could then pull and on top of which we could rebase your zpos series none of them would miss v4.8 then if I ack the zpos series now and if it gets merged in Dave's tree, there will be a conflict is v3 the latest zpos series version ? | [13:18] |
| benjiG | pinchartl: I guess on "drm: rcar-du: Add Z-order support for VSP planes" commit | [13:20] |
| pinchartl | correct
the problem is that that patch depends on a patch to the vsp driver (v4l2) and then patch "v4l: vsp1: Remove deprecated DRM API" depends on "drm: rcar-du: Add Z-order support for VSP planes" | [13:21] |
| benjiG | understand | [13:21] |
| pinchartl | nasty sandwich of dependencies :-/
I can rebase your v3 on top of my branch, test it and provide feedback and we could get it merged next week when Mauro merges my branch in his tree would that work for you ? | [13:21] |
| benjiG | pinchartl: sound good for me
pinchartl: just ping me when it will be done | [13:22] |
| pinchartl | thanks | [13:23] |
| benjiG | pinchartl: if you found any problem please tell me so I can fix them before rebasing my patches | [13:24] |
| pinchartl | v3 is the latest, right ? | [13:30] |
| benjiG | yes | [13:30] |
| ..... (idle for 23mn) | ||
| hverkuil | ribalda: good afternoon! | [13:53] |
| ......................... (idle for 2h4mn) | ||
| *** | benjiG has left | [15:57] |
| ................................................. (idle for 4h3mn) | ||
| awalls has left | [20:00] | |
| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |