Re: [PATCH v6 00/10] MPEG-2 stateless API cleanup and destaging

Daniel Almeida
2021-04-26T23:30:00Z

Hi Ezequiel, thanks for all your hard work on this!

On 26/04/2021 00:35, Ezequiel Garcia wrote:
> Hi everyone,
>
> Here's another round addressing Hans' feedback. More details
> about this can be found in the previous cover letter [1]:
>
> [1]
https://lore.kernel.org/linux-media/20210403180756.175881-11-ezequiel@collabora.com/T/
>
> The documentation looks good, and so does pahole,
> but it's really easy to miss some detail in this series,
> so we'd appreciate if more people could take a look.
>
> v6:
> * Reorder patch "media: controls: Log MPEG-2 stateless control in
.std_log"
> to avoid a new compile warning.
> * Remove "reserved" field in mpeg2 sequence control, noted by Hans.
> * Reorder "flags" field in mpeg2 picture control, noted by Hans.
> * Typos and comments fixes, noted by Hans.
>

I tested this series on rk3399 (rockpi4) + upcoming gstreamer v4l2
stateless mpeg2 decoder and it works just fine.

Tested-by: Daniel Almeida <daniel.almeida@collabora.com>

-- thanks
-- Daniel