[PATCH 2/3] imx-drm: imx-drm-core: avoid going the long route round for drm_device
Shawn Guo
shawn.guo at linaro.org
Wed Dec 18 01:39:17 UTC 2013
On Tue, Dec 17, 2013 at 07:10:47PM +0000, Russell King wrote:
> We have the drm_device available, so rather than storing it and then
> using the stored version, us the one we already have available to us.
>
> Signed-off-by: Russell King <rmk+kernel at arm.linux.org.uk>
> ---
> drivers/staging/imx-drm/imx-drm-core.c | 16 ++++++++--------
> 1 files changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/staging/imx-drm/imx-drm-core.c b/drivers/staging/imx-drm/imx-drm-core.c
> index 9aa5eaab6539..242e8db218ac 100644
> --- a/drivers/staging/imx-drm/imx-drm-core.c
> +++ b/drivers/staging/imx-drm/imx-drm-core.c
> @@ -88,9 +88,9 @@ static int imx_drm_driver_unload(struct drm_device *drm)
>
> imx_drm_device_put();
>
> - drm_vblank_cleanup(imxdrm->drm);
> - drm_kms_helper_poll_fini(imxdrm->drm);
> - drm_mode_config_cleanup(imxdrm->drm);
> + drm_vblank_cleanup(drm);
> + drm_kms_helper_poll_fini(drm);
> + drm_mode_config_cleanup(drm);
With the changes, 'imxdrm' in this function becomes an unused variable.
Other than that, for the series:
Acked-by: Shawn Guo <shawn.guo at linaro.org>
Tested-by: Shawn Guo <shawn.guo at linaro.org>
Shawn
>
> return 0;
> }
> @@ -424,15 +424,15 @@ static int imx_drm_driver_load(struct drm_device *drm, unsigned long flags)
>
> mutex_lock(&imxdrm->mutex);
>
> - drm_kms_helper_poll_init(imxdrm->drm);
> + drm_kms_helper_poll_init(drm);
>
> /* setup the grouping for the legacy output */
> - ret = drm_mode_group_init_legacy_group(imxdrm->drm,
> - &imxdrm->drm->primary->mode_group);
> + ret = drm_mode_group_init_legacy_group(drm,
> + &drm->primary->mode_group);
> if (ret)
> goto err_kms;
>
> - ret = drm_vblank_init(imxdrm->drm, MAX_CRTC);
> + ret = drm_vblank_init(drm, MAX_CRTC);
> if (ret)
> goto err_kms;
>
> @@ -441,7 +441,7 @@ static int imx_drm_driver_load(struct drm_device *drm, unsigned long flags)
> * by drm timer once a current process gives up ownership of
> * vblank event.(after drm_vblank_put function is called)
> */
> - imxdrm->drm->vblank_disable_allowed = true;
> + drm->vblank_disable_allowed = true;
>
> if (!imx_drm_device_get()) {
> ret = -EINVAL;
> --
> 1.7.4.4
>
More information about the devel
mailing list