Re: [igt-dev] [PATCH i-g-t v6 00/10] igt: chamelium: Test YUV buffers using the Chamelium

Paul Kocialkowski
2019-02-12T09:08:49Z

Hi,

On Fri, 2019-02-08 at 14:18 +0100, Maxime Ripard wrote:
> Hi,
>
> Here is a first attempt at expanding the current Chamelium formats
> test to YUV formats.
>
> It builds on top of the current infrastructure for the YUV convertions, and
> the format suppport of the Chamelium. It reworks igt_fb quite significantly
> to add generic support for YUV formats, and adds a bunch of new formats
> along the way.

The whole series was merged, thanks a lot!

Cheers,

Paul

> Let me know what you think,
> Maxime
>
> Changes from v5:
> - Fixed breakage on x86 platforms
> - Fixed typo in a comment
> - Split the size and strides check in a separate patch
> - Added the various tags from Lyude and Paul
> - Rebased on master
>
> Changes from v4:
> - Added Paul Reviewed-by's
> - Removed the patch that moves calc_fb_size for everybody, and added some
> asserts
>
> Changes from v3:
> - Fixed build breakage
>
> Changes from v2:
> - Added Paul Reviewed-by's
> - Fixed a stride issue on the buffer allocation
> - Take into account the plane bpp so that we don't overallocate
> - Rebased on master
>
> Changes from v1:
> - Support more YUV formats than just NV12
> - Rework the YUV convertion functions to add generic routines
> - Took the comments into account.
>
> Maxime Ripard (10):
> igt: fb: Refactor dumb buffer allocation path
> igt: fb: Add size checks and recalculation before dumb allocation
> igt: fb: Account for all planes bpp
> igt: fb: Don't pass the stride when allocating a dumb, multi-planar
> buffer
> igt: fb: Clear YUV dumb buffers
> igt: fb: Rework YUV i915 allocation path
> igt: fb: Add a bunch of new YUV formats
> igt: tests: chamelium: Start to unify tests
> igt: tests: chamelium: Convert VGA tests to do_test_display
> igt: tests: chamelium: Add YUV formats tests
>
> lib/igt_fb.c | 175 ++++++++++++++++++++++++-----
> tests/kms_chamelium.c | 254 ++++++++++++++++++++++--------------------
> 2 files changed, 282 insertions(+), 147 deletions(-)
>
--
Paul Kocialkowski, Bootlin
Embedded Linux and kernel engineering