Hans, On Tue 07 Nov 23, 16:00, Hans Verkuil wrote: > On 07/11/2023 15:25, Paul Kocialkowski wrote: > > Hi Hans, > > > > On Tue 07 Nov 23, 14:54, Hans Verkuil wrote: > >> Looking at the cedrus TODO: > >> > >> Before this stateless decoder driver can leave the staging area: > >> * The Request API needs to be stabilized; > >> * The codec-specific controls need to be thoroughly reviewed to ensure they > >> cover all intended uses cases; > >> * Userspace support for the Request API needs to be reviewed; > >> * Another stateless decoder driver should be submitted; > >> * At least one stateless encoder driver should be submitted. > >> > >> I would say that all items are done, except for the last item. > >> > >> But this is a decoder driver, so I'm not sure why the TODO would mention > >> something about encoders. > > > > I agree. This didn't prevent the hantro driver form being unstaged anyway. > > > >> Anything else that needs to be done before it can be moved out of staging? > > > > I'm actually working on a big rework of the driver, because its architecture > > is vastly sub-optimal and overall very messy. This is in particular needed for > > the work to support H.264 encoding. > > > > The plan for now is to publish this work in a downstream repository (at Bootlin) > > since we don't yet have a uAPI for encoding. The rework is also not split into > > nice transition commits but it's instead a big all-in-one commit for the moment. > > > > However I would like to see this rework hit mainline after the publication of > > the encoding work, because it's a much needed cleanup for the decode side too. > > I will of course split it into nice commits at that point. > > > > So I'm not sure whether this conflicts with unstaging or not. It could make > > sense to wait until the rework is achieved to unstage, but I would also be fine > > with unstaging first. It would just make me slightly happier to bring big > > changes to a staging driver instead of a fully-merged one. > > > > What do you think? > > Fine with me, but I would appreciate it if you can post a patch updating the TODO > file. That's really out of date, and you can mention the rework you are doing. Great, I've just sent out a patch which replaces the current TODO with a description of the rework. Thanks, Paul -- Paul Kocialkowski, Bootlin Embedded Linux and kernel engineering https://bootlin.com
Attachment:
signature.asc
Description: PGP signature
