From: Alexander Stein <alexander.stein@ew.tq-group.com>
To: Andrzej Hajda <andrzej.hajda@intel.com>,
Neil Armstrong <neil.armstrong@linaro.org>,
Robert Foss <robert.foss@linaro.org>,
Laurent Pinchart <Laurent.pinchart@ideasonboard.com>,
Jonas Karlman <jonas@kwiboo.se>,
Jernej Skrabec <jernej.skrabec@gmail.com>,
David Airlie <airlied@gmail.com>, Daniel Vetter <daniel@ffwll.ch>
Cc: dri-devel@lists.freedesktop.org
Subject: Re: [PATCH 1/1] drm: bridge: ldb: Warn if LDB clock does not match requested link frequency
Date: Thu, 19 Jan 2023 08:38:23 +0100 [thread overview]
Message-ID: <837062655.0ifERbkFSE@steina-w> (raw)
In-Reply-To: <20221208065538.1753666-1-alexander.stein@ew.tq-group.com>
Hi everyone,
Am Donnerstag, 8. Dezember 2022, 07:55:38 CET schrieb Alexander Stein:
> The LDB clock needs to be exactly 7-times the pixel clock used by the
> display.
Any feedback on this?
Thanks
Alexander
> Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
> ---
> i.MX8MP has a dedicated LDB clock which defines the actual LVDS link
> frequency. This has to be (exactly) the 7-time of the pixel clock.
> Although the clock min/max range is available, panel-simple does not (yet)
> use the range to find a (perfect) frequency which can be used down the
> chain, which is also in range.
> Depending on the pixel clock the exact multiple might not be configured.
> Raise a warning if there is a mismatch, which might cause an invalid display
> image.
>
> drivers/gpu/drm/bridge/fsl-ldb.c | 23 +++++++++++++++++++----
> 1 file changed, 19 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/bridge/fsl-ldb.c
> b/drivers/gpu/drm/bridge/fsl-ldb.c index f9e0f8d992680..9bcba8fc57e74
> 100644
> --- a/drivers/gpu/drm/bridge/fsl-ldb.c
> +++ b/drivers/gpu/drm/bridge/fsl-ldb.c
> @@ -66,6 +66,14 @@ static inline struct fsl_ldb *to_fsl_ldb(struct
> drm_bridge *bridge) return container_of(bridge, struct fsl_ldb, bridge);
> }
>
> +static unsigned long fsl_ldb_link_frequency(struct fsl_ldb *fsl_ldb, int
> clock) +{
> + if (fsl_ldb->lvds_dual_link)
> + return clock * 3500;
> + else
> + return clock * 7000;
> +}
> +
> static int fsl_ldb_attach(struct drm_bridge *bridge,
> enum drm_bridge_attach_flags flags)
> {
> @@ -85,6 +93,8 @@ static void fsl_ldb_atomic_enable(struct drm_bridge
> *bridge, const struct drm_display_mode *mode;
> struct drm_connector *connector;
> struct drm_crtc *crtc;
> + unsigned long configured_link_freq;
> + unsigned long requested_link_freq;
> bool lvds_format_24bpp;
> bool lvds_format_jeida;
> u32 reg;
> @@ -128,10 +138,15 @@ static void fsl_ldb_atomic_enable(struct drm_bridge
> *bridge, crtc_state = drm_atomic_get_new_crtc_state(state, crtc);
> mode = &crtc_state->adjusted_mode;
>
> - if (fsl_ldb->lvds_dual_link)
> - clk_set_rate(fsl_ldb->clk, mode->clock * 3500);
> - else
> - clk_set_rate(fsl_ldb->clk, mode->clock * 7000);
> + requested_link_freq = fsl_ldb_link_frequency(fsl_ldb, mode->clock);
> + clk_set_rate(fsl_ldb->clk, requested_link_freq);
> +
> + configured_link_freq = clk_get_rate(fsl_ldb->clk);
> + if (configured_link_freq != requested_link_freq)
> + dev_warn(fsl_ldb->dev, "Configured LDB clock (%lu Hz) does
not match
> requested LVDS clock: %lu Hz", +
configured_link_freq,
> + requested_link_freq);
> +
> clk_prepare_enable(fsl_ldb->clk);
>
> /* Program LDB_CTRL */
next prev parent reply other threads:[~2023-01-19 7:38 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-12-08 6:55 [PATCH 1/1] drm: bridge: ldb: Warn if LDB clock does not match requested link frequency Alexander Stein
2023-01-19 7:38 ` Alexander Stein [this message]
2023-01-19 7:49 ` Neil Armstrong
2023-01-19 7:53 ` Neil Armstrong
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=837062655.0ifERbkFSE@steina-w \
--to=alexander.stein@ew.tq-group.com \
--cc=Laurent.pinchart@ideasonboard.com \
--cc=airlied@gmail.com \
--cc=andrzej.hajda@intel.com \
--cc=daniel@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
--cc=jernej.skrabec@gmail.com \
--cc=jonas@kwiboo.se \
--cc=neil.armstrong@linaro.org \
--cc=robert.foss@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.