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

[Paul Kocialkowski <paul.kocialkowski@bootlin.com>]

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
https://bootlin.com

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev