Applied. Thanks!

Alex

On Tue, Aug 24, 2021 at 1:52 AM CGEL cgel.zte@gmail.com wrote:

From: Jing Yangyang jing.yangyang@zte.com.cn

./drivers/gpu/drm/amd/display/dc/dcn31/dcn31_panel_cntl.c:112:9-10:WARNING: return of 0/1 in function 'dcn31_is_panel_backlight_on' with return type bool

./drivers/gpu/drm/amd/display/dc/dcn31/dcn31_panel_cntl.c:122:9-10:WARNING: return of 0/1 in function 'dcn31_is_panel_powered_on' with return type bool

Return statements in functions returning bool should use true/false instead of 1/0.

Generated by: scripts/coccinelle/misc/boolreturn.cocci

Reported-by: Zeal Robot zealci@zte.com.cn Signed-off-by: Jing Yangyang jing.yangyang@zte.com.cn

---

drivers/gpu/drm/amd/display/dc/dcn31/dcn31_panel_cntl.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dc/dcn31/dcn31_panel_cntl.c b/drivers/gpu/drm/amd/display/dc/dcn31/dcn31_panel_cntl.c index 7db268d..3b37213 100644 --- a/drivers/gpu/drm/amd/display/dc/dcn31/dcn31_panel_cntl.c +++ b/drivers/gpu/drm/amd/display/dc/dcn31/dcn31_panel_cntl.c @@ -109,7 +109,7 @@ bool dcn31_is_panel_backlight_on(struct panel_cntl *panel_cntl) union dmub_rb_cmd cmd;

    if (!dcn31_query_backlight_info(panel_cntl, &cmd))

•           return 0;
  

•           return false;
  
    return cmd.panel_cntl.data.is_backlight_on;
  

} @@ -119,7 +119,7 @@ bool dcn31_is_panel_powered_on(struct panel_cntl *panel_cntl) union dmub_rb_cmd cmd;

    if (!dcn31_query_backlight_info(panel_cntl, &cmd))

•           return 0;
  

•           return false;
  
    return cmd.panel_cntl.data.is_powered_on;
  

}

1.8.3.1