15.11.2021 17:49, Hans Verkuil пишет: > On 15/11/2021 15:43, Nicolas Dufresne wrote: >> Hi Dmitry, >> >> Le lundi 15 novembre 2021 à 01:47 +0300, Dmitry Osipenko a écrit : >>> Video decoder of Tegra114/124 SoCs uses additional memory buffer required >>> for decoding of protected content. We won't support that content, but it >>> is impossible to disable access to the buffer, hence a stub buffer needs >>> to be provided. This series enables decoder driver only for Tegra114 >>> because Tegra124 support requires more non-trivial changes on both kernel >>> and userspace sides. >> >> I believe the stateless API is quite in place now, but I only see maintenance on >> this staging driver. I don't believe it really make sense to keep maintaining a >> staging driver without any step forward de-staging it. I believe it gives the >> wrong message on the Kernel staging purpose. >> >> I'm not criticizing your effort, I believe you are doing nice work for you >> community, but would prefer to see this driver be ported to the official kernel >> APIs rather then being maintain as staging till the end of time. > > I agree with Nicolas here. This driver only support H264 and the stateless API > for that is now in mainline. So there is no reason not to convert to the > stateless codec API and move this driver to mainline. > > It would be really nice to see that happen. > > Without any progress on that I am inclined to remove this driver some time > next year. I'll prioritize the v4l patches of this driver. The reason it wasn't done yet is because the current custom UAPI works perfectly fine and I was busy with a bit more important patches so far.