<!-- Some styling for better description lists --><style type='text/css'>dt { font-weight: bold;float: left;display:inline;margin-right: 1em} dd { display:block; margin-left: 2em}</style> hverkuil: <u>mchehab</u>: yes, I have a few questions. But that can be at a different time as well mchehab: well, I'm here now, but not sure if I'll be able to be here by the time of our usual meeting <br> s/by the time of our usual meeting/at our usual meeting time/ hverkuil: First question: is media_stage/master now the main repo we work with? I see several PRs of mine that are merged there, but not in media_tree/master. <br> Second question: any ETA of when you can review/merge this PR? https://patchwork.linuxtv.org/project/linux-media/patch/79eb0d83-1967-8caf-e205-b29beec6fa23@xs4all.nl/ <br> Once this is in, then I want to post a new PR to refactor the control framework. Since that is a large patch (no functionality changes, just splitting up v4l2-ctrls.c in smaller files), it is something that should be coordinated with you. mchehab: first question: no, The idea is to merge there first, and, after a couple of days, merge at media_tree. <br> I was planning to do such merge today, but there was a compilation breakage on -next, so I'll need to fix it at media_stage, apply the suff again and wait until monday to merge stuff back at media_tree <br> Btw, media_stage *can* be rebased <br> so don't rely on it hverkuil: OK, then I'll wait until next week when that PR is merged into media_tree/master. mchehab: ok <br> your second question: MPEG-2 stateless API cleanup and destaging <br> is this keeping part of the codecs at staging? I remember Ezequiel mentioned something like that <br> so, I ended skipping this PR to handle the less complex cases earlier <br> btw, it sounds that Greg's current view is that staging can't be used for "staging" APIs anymore hverkuil: in a meeting, sorry. mchehab: I'll let you know once I can confirm that pinchartl: <u>mchehab</u>: what can it be used for then ? :-) mchehab: anyway, I'll try to handle it tomorrow, if not sidetracked by something else <br> <u>pinchartl</u>: media-stage is meant to prevent patches that break -next to be reverted, as this tree can be rebased. It also allows picking patches while we're in the middle of a merge window <br> <u>pinchartl</u>: media-stage is meant to *allow* patches that break -next to be reverted, as this tree can be rebased. It also allows picking patches while we're in the middle of a merge window <br> anyway, I need to go. I should be returning back later today pinchartl: I meant staging, regarding your comment about Greg. but we can discuss this later, no worries hverkuil: <u>mchehab</u>: that PR moves the stateless codec MPEG API into uapi, i.e. it is now an officially supported public API, instead of a private API. Same as we did with H264 and VP8. After this only the HEVC support is still not stable (and thus also not part of uapi). How to proceed with that is something we can discuss separately. My preference would be to keep the HEVC support in staging somehow, but move the drivers themselves into m <br> ine. mchehab: <u>pinchartl</u>: yeah, agreed. anyway, I'm double checking it with Greg <br> <u>hverkuil</u>: ok. if the drivers using HEVC will be kept in staging, it shoudn't be a problem to move the ones that already use stable APIs out of staging hverkuil: Let's discuss that later with ezequiel in the loop. Right now I just want that PR merged so that I can respin my v4l2-ctrls refactor patch series. ***: sailus has left