* [PATCH 1/3] drm/tegra: Remove pixel clock rounding
@ 2014-05-23 3:32 Stéphane Marchesin
2014-05-23 3:32 ` [PATCH 2/3] drm/tegra: Fix copy/paste error Stéphane Marchesin
2014-05-23 3:32 ` [PATCH 3/3] drm/tegra: Fix sequencing Stéphane Marchesin
0 siblings, 2 replies; 4+ messages in thread
From: Stéphane Marchesin @ 2014-05-23 3:32 UTC (permalink / raw)
To: dri-devel; +Cc: Stéphane Marchesin, treding
The code currently rounds up the clock to the next MHZ, which is
rounding up a 69.5MHz clock to 70MHz on my machine. This in turn
prevents the display from syncing. Removing this rounding fixes eDP
for me.
Signed-off-by: Stéphane Marchesin <marcheu@chromium.org>
---
drivers/gpu/drm/tegra/sor.c | 7 ++-----
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/tegra/sor.c b/drivers/gpu/drm/tegra/sor.c
index 49ef572..75b8690 100644
--- a/drivers/gpu/drm/tegra/sor.c
+++ b/drivers/gpu/drm/tegra/sor.c
@@ -863,19 +863,16 @@ static int tegra_output_sor_setup_clock(struct tegra_output *output,
struct tegra_sor *sor = to_sor(output);
int err;
- /* round to next MHz */
- pclk = DIV_ROUND_UP(pclk / 2, 1000000) * 1000000;
-
err = clk_set_parent(clk, sor->clk_parent);
if (err < 0) {
dev_err(sor->dev, "failed to set parent clock: %d\n", err);
return err;
}
- err = clk_set_rate(sor->clk_parent, pclk);
+ err = clk_set_rate(sor->clk_parent, pclk / 2);
if (err < 0) {
dev_err(sor->dev, "failed to set base clock rate to %lu Hz\n",
- pclk * 2);
+ pclk);
return err;
}
--
1.9.1.423.g4596e3a
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 4+ messages in thread* [PATCH 2/3] drm/tegra: Fix copy/paste error
2014-05-23 3:32 [PATCH 1/3] drm/tegra: Remove pixel clock rounding Stéphane Marchesin
@ 2014-05-23 3:32 ` Stéphane Marchesin
2014-05-23 3:32 ` [PATCH 3/3] drm/tegra: Fix sequencing Stéphane Marchesin
1 sibling, 0 replies; 4+ messages in thread
From: Stéphane Marchesin @ 2014-05-23 3:32 UTC (permalink / raw)
To: dri-devel; +Cc: Stéphane Marchesin, treding
The comment above mentions link A/B but this isn't what the code does,
so let's fix that.
Signed-off-by: Stéphane Marchesin <marcheu@chromium.org>
---
drivers/gpu/drm/tegra/sor.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/tegra/sor.c b/drivers/gpu/drm/tegra/sor.c
index 75b8690..43709ee 100644
--- a/drivers/gpu/drm/tegra/sor.c
+++ b/drivers/gpu/drm/tegra/sor.c
@@ -624,7 +624,7 @@ static int tegra_output_sor_enable(struct tegra_output *output)
tegra_sor_writel(sor, 0x00000001, SOR_HEAD_STATE_5(0));
/* CSTM (LVDS, link A/B, upper) */
- value = SOR_CSTM_LVDS | SOR_CSTM_LINK_ACT_B | SOR_CSTM_LINK_ACT_B |
+ value = SOR_CSTM_LVDS | SOR_CSTM_LINK_ACT_A | SOR_CSTM_LINK_ACT_B |
SOR_CSTM_UPPER;
tegra_sor_writel(sor, value, SOR_CSTM);
--
1.9.1.423.g4596e3a
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH 3/3] drm/tegra: Fix sequencing
2014-05-23 3:32 [PATCH 1/3] drm/tegra: Remove pixel clock rounding Stéphane Marchesin
2014-05-23 3:32 ` [PATCH 2/3] drm/tegra: Fix copy/paste error Stéphane Marchesin
@ 2014-05-23 3:32 ` Stéphane Marchesin
2014-06-05 9:20 ` Thierry Reding
1 sibling, 1 reply; 4+ messages in thread
From: Stéphane Marchesin @ 2014-05-23 3:32 UTC (permalink / raw)
To: dri-devel; +Cc: Stéphane Marchesin, treding
I suspect this is a copy/paste error.
Signed-off-by: Stéphane Marchesin <marcheu@chromium.org>
---
drivers/gpu/drm/tegra/sor.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/tegra/sor.c b/drivers/gpu/drm/tegra/sor.c
index 43709ee..23fb9b1 100644
--- a/drivers/gpu/drm/tegra/sor.c
+++ b/drivers/gpu/drm/tegra/sor.c
@@ -740,7 +740,7 @@ static int tegra_sor_power_down(struct tegra_sor *sor)
tegra_sor_writel(sor, value, SOR_DP_PADCTL_0);
/* stop lane sequencer */
- value = SOR_LANE_SEQ_CTL_TRIGGER | SOR_LANE_SEQ_CTL_SEQUENCE_DOWN |
+ value = SOR_LANE_SEQ_CTL_TRIGGER | SOR_LANE_SEQ_CTL_SEQUENCE_UP |
SOR_LANE_SEQ_CTL_POWER_STATE_DOWN;
tegra_sor_writel(sor, value, SOR_LANE_SEQ_CTL);
--
1.9.1.423.g4596e3a
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 3/3] drm/tegra: Fix sequencing
2014-05-23 3:32 ` [PATCH 3/3] drm/tegra: Fix sequencing Stéphane Marchesin
@ 2014-06-05 9:20 ` Thierry Reding
0 siblings, 0 replies; 4+ messages in thread
From: Thierry Reding @ 2014-06-05 9:20 UTC (permalink / raw)
To: Stéphane Marchesin; +Cc: treding, dri-devel
[-- Attachment #1.1: Type: text/plain, Size: 691 bytes --]
On Thu, May 22, 2014 at 08:32:48PM -0700, Stéphane Marchesin wrote:
> I suspect this is a copy/paste error.
>
> Signed-off-by: Stéphane Marchesin <marcheu@chromium.org>
> ---
> drivers/gpu/drm/tegra/sor.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Would you mind if I extend the commit message with something like the
following:
Lanes are powered up in decreasing order, so they should be
powered down in increasing order.
Also does this fix any issues in particular that you were seeing or is
it just an "oddity" that you noticed during review? At least on Venice2
it doesn't seem to matter in what order the lanes are powered up or
down.
Thierry
[-- Attachment #1.2: Type: application/pgp-signature, Size: 836 bytes --]
[-- Attachment #2: Type: text/plain, Size: 159 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2014-06-05 9:23 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-05-23 3:32 [PATCH 1/3] drm/tegra: Remove pixel clock rounding Stéphane Marchesin
2014-05-23 3:32 ` [PATCH 2/3] drm/tegra: Fix copy/paste error Stéphane Marchesin
2014-05-23 3:32 ` [PATCH 3/3] drm/tegra: Fix sequencing Stéphane Marchesin
2014-06-05 9:20 ` Thierry Reding
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.