[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <2be9721e-62f3-41cc-83a8-5189b750a655@collabora.com>
Date: Tue, 17 Dec 2024 22:52:23 +0200
From: Cristian Ciocaltea <cristian.ciocaltea@...labora.com>
To: Derek Foreman <derek.foreman@...labora.com>,
Algea Cao <algea.cao@...k-chips.com>, Sandy Huang <hjc@...k-chips.com>,
Heiko Stübner <heiko@...ech.de>,
Andy Yan <andy.yan@...k-chips.com>,
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Maxime Ripard <mripard@...nel.org>, Thomas Zimmermann <tzimmermann@...e.de>,
David Airlie <airlied@...il.com>, Simona Vetter <simona@...ll.ch>
Cc: kernel@...labora.com, dri-devel@...ts.freedesktop.org,
linux-arm-kernel@...ts.infradead.org, linux-rockchip@...ts.infradead.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] drm/rockchip: Don't change hdmi reference clock rate
On 12/17/24 10:17 PM, Derek Foreman wrote:
> The code that changes hdmi->ref_clk was accidentally copied from
> downstream code that sets a different clock. We don't actually
> want to set any clock here at all.
>
> Setting this clock incorrectly leads to incorrect timings for
> DDC, CEC, and HDCP signal generation.
>
> No Fixes listed, as the theoretical timing error in DDC appears to
> still be within tolerances and harmless - and HDCP and CEC are not
> yet supported.
>
> Signed-off-by: Derek Foreman <derek.foreman@...labora.com>
> ---
> drivers/gpu/drm/rockchip/dw_hdmi_qp-rockchip.c | 13 -------------
> 1 file changed, 13 deletions(-)
Reviewed-by: Cristian Ciocaltea <cristian.ciocaltea@...labora.com>
Powered by blists - more mailing lists