* Greg Dietsche <Gregory.Dietsche@xxxxxxx>: > the code always returns ret regardless, so if(ret) check is unecessary. /unecessary/unnecessary/ > > Signed-off-by: Greg Dietsche <Gregory.Dietsche@xxxxxxx> Reviewed-by: Nicolas Kaiser <nikai@xxxxxxxxx> Best regards, Nicolas Kaiser > --- > drivers/gpu/drm/savage/savage_bci.c | 3 --- > 1 files changed, 0 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/savage/savage_bci.c b/drivers/gpu/drm/savage/savage_bci.c > index bf5f83e..cb1ee4e 100644 > --- a/drivers/gpu/drm/savage/savage_bci.c > +++ b/drivers/gpu/drm/savage/savage_bci.c > @@ -647,9 +647,6 @@ int savage_driver_firstopen(struct drm_device *dev) > ret = drm_addmap(dev, aperture_base, SAVAGE_APERTURE_SIZE, > _DRM_FRAME_BUFFER, _DRM_WRITE_COMBINING, > &dev_priv->aperture); > - if (ret) > - return ret; > - > return ret; > } > _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
