From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from sender4-pp-f112.zoho.com (sender4-pp-f112.zoho.com [136.143.188.112]) (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 3C80A3DFC94 for ; Thu, 18 Jun 2026 18:54:58 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=pass smtp.client-ip=136.143.188.112 ARC-Seal:i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781808899; cv=pass; b=MnJnsLA7rGYs4odRE8zk0HftptpQy8RLOwWGAJx8lokOY8t5J2809s6784XoP1jQlFcP2U1mn+w1SKrzq8R7ofUy32Du7BQwv4BFXNUDV+CpWLPsBft2+nqXV7enQe+NZJ59GnhzS8+LbvO/ZkCI9tJIO3e/8a1TVykSiizG0BA= ARC-Message-Signature:i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781808899; c=relaxed/simple; bh=fUlgWAfrB0ImI5eOewABDsSdi5p85EWj4wC3OpWMPQ4=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=ZzlgRqlSz1CpE1qJ55iOZcBdAptQpAtkjvnsyZ/itLiC7JOWN7CCupWjh1QIJmFn4Y3AMRj3SGuDbyNCo1TJ+lewFVTCiKz6k6S9o0SJN59ffIAsoizMXAIFn1OgUkXBUoKjd9Mkgxb1qJ9e9zUVliL/cxXHAVSDp9Qim/hEk2U= ARC-Authentication-Results:i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com; spf=pass smtp.mailfrom=collabora.com; dkim=pass (1024-bit key) header.d=collabora.com header.i=nicolas.frattaroli@collabora.com header.b=cXk0gvvg; arc=pass smtp.client-ip=136.143.188.112 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=collabora.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=collabora.com header.i=nicolas.frattaroli@collabora.com header.b="cXk0gvvg" ARC-Seal: i=1; a=rsa-sha256; t=1781808883; cv=none; d=zohomail.com; s=zohoarc; b=hgumXCdp+rH+fMYgh+RexDU0R1dIdf0T/afRh+Mk+VggjOJ4S5IqWe/H2cm6RLt3Xq+rMNeq2x6nKYvJq7ekzGRQ2ar9mqSx/r5RT55FswZL1g2+aLgfB6Utjy910DdnvebbC1ka04Dm8FfkVpH/vHnMpFnCdRvPdGS2uqgxWSE= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1781808883; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:MIME-Version:Message-ID:References:Subject:Subject:To:To:Message-Id:Reply-To; bh=5lPa3263Z6NZQL7A3UYYSb30v7v5maPBCkXVUvwi8H8=; b=QWZjYrUvJgHXXx4sYH8+wrHNneJ0pZTZb+Y3jazCj15QBg7zFkdjAaG0tCn3xafk/kSZM7vDAH/RlLkQu3fMICcrqH13mXU0sOb8ZMSHIV8KWgu7mp/vHML2ctaNW2OYjELJPOHqsWvi4K0iJ0aqUVx0UFI1CKWSdPn/a4CWpL8= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass header.i=collabora.com; spf=pass smtp.mailfrom=nicolas.frattaroli@collabora.com; dmarc=pass header.from= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1781808883; s=zohomail; d=collabora.com; i=nicolas.frattaroli@collabora.com; h=From:From:Date:Date:Subject:Subject:MIME-Version:Content-Type:Content-Transfer-Encoding:Message-Id:Message-Id:References:In-Reply-To:To:To:Cc:Cc:Reply-To; bh=5lPa3263Z6NZQL7A3UYYSb30v7v5maPBCkXVUvwi8H8=; b=cXk0gvvgQJdBg//Dok9QdRc3GU42iUr9dyPySHxM3bvbo+j96HK3UD8OzxUVgOO1 lJsA5y7UMGDHFJyHYi4OIBysKz3UZDaXENQQLQQZeyj19rDgrW+74rzCseWkX0HaD0F cWM6z5L5wXv+OXMQL2/yfdsmOFpWbaFccH5K3vao= Received: by mx.zohomail.com with SMTPS id 1781808881895879.5653375861743; Thu, 18 Jun 2026 11:54:41 -0700 (PDT) From: Nicolas Frattaroli Date: Thu, 18 Jun 2026 20:54:23 +0200 Subject: [PATCH v2 1/2] drm/display: hdmi: Only allow BPC values of 8, 10, 12 and 16 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: 7bit Message-Id: <20260618-hdmi-max-bpc-fix-v2-1-7b67f489b0cc@collabora.com> References: <20260618-hdmi-max-bpc-fix-v2-0-7b67f489b0cc@collabora.com> In-Reply-To: <20260618-hdmi-max-bpc-fix-v2-0-7b67f489b0cc@collabora.com> To: Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Dmitry Baryshkov Cc: dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, kernel@collabora.com, Nicolas Frattaroli X-Mailer: b4 0.15.2 As per the comment in sink_supports_format_bpc(), CTA-861-F defines that only bits-per-channel values of 8, 10, 12 and 16 are allowed for HDMI. Allowing more than this has surprising consequences for the atomic check phase. The HDMI state helpers may accidentally conclude that a sink supports 11bpc if a caller asks for it. Fix this by exiting early if the bpc value doesn't match one of those given in the standard. Fixes: 26ff1c38fc29 ("drm/connector: hdmi: Compute bpc and format automatically") Signed-off-by: Nicolas Frattaroli --- drivers/gpu/drm/display/drm_hdmi_state_helper.c | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/drivers/gpu/drm/display/drm_hdmi_state_helper.c b/drivers/gpu/drm/display/drm_hdmi_state_helper.c index ce17eeefc2da..0bf6102244f7 100644 --- a/drivers/gpu/drm/display/drm_hdmi_state_helper.c +++ b/drivers/gpu/drm/display/drm_hdmi_state_helper.c @@ -420,6 +420,16 @@ sink_supports_format_bpc(const struct drm_connector *connector, return false; } + switch (bpc) { + case 8: + case 10: + case 12: + case 16: + break; + default: + return false; + } + if (!info->is_hdmi && (format != DRM_OUTPUT_COLOR_FORMAT_RGB444 || bpc != 8)) { drm_dbg_kms(dev, "DVI Monitors require an RGB output at 8 bpc\n"); -- 2.54.0