From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id A519B41D22E for ; Mon, 3 Aug 2026 14:21:45 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785766907; cv=none; b=FuifrEBrgTix00g3L3b9oo6mKUnpqPXDcHbkkfQysHdOztPbPw0TieID0nAPgiinWAebfVNDptDCVFdMdi6EPn8d6YwIbeyRjsvtzUE15Pohyw8JxmgZXmAF5upwP+9q39+iyZhvA+AAT6NJVj+Kn3qgdXCbmBHmiMwLqvmFo9k= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785766907; c=relaxed/simple; bh=4CJhajy18FhKKEtxtQelzRWT70J1u2W4WYVGHD/S0C8=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=ufJuVYQwOJ0xjNn60wIQCiQ9Eb3hsvsMH6HZL/hPTh5ZPu+hkLnLK0viVOnV4kCvSqckrFwyhof1FkHDTFqJtP//cwmrwJiANp3jGDa76JA/sUkIN6T/0c+WmKcPdug4sUGMVKKWosAvLIruAPuZ2TLigfwNigBzxfIpDdK16G0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b=dJhPz/bf; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b="dJhPz/bf" Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id F2CCE1595; Mon, 3 Aug 2026 07:21:40 -0700 (PDT) Received: from e135116.arm.com (unknown [10.57.0.74]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id 17DAC3F632; Mon, 3 Aug 2026 07:21:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1785766905; bh=4CJhajy18FhKKEtxtQelzRWT70J1u2W4WYVGHD/S0C8=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=dJhPz/bfVnGss2yjBsBV116eXhaLoOkzluq9Yw7sYl74DIp/gALyANPDPWgXByl/c ljMDq4jhOTnQsQ2xuFEbfW5ic4SUTcQ1LBTrfW4nt2TVYVbjuQuWWjDqCZ/sXDSFTF Jsl6pEUexuInWjXNzGaKC9z3CI4WfY3SdxHuwTTs= From: Raveendra Talabattula To: dri-devel@lists.freedesktop.org Cc: linux-kernel@vger.kernel.org, Raveendra Talabattula , liviu.dudau@arm.com, maarten.lankhorst@linux.intel.com, mripard@kernel.org, tzimmermann@suse.de, airlied@gmail.com, simona@ffwll.ch, james.qian.wang@arm.com, asad.malik@arm.com, vincenzo.frascino@arm.com, nayden.kanchev@arm.com, charvi.mehta@arm.com Subject: [PATCH v2 2/2] drm/komeda: Set display/writeback clone masks Date: Mon, 3 Aug 2026 15:21:22 +0100 Message-ID: <20260803142122.4127851-3-raveendra.talabattula@arm.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20260803142122.4127851-1-raveendra.talabattula@arm.com> References: <20260803142122.4127851-1-raveendra.talabattula@arm.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit From: Vincenzo Frascino Since commit 41b4b11da0215 ("drm: Add valid clones check"), all encoders attached to the same CRTC must advertise each other through possible_clones. Komeda creates one writeback connector per CRTC. Allow the writeback encoder to be cloned only with encoders that can drive the same CRTC, and update those encoders reciprocally. Do this when the writeback connector is created rather than modifying every encoder registered with the DRM device from komeda_kms_attach(). This leaves encoders associated with other CRTCs untouched. Signed-off-by: Asad Malik Signed-off-by: Vincenzo Frascino Co-developed-by: Raveendra Talabattula Signed-off-by: Raveendra Talabattula --- .../arm/display/komeda/komeda_wb_connector.c | 33 +++++++++++++++++++ 1 file changed, 33 insertions(+) diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_wb_connector.c b/drivers/gpu/drm/arm/display/komeda/komeda_wb_connector.c index 41cc3e080dc9..4872253a2449 100644 --- a/drivers/gpu/drm/arm/display/komeda/komeda_wb_connector.c +++ b/drivers/gpu/drm/arm/display/komeda/komeda_wb_connector.c @@ -143,12 +143,17 @@ static int komeda_wb_connector_add(struct komeda_kms_dev *kms, struct komeda_wb_connector *kwb_conn; struct drm_writeback_connector *wb_conn; struct drm_display_info *info; + struct drm_encoder *encoder; + struct drm_encoder *wb_encoder; u32 *formats, n_formats = 0; + u32 crtc_mask; int err; if (!kcrtc->master->wb_layer) return 0; + crtc_mask = drm_crtc_mask(&kcrtc->base); + kwb_conn = kzalloc_obj(*kwb_conn); if (!kwb_conn) return -ENOMEM; @@ -176,6 +181,34 @@ static int komeda_wb_connector_add(struct komeda_kms_dev *kms, return err; } + wb_encoder = &wb_conn->encoder; + + /* + * The writeback connector is associated with a single CRTC. Make its + * encoder clone-compatible only with encoders that can drive that CRTC. + * + * possible_clones must contain the encoder's own bit whenever it is + * non-zero. Add both the encoder itself and the writeback encoder when + * updating the reciprocal clone relationship. + */ + wb_encoder->possible_clones = drm_encoder_mask(wb_encoder); + + drm_for_each_encoder(encoder, &kms->base) { + u32 encoder_mask; + + if (encoder == wb_encoder) + continue; + + if (!(encoder->possible_crtcs & crtc_mask)) + continue; + + encoder_mask = drm_encoder_mask(encoder); + + wb_encoder->possible_clones |= encoder_mask; + encoder->possible_clones |= encoder_mask | + drm_encoder_mask(wb_encoder); + } + drm_connector_helper_add(&wb_conn->base, &komeda_wb_conn_helper_funcs); info = &kwb_conn->base.base.display_info; -- 2.43.0