diff options
author | Jyri Sarha <jsarha@ti.com> | 2018-04-26 11:06:59 +0300 |
---|---|---|
committer | Thierry Reding <treding@nvidia.com> | 2018-05-18 11:22:06 +0200 |
commit | 38992c57c9c8425dc9cb75efe6f9b9255ea627a0 (patch) | |
tree | 6315380f51e94ee951fa3be75f5261302bf76864 /drivers/gpu/drm/panel/panel-simple.c | |
parent | 6553b123eae8dd117f6adb4a42e2e6ef8bff321d (diff) | |
download | linux-stable-38992c57c9c8425dc9cb75efe6f9b9255ea627a0.tar.gz linux-stable-38992c57c9c8425dc9cb75efe6f9b9255ea627a0.tar.bz2 linux-stable-38992c57c9c8425dc9cb75efe6f9b9255ea627a0.zip |
drm/panel: Remove drm_panel_detach() calls from all panel drivers
Remove all drm_panel_detach() calls from all panel drivers and update
the kerneldoc for drm_panel_detach().
Setting the connector and drm to NULL when the DRM panel device is going
away hardly serves any purpose. Usually the whole memory structure is
freed right after the remove call. However, calling the detach function
from the master DRM device, and setting the connector pointer to NULL,
has the logic of marking the panel again as available for another DRM
master to attach. The usual situation would be the same DRM master
device binding again.
Signed-off-by: Jyri Sarha <jsarha@ti.com>
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Thierry Reding <treding@nvidia.com>
Link: https://patchwork.freedesktop.org/patch/msgid/464b8d330d6b4c94cfb5aad2ca9ea7eb2c52d934.1524727888.git.jsarha@ti.com
Diffstat (limited to 'drivers/gpu/drm/panel/panel-simple.c')
-rw-r--r-- | drivers/gpu/drm/panel/panel-simple.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c index cbf1ab404ee7..062b6b416b89 100644 --- a/drivers/gpu/drm/panel/panel-simple.c +++ b/drivers/gpu/drm/panel/panel-simple.c @@ -364,7 +364,6 @@ static int panel_simple_remove(struct device *dev) { struct panel_simple *panel = dev_get_drvdata(dev); - drm_panel_detach(&panel->base); drm_panel_remove(&panel->base); panel_simple_disable(&panel->base); |