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

2023-03-06 Thread Joe Perches
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

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

2023-03-03 Thread Hamza Mahfooz
On 1/15/23 05:00, Deepak R Varma wrote: The if / else block code has same effect irrespective of the logical evaluation. Hence, simply the implementation by removing the unnecessary conditional evaluation. While at it, also fix the long line checkpatch complaint. Issue identified using

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

2023-03-03 Thread Harry Wentland
On 3/2/23 11:37, Harry Wentland wrote: > > > On 3/1/23 15:21, Deepak R Varma wrote: >> On Mon, Jan 23, 2023 at 12:23:19AM +0530, Deepak R Varma wrote: >>> On Sun, Jan 15, 2023 at 12:52:10PM -0800, Joe Perches wrote: On Sun, 2023-01-15 at 15:30 +0530, Deepak R Varma wrote: > The if /

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

2023-03-03 Thread Deepak R Varma
On Thu, Mar 02, 2023 at 11:37:30AM -0500, Harry Wentland wrote: > > > On 3/1/23 15:21, Deepak R Varma wrote: > > On Mon, Jan 23, 2023 at 12:23:19AM +0530, Deepak R Varma wrote: > >> On Sun, Jan 15, 2023 at 12:52:10PM -0800, Joe Perches wrote: > >>> On Sun, 2023-01-15 at 15:30 +0530, Deepak R

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

2023-03-02 Thread Harry Wentland
On 3/1/23 15:21, Deepak R Varma wrote: > On Mon, Jan 23, 2023 at 12:23:19AM +0530, Deepak R Varma wrote: >> On Sun, Jan 15, 2023 at 12:52:10PM -0800, Joe Perches wrote: >>> On Sun, 2023-01-15 at 15:30 +0530, Deepak R Varma wrote: The if / else block code has same effect irrespective of the

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

2023-03-01 Thread Deepak R Varma
On Mon, Jan 23, 2023 at 12:23:19AM +0530, Deepak R Varma wrote: > On Sun, Jan 15, 2023 at 12:52:10PM -0800, Joe Perches wrote: > > On Sun, 2023-01-15 at 15:30 +0530, Deepak R Varma wrote: > > > The if / else block code has same effect irrespective of the logical > > > evaluation. Hence, simply

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

2023-01-22 Thread Deepak R Varma
On Sun, Jan 15, 2023 at 12:52:10PM -0800, Joe Perches wrote: > On Sun, 2023-01-15 at 15:30 +0530, Deepak R Varma wrote: > > The if / else block code has same effect irrespective of the logical > > evaluation. Hence, simply the implementation by removing the unnecessary > > conditional evaluation.

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

2023-01-16 Thread Joe Perches
On Sun, 2023-01-15 at 15:30 +0530, Deepak R Varma wrote: > The if / else block code has same effect irrespective of the logical > evaluation. Hence, simply the implementation by removing the unnecessary > conditional evaluation. While at it, also fix the long line checkpatch > complaint. Issue

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

2023-01-15 Thread Deepak R Varma
The if / else block code has same effect irrespective of the logical evaluation. Hence, simply the implementation by removing the unnecessary conditional evaluation. While at it, also fix the long line checkpatch complaint. Issue identified using cond_no_effect.cocci Coccinelle semantic patch