[PATCH] drm/msm/dpu: merge two CRTC debugfs dirs

[Abhinav Kumar]

On 1/11/2023 9:36 PM, Dmitry Baryshkov wrote:
> 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 at linaro.org>

Makes sense to me and also verified it on my chromebook.

Reviewed-by: Abhinav Kumar <quic_abhinavk at quicinc.com>
Tested-by: Abhinav Kumar <quic_abhinavk at quicinc.com>


> ---
>   drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 8 ++------
>   1 file changed, 2 insertions(+), 6 deletions(-)
> 
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c
> index 7f0f467dbabd..659fdfec5346 100644
> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c
> @@ -1380,16 +1380,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);
>