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@xxxxxxxxx> 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 -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html