[PATCH] drm/rockchip: Cleanup dangling devm pointers
Daniel Kurtz
djkurtz at chromium.org
Mon Sep 19 04:14:41 UTC 2016
Hi Sean,
On Sat, Sep 17, 2016 at 2:22 AM, Sean Paul <seanpaul at chromium.org> wrote:
>
> Instead of assigning device managed resources to local variables,
> keep track of them in the vop struct.
Why this patch?
Is it fixing an issue?
Or, is it preparing for some future use of ahb_rst outside of vop_initial?
I thought that one of the nice features of using devm is you do not
need to carry around pointers to devm allocated resources in the
driver local device struct.
-Dan
>
> Signed-off-by: Sean Paul <seanpaul at chromium.org>
> ---
> drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c
> index 131ae0f..bed782e 100644
> --- a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c
> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c
> @@ -142,6 +142,7 @@ struct vop {
>
> /* vop dclk reset */
> struct reset_control *dclk_rst;
> + struct reset_control *ahb_rst;
>
> struct vop_win win[];
> };
> @@ -1333,7 +1334,6 @@ static int vop_initial(struct vop *vop)
> {
> const struct vop_data *vop_data = vop->data;
> const struct vop_reg_data *init_table = vop_data->init_table;
> - struct reset_control *ahb_rst;
> int i, ret;
>
> vop->hclk = devm_clk_get(vop->dev, "hclk_vop");
> @@ -1374,15 +1374,15 @@ static int vop_initial(struct vop *vop)
> /*
> * do hclk_reset, reset all vop registers.
> */
> - ahb_rst = devm_reset_control_get(vop->dev, "ahb");
> - if (IS_ERR(ahb_rst)) {
> + vop->ahb_rst = devm_reset_control_get(vop->dev, "ahb");
> + if (IS_ERR(vop->ahb_rst)) {
> dev_err(vop->dev, "failed to get ahb reset\n");
> - ret = PTR_ERR(ahb_rst);
> + ret = PTR_ERR(vop->ahb_rst);
> goto err_disable_aclk;
> }
> - reset_control_assert(ahb_rst);
> + reset_control_assert(vop->ahb_rst);
> usleep_range(10, 20);
> - reset_control_deassert(ahb_rst);
> + reset_control_deassert(vop->ahb_rst);
>
> memcpy(vop->regsbak, vop->regs, vop->len);
>
> --
> 2.8.0.rc3.226.g39d4020
>
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
More information about the dri-devel
mailing list