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

Jernej Škrabec
2021-04-26T19:36:21Z

Hi!

Dne ponedeljek, 26. april 2021 ob 05:35:12 CEST je Ezequiel Garcia napisal(a):
> 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.

Thanks for working on this! I tested everything with Cedrus and FFmpeg and it
works without a problem. So, whole series is:

Tested-by: Jernej Skrabec <jernej.skrabec@siol.net>
Reviewed-by: Jernej Skrabec <jernej.skrabec@siol.net>

Best regards,
Jernej