From nobody Mon Feb 9 07:06:01 2026 Received: from relay5-d.mail.gandi.net (relay5-d.mail.gandi.net [217.70.183.197]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 908211DFD96 for ; Fri, 22 Nov 2024 17:20:28 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.70.183.197 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1732296030; cv=none; b=moHiQqT38FdmLngWcbzanKNHW3SwgXj3qEe9CXc8uSaxBGzi6NlsazzSETiuiejOh6odHCb3bv9jtS9RpBb369O028+k6OmRG+x6WQCE1qupjMege5tiJ5neAS6hxZZ5m/4JB8kVfAUe1yHkxyHmz0FKDYPpS4bnD7WdmMgvlk8= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1732296030; c=relaxed/simple; bh=BF0yRjAMyDjPU9RolEhuXzgfCND9f1KlhkLeU6VUXIA=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=SF7VmrbcohdW7nH7bI7Sa9ZHkYsBavmeCB1RoqU8mRg9//atKUXFFk2pDx8E1QMpmz7ee4BDC1ZdxwIk9Zc2janF4/anhhvcGtG4ag1hk8y76XDCBbU0+Fn3vcIZE1xKR3MmWJ8LDzCHOstUWfTQPrqOnMHCp05P3HrU36MbEXo= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com; spf=pass smtp.mailfrom=bootlin.com; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b=oMtAb9H/; arc=none smtp.client-ip=217.70.183.197 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bootlin.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b="oMtAb9H/" Received: by mail.gandi.net (Postfix) with ESMTPSA id 4F2E61C0004; Fri, 22 Nov 2024 17:20:26 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1732296027; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=ktawcOolQxrYmPaqwiDnraP2XPOZ45oKEBQAwhE213s=; b=oMtAb9H/2mSXg4pB/kN/CZe1IBf0mJQ2Y9rBPOMS1odtIZac57aixaETv1P7HBdetsX6Mr FxJqxi+y4FQdwzB6SYaurDM2w0AoIKk1FnG6okiXkVuuxNT9fb0sW9SGyMDDGq9XFbgJiE TER3Mg74kpJYBa8WOvVPm+/nb8Hh328iDymbXov5Ly2wakjkD0tnxOxHiow2tfmDZVFJlo CAF9qPTjvo9cl2yclVINnQcpmq3LMS94Tvcarb8M7ICyuK21/tTDdcE50x/53Xvs7Aqu5c i+zePwCXNBVIl285/lKvNPzPd/7+FmKlAQgyL3rCsFdwPPuL4MvbxdJjwV5x5A== From: Louis Chauvet Date: Fri, 22 Nov 2024 18:20:11 +0100 Subject: [PATCH RFC v2 07/18] drm/vkms: Introduce config for plane color encoding Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20241122-google-remove-crtc-index-from-parameter-v2-7-81540742535a@bootlin.com> References: <20241122-google-remove-crtc-index-from-parameter-v2-0-81540742535a@bootlin.com> In-Reply-To: <20241122-google-remove-crtc-index-from-parameter-v2-0-81540742535a@bootlin.com> To: =?utf-8?q?Ma=C3=ADra_Canal?= , Haneen Mohammed , Simona Vetter , Melissa Wen , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie Cc: arthurgrillo@riseup.net, jeremie.dautheribes@bootlin.com, miquel.raynal@bootlin.com, thomas.petazzoni@bootlin.com, seanpaul@google.com, nicolejadeyee@google.com, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Louis Chauvet X-Mailer: b4 0.15-dev X-Developer-Signature: v=1; a=openpgp-sha256; l=4434; i=louis.chauvet@bootlin.com; h=from:subject:message-id; bh=BF0yRjAMyDjPU9RolEhuXzgfCND9f1KlhkLeU6VUXIA=; b=owEBbQKS/ZANAwAIASCtLsZbECziAcsmYgBnQL1OpfzvGPhi5nbgVTIP6JnGt047EF7mfamOi pxenGUuXs6JAjMEAAEIAB0WIQRPj7g/vng8MQxQWQQgrS7GWxAs4gUCZ0C9TgAKCRAgrS7GWxAs 4jn3D/9XTXmzSLdEBdnwiOtk+vlzWX4GjMEwjVySVgf6P1aTfE14F2h3zwVSfpEySyzf8oqZkwf dnPI6vWOo27Wgcr7d71tRugyArDcMAH5BA00ZrjvrhgG2ucssqDjdABMFZGKeHK9tfA4AmBFiQn bx7fHjyc4VUoCrr7SIMefcSY45kw2Z7smbAnqawcaISuEG4eofR895J3ONpQy1J7tMSYEG9RKJW 5HdJvtnY+50HEhApspf6OxGjaf+q/DjRigUVRxC3R1B+FzHYbEkOiVhfDMRmvZAfWTnIK8wTVJA 5hH3irC8CH/+RCSzl8L0uryzzGnLpw5npMQGIXeW/SMSeB4/in+ZiOF/Nf1vsHFSOvgL8DSx1oN UFOIUn77SOup0RscPxmScSAzSHJxaq5t2DYZuNcGvRsalP2hbgnt/DM1OOiTQhdxvbej/6AX5Qp k+YclDnR9D4zU6kVEa+CHhUffz7/zaWXV79WJwzCK7AjTcWV1ltJYy6DubrcpRJ4LQFudvg5nF1 aIonJ4z+RkhZOmYLeB22TUDLS7bYyGzsY9FYn9VmPpOIxRwB4qaZ1sj2JYs69pDI0BXo+i4Zv9k ljR6Bp6QLGksigmtAwkoIVO3q5nQR22c8BgkF/EpdF3fE+Hzw4n2P2sCQJ0Lh+1yjddt3lyKWdE vz9TpuJOfiMeOdQ== X-Developer-Key: i=louis.chauvet@bootlin.com; a=openpgp; fpr=8B7104AE9A272D6693F527F2EC1883F55E0B40A5 X-GND-Sasl: louis.chauvet@bootlin.com VKMS driver supports all the color encoding on planes, but for testing it can be useful to only advertise few of them. This new configuration interface will allow configuring the color encoding per planes. Signed-off-by: Louis Chauvet --- drivers/gpu/drm/vkms/vkms_config.c | 15 +++++++++++++++ drivers/gpu/drm/vkms/vkms_config.h | 4 ++++ drivers/gpu/drm/vkms/vkms_plane.c | 6 ++---- 3 files changed, 21 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/vkms/vkms_config.c b/drivers/gpu/drm/vkms/vkms= _config.c index 6a2e5f0c580a348effc12b474a7f87dcd4e36642..9f9ebe674f6cbe03ac15e5380e4= 584c40a8bdea7 100644 --- a/drivers/gpu/drm/vkms/vkms_config.c +++ b/drivers/gpu/drm/vkms/vkms_config.c @@ -84,6 +84,11 @@ struct vkms_config_plane *vkms_config_create_plane(struc= t vkms_config *vkms_conf vkms_config_overlay->type =3D DRM_PLANE_TYPE_OVERLAY; vkms_config_overlay->supported_rotations =3D DRM_MODE_ROTATE_MASK | DRM_M= ODE_REFLECT_MASK; vkms_config_overlay->default_rotation =3D DRM_MODE_ROTATE_0; + vkms_config_overlay->supported_color_encoding =3D BIT(DRM_COLOR_YCBCR_BT6= 01) | + BIT(DRM_COLOR_YCBCR_BT709) | + BIT(DRM_COLOR_YCBCR_BT2020); + vkms_config_overlay->default_color_encoding =3D DRM_COLOR_YCBCR_BT601; + =20 list_add(&vkms_config_overlay->link, &vkms_config->planes); =20 @@ -125,6 +130,12 @@ bool vkms_config_is_valid(struct vkms_config *config) config_plane->default_rotation) return false; =20 + // Default color range not in supported color range + if ((BIT(config_plane->default_color_encoding) & + config_plane->supported_color_encoding) !=3D + BIT(config_plane->default_color_encoding)) + return false; + if (config_plane->type =3D=3D DRM_PLANE_TYPE_PRIMARY) { // Multiple primary planes for only one CRTC if (has_primary) @@ -160,6 +171,10 @@ static int vkms_config_show(struct seq_file *m, void *= data) seq_printf(m, "\ttype: %d\n", config_plane->type); seq_printf(m, "\tsupported rotations: 0x%x\n", config_plane->supported_r= otations); seq_printf(m, "\tdefault rotation: 0x%x\n", config_plane->default_rotati= on); + seq_printf(m, "\tsupported color encoding: 0x%x\n", + config_plane->supported_color_encoding); + seq_printf(m, "\tdefault color encoding: %d\n", + config_plane->default_color_encoding); } =20 return 0; diff --git a/drivers/gpu/drm/vkms/vkms_config.h b/drivers/gpu/drm/vkms/vkms= _config.h index fd5e0df11647f145d53e71a1b6a244951af3ec0d..6cc9b89a996fe26c04951981cfe= ff7b603c6aec4 100644 --- a/drivers/gpu/drm/vkms/vkms_config.h +++ b/drivers/gpu/drm/vkms/vkms_config.h @@ -32,6 +32,8 @@ struct vkms_config { * plane is primary. * @default_rotation: Default rotation that should be used by this plane * @supported_rotation: Rotation that this plane will support + * @default_color_encoding: Default color encoding that should be used by = this plane + * @supported_color_encoding: Color encoding that this plane will support * @plane: Internal usage. This pointer should never be considered as vali= d. It can be used to * store a temporary reference to a vkms plane during device creat= ion. This pointer is * not managed by the configuration and must be managed by other m= eans. @@ -43,6 +45,8 @@ struct vkms_config_plane { enum drm_plane_type type; unsigned int default_rotation; unsigned int supported_rotations; + enum drm_color_encoding default_color_encoding; + unsigned int supported_color_encoding; =20 /* Internal usage */ struct vkms_plane *plane; diff --git a/drivers/gpu/drm/vkms/vkms_plane.c b/drivers/gpu/drm/vkms/vkms_= plane.c index 306af498543cbc39b8af3e5b5879f8b3972d5a4a..77704ee2d49d2ce1551f67cab63= 08dc69b329ef5 100644 --- a/drivers/gpu/drm/vkms/vkms_plane.c +++ b/drivers/gpu/drm/vkms/vkms_plane.c @@ -239,12 +239,10 @@ struct vkms_plane *vkms_plane_init(struct vkms_device= *vkmsdev, config->supported_rotations); =20 drm_plane_create_color_properties(&plane->base, - BIT(DRM_COLOR_YCBCR_BT601) | - BIT(DRM_COLOR_YCBCR_BT709) | - BIT(DRM_COLOR_YCBCR_BT2020), + config->supported_color_encoding, BIT(DRM_COLOR_YCBCR_LIMITED_RANGE) | BIT(DRM_COLOR_YCBCR_FULL_RANGE), - DRM_COLOR_YCBCR_BT601, + config->default_color_encoding, DRM_COLOR_YCBCR_FULL_RANGE); =20 return plane; --=20 2.47.0