Skip to content

Commit 0c2dda8

Browse files
lumagRob Clark
authored andcommitted
drm/msm: move helper calls to msm_kms.c
Extract two more KMS-related codepieces to msm_kms.c, removing last pieces of KMS code from msm_drv.c. Reviewed-by: Abhinav Kumar <[email protected]> Signed-off-by: Dmitry Baryshkov <[email protected]> Patchwork: https://patchwork.freedesktop.org/patch/662574/ Signed-off-by: Rob Clark <[email protected]>
1 parent a409b78 commit 0c2dda8

File tree

3 files changed

+25
-6
lines changed

3 files changed

+25
-6
lines changed

drivers/gpu/drm/msm/msm_drv.c

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
#include <linux/of_address.h>
1212
#include <linux/uaccess.h>
1313

14-
#include <drm/clients/drm_client_setup.h>
1514
#include <drm/drm_drv.h>
1615
#include <drm/drm_file.h>
1716
#include <drm/drm_ioctl.h>
@@ -74,7 +73,7 @@ static int msm_drm_uninit(struct device *dev)
7473
if (ddev->registered) {
7574
drm_dev_unregister(ddev);
7675
if (priv->kms)
77-
drm_atomic_helper_shutdown(ddev);
76+
msm_drm_kms_unregister(dev);
7877
}
7978

8079
msm_gem_shrinker_cleanup(ddev);
@@ -167,10 +166,8 @@ static int msm_drm_init(struct device *dev, const struct drm_driver *drv)
167166
if (ret)
168167
goto err_msm_uninit;
169168

170-
if (priv->kms_init) {
171-
drm_kms_helper_poll_init(ddev);
172-
drm_client_setup(ddev, NULL);
173-
}
169+
if (priv->kms_init)
170+
msm_drm_kms_post_init(dev);
174171

175172
return 0;
176173

drivers/gpu/drm/msm/msm_kms.c

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
#include <drm/drm_drv.h>
1414
#include <drm/drm_mode_config.h>
1515
#include <drm/drm_vblank.h>
16+
#include <drm/clients/drm_client_setup.h>
1617

1718
#include "disp/msm_disp_snapshot.h"
1819
#include "msm_drv.h"
@@ -217,6 +218,15 @@ struct drm_gpuvm *msm_kms_init_vm(struct drm_device *dev)
217218
return vm;
218219
}
219220

221+
void msm_drm_kms_unregister(struct device *dev)
222+
{
223+
struct platform_device *pdev = to_platform_device(dev);
224+
struct msm_drm_private *priv = platform_get_drvdata(pdev);
225+
struct drm_device *ddev = priv->dev;
226+
227+
drm_atomic_helper_shutdown(ddev);
228+
}
229+
220230
void msm_drm_kms_uninit(struct device *dev)
221231
{
222232
struct platform_device *pdev = to_platform_device(dev);
@@ -366,3 +376,13 @@ void msm_kms_shutdown(struct platform_device *pdev)
366376
if (drm && drm->registered && priv->kms)
367377
drm_atomic_helper_shutdown(drm);
368378
}
379+
380+
void msm_drm_kms_post_init(struct device *dev)
381+
{
382+
struct platform_device *pdev = to_platform_device(dev);
383+
struct msm_drm_private *priv = platform_get_drvdata(pdev);
384+
struct drm_device *ddev = priv->dev;
385+
386+
drm_kms_helper_poll_init(ddev);
387+
drm_client_setup(ddev, NULL);
388+
}

drivers/gpu/drm/msm/msm_kms.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -200,6 +200,8 @@ static inline void msm_kms_destroy(struct msm_kms *kms)
200200
for_each_if (drm_crtc_mask(crtc) & (crtc_mask))
201201

202202
int msm_drm_kms_init(struct device *dev, const struct drm_driver *drv);
203+
void msm_drm_kms_post_init(struct device *dev);
204+
void msm_drm_kms_unregister(struct device *dev);
203205
void msm_drm_kms_uninit(struct device *dev);
204206

205207
#endif /* __MSM_KMS_H__ */

0 commit comments

Comments
 (0)