All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Timur Kristóf" <timur.kristof@gmail.com>
To: Alex Deucher <alexdeucher@gmail.com>, Alex Hung <alex.hung@amd.com>
Cc: amd-gfx@lists.freedesktop.org,
	Harry Wentland <Harry.Wentland@amd.com>,
	Wenjing Liu <wenjing.liu@amd.com>,
	alexander.deucher@amd.com, siqueira@igalia.com
Subject: Re: [PATCH 0/5] drm/amd/display: DC analog connector fixes
Date: Tue, 16 Dec 2025 18:26:31 -0600	[thread overview]
Message-ID: <5017406.vXUDI8C0e8@timur-max> (raw)
In-Reply-To: <96ac728e-1d58-46e8-a4a4-b722023f6a45@amd.com>

> >> 
> >> Did you have an update for this series, or are they ready to be applied?
> >> 
> >> Alex
> > 
> > Hi,
> > 
> > I re-reviewed my patches in this series again and I realized that my
> > comment was mistaken (I confused link encoder with stream encoder). So
> > the patches should be good as-is.
> > 
> > However, in the meantime Mauro got back to me this weekend, and told me
> > that he determined that the patch "Setup DAC encoder before using it" not
> > only doesn't fix it, but regresses the analog connector on his HD 7790
> > further. Before this patch, he got a black screen on boot, but the DAC
> > started working after suspend/resume. After this patch, it no longer
> > works for him even after suspend/resume.
> > 
> > What do you think is the way to proceed?
> > 
> > A) Merge all patches but that one. I'll submit a corrected patch once we
> > figure out the HD 7790.
> > B) Merge the whole series as-is and I can submit a fix once we figured out
> > what is going on with the HD 7790.
> 
> Plan A sounds better. Let's not merge a patch with a known issue. I will
> also share test results by the end of the week.
> 

Hi Alex,

Thank you, that sounds like a good plan to me.

Timur




  reply	other threads:[~2025-12-17  0:26 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-12-06  2:31 [PATCH 0/5] drm/amd/display: DC analog connector fixes Timur Kristóf
2025-12-06  2:31 ` [PATCH 1/5] drm/amd/display: Pass proper DAC encoder ID to VBIOS Timur Kristóf
2025-12-06  2:31 ` [PATCH 2/5] drm/amd/display: Correct color depth for SelectCRTC_Source Timur Kristóf
2025-12-06  2:31 ` [PATCH 3/5] drm/amd/display: Add missing encoder setup to DACnEncoderControl Timur Kristóf
2025-12-06  2:31 ` [PATCH 4/5] drm/amd/display: Setup DAC encoder before using it Timur Kristóf
2025-12-08 15:21   ` timur.kristof
2025-12-08 15:27     ` Alex Deucher
2025-12-08 15:32       ` timur.kristof
2025-12-08 15:46         ` Alex Deucher
2025-12-06  2:31 ` [PATCH 5/5] drm/amd/display: Don't repeat DAC load detection Timur Kristóf
2025-12-16 15:27 ` [PATCH 0/5] drm/amd/display: DC analog connector fixes Alex Deucher
2025-12-16 17:15   ` Alex Hung
2025-12-16 18:21   ` Timur Kristóf
2025-12-16 19:10     ` Alex Hung
2025-12-17  0:26       ` Timur Kristóf [this message]
2025-12-22  7:02         ` Alex Hung

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=5017406.vXUDI8C0e8@timur-max \
    --to=timur.kristof@gmail.com \
    --cc=Harry.Wentland@amd.com \
    --cc=alex.hung@amd.com \
    --cc=alexander.deucher@amd.com \
    --cc=alexdeucher@gmail.com \
    --cc=amd-gfx@lists.freedesktop.org \
    --cc=siqueira@igalia.com \
    --cc=wenjing.liu@amd.com \
    /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.