diff mbox series

[v1,3/8] drm/msm/dpu: make danger_status/safe_status readable

Message ID 20211201222633.2476780-4-dmitry.baryshkov@linaro.org
State Accepted
Commit 1a24e099c382adc062966e5e45e977e0283303db
Headers show
Series [v1,1/8] drm/msm/dpu: move disable_danger out of plane subdir | expand

Commit Message

Dmitry Baryshkov Dec. 1, 2021, 10:26 p.m. UTC 
Change \t to \n in the print format to stop putting all SSPP status in a
single line. Splitting it to one SSPP per line is much more readable.

Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
---
 drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Abhinav Kumar Dec. 9, 2021, 9:10 p.m. UTC | #1 
On 12/1/2021 2:26 PM, Dmitry Baryshkov wrote:
> Change \t to \n in the print format to stop putting all SSPP status in a
> single line. Splitting it to one SSPP per line is much more readable.
> 
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Reviewed-by: Abhinav Kumar <quic_abhinavk@quicinc.com>
> ---
>   drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
> index e7f0cded2c6b..4c04982c71b2 100644
> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
> @@ -82,7 +82,7 @@ static int _dpu_danger_signal_status(struct seq_file *s,
>   	seq_printf(s, "MDP     :  0x%x\n", status.mdp);
>   
>   	for (i = SSPP_VIG0; i < SSPP_MAX; i++)
> -		seq_printf(s, "SSPP%d   :  0x%x  \t", i - SSPP_VIG0,
> +		seq_printf(s, "SSPP%d   :  0x%x  \n", i - SSPP_VIG0,
>   				status.sspp[i]);
>   	seq_puts(s, "\n");
>   
>
diff mbox series

Patch

diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
index e7f0cded2c6b..4c04982c71b2 100644
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
@@ -82,7 +82,7 @@  static int _dpu_danger_signal_status(struct seq_file *s,
 	seq_printf(s, "MDP     :  0x%x\n", status.mdp);
 
 	for (i = SSPP_VIG0; i < SSPP_MAX; i++)
-		seq_printf(s, "SSPP%d   :  0x%x  \t", i - SSPP_VIG0,
+		seq_printf(s, "SSPP%d   :  0x%x  \n", i - SSPP_VIG0,
 				status.sspp[i]);
 	seq_puts(s, "\n");