* linux-4.7-rc3/drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c: 2 * pointless tests ?
@ 2016-06-13 14:51 David Binderman
2016-06-13 15:27 ` Rob Clark
0 siblings, 1 reply; 3+ messages in thread
From: David Binderman @ 2016-06-13 14:51 UTC (permalink / raw)
To: robdclark, airlied, linux-arm-msm, dri-devel, freedreno,
linux-kernel, dcb314
Hello there,
1.
linux-4.7-rc3/drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c:545]: (style)
Checking if unsigned variable 'pipe' is less than zero.
Source code is
if (pipe < 0 || pipe >= priv->num_crtcs) {
2.
linux-4.7-rc3/drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c:567]: (style)
Checking if unsigned variable 'pipe' is less than zero.
Duplicate a few lines further down.
Regards
David Binderman
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: linux-4.7-rc3/drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c: 2 * pointless tests ?
2016-06-13 14:51 linux-4.7-rc3/drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c: 2 * pointless tests ? David Binderman
@ 2016-06-13 15:27 ` Rob Clark
2016-06-15 10:03 ` David Binderman
0 siblings, 1 reply; 3+ messages in thread
From: Rob Clark @ 2016-06-13 15:27 UTC (permalink / raw)
To: David Binderman
Cc: David Airlie, linux-arm-msm, dri-devel, freedreno,
Linux Kernel Mailing List, dcb314
yup, looks like we can drop the two pipe<0 checks. Care to send a patch?
BR,
-R
On Mon, Jun 13, 2016 at 10:51 AM, David Binderman
<linuxdev.baldrick@gmail.com> wrote:
> Hello there,
>
> 1.
>
> linux-4.7-rc3/drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c:545]: (style)
> Checking if unsigned variable 'pipe' is less than zero.
>
> Source code is
>
> if (pipe < 0 || pipe >= priv->num_crtcs) {
>
> 2.
>
> linux-4.7-rc3/drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c:567]: (style)
> Checking if unsigned variable 'pipe' is less than zero.
>
> Duplicate a few lines further down.
>
>
> Regards
>
> David Binderman
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: linux-4.7-rc3/drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c: 2 * pointless tests ?
2016-06-13 15:27 ` Rob Clark
@ 2016-06-15 10:03 ` David Binderman
0 siblings, 0 replies; 3+ messages in thread
From: David Binderman @ 2016-06-15 10:03 UTC (permalink / raw)
To: Rob Clark
Cc: David Airlie, linux-arm-msm, dri-devel, freedreno,
Linux Kernel Mailing List, dcb314
Hello there,
>yup, looks like we can drop the two pipe<0 checks.
Righto.
>Care to send a patch?
Oh dear. My success rate with patches is near zero. Maybe something
like this might be suitable:
*** linux-3.7-rc3/drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c.sav
2016-06-15 10:58:04.868619030 +0100
--- linux-4.7-rc3/drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c 2016-06-15
10:58:25.401942157 +0100
***************
*** 542,548 ****
struct msm_drm_private *priv = dev->dev_private;
struct drm_crtc *crtc;
! if (pipe < 0 || pipe >= priv->num_crtcs) {
DRM_ERROR("Invalid crtc %d\n", pipe);
return -EINVAL;
}
--- 542,548 ----
struct msm_drm_private *priv = dev->dev_private;
struct drm_crtc *crtc;
! if (pipe >= priv->num_crtcs) {
DRM_ERROR("Invalid crtc %d\n", pipe);
return -EINVAL;
}
***************
*** 564,570 ****
struct drm_crtc *crtc;
struct drm_encoder *encoder;
! if (pipe < 0 || pipe >= priv->num_crtcs)
return 0;
crtc = priv->crtcs[pipe];
--- 564,570 ----
struct drm_crtc *crtc;
struct drm_encoder *encoder;
! if (pipe >= priv->num_crtcs)
return 0;
crtc = priv->crtcs[pipe];
Feel free to mess this tentative patch about in any way you see fit.
Regards
David Binderman
On Mon, Jun 13, 2016 at 4:27 PM, Rob Clark <robdclark@gmail.com> wrote:
> yup, looks like we can drop the two pipe<0 checks. Care to send a patch?
>
> BR,
> -R
>
> On Mon, Jun 13, 2016 at 10:51 AM, David Binderman
> <linuxdev.baldrick@gmail.com> wrote:
>> Hello there,
>>
>> 1.
>>
>> linux-4.7-rc3/drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c:545]: (style)
>> Checking if unsigned variable 'pipe' is less than zero.
>>
>> Source code is
>>
>> if (pipe < 0 || pipe >= priv->num_crtcs) {
>>
>> 2.
>>
>> linux-4.7-rc3/drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c:567]: (style)
>> Checking if unsigned variable 'pipe' is less than zero.
>>
>> Duplicate a few lines further down.
>>
>>
>> Regards
>>
>> David Binderman
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-06-15 10:09 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-06-13 14:51 linux-4.7-rc3/drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c: 2 * pointless tests ? David Binderman
2016-06-13 15:27 ` Rob Clark
2016-06-15 10:03 ` David Binderman
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).