The decoder does use the same message box type interface as the encoder, reference implementation would be available here: https://github.com/Xilinx/vcu-ctrl-sw We are currently investigating on what needs to be done, first issue was a change in the interface with the later firmware images, we will start with the older firmware since the encoder is working for that one. -----Original Message----- From: Michael Tretter <m.tretter@xxxxxxxxxxxxxx> Sent: Freitag, 10. Februar 2023 18:40 To: Nicolas Dufresne <nicolas@xxxxxxxxxxxx> Cc: Deji Aribuki Ext <deji.aribuki.ext@xxxxxxxxxxxx>; linux-media@xxxxxxxxxxxxxxx; Laurenz Eschwey <laurenz.eschwey@xxxxxxxxxxxx>; kernel@xxxxxxxxxxxxxx Subject: Re: Mainline zyncmp vcu driver: support for the video decoder [You don't often get email from m.tretter@xxxxxxxxxxxxxx. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ] On Fri, 10 Feb 2023 10:58:01 -0500, Nicolas Dufresne wrote: > Le jeudi 09 février 2023 à 14:54 +0100, Michael Tretter a écrit : > > Hi Deji, > > > > On Thu, 09 Feb 2023 10:03:19 +0000, Deji Aribuki Ext wrote: > > > I would like to know if there is a plan (or if there is already > > > have a > > > patch-set) for adding support for allegro dvt video decoder. > > > > I currently don't have any plans or patches to add decoder support. > > > > A while ago, there was a bit of interest in decoder support, but I > > haven't heard of anything new for a long time. > > > > Are you interested in adding decoder support to the driver? I would > > be happy to review and test patches. > > Isn't the decoder side stateless ? That could be difficult to achieve > without reference code or documentation. There is at least some firmware running on the decoder. And I am pretty sure that it provides a similar mailbox interface as the encoder that the driver must use. However, I don't know, what messages the firmware expects. Thus, the decoder maybe stateless. Maybe it even keeps some state in the firmware, but still requires the software to parse the stream and explicitly set the stream parameters. Michael
