From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 68D93CD6E6B for ; Tue, 2 Jun 2026 21:58:38 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 7FE17113B58; Tue, 2 Jun 2026 21:58:32 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=fail reason="signature verification failed" (2048-bit key; unprotected) header.d=igalia.com header.i=@igalia.com header.b="dIIxaNJm"; dkim-atps=neutral Received: from fanzine2.igalia.com (fanzine2.igalia.com [213.97.179.56]) by gabe.freedesktop.org (Postfix) with ESMTPS id 04983113B47; Tue, 2 Jun 2026 21:58:27 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=igalia.com; s=20170329; h=Content-Transfer-Encoding:MIME-Version:Message-ID:Date:Subject: Cc:To:From:Sender:Reply-To:Content-Type:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: In-Reply-To:References:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=O8PrixyJlh3bL7brTHTRmC6t8kgBXRWn3cbWulgmC44=; b=dIIxaNJmzt6o+tpzvyj7QjK/es R+DDw8S9HoXOg5mK/loo885OggIbcMO5vw1/Ahi73dYJm9JbUhDmjEY8laFraZej5V+tVoN45j4ii bnm0ModbfP7k8adkLDIg2cjx4hMnIL9UfxQOnL04u2aZsU9Ya1Y2XsGenur5gDEhm8owIvkd55a75 6dGYd71E3afovYRBKUA0dmeNkA1aQCGrvbrMf/95gu0+dBITsglc7vw8f1I4hyqs7zCI9PXMJiT+o 7q0AC44TX1J74ntDfqKFI8Iq4Ke+6ihWLqdbFIk2BJpYzEIXLEK7WpanjHOqCm2OHiCkjlKPnkLca NAVT1jNA==; Received: from [79.117.146.159] (helo=killbill.home) by fanzine2.igalia.com with esmtpsa (Cipher TLS1.3:ECDHE_X25519__RSA_PSS_RSAE_SHA256__AES_256_GCM:256) (Exim) id 1wUX7R-00BuCE-Vn; Tue, 02 Jun 2026 23:57:54 +0200 From: Melissa Wen To: airlied@gmail.com, alexander.deucher@amd.com, alex.hung@amd.com, christian.koenig@amd.com, contact@emersion.fr, daniels@collabora.com, harry.wentland@amd.com, louis.chauvet@bootlin.com, maarten.lankhorst@linux.intel.com, mripard@kernel.org, mwen@igalia.com, sebastian.wick@redhat.com, simona@ffwll.ch, siqueira@igalia.com, sunpeng.li@amd.com, tzimmermann@suse.de Cc: Uma Shankar , Chaitanya Kumar Borah , Xaver Hugl , Pekka Paalanen , Matthew Schwartz , amd-gfx@lists.freedesktop.org, kernel-dev@igalia.com, Rob Clark , Dmitry Baryshkov , Abhinav Kumar , Jessica Zhang , Sean Paul , Marijn Suijten , linux-arm-msm@vger.kernel.org, freedreno@lists.freedesktop.org, intel-xe@lists.freedesktop.org, intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org Subject: [PATCH v8 0/4] drm/atomic: track individual colorop updates Date: Tue, 2 Jun 2026 23:53:52 +0200 Message-ID: <20260602215743.914265-1-mwen@igalia.com> X-Mailer: git-send-email 2.53.0 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" This is a partial of [1], only with patches related to individual colorop update tracking. I.e., I'm detaching from here fixes regarding attempts of changing colorops that are not part of an active color pipeline, or in the transition between active and inactive color pipelines. This series focus on tracking updates for each individual color operation, allowing the driver to react accordingly. This new version just adds r-b and Fixes tag accordingly. During the Display Next Hackfest we also agree that it should be applied to drm-misc-fixes and it can be backported by AMD after. - Patches 1 and 2 make lut1d_interpolation and lut3d_interpolation colorops correctly behave as mutable, handling their changes via drm_colorop_state. - Patches 3 and 4 track colorop updates of a given plane color pipeline by setting plane `color_mgmt_changed` flag, similar to what is done for tracking CRTC color mgmt property changes with CRTC `color_mgmt_changed` flag. The flag also tracks when a different color pipeline is set to a given plane, but doesn't consider as a change when the same color pipeline value is set to the plane COLOR_PIPELINE prop. That way, the driver can react accordingly and update their color blocks. As interpolation properties become mutable, they are also tracked here. It also fixes shaper/3D LUT updates when changing night mode settings on gamescope with a custom branch that supports `COLOR_PIPELINE`: - https://github.com/ValveSoftware/gamescope/pull/2113 v1: https://lore.kernel.org/dri-devel/20260318162348.299807-1-mwen@igalia.com/ Changes: - include linux types for function's bool return type (kernel bot on MSM driver) - add Harry's r-b tags v2: https://lore.kernel.org/dri-devel/20260323131942.494217-1-mwen@igalia.com/ Changes: - [NEW] two patches to only consider colorop updates from active color pipelines (Chaitanya) - [NEW] make lut interpolation properties mutable + Alex H patch for kernel docs - track lut(1/3)d_interpolation updates (Chaitanya) - rebase changes according to new patches v3: https://lore.kernel.org/dri-devel/20260403135909.214378-1-mwen@igalia.com/ Changes: rebase on drm-misc-next v4: https://lore.kernel.org/dri-devel/20260501132527.522320-1-mwen@igalia.com/ Changes: fix kernel doc (kernel bot) v5: https://lore.kernel.org/dri-devel/20260506192633.16066-1-mwen@igalia.com/ Changes: - rebase on drm-misc-next - fix kernel-doc and correctly reword (atomic) state to plane_state (Chaitanya) - reject inactive colorop updates in atomic check time, instead of during property's setup, to avoid ordering dependency as pointed out by Chaitanya - use `|= replaced` for consistency (Chaitanya) - add Chaitanya's r-b tags to patches 1,3-5 [1] v6: https://lore.kernel.org/dri-devel/20260519211111.228303-1-mwen@igalia.com/ Changes: - detach patches that implement individual tracking from those related to inactive colorop updates. v7: https://lore.kernel.org/dri-devel/20260525100524.304263-1-mwen@igalia.com/ Changes: - add Fixes and r-b tags Alex Hung (1): drm/colorop: Remove read-only comments from interpolation fields Melissa Wen (3): drm/colorop: make lut(1/3)d_interpolation props correctly behave as mutable drm/atomic: track individual colorop updates drm/amd/display: use plane color_mgmt_changed to track colorop changes .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 6 +- drivers/gpu/drm/drm_atomic.c | 4 +- drivers/gpu/drm/drm_atomic_uapi.c | 68 +++++++++++++++---- drivers/gpu/drm/drm_colorop.c | 16 ++++- include/drm/drm_atomic_uapi.h | 4 +- include/drm/drm_colorop.h | 34 +++++----- 6 files changed, 93 insertions(+), 39 deletions(-) -- 2.53.0