On Mon, Sep 16, 2013 at 5:19 PM, Thomas Meyer <thomas@xxxxxxxx> wrote: > The variable priv->kms is not initialized yet. > > Found by "scripts/coccinelle/tests/odd_ptr_err.cocci". > PTR_ERR should access the value just tested by IS_ERR. > > Signed-off-by: Thomas Meyer <thomas@xxxxxxxx> thanks, I've got it queued up BR, -R > --- > > diff -u -p a/drivers/gpu/drm/msm/msm_drv.c b/drivers/gpu/drm/msm/msm_drv.c > --- a/drivers/gpu/drm/msm/msm_drv.c > +++ b/drivers/gpu/drm/msm/msm_drv.c > @@ -199,7 +199,7 @@ static int msm_load(struct drm_device *d > * imx drm driver on iMX5 > */ > dev_err(dev->dev, "failed to load kms\n"); > - ret = PTR_ERR(priv->kms); > + ret = PTR_ERR(kms); > goto fail; > } > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
