From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from gloria.sntech.de (gloria.sntech.de [185.11.138.130]) (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 924243BBA0F for ; Mon, 27 Apr 2026 19:10:03 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.11.138.130 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777317007; cv=none; b=XQURO9O8DPJHhz/3oB1RWLuW1yc5NyigeWr2Ira5SVL0GiytNaS90G7VqZ1gLFEoCfxcZHGiwr8mRkPWpmp3FBLPLUhJpfQiWw7VfsByc7MldbjWlG237JO0xUkzEeLct3XoRrSDzAaC5VWW/FdSSRdtfQoWWiR6O6GOjDgbeYI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777317007; c=relaxed/simple; bh=1fg5MOLosMQgyohj3J3j1cqdfRQ937B+FWqdHT3fFVU=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=PwlEIyll1H6DyFS5pukokN6N9eFT8L8vxmpEheZ+ojRLjKtHSwKr2u71Ixsmm0PdJGZxXG1G6R/RplbbhqI7rssLmg1UlFqGUGk7boRMUWiGXip0nSZ9RhCDAiQffkH1tTqOSQKXILuwPKhSX1uHT1PZYbK7I1imxN9JYoLlN5c= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=sntech.de; spf=pass smtp.mailfrom=sntech.de; dkim=pass (2048-bit key) header.d=sntech.de header.i=@sntech.de header.b=VaiZ0EuM; arc=none smtp.client-ip=185.11.138.130 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=sntech.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=sntech.de Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=sntech.de header.i=@sntech.de header.b="VaiZ0EuM" DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sntech.de; s=gloria202408; h=Content-Type:Content-Transfer-Encoding:MIME-Version: References:In-Reply-To:Message-ID:Date:Subject:Cc:To:From:Reply-To; bh=Rb4LC4RoKjMwkAoMYkZMORsdxxUBkkBcuhhkQcQ9qHU=; b=VaiZ0EuMs/V/UpaUqhIxQT4kOQ m2fmNRxLnBGy786PBF2Z0XkvBMAdhyEjrVoKui6GQ8giNSHAgxOJ65AnNm08rjG9ZcemAVGmlbYo/ x0vLZwJUF06BH6iWuyebP6piC19SdZ3GLQlXRGSEATGAP7cM7uguCYD7l8bg82hup8QcS5AT6NLAK O1rudfGXB/blSVVheseL8grFMm70H+C1ElnAg1J4VOHI1ueOm2U+FYROlQkL8Vzg77Rlg4y8VwKlq IekUEQ9uAOQ5SFiBYkhiDulThCYOT1lPRHpbX2efjvngHBt09pilTEDB+hXdcnKs1SbNOdXWus5zo 4MNyAfbg==; From: Heiko Stuebner To: Andrzej Hajda , Neil Armstrong , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Sandy Huang , Andy Yan , Cristian Ciocaltea Cc: kernel@collabora.com, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, Diederik de Haas , Maud Spierings Subject: Re: [PATCH v5 05/10] drm/bridge: dw-hdmi-qp: Add HDMI 2.0 SCDC scrambling and high TMDS clock ratio support Date: Mon, 27 Apr 2026 21:09:09 +0200 Message-ID: <890451008.0ifERbkFSE@phil> In-Reply-To: References: <20260426-dw-hdmi-qp-scramb-v5-0-d778e70c317b@collabora.com> <3826043.PYKUYFuaPT@phil> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Am Montag, 27. April 2026, 18:39:44 Mitteleurop=C3=A4ische Sommerzeit schri= eb Cristian Ciocaltea: > Hello Heiko, >=20 > Thanks for the quick reviews! >=20 > On 4/27/26 1:49 PM, Heiko Stuebner wrote: > > Am Sonntag, 26. April 2026, 02:20:17 Mitteleurop=C3=A4ische Sommerzeit = schrieb Cristian Ciocaltea: > >> Enable HDMI 2.0 display modes (e.g. 4K@60Hz) by adding SCDC management > >> for the high TMDS clock ratio and scrambling, required when the TMDS > >> character rate exceeds the 340 MHz HDMI 1.4b limit. > >> > >> A periodic work item monitors the sink's scrambling status to recover > >> from sink-side resets. On hotplug detect, if SCDC scrambling state is > >> out of sync with the driver, trigger a CRTC reset to re-establish the > >> link. > >> > >> Reject modes requiring TMDS rates above 600 MHz, as those fall in the > >> HDMI 2.1 FRL domain which is not supported. In no_hpd configurations, > >> further restrict to 340 MHz since SCDC requires a connected sink. > >> > >> Tested-by: Diederik de Haas > >> Tested-by: Maud Spierings > >> Signed-off-by: Cristian Ciocaltea > >> --- > >> drivers/gpu/drm/bridge/synopsys/dw-hdmi-qp.c | 188 ++++++++++++++++++= ++++++--- > >> 1 file changed, 172 insertions(+), 16 deletions(-) > >> > >> diff --git a/drivers/gpu/drm/bridge/synopsys/dw-hdmi-qp.c b/drivers/gp= u/drm/bridge/synopsys/dw-hdmi-qp.c > >> index d649a1cf07f5..c482a8e7da25 100644 > >> --- a/drivers/gpu/drm/bridge/synopsys/dw-hdmi-qp.c > >> +++ b/drivers/gpu/drm/bridge/synopsys/dw-hdmi-qp.c > >> @@ -2,6 +2,7 @@ > >> /* > >> * Copyright (c) 2021-2022 Rockchip Electronics Co., Ltd. > >> * Copyright (c) 2024 Collabora Ltd. > >> + * Copyright (c) 2025 Amazon.com, Inc. or its affiliates. > >=20 > > I have no real clue about the inner workings of hdmi, but that line > > does make me curious, because no part of the patch authorship does > > mention Amazon ;-) . >=20 > That was intentional: although the code was written by Collabora, the cop= yright > belongs to Amazon. They selected a nice chip then :-D > > Also I think "or it's affiliates" might be way too broad, because this > > can be essentially everyone. So noone in the future will know who to > > ask on copyright questions. > >=20 > > Similarly when _somebody_ comes forward with "I hold a copyright on thi= s" > > no-one could verify this claim as well. > >=20 > > I somehow expect _one_ entity being specified, not possibly hundreds. >=20 > Hmm, I followed the wording already used elsewhere in the kernel tree: >=20 > $ git grep 'Copyright.*Amazon.*affiliates' | wc -l > 93 >=20 > There appears to be only a single instance that omits =E2=80=9Caffiliates= =E2=80=9D: >=20 > $ git grep 'Copyright.*Amazon' | grep -v affiliates > drivers/power/supply/max14656_charger_detector.c: * Copyright (C) Amazon = Technologies Inc. All rights reserved. >=20 > Which was added nearly ten years ago, so I=E2=80=99m not sure it makes se= nse to adopt > that form now. >=20 > Actually, this seems to be fairly common among other vendors as well: >=20 > $ git grep 'Copyright.*affiliates' | wc -l > 1014 okay then ... if that is already used a lot, and nobody else complained then it should be fine I guess :-) . It just sounded strange to me when reading that earlier. Thanks a lot for digging those stats up Heiko