| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |
| Who | What | When |
|---|---|---|
| *** | rellla has quit IRC (Quit: see you later) | [07:34] |
| ................... (idle for 1h33mn) | ||
| bbrezillon | hverkuil: just went through the patch you pointed to in your reply, and I see "The decoder expects one Access Unit per buffer." for H264
not sure what access unit is, but if it's a NAL, then this statement will no longer be true after the introduction of the decoding-mode ctrl | [09:07] |
| hverkuil | It's definition 3.1 in the spec :-)
Note that this is for stateful decoders, not stateless. | [09:16] |
| .... (idle for 19mn) | ||
| bbrezillon: your hantro series adding H264 support doesn't have support for held capture buffers, so does that mean that it doesn't yet support one-slice-per-output-buffer? I assume so.
Not a problem, just wanted to confirm this. bbrezillon: please repost your "[PATCH 0/9] media: hantro: Add support for H264 decoding" series on top of the latest master. Some of the patches already came in with the hantro vp8 series, and others don't quite apply cleanly. I prefer to have a v2. | [09:35] | |
| bbrezillon | hverkuil: HOLD_CAP_BUF => no, the IP does not support slice-based decoding
we can hack something, but it hurts multiplexing so we just gave up hverkuil: regarding the H264, I already rebased on top of media master ;) *H264 series | [09:41] |
| hverkuil | bbrezillon: that was yesterday's master, not today's :-)
This got merged yesterday: "[PATCH v2 0/7] hantro: Add RK3399 VP8 decoding suppor" | [09:44] |
| bbrezillon | hverkuil: I'm talking about that one yes :), I had already rebased the new version on top of ezequielg changes | [09:48] |
| hverkuil | Well, not quite, because it has patches that are also in ezequielg's changes. Please just rebase and repost. That way I am sure it is a clean series. | [09:51] |
| bbrezillon | still waiting for paulk-leonov's feedback on the new ctrls
do you want me to send a new version with the NAL-type header we were talking about? | [09:54] |
| hverkuil | bbrezillon: I'm sorry, I missed that it had another dependency which is still in progress.
Never mind. I'll mark this series as 'Changes Requested' and just wait for a v2. I was under the impression that this series was ready to be merged. My mistake. | [09:55] |
| bbrezillon | nope | [10:02] |
| .... (idle for 16mn) | ||
| paulk-leonov | bbrezillon: the new ctrls are part of the hantro series, or did I miss another series? | [10:18] |
| bbrezillon | paulk-leonov: haven't posted it yet
and they'll be part "H264 uAPI adjustments" v3 | [10:19] |
| paulk-leonov | oh right | [10:20] |
| bbrezillon | *part of | [10:20] |
| paulk-leonov | (I'm not at work today, but taking the train to Paris later so I'll try to catch up then) | [10:20] |
| ............................................................. (idle for 5h3mn) | ||
| *** | benjiG has left | [15:23] |
| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |