AMD-GFX Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Jiri Slaby <jirislaby@kernel.org>
To: "Li, Roman" <Roman.Li@amd.com>,
	"amd-gfx@lists.freedesktop.org" <amd-gfx@lists.freedesktop.org>
Cc: "Wentland, Harry" <Harry.Wentland@amd.com>,
	"Li, Sun peng (Leo)" <Sunpeng.Li@amd.com>,
	"Siqueira, Rodrigo" <Rodrigo.Siqueira@amd.com>,
	"Pillai, Aurabindo" <Aurabindo.Pillai@amd.com>,
	"Lin, Wayne" <Wayne.Lin@amd.com>,
	"Gutierrez, Agustin" <Agustin.Gutierrez@amd.com>,
	"Chung, ChiaHsuan (Tom)" <ChiaHsuan.Chung@amd.com>,
	"Zuo, Jerry" <Jerry.Zuo@amd.com>,
	"Mohamed, Zaeem" <Zaeem.Mohamed@amd.com>,
	"Limonciello, Mario" <Mario.Limonciello@amd.com>,
	"Deucher, Alexander" <Alexander.Deucher@amd.com>,
	"stable@vger.kernel.org" <stable@vger.kernel.org>
Subject: Re: [PATCH 12/13] drm/amd/display: Fix a typo in revert commit
Date: Tue, 20 Aug 2024 06:49:02 +0200	[thread overview]
Message-ID: <0d12eb50-c51b-496e-a931-9ce8fb6a1455@kernel.org> (raw)
In-Reply-To: <CY8PR12MB819387431D6D6E754929ECB9898C2@CY8PR12MB8193.namprd12.prod.outlook.com>

On 19. 08. 24, 16:29, Li, Roman wrote:
> Thank you, Jiri, for your feedback.
> I've dropped this patch from DC v.3.2.297.
> We will  follow-up on this separately and merge it after you do confirm the issue you reported is fixed.

The patch is all fine and very welcome to be upstream as soon as 
possible. With this patch, it works as expected.

But the process is broken. Your and Fangzhi Zuo's send-email setup to 
the least.


BTW you write in the commit log:
Fixes: 4b6564cb120c ("drm/amd/display: Fix MST BW calculation
Regression")

But:
$ git show 4b6564cb120c
fatal: ambiguous argument '4b6564cb120c': unknown revision or path not 
in the working tree.


Instead, it is:
commit 338567d17627064dba63cf063459605e782f71d2
Author: Fangzhi Zuo <Jerry.Zuo@amd.com>
Date:   Mon Jul 29 10:23:03 2024 -0400

     drm/amd/display: Fix MST BW calculation Regression


So apparently, someone in the process rebases the tree or something. 
Which is another breakage (non-reliable SHAs).

thanks,
-- 
js
suse labs


  reply	other threads:[~2024-08-20  4:49 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-08-15 22:45 [PATCH 00/13] DC Patches August 15, 2024 Roman.Li
2024-08-15 22:45 ` [PATCH 01/13] Revert "drm/amd/display: Update to using new dccg callbacks" Roman.Li
2024-08-15 22:45 ` [PATCH 02/13] drm/amd/display: Update HPO I/O When Handling Link Retrain Automation Request Roman.Li
2024-08-15 22:45 ` [PATCH 03/13] drm/amd/display: remove an extraneous call for checking dchub clock Roman.Li
2024-08-15 22:45 ` [PATCH 04/13] drm/amd/display: Remove redundant check in DCN35 hwseq Roman.Li
2024-08-15 22:45 ` [PATCH 05/13] drm/amd/display: Allow UHBR Interop With eDP Supported Link Rates Table Roman.Li
2024-08-15 22:45 ` [PATCH 06/13] drm/amd/display: Hardware cursor changes color when switched to software cursor Roman.Li
2024-08-15 22:45 ` [PATCH 07/13] drm/amd/display: Support UHBR10 link rate on eDP Roman.Li
2024-08-15 22:45 ` [PATCH 08/13] drm/amd/display: Fix construct_phy with MXM connector Roman.Li
2024-08-15 22:45 ` [PATCH 09/13] drm/amd/display: DCN35 set min dispclk to 50Mhz Roman.Li
2024-08-15 22:45 ` [PATCH 10/13] drm/amd/display: fix double free issue during amdgpu module unload Roman.Li
2024-08-15 22:45 ` [PATCH 11/13] drm/amd/display: DML2.1 Reintegration for Various Fixes Roman.Li
2024-08-15 22:45 ` [PATCH 12/13] drm/amd/display: Fix a typo in revert commit Roman.Li
2024-08-16 19:30   ` Li, Roman
2024-08-19  8:37     ` Jiri Slaby
2024-08-19 14:29       ` Li, Roman
2024-08-20  4:49         ` Jiri Slaby [this message]
2024-08-20 18:28           ` Zuo, Jerry
2024-08-21  4:24             ` Jiri Slaby
2024-08-21 19:50           ` Zuo, Jerry
2024-08-15 22:45 ` [PATCH 13/13] drm/amd/display: Promote DC to 3.2.297 Roman.Li
2024-08-16 18:47 ` [PATCH 00/13] DC Patches August 15, 2024 Wheeler, Daniel

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=0d12eb50-c51b-496e-a931-9ce8fb6a1455@kernel.org \
    --to=jirislaby@kernel.org \
    --cc=Agustin.Gutierrez@amd.com \
    --cc=Alexander.Deucher@amd.com \
    --cc=Aurabindo.Pillai@amd.com \
    --cc=ChiaHsuan.Chung@amd.com \
    --cc=Harry.Wentland@amd.com \
    --cc=Jerry.Zuo@amd.com \
    --cc=Mario.Limonciello@amd.com \
    --cc=Rodrigo.Siqueira@amd.com \
    --cc=Roman.Li@amd.com \
    --cc=Sunpeng.Li@amd.com \
    --cc=Wayne.Lin@amd.com \
    --cc=Zaeem.Mohamed@amd.com \
    --cc=amd-gfx@lists.freedesktop.org \
    --cc=stable@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox