Re: [PATCH] drm/amd/display: Simplify same effect if/else blocks

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: Re: [PATCH] drm/amd/display: Simplify same effect if/else blocks
From: Joe Perches <joe@xxxxxxxxxxx>
Date: Sat, 04 Mar 2023 15:48:05 -0800
Cc: Saurabh Singh Sengar <ssengar@xxxxxxxxxxxxx>, Leo Li <sunpeng.li@xxxxxxx>, "Pan, Xinhui" <Xinhui.Pan@xxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx, amd-gfx@xxxxxxxxxxxxxxxxxxxxx, Praveen Kumar <kumarpraveen@xxxxxxxxxxxxxxxxxxx>, dri-devel@xxxxxxxxxxxxxxxxxxxxx, Christian König <christian.koenig@xxxxxxx>
In-reply-to: <4d670a5c-b680-ba76-a640-88b3d945c0b0@amd.com>
References: <Y8POxreeC3EvOXhC@ubun2204.myguest.virtualbox.org> <33ecbe8971bb9c90d72c67d43ca740abac160908.camel@perches.com> <Y82GHzOUHgEqTUq7@ubun2204.myguest.virtualbox.org> <Y/+z22xM7NNjX8VP@ubun2204.myguest.virtualbox.org> <f15b10a8-11f6-6c1f-c94e-71796aad9155@amd.com> <4d670a5c-b680-ba76-a640-88b3d945c0b0@amd.com>
User-agent: Evolution 3.44.4 (3.44.4-2.fc36)

On Fri, 2023-03-03 at 15:35 -0500, Harry Wentland wrote:
> Actually I was wrong. Too many similar-looking snippets in this
> function made me look at the wrong thing. This change is fine and
> Reviewed-by: Harry Wentland <harry.wentland@xxxxxxx.

So why was the change made in the first place?

Please explain commit 9114b55fabae.

References:
- [PATCH] drm/amd/display: Simplify same effect if/else blocks
  - From: Deepak R Varma
- Re: [PATCH] drm/amd/display: Simplify same effect if/else blocks
  - From: Joe Perches
- Re: [PATCH] drm/amd/display: Simplify same effect if/else blocks
  - From: Deepak R Varma
- Re: [PATCH] drm/amd/display: Simplify same effect if/else blocks
  - From: Deepak R Varma
- Re: [PATCH] drm/amd/display: Simplify same effect if/else blocks
  - From: Harry Wentland
- Re: [PATCH] drm/amd/display: Simplify same effect if/else blocks
  - From: Harry Wentland

Prev by Date: Re: [PATCH] MAINTAINERS: Update the URI for MSM DRM bugs
Next by Date: [PATCH] drm/amdgpu: Use uncached ioremap() for LoongArch
Previous by thread: Re: [PATCH] drm/amd/display: Simplify same effect if/else blocks
Next by thread: Re: [PATCH] drm/amd/display: Simplify same effect if/else blocks
Index(es):
- Date
- Thread

[Index of Archives] [Linux DRI Users] [Linux Intel Graphics] [Linux USB Devel] [Video for Linux] [Linux Audio Users] [Yosemite News] [Linux Kernel] [Linux SCSI] [XFree86] [Linux USB Devel] [Video for Linux] [Linux Audio Users] [Linux Kernel] [Linux SCSI] [XFree86]