Igt-dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [igt-dev] [PATCH i-g-t v3 0/3] Writeback fixes and improvements
@ 2022-06-10 12:16 Maxime Ripard
  2022-06-10 12:16 ` [igt-dev] [PATCH i-g-t v3 1/3] tests/kms_writeback: Use endianness accessor to fill pixels Maxime Ripard
                   ` (4 more replies)
  0 siblings, 5 replies; 7+ messages in thread
From: Maxime Ripard @ 2022-06-10 12:16 UTC (permalink / raw)
  To: igt-dev, Petri Latvala, Arkadiusz Hiler; +Cc: Pekka Paalanen, Maxime Ripard

Hi,

Here's a series addressing the writeback-check-output test not passing
on vc4. This is due to the fact that the writeback controller will force
the X component to 0, while we expect the output and input buffers to be
identical, and the input buffer has 0xff.

This series tackles this by adding some support to dump the faulty
frames in PNG and raw, which proved to be useful while debugging this
series. Then, we rework the hash function to ignore the X component of
our buffer and only hash the pixels themselves.

Finally, we change the input buffer X value to a garbage, different from
0 and 0xff, value to make sure the hardware properly ignores the
component and doesn't treat it as alpha.

Let me know what you think,
Maxime

Changes from v2:
  - Rebased on master
  - Dropped the frame dumping patches

Changes from v1:
  - Rebased on master
  - Added maintainers to the recipients

Maxime Ripard (3):
  tests/kms_writeback: Use endianness accessor to fill pixels
  lib/igt_fb: Ignore the X component when computing CRC
  tests/kms_writeback: Use a garbage X value to create fill our test
    buffer

 lib/igt_core.h        |  9 +++++++++
 lib/igt_fb.c          | 20 ++++++++++++++++----
 tests/kms_writeback.c |  4 ++--
 3 files changed, 27 insertions(+), 6 deletions(-)

-- 
2.36.1

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2022-06-28 10:24 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-06-10 12:16 [igt-dev] [PATCH i-g-t v3 0/3] Writeback fixes and improvements Maxime Ripard
2022-06-10 12:16 ` [igt-dev] [PATCH i-g-t v3 1/3] tests/kms_writeback: Use endianness accessor to fill pixels Maxime Ripard
2022-06-28 10:24   ` Pekka Paalanen
2022-06-10 12:16 ` [igt-dev] [PATCH i-g-t v3 2/3] lib/igt_fb: Ignore the X component when computing CRC Maxime Ripard
2022-06-10 12:16 ` [igt-dev] [PATCH i-g-t v3 3/3] tests/kms_writeback: Use a garbage X value to create fill our test buffer Maxime Ripard
2022-06-10 17:12 ` [igt-dev] ✓ Fi.CI.BAT: success for Writeback fixes and improvements (rev3) Patchwork
2022-06-14 18:24 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox