drm/msm/dpu: merge two CRTC debugfs dirs
authorDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Thu, 12 Jan 2023 05:36:59 +0000 (07:36 +0200)
committerDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Wed, 18 Jan 2023 01:34:45 +0000 (03:34 +0200)
For each CRTC we are creating two different debugfs directories one
using crtc index (created automatically for the CRC files) and another
one using CRTC name/object ID (for state and status files).

This can be confusing, so move our custom files to crtc->debugfs_entry,
effetively merging two debugfs dirs.

Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Reviewed-by: Abhinav Kumar <quic_abhinavk@quicinc.com>
Tested-by: Abhinav Kumar <quic_abhinavk@quicinc.com>
Patchwork: https://patchwork.freedesktop.org/patch/518065/
Link: https://lore.kernel.org/r/20230112053659.1361298-1-dmitry.baryshkov@linaro.org
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c

index 13ce321..c264801 100644 (file)
@@ -1517,16 +1517,12 @@ DEFINE_SHOW_ATTRIBUTE(dpu_crtc_debugfs_state);
 static int _dpu_crtc_init_debugfs(struct drm_crtc *crtc)
 {
        struct dpu_crtc *dpu_crtc = to_dpu_crtc(crtc);
-       struct dentry *debugfs_root;
-
-       debugfs_root = debugfs_create_dir(dpu_crtc->name,
-                       crtc->dev->primary->debugfs_root);
 
        debugfs_create_file("status", 0400,
-                       debugfs_root,
+                       crtc->debugfs_entry,
                        dpu_crtc, &_dpu_debugfs_status_fops);
        debugfs_create_file("state", 0600,
-                       debugfs_root,
+                       crtc->debugfs_entry,
                        &dpu_crtc->base,
                        &dpu_crtc_debugfs_state_fops);