mbox series

[v1,0/6] drm/amd/display: Use max/min macro

Message ID 20240822065723.1336181-1-yujiaoliang@vivo.com (mailing list archive)
Headers show
Series drm/amd/display: Use max/min macro | expand

Message

Yu Jiaoliang Aug. 22, 2024, 6:57 a.m. UTC
*** BLURB HERE ***

Yu Jiaoliang (6):
  drm/amd/display: Use max/min macro
  drm/amd/display: Use max/min macro
  drm/amd/display: Use max/min macro
  drm/amd/display: Use max/min macro
  drm/amd/display: Use max/min macro
  drm/amd/display: Use max/min macro

 drivers/gpu/drm/amd/display/dc/bios/bios_parser.c            | 4 ++--
 drivers/gpu/drm/amd/display/dc/clk_mgr/dcn20/dcn20_clk_mgr.c | 4 +++-
 drivers/gpu/drm/amd/display/dc/core/dc_resource.c            | 4 ++--
 drivers/gpu/drm/amd/display/dc/dc_dmub_srv.c                 | 2 +-
 drivers/gpu/drm/amd/display/dc/spl/dc_spl.c                  | 4 ++--
 drivers/gpu/drm/amd/display/modules/freesync/freesync.c      | 4 ++--
 6 files changed, 12 insertions(+), 10 deletions(-)

Comments

Yu Jiaoliang Aug. 22, 2024, 9:45 a.m. UTC | #1
在 2024/8/22 14:57, Yu Jiaoliang 写道:
> *** BLURB HERE ***
>
> Yu Jiaoliang (6):
>    drm/amd/display: Use max/min macro
>    drm/amd/display: Use max/min macro
>    drm/amd/display: Use max/min macro
>    drm/amd/display: Use max/min macro
>    drm/amd/display: Use max/min macro
>    drm/amd/display: Use max/min macro
>
>   drivers/gpu/drm/amd/display/dc/bios/bios_parser.c            | 4 ++--
>   drivers/gpu/drm/amd/display/dc/clk_mgr/dcn20/dcn20_clk_mgr.c | 4 +++-
>   drivers/gpu/drm/amd/display/dc/core/dc_resource.c            | 4 ++--
>   drivers/gpu/drm/amd/display/dc/dc_dmub_srv.c                 | 2 +-
>   drivers/gpu/drm/amd/display/dc/spl/dc_spl.c                  | 4 ++--
>   drivers/gpu/drm/amd/display/modules/freesync/freesync.c      | 4 ++--
>   6 files changed, 12 insertions(+), 10 deletions(-)
>
Sorry, please ignore this patch for now. As the corresponding header 
file is not included, compilation errors may occur. This patch will be 
updated later.

Br,

Yu
Alex Deucher Aug. 22, 2024, 6:34 p.m. UTC | #2
On Thu, Aug 22, 2024 at 2:57 AM Yu Jiaoliang <yujiaoliang@vivo.com> wrote:
>
> *** BLURB HERE ***
>
> Yu Jiaoliang (6):
>   drm/amd/display: Use max/min macro
>   drm/amd/display: Use max/min macro
>   drm/amd/display: Use max/min macro
>   drm/amd/display: Use max/min macro
>   drm/amd/display: Use max/min macro
>   drm/amd/display: Use max/min macro

Is this the preferred kernel coding style for these type of
comparisons?  Otherwise this just seems like a lot of churn for not
much gain.

Alex

>
>  drivers/gpu/drm/amd/display/dc/bios/bios_parser.c            | 4 ++--
>  drivers/gpu/drm/amd/display/dc/clk_mgr/dcn20/dcn20_clk_mgr.c | 4 +++-
>  drivers/gpu/drm/amd/display/dc/core/dc_resource.c            | 4 ++--
>  drivers/gpu/drm/amd/display/dc/dc_dmub_srv.c                 | 2 +-
>  drivers/gpu/drm/amd/display/dc/spl/dc_spl.c                  | 4 ++--
>  drivers/gpu/drm/amd/display/modules/freesync/freesync.c      | 4 ++--
>  6 files changed, 12 insertions(+), 10 deletions(-)
>
> --
> 2.34.1
>