diff options
author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2021-01-12 11:43:50 +0300 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2021-02-25 14:14:23 +0300 |
commit | c9e88a2ab5b411b5b41940e9e5b4b15d647d64d6 (patch) | |
tree | bc01742f43ed0d7d9cc9fbee127045014c6fa926 /drivers/gpu/drm/arc/arcpgu_crtc.c | |
parent | be5207ae897ed394d55988e001eba6f10f554040 (diff) | |
download | linux-c9e88a2ab5b411b5b41940e9e5b4b15d647d64d6.tar.xz |
drm/arc: Use drmm_mode_config_cleanup
With autocleanup through drm_device management we can delete all the
code. Possible now that there's no confusion against devm_kzalloc'ed
structures anymore.
I inlined arcpgu_setup_mode_config because it's tiny and the newly
needed return value handling would have been more ...
Acked-by: Sam Ravnborg <sam@ravnborg.org>
Acked-by: Thomas Zimmermann <tzimmermann@suse.de>
Cc: Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Cc: Alexey Brodkin <abrodkin@synopsys.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20210112084358.2771527-7-daniel.vetter@ffwll.ch
Diffstat (limited to 'drivers/gpu/drm/arc/arcpgu_crtc.c')
-rw-r--r-- | drivers/gpu/drm/arc/arcpgu_crtc.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/gpu/drm/arc/arcpgu_crtc.c b/drivers/gpu/drm/arc/arcpgu_crtc.c index bcf3c3f077a6..1cf29dea931e 100644 --- a/drivers/gpu/drm/arc/arcpgu_crtc.c +++ b/drivers/gpu/drm/arc/arcpgu_crtc.c @@ -207,10 +207,8 @@ int arc_pgu_setup_crtc(struct drm_device *drm) ret = drm_crtc_init_with_planes(drm, &arcpgu->pipe.crtc, primary, NULL, &arc_pgu_crtc_funcs, NULL); - if (ret) { - drm_plane_cleanup(primary); + if (ret) return ret; - } drm_crtc_helper_add(&arcpgu->pipe.crtc, &arc_pgu_crtc_helper_funcs); return 0; |