drm/exynos: remove superfluous checks in g2d_check_reg_offset()
Tobias Jakobi
tjakobi at math.uni-bielefeld.de
Tue Aug 25 08:50:07 PDT 2015
Gentle ping!
On 2015-08-18 00:51, Tobias Jakobi wrote:
> The cases of the switch statement ensure that reg_type
> can never be REG_TYPE_NONE here.
>
> Signed-off-by: Tobias Jakobi <tjakobi at math.uni-bielefeld.de>
> ---
> drivers/gpu/drm/exynos/exynos_drm_g2d.c | 8 --------
> 1 file changed, 8 deletions(-)
>
> diff --git a/drivers/gpu/drm/exynos/exynos_drm_g2d.c
> b/drivers/gpu/drm/exynos/exynos_drm_g2d.c
> index ba00839..211af37 100644
> --- a/drivers/gpu/drm/exynos/exynos_drm_g2d.c
> +++ b/drivers/gpu/drm/exynos/exynos_drm_g2d.c
> @@ -973,8 +973,6 @@ static int g2d_check_reg_offset(struct device *dev,
> goto err;
>
> reg_type = g2d_get_reg_type(reg_offset);
> - if (reg_type == REG_TYPE_NONE)
> - goto err;
>
> /* check userptr buffer type. */
> if ((cmdlist->data[index] & ~0x7fffffff) >> 31) {
> @@ -989,8 +987,6 @@ static int g2d_check_reg_offset(struct device *dev,
> goto err;
>
> reg_type = g2d_get_reg_type(reg_offset);
> - if (reg_type == REG_TYPE_NONE)
> - goto err;
>
> buf_desc = &buf_info->descs[reg_type];
> value = cmdlist->data[index + 1];
> @@ -1003,8 +999,6 @@ static int g2d_check_reg_offset(struct device
> *dev,
> goto err;
>
> reg_type = g2d_get_reg_type(reg_offset);
> - if (reg_type == REG_TYPE_NONE)
> - goto err;
>
> buf_desc = &buf_info->descs[reg_type];
> value = cmdlist->data[index + 1];
> @@ -1018,8 +1012,6 @@ static int g2d_check_reg_offset(struct device
> *dev,
> goto err;
>
> reg_type = g2d_get_reg_type(reg_offset);
> - if (reg_type == REG_TYPE_NONE)
> - goto err;
>
> buf_desc = &buf_info->descs[reg_type];
> value = cmdlist->data[index + 1];
More information about the dri-devel
mailing list