On Tue, May 9, 2023 at 1:17 AM SHANMUGAM, SRINIVASAN <[email protected]> wrote: > > [AMD Official Use Only - General] > > > > -----Original Message----- > From: Alex Deucher <[email protected]> > Sent: Monday, May 8, 2023 9:27 PM > To: SHANMUGAM, SRINIVASAN <[email protected]> > Cc: Koenig, Christian <[email protected]>; Deucher, Alexander > <[email protected]>; [email protected] > Subject: Re: [PATCH] drm/amd/amdgpu: Remove redundant else branch in > amdgpu_encoders.c > > On Mon, May 8, 2023 at 11:29 AM Srinivasan Shanmugam > <[email protected]> wrote: > > > > Adhere to Linux kernel coding style. > > > > Reported by checkpatch: > > > > WARNING: else is not generally useful after a break or return > > > > What about the else in the previous case statement? > > Alex > > Hi Alex, > > Thanks a lot for your feedbacks, > > the else in the previous case ie., is binded to if statement ie., "if > (amdgpu_connector->use_digital) {", am I correct please?, please correct me, > if my understanding is wrong? & the best solution with your tips pls, so that > I can edit & resend the patch please? >
Yes that one. It follows a similar pattern to the case you changed. Shouldn't checkpatch warn on both? Alex > Much appreciate for your help in advance, > > > Cc: Christian König <[email protected]> > > Cc: Alex Deucher <[email protected]> > > Signed-off-by: Srinivasan Shanmugam <[email protected]> > > --- > > drivers/gpu/drm/amd/amdgpu/amdgpu_encoders.c | 26 > > ++++++++++---------- > > 1 file changed, 13 insertions(+), 13 deletions(-) > > > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_encoders.c > > b/drivers/gpu/drm/amd/amdgpu/amdgpu_encoders.c > > index c96e458ed088..049e9976ff34 100644 > > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_encoders.c > > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_encoders.c > > @@ -242,19 +242,18 @@ bool amdgpu_dig_monitor_is_duallink(struct > > drm_encoder *encoder, > > if ((dig_connector->dp_sink_type == > > CONNECTOR_OBJECT_ID_DISPLAYPORT) || > > (dig_connector->dp_sink_type == > > CONNECTOR_OBJECT_ID_eDP)) > > return false; > > - else { > > - /* HDMI 1.3 supports up to 340 Mhz over single link > > */ > > - if (connector->display_info.is_hdmi) { > > - if (pixel_clock > 340000) > > - return true; > > - else > > - return false; > > - } else { > > - if (pixel_clock > 165000) > > - return true; > > - else > > - return false; > > - } > > + > > + /* HDMI 1.3 supports up to 340 Mhz over single link */ > > + if (connector->display_info.is_hdmi) { > > + if (pixel_clock > 340000) > > + return true; > > + else > > + return false; > > + } else { > > + if (pixel_clock > 165000) > > + return true; > > + else > > + return false; > > } > > default: > > return false; > > -- > > 2.25.1 > >
