Received-SPF: pass (google.com: domain of linux-kernel+bounces-29130-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
From: Detlev Casanova <detlev.casanova@collabora.com>
To: linux-kernel@vger.kernel.org
Cc: linux-media@vger.kernel.org,
	Daniel Almeida <daniel.almeida@collabora.com>,
	Mauro Carvalho Chehab <mchehab@kernel.org>,
	Hans Verkuil <hverkuil@xs4all.nl>,
	Detlev Casanova <detlev.casanova@collabora.com>
Subject: [PATCH v5 0/4] visl: Add a tpg_verbose parameter for reference comparison
Date: Wed, 17 Jan 2024 10:10:14 -0500
Message-ID: <20240117151202.405426-1-detlev.casanova@collabora.com>
Precedence: bulk
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit

When using visl in automated tests, we need to have output frames that
can be compared to reference frames or hash of those to validate that
the whole pipeline is working properly.

Make sure that a given input stream always outputs the same frames.
This is done by removing TPG information like queues status and pointer
values.

Introduce the tpg_verbose parameter to allow adding the more verbose
information that this patchset removes

This also adds some stable variation in the frames so that different
input give more different output.

Changes since v4:
 - Remove unnecessary changes in visl_get_ref_frames
 - Fix missing space after &&
Changes since v3:
 - Fix some more permissions in vidtv
 - Rename stable_output to tpg_verbose
 - Rebase on media_stage/master to include AV1 support
 - Remove code churn in visl_get_ref_frames()
 - Remove code duplication in visl_tpg_fill_sequence()
Changes since v2:
 - Set parameters permissions to 0444 in visl and vidtv
 - Make stable_output true by default
 - Keep showing frames timestamps and remove vb2 buffer indexes only in
   stable output mode
 - Make codec_variablility non optional (remove parameter, keep
   implementation)
 - Add details on used variability fields
Changes since v1:
 - Fix typo in parameter documentation

Detlev Casanova (4):
  media: visl,vidtv: Set parameters permissions to 0444
  media: visl: Add a tpg_verbose parameter
  doc: visl: Document tpg_verbose parameter
  visl: Add codec specific variability on output frames

 Documentation/admin-guide/media/visl.rst      |  12 +-
 .../media/test-drivers/vidtv/vidtv_bridge.c   |  26 +-
 drivers/media/test-drivers/visl/visl-core.c   |  15 +-
 drivers/media/test-drivers/visl/visl-dec.c    | 301 +++++++++++-------
 drivers/media/test-drivers/visl/visl.h        |   1 +
 5 files changed, 217 insertions(+), 138 deletions(-)

-- 
2.43.0