Re: [PATCH v6 00/10] MPEG-2 stateless API cleanup and destaging
From: Jernej Škrabec
Date: Mon Apr 26 2021 - 15:36:21 EST
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@xxxxxxxxxxxxx/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@xxxxxxxx>
Reviewed-by: Jernej Skrabec <jernej.skrabec@xxxxxxxx>
Best regards,
Jernej