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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (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 BCECAC2BD09 for ; Mon, 24 Jun 2024 09:19:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Type: Content-Transfer-Encoding:MIME-Version:References:In-Reply-To:Message-ID:Date :Subject:Cc:To:From:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=uJW3zLdz7y8+GEGkB3RFqz/8yBoXrC/7C7PphqWWonM=; b=yxVTpcwuziGHw8mk9h6DIevYxs 2/E3Grl2ekWmkUg5a7Mf4PXPSPBlkQG0edeqtyAtgMAoHQxsdhxErIZ1Lf8o4yHigWRt6tIcCs37n KZcEod1c3oSWjavqhjK/AMPNpooGGF/5//1AfqG7clNRyuzn7Z/mLALw8LeGrmwu1/9Samd8cWp0i PUDh3IPdx0tqZJCvkA1uAAXCb0lD8HqRMPTrAUDevNtKfcSvLQZi6BgAt5nQRvXmKKsHU3bStNh+P ChKW4UnUeKOmNW1cTTn9Jn15xeKLi485PK7Je9hr+MsfeUXv6xNjErWZk98IuHIOdOc8F6V1+IbJh jyRdMfOA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sLfrV-0000000GE76-3jKr; Mon, 24 Jun 2024 09:19:45 +0000 Received: from mx1.tq-group.com ([93.104.207.81]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sLfrP-0000000GE5N-3Rtm for linux-arm-kernel@lists.infradead.org; Mon, 24 Jun 2024 09:19:42 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tq-group.com; i=@tq-group.com; q=dns/txt; s=key1; t=1719220779; x=1750756779; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=uJW3zLdz7y8+GEGkB3RFqz/8yBoXrC/7C7PphqWWonM=; b=BTebwJMQpQa2PqGydvSIS3oecPNQMPMHvOewm4mEhCY+Os8Ah/6dfpnN DXyb3imFdITUugsYNhk0sDG39ajkBZ7LlTdh7UubZV1cLNieXufnX1vnL WPStJtHKX+PQqyl0LWxJD84jk2ncsof5fpsz6OeeOtixlJXNWahwaueiC jweuzqcTex1LwYnR/gnJx6rq0rAddn5xGUJB2Ea8/pIxvorFctvR3gw7C +5L2Nhlb2d5RINhDhjUsg59QJZqi5UAKziAF05Lct4JW0M29SCIaBnSQG vVNxzZfxtOS9O6RJDnZabLP/Mja+Ta9ah75SfVSSsIbzo7y4wBFtE++3H Q==; X-CSE-ConnectionGUID: BkJsueitROW81l3R/C86nw== X-CSE-MsgGUID: J+yHdkulRb+CPQUvJuzl1Q== X-IronPort-AV: E=Sophos;i="6.08,261,1712613600"; d="scan'208";a="37547624" Received: from vmailcow01.tq-net.de ([10.150.86.48]) by mx1.tq-group.com with ESMTP; 24 Jun 2024 11:19:35 +0200 Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id 29F94165926; Mon, 24 Jun 2024 11:19:29 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ew.tq-group.com; s=dkim; t=1719220770; h=from:subject:date:message-id:to:cc:mime-version:content-type: content-transfer-encoding:in-reply-to:references; bh=uJW3zLdz7y8+GEGkB3RFqz/8yBoXrC/7C7PphqWWonM=; b=mk0rGT56lQculyO1b1QLG4ZJ+l8y6ZfvbcMypcZZz5GIc5+yGTh6KVJMQ8XqA8d+DzA8sx Qxwpsk6RsbOu9tf42+v76lCnXAmBeu3gdJmKSqIGuIU2IIMhshqabxw1yJiooilm8saFfy iVAhsz02HQg8TqN1V0VblkWTEMJ829EI3KG2/Qv9HGa8yOiGJDyWnblaRbVBGeKDzHxNVr G/yl6p7mTd0Izeil+uXyxSjqE0gQB75Dc7ehw2WmcUrtMWPEaNU9oFep0VxGRAgWxcdOo5 zfTCcmNysdQKRbuPvvadCNkXIVRTEiefKLSuyROTNohGMprwzONlnxLVWuNjLQ== From: Alexander Stein To: dri-devel@lists.freedesktop.org Cc: Marek Vasut , Daniel Vetter , David Airlie , Fabio Estevam , Lucas Stach , "Lukas F . Hartmann" , Maarten Lankhorst , Maxime Ripard , Pengutronix Kernel Team , Sascha Hauer , Shawn Guo , Stefan Agner , Thomas Zimmermann , imx@lists.linux.dev, kernel@dh-electronics.com, linux-arm-kernel@lists.infradead.org, Marek Vasut Subject: Re: [PATCH] drm: lcdif: Use adjusted_mode .clock instead of .crtc_clock Date: Mon, 24 Jun 2024 11:19:33 +0200 Message-ID: <1897634.CQOukoFCf9@steina-w> Organization: TQ-Systems GmbH In-Reply-To: <20240531202813.277109-1-marex@denx.de> References: <20240531202813.277109-1-marex@denx.de> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="iso-8859-1" X-Last-TLS-Session-Version: TLSv1.3 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240624_021940_520210_DED135CD X-CRM114-Status: GOOD ( 14.93 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Am Freitag, 31. Mai 2024, 22:27:21 CEST schrieb Marek Vasut: > In case an upstream bridge modified the required clock frequency > in its .atomic_check callback by setting adjusted_mode.clock , > make sure that clock frequency is generated by the LCDIFv3 block. >=20 > This is useful e.g. when LCDIFv3 feeds DSIM which feeds TC358767 > with (e)DP output, where the TC358767 expects precise timing on > its input side, the precise timing must be generated by the LCDIF. >=20 > Signed-off-by: Marek Vasut With the other rc358767 patches in place, this does the trick. Reviewed-by: Alexander Stein > --- > Cc: Daniel Vetter > Cc: David Airlie > Cc: Fabio Estevam > Cc: Lucas Stach > Cc: Lukas F. Hartmann > Cc: Maarten Lankhorst > Cc: Maxime Ripard > Cc: Pengutronix Kernel Team > Cc: Sascha Hauer > Cc: Shawn Guo > Cc: Stefan Agner > Cc: Thomas Zimmermann > Cc: dri-devel@lists.freedesktop.org > Cc: imx@lists.linux.dev > Cc: kernel@dh-electronics.com > Cc: linux-arm-kernel@lists.infradead.org > --- > drivers/gpu/drm/mxsfb/lcdif_kms.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) >=20 > diff --git a/drivers/gpu/drm/mxsfb/lcdif_kms.c b/drivers/gpu/drm/mxsfb/lc= dif_kms.c > index 2541d2de4e45f..dbd42cc1da87f 100644 > --- a/drivers/gpu/drm/mxsfb/lcdif_kms.c > +++ b/drivers/gpu/drm/mxsfb/lcdif_kms.c > @@ -407,8 +407,7 @@ static void lcdif_crtc_mode_set_nofb(struct drm_crtc_= state *crtc_state, > struct drm_display_mode *m =3D &crtc_state->adjusted_mode; > =20 > DRM_DEV_DEBUG_DRIVER(drm->dev, "Pixel clock: %dkHz (actual: %dkHz)\n", > - m->crtc_clock, > - (int)(clk_get_rate(lcdif->clk) / 1000)); > + m->clock, (int)(clk_get_rate(lcdif->clk) / 1000)); > DRM_DEV_DEBUG_DRIVER(drm->dev, "Bridge bus_flags: 0x%08X\n", > lcdif_crtc_state->bus_flags); > DRM_DEV_DEBUG_DRIVER(drm->dev, "Mode flags: 0x%08X\n", m->flags); > @@ -538,7 +537,7 @@ static void lcdif_crtc_atomic_enable(struct drm_crtc = *crtc, > struct drm_device *drm =3D lcdif->drm; > dma_addr_t paddr; > =20 > - clk_set_rate(lcdif->clk, m->crtc_clock * 1000); > + clk_set_rate(lcdif->clk, m->clock * 1000); > =20 > pm_runtime_get_sync(drm->dev); > =20 >=20 =2D-=20 TQ-Systems GmbH | M=FChlstra=DFe 2, Gut Delling | 82229 Seefeld, Germany Amtsgericht M=FCnchen, HRB 105018 Gesch=E4ftsf=FChrer: Detlef Schneider, R=FCdiger Stahl, Stefan Schneider http://www.tq-group.com/